--- loncom/interface/lonmeta.pm	2004/08/24 21:40:00	1.86
+++ loncom/interface/lonmeta.pm	2005/01/19 01:28:33	1.89
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Metadata display handler
 #
-# $Id: lonmeta.pm,v 1.86 2004/08/24 21:40:00 albertel Exp $
+# $Id: lonmeta.pm,v 1.89 2005/01/19 01:28:33 banghart Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -537,6 +537,11 @@ sub handler {
         $r->print(&Apache::loncommon::bodytag('Error Messages'));
         # Looking for all bombs?
         &report_bombs($r,$uri);
+    } elsif ($uri=~/\/portfolio\//) {
+        $r->print(&Apache::loncommon::bodytag
+          ('Edit Portfolio File Information','','','',$resdomain));
+        &present_editable_metadata($r,$uri);
+        
     } elsif ($uri=~/^\/\~/) { 
         # Construction space
         $r->print(&Apache::loncommon::bodytag
@@ -665,7 +670,7 @@ $disuri<br />
 $obsoletewarning
 $versiondisplay
 </p>
-<table cellspacing=2 border=0>
+<table cellspacing="2" border="0">
 $table
 </table>
 ENDHEAD
@@ -695,7 +700,7 @@ sub print_dynamic_metadata {
         exists($dynmeta{'goto'}) ||
         exists($dynmeta{'course'})) {
         $r->print('<h4>'.&mt('Access and Usage Statistics').'</h4>'.
-                  '<table cellspacing=2 border=0>');
+                  '<table cellspacing="2" border="0">');
         foreach ('count',
                  'sequsage','sequsage_list',
                  'comefrom','comefrom_list',
@@ -720,7 +725,7 @@ sub print_dynamic_metadata {
             $r->print('<h4>'.
                       &mt('Overall Assessment Statistical Data').
                       '</h4>'.
-                      '<table cellspacing=2 border=0>');
+                      '<table cellspacing="2" border="0">');
             $r->print('<tr><td bgcolor="#AAAAAA">'.$lt{'stdno'}.'</td>'.
                       '<td bgcolor="#CCCCCC">'.
                       &prettyprint('stdno',$dynmeta{'stdno'}).
@@ -739,7 +744,7 @@ sub print_dynamic_metadata {
             $r->print('<h4>'.
                       &mt('Detailed Assessment Statistical Data').
                       '</h4>');
-            my $table = '<table cellspacing=2 border=0>'.
+            my $table = '<table cellspacing="2" border="0">'.
                 '<tr>'.
                 '<th>Course</th>'.
                 '<th>Section(s)</th>'.
@@ -799,7 +804,7 @@ sub print_dynamic_metadata {
         exists($dynmeta{'correct'}) || 
         exists($dynmeta{'technical'})){ 
         $r->print('<h4>'.&mt('Evaluation Data').'</h4>'.
-                  '<table cellspacing=2 border=0>');
+                  '<table cellspacing="2" border="0">');
         foreach ('clear','depth','helpful','correct','technical') {
             $r->print('<tr><td bgcolor="#AAAAAA">'.$lt{$_}.'</td>'.
                       '<td bgcolor="#CCCCCC">'.
@@ -916,6 +921,7 @@ ENDEDIT
                   &Apache::loncommon::browser_and_searcher_javascript().
                   '</script>');
         my %lt=&fieldnames();
+	my $output;
         foreach ('author','title','subject','keywords','abstract','notes',
                  'copyright','customdistributionfile','language',
                  'standards',
@@ -929,17 +935,17 @@ ENDEDIT
                 $Apache::lonpublisher::metadatafields{'copyright'}=
                     'default';
             }
-            $r->print('<p>'.$lt{$_}.': '.
-                      &prettyinput
-                      ($_,$Apache::lonpublisher::metadatafields{$_},
-                       'new_'.$_,'defaultmeta').'</p>');
+            $output.=('<p>'.$lt{$_}.': '.
+                      &prettyinput($_,
+				   $Apache::lonpublisher::metadatafields{$_},
+				   'new_'.$_,'defaultmeta').'</p>');
         }
         if ($ENV{'form.store'}) {
             my $mfh;
             if (!  ($mfh=Apache::File->new('>'.$fn))) {
-                $r->print('<p><font color=red>'.
+                $r->print('<p><font color="red">'.
                           &mt('Could not write metadata').', '.
-                          &mt('FAIL').'</font>');
+                          &mt('FAIL').'</font></p>');
             } else {
                 foreach (sort keys %Apache::lonpublisher::metadatafields) {
                     next if ($_ =~ /\./);
@@ -962,10 +968,12 @@ ENDEDIT
                          '<>&"').
                          '</'.$tag.'>';
                 }
-                $r->print('<p>'.&mt('Wrote Metadata'));
+                $r->print('<p><font color="blue">'.&mt('Wrote Metadata').
+			  ' '.&Apache::lonlocal::locallocaltime(time).
+			  '</font></p>');
             }
         }
-        $r->print('<br /><input type="submit" name="store" value="'.
+	$r->print($output.'<br /><input type="submit" name="store" value="'.
                   &mt('Store Catalog Information').'">');
     }
     $r->print('</form>');