--- loncom/interface/lonmeta.pm	2004/04/13 14:42:24	1.64
+++ loncom/interface/lonmeta.pm	2005/11/02 01:02:31	1.122
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Metadata display handler
 #
-# $Id: lonmeta.pm,v 1.64 2004/04/13 14:42:24 matthew Exp $
+# $Id: lonmeta.pm,v 1.122 2005/11/02 01:02:31 banghart Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -17,7 +17,7 @@
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU General Public License for more details.
 #
-# You should have received a copy of the GNU General Public License
+# You should have received a copy of the GNU General Public License 
 # along with LON-CAPA; if not, write to the Free Software
 # Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 #
@@ -31,18 +31,96 @@ package Apache::lonmeta;
 use strict;
 use LONCAPA::lonmetadata();
 use Apache::Constants qw(:common);
-use Apache::lonnet();
+use Apache::lonnet;
 use Apache::loncommon();
-use Apache::lonhtmlcommon();
+use Apache::lonhtmlcommon(); 
 use Apache::lonmsg;
 use Apache::lonpublisher;
 use Apache::lonlocal;
 use Apache::lonmysql;
 use Apache::lonmsg;
 
-# MySQL table columns
 
-my @columns;
+############################################################
+############################################################
+##
+## &get_dynamic_metadata_from_sql($url)
+## 
+## Queries sql database for dynamic metdata
+## Returns a hash of hashes, with keys of urls which match $url
+## Returned fields are given below.
+##
+## Examples:
+## 
+## %DynamicMetadata = &Apache::lonmeta::get_dynmaic_metadata_from_sql
+##     ('/res/msu/korte/');
+##
+## $DynamicMetadata{'/res/msu/korte/example.problem'}->{$field}
+##
+############################################################
+############################################################
+sub get_dynamic_metadata_from_sql {
+    my ($url) = shift();
+    my ($authordom,$author)=($url=~m:^/res/(\w+)/(\w+)/:);
+    if (! defined($authordom)) {
+        $authordom = shift();
+    }
+    if  (! defined($author)) { 
+        $author = shift();
+    }
+    if (! defined($authordom) || ! defined($author)) {
+        return ();
+    }
+    my @Fields = ('url','count','course',
+                  'goto','goto_list',
+                  'comefrom','comefrom_list',
+                  'sequsage','sequsage_list',
+                  'stdno','stdno_list',
+		  'dependencies',
+                  'avetries','avetries_list',
+                  'difficulty','difficulty_list',
+                  'disc','disc_list',
+                  'clear','technical','correct',
+                  'helpful','depth');
+    #
+    my $query = 'SELECT '.join(',',@Fields).
+        ' FROM metadata WHERE url LIKE "'.$url.'%"';
+    my $server = &Apache::lonnet::homeserver($author,$authordom);
+    my $reply = &Apache::lonnet::metadata_query($query,undef,undef,
+                                                ,[$server]);
+    return () if (! defined($reply) || ref($reply) ne 'HASH');
+    my $filename = $reply->{$server};
+    if (! defined($filename) || $filename =~ /^error/) {
+        return ();
+    }
+    my $max_time = time + 10; # wait 10 seconds for results at most
+    my %ReturnHash;
+    #
+    # Look for results
+    my $finished = 0;
+    while (! $finished && time < $max_time) {
+        my $datafile=$Apache::lonnet::perlvar{'lonDaemons'}.'/tmp/'.$filename;
+        if (! -e "$datafile.end") { next; }
+        my $fh;
+        if (!($fh=Apache::File->new($datafile))) { next; }
+        while (my $result = <$fh>) {
+            chomp($result);
+            next if (! $result);
+            my @Data = 
+                map { 
+                    &Apache::lonnet::unescape($_); 
+                } split(',',$result);
+            my $url = $Data[0];
+            for (my $i=0;$i<=$#Fields;$i++) {
+                $ReturnHash{$url}->{$Fields[$i]}=$Data[$i];
+            }
+        }
+        $finished = 1;
+    }
+    #
+    return %ReturnHash;
+}
+
 
 # Fetch and evaluate dynamic metadata
 sub dynamicmeta {
@@ -60,6 +138,15 @@ sub dynamicmeta {
     #
     # Deal with 'count' separately
     $Data{'count'} = &access_count($url,$aauthor,$adomain);
+    #
+    # Debugging code I will probably need later
+    if (0) {
+        &Apache::lonnet::logthis('Dynamic Metadata');
+        while(my($k,$v)=each(%Data)){
+            &Apache::lonnet::logthis('    "'.$k.'"=>"'.$v.'"');
+        }
+        &Apache::lonnet::logthis('-------------------');
+    }
     return %Data;
 }
 
@@ -104,59 +191,54 @@ sub authordisplay {
 # Pretty display
 sub evalgraph {
     my $value=shift;
-    unless ($value) { return ''; }
+    if (! $value) { 
+        return '';
+    }
     my $val=int($value*10.+0.5)-10;
-    my $output='<table border=0 cellpadding=0 cellspacing=0><tr>';
+    my $output='<table border="0" cellpadding="0" cellspacing="0"><tr>';
     if ($val>=20) {
-	$output.='<td width=20 bgcolor="#555555">&nbsp&nbsp;</td>';
+	$output.='<td width="20" bgcolor="#555555">&nbsp&nbsp;</td>';
     } else {
-        $output.='<td width='.($val).' bgcolor="#555555">&nbsp;</td>'.
-                 '<td width='.(20-$val).' bgcolor="#FF3333">&nbsp;</td>';
+        $output.='<td width="'.($val).'" bgcolor="#555555">&nbsp;</td>'.
+                 '<td width="'.(20-$val).'" bgcolor="#FF3333">&nbsp;</td>';
     }
     $output.='<td bgcolor="#FFFF33">&nbsp;</td>';
     if ($val>20) {
-	$output.='<td width='.($val-20).' bgcolor="#33FF33">&nbsp;</td>'.
-                 '<td width='.(40-$val).' bgcolor="#555555">&nbsp;</td>';
+	$output.='<td width="'.($val-20).'" bgcolor="#33FF33">&nbsp;</td>'.
+                 '<td width="'.(40-$val).'" bgcolor="#555555">&nbsp;</td>';
     } else {
-        $output.='<td width=20 bgcolor="#555555">&nbsp&nbsp;</td>';
+        $output.='<td width="20" bgcolor="#555555">&nbsp&nbsp;</td>';
     }
-    $output.='<td> ('.$value.') </td></tr></table>';
+    $output.='<td> ('.sprintf("%5.2f",$value).') </td></tr></table>';
     return $output;
 }
 
 sub diffgraph {
     my $value=shift;
-    unless ($value) { return ''; }
+    if (! $value) { 
+        return '';
+    }
     my $val=int(40.0*$value+0.5);
     my @colors=('#FF9933','#EEAA33','#DDBB33','#CCCC33',
                 '#BBDD33','#CCCC33','#DDBB33','#EEAA33');
-    my $output='<table border=0 cellpadding=0 cellspacing=0><tr>';
+    my $output='<table border="0" cellpadding="0" cellspacing="0"><tr>';
     for (my $i=0;$i<8;$i++) {
 	if ($val>$i*5) {
-            $output.='<td width=5 bgcolor="'.$colors[$i].'">&nbsp;</td>';
+            $output.='<td width="5" bgcolor="'.$colors[$i].'">&nbsp;</td>';
         } else {
-	    $output.='<td width=5 bgcolor="#555555">&nbsp;</td>';
+	    $output.='<td width="5" bgcolor="#555555">&nbsp;</td>';
 	}
     }
-    $output.='<td> ('.$value.') </td></tr></table>';
+    $output.='<td> ('.sprintf("%3.2f",$value).') </td></tr></table>';
     return $output;
 }
 
-# Turn MySQL row into hash
-sub metadata_col_to_hash {
-    my @cols=@_;
-    my %hash=();
-    for (my $i=0; $i<=$#columns; $i++) {
-	$hash{$columns[$i]}=$cols[$i];
-    }
-    return %hash;
-}
 
 # The field names
 sub fieldnames {
-    return &Apache::lonlocal::texthash
-        (
-         'title' => 'Title',
+    my $file_type=shift;
+    my %fields = 
+        ('title' => 'Title',
          'author' =>'Author(s)',
          'authorspace' => 'Author Space',
          'modifyinguser' => 'Last Modifying User',
@@ -166,6 +248,12 @@ sub fieldnames {
          'abstract' => 'Abstract',
          'lowestgradelevel' => 'Lowest Grade Level',
          'highestgradelevel' => 'Highest Grade Level',
+         'courserestricted' => 'Course Restricting Metadata');
+         
+    if (! defined($file_type) || $file_type ne 'portfolio') {
+        %fields = 
+        (%fields,
+         'domain' => 'Domain',
          'standards' => 'Standards',
          'mime' => 'MIME Type',
          'language' => 'Language',
@@ -174,6 +262,8 @@ sub fieldnames {
          'owner' => 'Publisher/Owner',
          'copyright' => 'Copyright/Distribution',
          'customdistributionfile' => 'Custom Distribution File',
+         'sourceavail' => 'Source Available',
+         'sourcerights' => 'Source Custom Distribution File',
          'obsolete' => 'Obsolete',
          'obsoletereplacement' => 'Suggested Replacement for Obsolete File',
          'count'      => 'Network-wide number of accesses (hits)',
@@ -192,15 +282,59 @@ sub fieldnames {
          'technical'  => 'Resource is technically correct', 
          'avetries'   => 'Average number of tries till solved',
          'stdno'      => 'Total number of students who have worked on this problem',
-         'difficulty' => 'Degree of difficulty'
+         'difficulty' => 'Degree of difficulty',
+         'disc'       => 'Degree of discrimination',
+	 'dependencies' => 'Resources used by this resource',
          );
+    }
+    return &Apache::lonlocal::texthash(%fields);
 }
 
+sub select_course {
+    my ($r)=@_;
+    my %courses;
+    foreach my $key (keys (%env)) { 
+        if ($key =~ m/\.metadata\./) {
+            $key =~ m/^course\.(.+)(\.metadata.+$)/;
+            my $course = $1;
+            my $coursekey = 'course.'.$course.'.description';
+            my $value = $env{$coursekey};
+            $courses{$coursekey} = $value;
+        }
+    }
+    $r->print('<h3>Course Related Meta-Data</h3><br />');
+    $r->print('<form action="" method="post">');
+    $r->print('Select your test course<br />');
+    $r->print('<select name="metacourse" >');
+    my $meta_not_found = 1;
+    foreach my $key (keys (%courses)) {    
+        if ($meta_not_found) {
+            undef($meta_not_found);
+            $r->print('<h3>Portfolio Meta-Data</h3><br />');
+            $r->print('<form action="" method="post">');
+            $r->print('Select your course<br />');
+            $r->print('<select name="metacourse" >');
+        }
+        $key =~ m/(^.+)\.description$/;
+        $r->print('<option value="'.$1.'">');
+        $r->print($courses{$key});
+        $r->print('</option>');
+    }
+    unless ($meta_not_found) {
+        $r->print('</select><br />');
+        $r->print('<input type="submit" value="Assign Portfolio Metadata" />');
+        $r->print('</form>');
+    }
+    return 'ok';
+}
 # Pretty printing of metadata field
 
 sub prettyprint {
-    my ($type,$value)=@_;
-    unless (defined($value)) { return '&nbsp;'; }
+    my ($type,$value,$target,$prefix,$form,$noformat)=@_;
+# $target,$prefix,$form are optional and for filecrumbs only
+    if (! defined($value)) { 
+        return '&nbsp;'; 
+    }
     # Title
     if ($type eq 'title') {
 	return '<font size="+1" face="arial">'.$value.'</font>';
@@ -220,6 +354,10 @@ sub prettyprint {
     if ($type eq 'copyright') {
 	return &Apache::loncommon::copyrightdescription($value);
     }
+    # Copyright
+    if ($type eq 'sourceavail') {
+	return &Apache::loncommon::source_copyrightdescription($value);
+    }
     # MIME
     if ($type eq 'mime') {
         return '<img src="'.&Apache::loncommon::icon($value).'" />&nbsp;'.
@@ -239,18 +377,37 @@ sub prettyprint {
 	return &Apache::loncommon::gradeleveldescription($value);
     }
     # Only for advance users below
-    unless ($ENV{'user.adv'}) { return '<i>- '.&mt('not displayed').' -</i>' };
+    if (! $env{'user.adv'}) { 
+        return '<i>- '.&mt('not displayed').' -</i>';
+    }
     # File
     if (($type eq 'customdistributionfile') ||
 	($type eq 'obsoletereplacement') ||
 	($type eq 'goto_list') ||
 	($type eq 'comefrom_list') ||
-	($type eq 'sequsage_list')) {
-	return join('<br />',map {
-            my $url=&Apache::lonnet::clutter($_);
-            '<b>'.&Apache::lonnet::gettitle($url).'</b>'.
-                &Apache::lonhtmlcommon::crumbs($url,'preview','',undef,'+0');
-        } split(/\s*\,\s*/,$value));
+	($type eq 'sequsage_list') ||
+	($type eq 'dependencies')) {
+	return '<ul><font size="-1">'.join("\n",map {
+            my $url = &Apache::lonnet::clutter($_);
+            my $title = &Apache::lonnet::gettitle($url);
+            if ($title eq '') {
+                $title = 'Untitled';
+                if ($url =~ /\.sequence$/) {
+                    $title .= ' Sequence';
+                } elsif ($url =~ /\.page$/) {
+                    $title .= ' Page';
+                } elsif ($url =~ /\.problem$/) {
+                    $title .= ' Problem';
+                } elsif ($url =~ /\.html$/) {
+                    $title .= ' HTML document';
+                } elsif ($url =~ m:/syllabus$:) {
+                    $title .= ' Syllabus';
+                } 
+            }
+            $_ = '<li>'.$title.' '.
+		&Apache::lonhtmlcommon::crumbs($url,$target,$prefix,$form,'-1',$noformat).
+                '</li>'
+	    } split(/\s*\,\s*/,$value)).'</ul></font>';
     }
     # Evaluations
     if (($type eq 'clear') ||
@@ -261,17 +418,24 @@ sub prettyprint {
 	return &evalgraph($value);
     }
     # Difficulty
-    if ($type eq 'difficulty') {
+    if ($type eq 'difficulty' || $type eq 'disc') {
 	return &diffgraph($value);
     }
     # List of courses
     if ($type=~/\_list/) {
-	return join('<br />',map {
-	    my %courseinfo=&Apache::lonnet::coursedescription($_);  
-	    '<a href="/public/'.
-		$courseinfo{'domain'}.'/'.$courseinfo{'num'}.'/syllabus" target="preview">'.
-		$courseinfo{'description'}.'</a>';
-	} split(/\s*\,\s*/,$value));
+        my @Courses = split(/\s*\,\s*/,$value);
+        my $Str;
+        foreach my $course (@Courses) {
+            my %courseinfo = &Apache::lonnet::coursedescription($course);
+            if (! exists($courseinfo{'num'}) || $courseinfo{'num'} eq '') {
+                next;
+            }
+            if ($Str ne '') { $Str .= '<br />'; }
+            $Str .= '<a href="/public/'.$courseinfo{'domain'}.'/'.
+                $courseinfo{'num'}.'/syllabus" target="preview">'.
+                $courseinfo{'description'}.'</a>';
+        }
+	return $Str;
     }
     # No pretty print found
     return $value;
@@ -284,7 +448,9 @@ sub direct {
 
 sub selectbox {
     my ($name,$value,$functionref,@idlist)=@_;
-    unless (defined($functionref)) { $functionref=\&direct; }
+    if (! defined($functionref)) {
+        $functionref=\&direct;
+    }
     my $selout='<select name="'.$name.'">';
     foreach (@idlist) {
         $selout.='<option value=\''.$_.'\'';
@@ -298,16 +464,71 @@ sub selectbox {
 
 sub relatedfield {
     my ($show,$relatedsearchflag,$relatedsep,$fieldname,$relatedvalue)=@_;
-    unless ($relatedsearchflag) { return ''; }
-    unless (defined($relatedsep)) { $relatedsep=' '; }
-    unless ($show) { return $relatedsep.'&nbsp;'; }
+    if (! $relatedsearchflag) { 
+        return '';
+    }
+    if (! defined($relatedsep)) {
+        $relatedsep=' ';
+    }
+    if (! $show) {
+        return $relatedsep.'&nbsp;';
+    }
     return $relatedsep.'<input type="checkbox" name="'.$fieldname.'_related"'.
 	($relatedvalue?' checked="1"':'').' />';
 }
 
 sub prettyinput {
     my ($type,$value,$fieldname,$formname,
-	$relatedsearchflag,$relatedsep,$relatedvalue)=@_;
+	$relatedsearchflag,$relatedsep,$relatedvalue,$size,$course_key)=@_;
+    if (! defined($size)) {
+        $size = 80;
+    }
+    if (defined($course_key)) {
+        my $stu_add;
+        my $only_one;
+        my $values = $env{$course_key.'.metadata.'.$_.'.values'};
+        if ($env{$course_key.'.metadata.'.$_.'.options'} =~ m/stuadd/) {
+            $stu_add = 'true';
+        }
+        if ($env{$course_key.'.metadata.'.$_.'.options'} =~ m/onlyone/) {
+            $only_one = 'true';
+        }
+        if ($type eq 'author') {
+        }
+        if ($type eq 'courserestricted') {
+            return ('<input type="hidden" name="new_courserestricted" value="'.$course_key.'" />');
+        }
+        if ($type eq 'authorspace') {
+        }
+        if ($type eq 'modifyinguser') {
+        }
+        if ($type eq 'subject') {
+        }
+        if ($type eq 'keywords') {
+            my %hash;
+            my $def;
+            foreach (split(/,/,$values)) {
+                $_ =~ s/^\s+//;
+                $hash{$_} = $_;
+            }
+            if ($only_one) {
+                return(&Apache::loncommon::select_form($def,'new_keywords',%hash));
+            } else {
+                return(&Apache::loncommon::multiple_select_form('new_keywords',$value,undef,%hash));
+            }
+        }
+        if ($type eq  'notes') {
+        }
+        if ($type eq  'abstract') {
+	}
+
+        if (($type eq 'lowestgradelevel') ||
+	    ($type eq 'highestgradelevel')) {
+	    return &Apache::loncommon::select_level_form($value,$fieldname).
+            &relatedfield(0,$relatedsearchflag,$relatedsep); 
+        }
+        return(); 
+    }
     # Language
     if ($type eq 'language') {
 	return &selectbox($fieldname,
@@ -324,6 +545,14 @@ sub prettyinput {
 			  (&Apache::loncommon::copyrightids)).
                               &relatedfield(0,$relatedsearchflag,$relatedsep);
     }
+    # Source Copyright
+    if ($type eq 'sourceavail') {
+	return &selectbox($fieldname,
+			  $value,
+			  \&Apache::loncommon::source_copyrightdescription,
+			  (&Apache::loncommon::source_copyrightids)).
+                              &relatedfield(0,$relatedsearchflag,$relatedsep);
+    }
     # Gradelevels
     if (($type eq 'lowestgradelevel') ||
 	($type eq 'highestgradelevel')) {
@@ -352,6 +581,14 @@ sub prettyinput {
 	    ",'rights')\">".&mt('Select').'</a>'.
             &relatedfield(0,$relatedsearchflag,$relatedsep); 
     }
+    # Source Customdistribution file
+    if ($type eq 'sourcerights') {
+	return '<input type="text" name="'.$fieldname.
+	    '" size="60" value="'.$value.'" /><a href="javascript:openbrowser'.
+	    "('".$formname."','".$fieldname."'".
+	    ",'rights')\">".&mt('Select').'</a>'.
+            &relatedfield(0,$relatedsearchflag,$relatedsep); 
+    }
     # Dates
     if (($type eq 'creationdate') ||
 	($type eq 'lastrevisiondate')) {
@@ -363,9 +600,9 @@ sub prettyinput {
     $value=~s/^\s+//gs;
     $value=~s/\s+$//gs;
     $value=~s/\s+/ /gs;
-    $value=~s/\"/\&quod\;/gs;
+    $value=~s/\"/\&quot\;/gs;
     return 
-        '<input type="text" name="'.$fieldname.'" size="80" '.
+        '<input type="text" name="'.$fieldname.'" size="'.$size.'" '.
         'value="'.$value.'" />'.
         &relatedfield(1,$relatedsearchflag,$relatedsep,$fieldname,
                       $relatedvalue); 
@@ -375,326 +612,566 @@ sub prettyinput {
 sub handler {
     my $r=shift;
     #
-    my $loaderror=&Apache::lonnet::overloaderror($r);
-    if ($loaderror) { return $loaderror; }
-    #
     my $uri=$r->uri;
     #
-    # Looking for all bombs?
-    if ($uri=~/\/adm\/bombs\/(.*)$/) {
-        # Set document type
-        $uri=&Apache::lonnet::declutter($1);
-        &Apache::loncommon::content_type($r,'text/html');
-        $r->send_http_header;
-        #
-        return OK if $r->header_only;
+    # Set document type
+    &Apache::loncommon::content_type($r,'text/html');
+    $r->send_http_header;
+    return OK if $r->header_only;
+    #
+    my ($resdomain,$resuser)=
+        (&Apache::lonnet::declutter($uri)=~/^(\w+)\/(\w+)\//);
+    my $html=&Apache::lonxml::xmlbegin();
+    $r->print($html.'<head><title>'.
+              'Catalog Information'.
+              '</title></head>');
+    if ($uri=~m:/adm/bombs/(.*)$:) {
         $r->print(&Apache::loncommon::bodytag('Error Messages'));
-        $r->print('<h1>'.&Apache::lonnet::clutter($uri).'</h1>');
-        my ($domain,$author)=($uri=~/^(\w+)\/(\w+)\//);
-        if (&Apache::loncacc::constructaccess('/~'.$author.'/',$domain)) {
-            my %brokenurls=&Apache::lonmsg::all_url_author_res_msg($author,
-                                                                   $domain);
-            foreach (sort keys %brokenurls) {
-                if ($_=~/^\Q$uri\E/) {
-                    $r->print(&Apache::lonhtmlcommon::crumbs
-                              (&Apache::lonnet::clutter($_)).
-                              &Apache::lonmsg::retrieve_author_res_msg($_).
-                              '<hr />');
-                }
+        # Looking for all bombs?
+        &report_bombs($r,$uri);
+    } elsif ($uri=~/\/portfolio\//) {
+	($resdomain,$resuser)=
+	    (&Apache::lonnet::declutter($uri)=~m|^(\w+)/(\w+)/portfolio|);
+        $r->print(&Apache::loncommon::bodytag
+          ('Edit Portfolio File Information','','','',$resdomain));
+        &present_editable_metadata($r,$uri,'portfolio');
+        &select_course($r);
+    } elsif ($uri=~/^\/\~/) { 
+        # Construction space
+        $r->print(&Apache::loncommon::bodytag
+                  ('Edit Catalog Information','','','',$resdomain));
+        &present_editable_metadata($r,$uri);
+    } else {
+        $r->print(&Apache::loncommon::bodytag
+		  ('Catalog Information','','','',$resdomain));
+        &present_uneditable_metadata($r,$uri);
+    }
+    $r->print('</body></html>');
+    return OK;
+}
+
+#####################################################
+#####################################################
+###                                               ###
+###                Report Bombs                   ###
+###                                               ###
+#####################################################
+#####################################################
+sub report_bombs {
+    my ($r,$uri) = @_;
+    # Set document type
+    $uri =~ s:/adm/bombs/::;
+    $uri = &Apache::lonnet::declutter($uri);
+    $r->print('<h1>'.&Apache::lonnet::clutter($uri).'</h1>');
+    my ($domain,$author)=($uri=~/^(\w+)\/(\w+)\//);
+    if (&Apache::loncacc::constructaccess('/~'.$author.'/',$domain)) {
+	if ($env{'form.clearbombs'}) {
+	    &Apache::lonmsg::clear_author_res_msg($uri);
+	}
+        my $clear=&mt('Clear all Messages in Subdirectory');
+	$r->print(<<ENDCLEAR);
+<form method="post">
+<input type="submit" name="clearbombs" value="$clear" />
+</form>
+ENDCLEAR
+        my %brokenurls = 
+            &Apache::lonmsg::all_url_author_res_msg($author,$domain);
+        foreach (sort(keys(%brokenurls))) {
+            if ($_=~/^\Q$uri\E/) {
+                $r->print
+                    ('<a href="'.&Apache::lonnet::clutter($_).'">'.$_.'</a>'.
+                     &Apache::lonmsg::retrieve_author_res_msg($_).
+                     '<hr />');
             }
-        } else {
-            $r->print(&mt('Not authorized'));
         }
-        $r->print('</body></html>');
-    } elsif ($uri!~/^\/\~/) { 
-        # This is not in construction space
-        my ($resdomain,$resuser)=
-            (&Apache::lonnet::declutter($uri)=~/^(\w+)\/(\w+)\//);
-        $loaderror=&Apache::lonnet::overloaderror
-            ($r,
-             &Apache::lonnet::homeserver($resuser,$resdomain));
-        if ($loaderror) { return $loaderror; }
-        #
-        my %content=();
-        # Set document type
-        &Apache::loncommon::content_type($r,'text/html');
-        $r->send_http_header;
-        return OK if $r->header_only;
-        # Read file
-        foreach (split(/\,/,&Apache::lonnet::metadata($uri,'keys'))) {
-            $content{$_}=&Apache::lonnet::metadata($uri,$_);
-        }
-        # Render Output
-        # displayed url
-        my ($thisversion)=($uri=~/\.(\d+)\.(\w+)\.meta$/);
-        $uri=~s/\.meta$//;
-        my $disuri=&Apache::lonnet::clutter($uri);
-        # version
-        my $currentversion=&Apache::lonnet::getversion($disuri);
-        my $versiondisplay='';
-        if ($thisversion) {
-            $versiondisplay=&mt('Version').': '.$thisversion.
-                ' ('.&mt('most recent version').': '.
-                ($currentversion>0 ? 
-                 $currentversion   :
-                 &mt('information not available')).')';
-        } else {
-            $versiondisplay='Version: '.$currentversion;
-        }
-        # crumbify displayed URL
-        $disuri=&Apache::lonhtmlcommon::crumbs($disuri);
-        # obsolete
-        my $obsolete=$content{'obsolete'};
-        my $obsoletewarning='';
-        if (($obsolete) && ($ENV{'user.adv'})) {
-            $obsoletewarning='<p><font color="red">'.
-                &mt('This resource has been marked obsolete by the author(s)').
-                '</font></p>';
-        }
-        #
-        my %lt=&fieldnames();
-        my $table='';
-        my $bodytag=&Apache::loncommon::bodytag
-            ('Catalog Information','','','',$resdomain);
-        foreach ('title', 
-                 'author', 
-                 'subject', 
-                 'keywords', 
-                 'notes', 
-                 'abstract',
-                 'lowestgradelevel',
-                 'highestgradelevel',
-                 'standards', 
-                 'mime', 
-                 'language', 
-                 'creationdate', 
-                 'lastrevisiondate', 
-                 'owner', 
-                 'copyright', 
-                 'customdistributionfile', 
-                 'obsolete', 
-                 'obsoletereplacement') {
-            $table.='<tr><td bgcolor="#AAAAAA">'.$lt{$_}.
-	        '</td><td bgcolor="#CCCCCC">'.
-                &prettyprint($_,$content{$_}).'</td></tr>';
-            delete $content{$_};
-        }
-        #
-        $r->print(<<ENDHEAD);
-<html><head><title>Catalog Information</title></head>
-$bodytag
-<h2>$content{'title'}</h2>
-<h3><tt>$disuri</tt></h3>
+    } else {
+        $r->print(&mt('Not authorized'));
+    }
+    return;
+}
+
+#####################################################
+#####################################################
+###                                               ###
+###        Uneditable Metadata Display            ###
+###                                               ###
+#####################################################
+#####################################################
+sub present_uneditable_metadata {
+    my ($r,$uri) = @_;
+    #
+    my %content=();
+    # Read file
+    foreach (split(/\,/,&Apache::lonnet::metadata($uri,'keys'))) {
+        $content{$_}=&Apache::lonnet::metadata($uri,$_);
+    }
+    # Render Output
+    # displayed url
+    my ($thisversion)=($uri=~/\.(\d+)\.(\w+)\.meta$/);
+    $uri=~s/\.meta$//;
+    my $disuri=&Apache::lonnet::clutter($uri);
+    # version
+    my $currentversion=&Apache::lonnet::getversion($disuri);
+    my $versiondisplay='';
+    if ($thisversion) {
+        $versiondisplay=&mt('Version').': '.$thisversion.
+            ' ('.&mt('most recent version').': '.
+            ($currentversion>0 ? 
+             $currentversion   :
+             &mt('information not available')).')';
+    } else {
+        $versiondisplay='Version: '.$currentversion;
+    }
+    # crumbify displayed URL               uri     target prefix form  size
+    $disuri=&Apache::lonhtmlcommon::crumbs($disuri,undef, undef, undef,'+1');
+    $disuri =~ s:<br />::g;
+    # obsolete
+    my $obsolete=$content{'obsolete'};
+    my $obsoletewarning='';
+    if (($obsolete) && ($env{'user.adv'})) {
+        $obsoletewarning='<p><font color="red">'.
+            &mt('This resource has been marked obsolete by the author(s)').
+            '</font></p>';
+    }
+    #
+    my %lt=&fieldnames();
+    my $table='';
+    my $title = $content{'title'};
+    if (! defined($title)) {
+        $title = 'Untitled Resource';
+    }
+    foreach ('title', 
+             'author', 
+             'subject', 
+             'keywords', 
+             'notes', 
+             'abstract',
+             'lowestgradelevel',
+             'highestgradelevel',
+             'standards', 
+             'mime', 
+             'language', 
+             'creationdate', 
+             'lastrevisiondate', 
+             'owner', 
+             'copyright', 
+             'customdistributionfile',
+             'sourceavail',
+             'sourcerights', 
+             'obsolete', 
+             'obsoletereplacement') {
+        $table.='<tr><td bgcolor="#AAAAAA">'.$lt{$_}.
+            '</td><td bgcolor="#CCCCCC">'.
+            &prettyprint($_,$content{$_}).'</td></tr>';
+        delete $content{$_};
+    }
+    #
+    $r->print(<<ENDHEAD);
+<h2>$title</h2>
+<p>
+$disuri<br />
 $obsoletewarning
-$versiondisplay<br />
-<table cellspacing=2 border=0>
+$versiondisplay
+</p>
+<table cellspacing="2" border="0">
 $table
 </table>
 ENDHEAD
-        if ($ENV{'user.adv'}) {
-            # Dynamic Metadata
-            $r->print(
-                      '<h3>'.&mt('Dynamic Metadata').' ('.
-                      &mt('updated periodically').')</h3>'.&mt('Processing').
-                      ' ...<br />');
-            $r->rflush();
-            my %items=&fieldnames();
-            my %dynmeta=&dynamicmeta($uri);
-            # General Access and Usage Statistics
-            $r->print('<h4>'.&mt('Access and Usage Statistics').'</h4>'.
-                      '<table cellspacing=2 border=0>');
-            foreach ('count',
-                     'sequsage','sequsage_list',
-                     'comefrom','comefrom_list',
-                     'goto','goto_list',
-                     'course','course_list') {
+    if ($env{'user.adv'}) {
+        &print_dynamic_metadata($r,$uri,\%content);
+    }
+    return;
+}
+
+sub print_dynamic_metadata {
+    my ($r,$uri,$content) = @_;
+    #
+    my %content = %$content;
+    my %lt=&fieldnames();
+    #
+    my $description = 'Dynamic Metadata (updated periodically)';
+    $r->print('<h3>'.&mt($description).'</h3>'.
+              &mt('Processing'));
+    $r->rflush();
+    my %items=&fieldnames();
+    my %dynmeta=&dynamicmeta($uri);
+    #
+    # General Access and Usage Statistics
+    if (exists($dynmeta{'count'}) ||
+        exists($dynmeta{'sequsage'}) ||
+        exists($dynmeta{'comefrom'}) ||
+        exists($dynmeta{'goto'}) ||
+        exists($dynmeta{'course'})) {
+        $r->print('<h4>'.&mt('Access and Usage Statistics').'</h4>'.
+                  '<table cellspacing="2" border="0">');
+        foreach ('count',
+                 'sequsage','sequsage_list',
+                 'comefrom','comefrom_list',
+                 'goto','goto_list',
+                 'course','course_list') {
+            $r->print('<tr><td bgcolor="#AAAAAA">'.$lt{$_}.'</td>'.
+                      '<td bgcolor="#CCCCCC">'.
+                      &prettyprint($_,$dynmeta{$_})."</td></tr>\n");
+        }
+        $r->print('</table>');
+    } else {
+        $r->print('<h4>'.&mt('No Access or Usages Statistics are available for this resource.').'</h4>');
+    }
+    #
+    # Assessment statistics
+    if ($uri=~/\.(problem|exam|quiz|assess|survey|form)$/) {
+        if (exists($dynmeta{'stdno'}) ||
+            exists($dynmeta{'avetries'}) ||
+            exists($dynmeta{'difficulty'}) ||
+            exists($dynmeta{'disc'})) {
+            # This is an assessment, print assessment data
+            $r->print('<h4>'.
+                      &mt('Overall Assessment Statistical Data').
+                      '</h4>'.
+                      '<table cellspacing="2" border="0">');
+            $r->print('<tr><td bgcolor="#AAAAAA">'.$lt{'stdno'}.'</td>'.
+                      '<td bgcolor="#CCCCCC">'.
+                      &prettyprint('stdno',$dynmeta{'stdno'}).
+                      '</td>'."</tr>\n");
+            foreach ('avetries','difficulty','disc') {
                 $r->print('<tr><td bgcolor="#AAAAAA">'.$lt{$_}.'</td>'.
                           '<td bgcolor="#CCCCCC">'.
-                          &prettyprint($_,$dynmeta{$_})."</td></tr>\n");
+                          &prettyprint($_,sprintf('%5.2f',$dynmeta{$_})).
+                          '</td>'."</tr>\n");
             }
-            $r->print('</table>');
-            if ($uri=~/\.(problem|exam|quiz|assess|survey|form)\.meta$/) {
-                # This is an assessment, print assessment data
-                $r->print(
-                          '<h4>'.&mt('Assessment Statistical Data').'</h4>'.
-                          '<table cellspacing=2 border=0>');
-                foreach ('stdno','avetries','difficulty') {
-                    $r->print('<tr><td bgcolor="#AAAAAA">'.$lt{$_}.'</td>'.
-                              '<td bgcolor="#CCCCCC">'.
-                              &prettyprint($_,$dynmeta{$_})."</td></tr>\n");
+            $r->print('</table>');    
+        }
+        if (exists($dynmeta{'stats'})) {
+            #
+            # New assessment statistics
+            $r->print('<h4>'.
+                      &mt('Detailed Assessment Statistical Data').
+                      '</h4>');
+            my $table = '<table cellspacing="2" border="0">'.
+                '<tr>'.
+                '<th>Course</th>'.
+                '<th>Section(s)</th>'.
+                '<th>Num Students</th>'.
+                '<th>Mean Tries</th>'.
+                '<th>Degree of Difficulty</th>'.
+                '<th>Degree of Discrimination</th>'.
+                '<th>Time of computation</th>'.
+                '</tr>'.$/;
+            foreach my $identifier (sort(keys(%{$dynmeta{'stats'}}))) {
+                my $data = $dynmeta{'stats'}->{$identifier};
+                my $course = $data->{'course'};
+                my %courseinfo = &Apache::lonnet::coursedescription($course);
+                if (! exists($courseinfo{'num'}) || $courseinfo{'num'} eq '') {
+                    &Apache::lonnet::logthis('lookup for '.$course.' failed');
+                    next;
                 }
-                $r->print('</table>');    
-            }
-            $r->print('<h4>'.&mt('Evaluation Data').'</h4>'.
-                      '<table cellspacing=2 border=0>');
-            foreach ('clear','depth','helpful','correct','technical') {
-                $r->print('<tr><td bgcolor="#AAAAAA">'.$lt{$_}.'</td>'.
-                          '<td bgcolor="#CCCCCC">'.
-                          &prettyprint($_,$dynmeta{$_})."</td></tr>\n");
-            }
-            $r->print('</table>');
-            $uri=~/^\/res\/(\w+)\/(\w+)\//; 
-            if ((($ENV{'user.domain'} eq $1) && ($ENV{'user.name'} eq $2))
-                || ($ENV{'user.role.ca./'.$1.'/'.$2})) {
-                $r->print('<h4>'.&mt('Evaluation Comments').' ('.
-                          &mt('visible to author and co-authors only').
-                          ')</h4>'.
-                          '<blockquote>'.$dynmeta{'comments'}.'</blockquote>');
-                $r->print('<a name="bombs" /><h4>'.&mt('Error Messages').' ('.
-                          &mt('visible to author and co-authors only').')'.
-                          '</h4>'.
-                          &Apache::lonmsg::retrieve_author_res_msg($uri));
-            }
-            # All other stuff
-            $r->print('<h3>'.
-                &mt('Additional Metadata (non-standard, parameters, exports)').
-                      '</h3>');
-            foreach (sort keys %content) {
-                my $name=$_;
-                unless ($name=~/\.display$/) {
-                    my $display=&Apache::lonnet::metadata($uri,
-                                                          $name.'.display');
-                    unless ($display) { $display=$name; };
-                    my $otherinfo='';
-                    foreach ('name','part','type','default') {
-                        if (defined(&Apache::lonnet::metadata($uri,
-                                                              $name.'.'.$_))) {
-                            $otherinfo.=' '.$_.'='.
-                                &Apache::lonnet::metadata($uri,
-                                                          $name.'.'.$_).'; ';
-                        }
-                    }
-                    $r->print('<b>'.$display.':</b> '.$content{$name});
-                    if ($otherinfo) {
-                        $r->print(' ('.$otherinfo.')');
+                $table .= '<tr>';
+                $table .= 
+                    '<td><nobr>'.$courseinfo{'description'}.'</nobr></td>';
+                $table .= 
+                    '<td align="right">'.$data->{'sections'}.'</td>';
+                $table .=
+                    '<td align="right">'.$data->{'stdno'}.'</td>';
+                foreach ('avetries','difficulty','disc') {
+                    $table .= '<td align="right">';
+                    if (exists($data->{$_})) {
+                        $table .= sprintf('%.2f',$data->{$_}).'&nbsp;';
+                    } else {
+                        $table .= '';
                     }
-                    $r->print("<br />\n");
+                    $table .= '</td>';
                 }
+                $table .=
+                    '<td><nobr>'.
+                    &Apache::lonlocal::locallocaltime($data->{'timestamp'}).
+                    '</nobr></td>';
+                $table .=
+                    '</tr>'.$/;
             }
+            $table .= '</table>'.$/;
+            $r->print($table);
+        } else {
+            $r->print('No new dynamic data found.');
         }
-        # End Resource Space Call
     } else {
-        # Construction Space Call
-        # Set document type
-        &Apache::loncommon::content_type($r,'text/html');
-        $r->send_http_header;
-        #
-        return OK if $r->header_only;
-        # Header
-        my $disuri=$uri;
-        my $fn=&Apache::lonnet::filelocation('',$uri);
-        $disuri=~s/^\/\~/\/priv\//;
-        $disuri=~s/\.meta$//;
-        my $target=$uri;
-        $target=~s/^\/\~/\/res\/$ENV{'request.role.domain'}\//;
-        $target=~s/\.meta$//;
-        my $bombs=&Apache::lonmsg::retrieve_author_res_msg($target);
-        if ($bombs) {
-            if ($ENV{'form.delmsg'}) {
-                if (&Apache::lonmsg::del_url_author_res_msg($target) eq 'ok') {
-                    $bombs=&mt('Messages deleted.');
-                } else {
-                    $bombs=&mt('Error deleting messages');
+        $r->print('<h4>'.
+          &mt('No Assessment Statistical Data is available for this resource').
+                  '</h4>');
+    }
+
+    #
+    #
+    if (exists($dynmeta{'clear'})   || 
+        exists($dynmeta{'depth'})   || 
+        exists($dynmeta{'helpful'}) || 
+        exists($dynmeta{'correct'}) || 
+        exists($dynmeta{'technical'})){ 
+        $r->print('<h4>'.&mt('Evaluation Data').'</h4>'.
+                  '<table cellspacing="2" border="0">');
+        foreach ('clear','depth','helpful','correct','technical') {
+            $r->print('<tr><td bgcolor="#AAAAAA">'.$lt{$_}.'</td>'.
+                      '<td bgcolor="#CCCCCC">'.
+                      &prettyprint($_,$dynmeta{$_})."</td></tr>\n");
+        }
+        $r->print('</table>');
+    } else {
+        $r->print('<h4>'.&mt('No Evaluation Data is available for this resource.').'</h4>');
+    }
+    $uri=~/^\/res\/(\w+)\/(\w+)\//; 
+    if ((($env{'user.domain'} eq $1) && ($env{'user.name'} eq $2))
+        || ($env{'user.role.ca./'.$1.'/'.$2})) {
+        if (exists($dynmeta{'comments'})) {
+            $r->print('<h4>'.&mt('Evaluation Comments').' ('.
+                      &mt('visible to author and co-authors only').
+                      ')</h4>'.
+                      '<blockquote>'.$dynmeta{'comments'}.'</blockquote>');
+        } else {
+            $r->print('<h4>'.&mt('There are no Evaluation Comments on this resource.').'</h4>');
+        }
+        my $bombs = &Apache::lonmsg::retrieve_author_res_msg($uri);
+        if (defined($bombs) && $bombs ne '') {
+            $r->print('<a name="bombs" /><h4>'.&mt('Error Messages').' ('.
+                      &mt('visible to author and co-authors only').')'.
+                      '</h4>'.$bombs);
+        } else {
+            $r->print('<h4>'.&mt('There are currently no Error Messages for this resource.').'</h4>');
+        }
+    }
+    #
+    # All other stuff
+    $r->print('<h3>'.
+              &mt('Additional Metadata (non-standard, parameters, exports)').
+              '</h3><table border="0" cellspacing="1">');
+    foreach (sort(keys(%content))) {
+        my $name=$_;
+        if ($name!~/\.display$/) {
+            my $display=&Apache::lonnet::metadata($uri,
+                                                  $name.'.display');
+            if (! $display) { 
+                $display=$name;
+            };
+            my $otherinfo='';
+            foreach ('name','part','type','default') {
+                if (defined(&Apache::lonnet::metadata($uri,
+                                                      $name.'.'.$_))) {
+                    $otherinfo.=' '.$_.'='.
+                        &Apache::lonnet::metadata($uri,
+                                                  $name.'.'.$_).'; ';
                 }
             }
-            my $bodytag=&Apache::loncommon::bodytag('Error Messages');
-            my $del=&mt('Delete Messages');
-            $r->print(<<ENDBOMBS);
-<html><head><title>Edit Catalog Information</title></head>
-$bodytag
+            $r->print('<tr><td bgcolor="#bbccbb"><font size="-1" color="#556655">'.$display.'</font></td><td bgcolor="#ccddcc"><font size="-1" color="#556655">'.$content{$name});
+            if ($otherinfo) {
+                $r->print(' ('.$otherinfo.')');
+            }
+            $r->print("</font></td></tr>\n");
+        }
+    }
+    $r->print("</table>");
+    return;
+}
+
+
+
+#####################################################
+#####################################################
+###                                               ###
+###          Editable metadata display            ###
+###                                               ###
+#####################################################
+#####################################################
+sub present_editable_metadata {
+    my ($r,$uri, $file_type) = @_;
+    # Construction Space Call
+    # Header
+    my $disuri=$uri;
+    my $fn=&Apache::lonnet::filelocation('',$uri);
+    $disuri=~s/^\/\~/\/priv\//;
+    $disuri=~s/\.meta$//;
+    $disuri=~s|^/editupload||;
+    my $target=$uri;
+    $target=~s/^\/\~/\/res\/$env{'request.role.domain'}\//;
+    $target=~s/\.meta$//;
+    my $bombs=&Apache::lonmsg::retrieve_author_res_msg($target);
+    if ($bombs) {
+        my $showdel=1;
+        if ($env{'form.delmsg'}) {
+            if (&Apache::lonmsg::del_url_author_res_msg($target) eq 'ok') {
+                $bombs=&mt('Messages deleted.');
+		$showdel=0;
+            } else {
+                $bombs=&mt('Error deleting messages');
+            }
+        }
+        if ($env{'form.clearmsg'}) {
+	    my $cleardir=$target;
+	    $cleardir=~s/\/[^\/]+$/\//;
+            if (&Apache::lonmsg::clear_author_res_msg($cleardir) eq 'ok') {
+                $bombs=&mt('Messages cleared.');
+		$showdel=0;
+            } else {
+                $bombs=&mt('Error clearing messages');
+            }
+        }
+        my $del=&mt('Delete Messages for this Resource');
+	my $clear=&mt('Clear all Messages in Subdirectory');
+	my $goback=&mt('Back to Source File');
+        $r->print(<<ENDBOMBS);
 <h1>$disuri</h1>
 <form method="post" name="defaultmeta">
-<input type="submit" name="delmsg" value="$del" />
-<br />$bombs
-</form>
-</body>
-</html>
 ENDBOMBS
+        if ($showdel) {
+	    $r->print(<<ENDDEL);
+<input type="submit" name="delmsg" value="$del" />
+<input type="submit" name="clearmsg" value="$clear" />
+ENDDEL
+        } else {
+            $r->print('<a href="'.$disuri.'" />'.$goback.'</a>');
+	}
+	$r->print('<br />'.$bombs);
+    } else {
+        my $displayfile='Catalog Information for '.$disuri;
+        if ($disuri=~/\/default$/) {
+            my $dir=$disuri;
+            $dir=~s/default$//;
+            $displayfile=
+                &mt('Default Cataloging Information for Directory').' '.
+                $dir;
+        }
+        %Apache::lonpublisher::metadatafields=();
+        %Apache::lonpublisher::metadatakeys=();
+        my $result=&Apache::lonnet::getfile($fn);
+        if ($result == -1){
+            $r->print('Creating new '.$disuri);
         } else {
-            my $displayfile='Catalog Information for '.$disuri;
-            if ($disuri=~/\/default$/) {
-                my $dir=$disuri;
-                $dir=~s/default$//;
-                $displayfile=
-                    &mt('Default Cataloging Information for Directory').' '.
-                    $dir;
-            }
-            my $bodytag=
-                &Apache::loncommon::bodytag('Edit Catalog Information');
-            %Apache::lonpublisher::metadatafields=();
-            %Apache::lonpublisher::metadatakeys=();
-            &Apache::lonpublisher::metaeval(&Apache::lonnet::getfile($fn));
-            $r->print(<<ENDEDIT);
-<html><head><title>Edit Catalog Information</title></head>
-$bodytag
+            &Apache::lonpublisher::metaeval($result);
+        }
+        $r->print(<<ENDEDIT);
 <h1>$displayfile</h1>
 <form method="post" name="defaultmeta">
 ENDEDIT
-            $r->print('<script language="JavaScript">'.
-                      &Apache::loncommon::browser_and_searcher_javascript.
-                      '</script>');
-            my %lt=&fieldnames();
-            foreach ('author','title','subject','keywords','abstract','notes',
-                     'copyright','customdistributionfile','language',
-                     'standards',
-                     'lowestgradelevel','highestgradelevel',
-                     'obsolete','obsoletereplacement') {
-                if (defined($ENV{'form.new_'.$_})) {
-                    $Apache::lonpublisher::metadatafields{$_}=
-                        $ENV{'form.new_'.$_};
+        $r->print('<script language="JavaScript">'.
+                  &Apache::loncommon::browser_and_searcher_javascript().
+                  '</script>');
+        my %lt=&fieldnames($file_type);
+	my $output;
+	my @fields;
+	if ($file_type eq 'portfolio') {
+	    @fields =  ('author','title','subject','keywords','abstract','notes','lowestgradelevel',
+	                'highestgradelevel','courserestricted');
+	} else {
+	    @fields = ('author','title','subject','keywords','abstract','notes',
+                 'copyright','customdistributionfile','language',
+                 'standards',
+                 'lowestgradelevel','highestgradelevel','sourceavail','sourcerights',
+                 'obsolete','obsoletereplacement');
+        }
+        my $metacourse;
+        if ($env{'form.metacourse'} ) {
+            $Apache::lonpublisher::metadatafields{'courserestricted'} = $env{'form.metacourse'};
+            $metacourse = $env{'form.metacourse'};
+         } else {
+            if (! $Apache::lonpublisher::metadatafields{'courserestricted'}) {
+                $Apache::lonpublisher::metadatafields{'courserestricted'}=
+                    'none';
+                $metacourse = 'none';
+            } else {
+                $metacourse = $Apache::lonpublisher::metadatafields{'courserestricted'};
+            }
+        }
+        if (! $Apache::lonpublisher::metadatafields{'copyright'}) {
+                $Apache::lonpublisher::metadatafields{'copyright'}=
+                'default';
+        }
+        foreach (@fields) {
+            if (defined($env{'form.new_'.$_})) {
+                $Apache::lonpublisher::metadatafields{$_}=
+                    $env{'form.new_'.$_};
+            }
+            if ($metacourse ne 'none') {
+                # handle restrictions here
+                if (($env{$metacourse.'.metadata.'.$_.'.options'} =~ m/active/)||
+                    ($_ eq 'courserestricted'))  {
+                    $output.=('<p>'.$lt{$_}.': '.
+                            &prettyinput($_,
+				   $Apache::lonpublisher::metadatafields{$_},
+				   'new_'.$_,'defaultmeta',undef,undef,undef,undef,$metacourse).'</p>');
+                }
+            } else {
+                $output.=('<p>'.$lt{$_}.': '.
+                            &prettyinput($_,
+				   $Apache::lonpublisher::metadatafields{$_},
+				   'new_'.$_,'defaultmeta').'</p>');
+            }
+        }
+        if ($env{'form.store'}) {
+            my $mfh;
+            my $formname='store'; 
+            my $file_content;
+            foreach my $meta_field (keys %env) {
+                if (&Apache::loncommon::get_env_multiple('form.new_keywords')) {
+                $Apache::lonpublisher::metadatafields{'keywords'} = 
+                        join (',', &Apache::loncommon::get_env_multiple('form.new_keywords'));
                 }
-                unless ($Apache::lonpublisher::metadatafields{'copyright'}) {
-                    $Apache::lonpublisher::metadatafields{'copyright'}=
-                        'default';
+            }
+            foreach (sort keys %Apache::lonpublisher::metadatafields) {
+                next if ($_ =~ /\./);
+                my $unikey=$_;
+                $unikey=~/^([A-Za-z]+)/;
+                my $tag=$1;
+                $tag=~tr/A-Z/a-z/;
+                $file_content.= "\n\<$tag";
+                foreach (split(/\,/,
+                             $Apache::lonpublisher::metadatakeys{$unikey})
+                         ) {
+                    my $value=
+                     $Apache::lonpublisher::metadatafields{$unikey.'.'.$_};
+                    $value=~s/\"/\'\'/g;
+                    $file_content.=' '.$_.'="'.$value.'"' ;
+                    # print $mfh ' '.$_.'="'.$value.'"';
                 }
-                $r->print('<p>'.$lt{$_}.': '.
-                          &prettyinput
-                          ($_,$Apache::lonpublisher::metadatafields{$_},
-                           'new_'.$_,'defaultmeta').'</p>');
-            }
-            if ($ENV{'form.store'}) {
-                my $mfh;
-                unless ($mfh=Apache::File->new('>'.$fn)) {
-                    $r->print('<p><font color=red>'.
-                              &mt('Could not write metadata').', '.
-                              &mt('FAIL').'</font>');
+                $file_content.= '>'.
+                    &HTML::Entities::encode
+                    ($Apache::lonpublisher::metadatafields{$unikey},
+                     '<>&"').
+                     '</'.$tag.'>';
+            }
+            if ($fn =~ /\/portfolio\//) {
+                $fn =~ /\/portfolio\/(.*)$/;
+                my $new_fn = '/'.$1;
+                $env{'form.'.$formname}=$file_content;
+                $env{'form.'.$formname.'.filename'}=$new_fn;
+                &Apache::lonnet::userfileupload('uploaddoc','',
+	        	 'portfolio'.$env{'form.currentpath'});
+                if (&Apache::lonnet::userfileupload($formname,'','portfolio') eq 'error: no uploaded file') {
+                    $r->print('<p><font color="red">'.
+                      &mt('Could not write metadata').', '.
+                     &mt('FAIL').'</font></p>');
                 } else {
-                    foreach (sort keys %Apache::lonpublisher::metadatafields) {
-                        unless ($_=~/\./) {
-                            my $unikey=$_;
-                            $unikey=~/^([A-Za-z]+)/;
-                            my $tag=$1;
-                            $tag=~tr/A-Z/a-z/;
-                            print $mfh "\n\<$tag";
-                            foreach (split(/\,/,
-                                 $Apache::lonpublisher::metadatakeys{$unikey})
-                                     ) {
-                                my $value=
-                        $Apache::lonpublisher::metadatafields{$unikey.'.'.$_};
-                                $value=~s/\"/\'\'/g;
-                                print $mfh ' '.$_.'="'.$value.'"';
-                            }
-                            print $mfh '>'.
-                                &HTML::Entities::encode($Apache::lonpublisher::metadatafields{$unikey},'<>&"').
-                                '</'.$tag.'>';
-                        }
-                    }
-                    $r->print('<p>'.&mt('Wrote Metadata'));
+                    $r->print('<p><font color="blue">'.&mt('Wrote Metadata').
+		  ' '.&Apache::lonlocal::locallocaltime(time).
+		  '</font></p>');
+                }
+            } else {
+                if (!  ($mfh=Apache::File->new('>'.$fn))) {
+                    $r->print('<p><font color="red">'.
+                        &mt('Could not write metadata').', '.
+                        &mt('FAIL').'</font></p>');
+                } else {
+                    print $mfh $file_content;
+		    $r->print('<p><font color="blue">'.&mt('Wrote Metadata').
+			      ' '.&Apache::lonlocal::locallocaltime(time).
+			      '</font></p>');
                 }
             }
-            $r->print('<br /><input type="submit" name="store" value="'.
-                      &mt('Store Catalog Information').'"></form>'.
-                      '</body></html>');
         }
+	$r->print($output.'<br /><input type="submit" name="store" value="'.
+                  &mt('Store Catalog Information').'">');
     }
-    return OK;
-}
-
-# BEGIN Block
-BEGIN {
-    # Get columns of MySQL metadata table
-    @columns=&Apache::lonmysql::col_order('metadata');
+    $r->print('</form>');
+    return;
 }
 
 1;
 __END__
+
+