--- loncom/interface/lonmeta.pm	2004/12/03 21:37:29	1.87
+++ loncom/interface/lonmeta.pm	2005/02/01 17:37:23	1.90
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Metadata display handler
 #
-# $Id: lonmeta.pm,v 1.87 2004/12/03 21:37:29 albertel Exp $
+# $Id: lonmeta.pm,v 1.90 2005/02/01 17:37:23 banghart Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -236,6 +236,17 @@ sub diffgraph {
 
 # The field names
 sub fieldnames {
+    my $file_type=shift;
+    if ($file_type eq 'portfolio') {
+    return &Apache::lonlocal::texthash
+        (
+         'title' => 'Title',
+         'author' =>'Author(s)',
+         'authorspace' => 'Author Space',
+         'modifyinguser' => 'Last Modifying User',
+         'subject' => 'Subject',
+         'keywords' => 'Keyword(s)');
+    } else {
     return &Apache::lonlocal::texthash
         (
          'title' => 'Title',
@@ -280,6 +291,7 @@ sub fieldnames {
          'disc'       => 'Degree of discrimination',
 	 'dependencies' => 'Resources used by this resource',
          );
+        }
 }
 
 # Pretty printing of metadata field
@@ -537,6 +549,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,'portfolio');
+        
     } elsif ($uri=~/^\/\~/) { 
         # Construction space
         $r->print(&Apache::loncommon::bodytag
@@ -665,7 +682,7 @@ $disuri<br />
 $obsoletewarning
 $versiondisplay
 </p>
-<table cellspacing=2 border=0>
+<table cellspacing="2" border="0">
 $table
 </table>
 ENDHEAD
@@ -695,7 +712,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 +737,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 +756,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 +816,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">'.
@@ -870,7 +887,7 @@ sub print_dynamic_metadata {
 #####################################################
 #####################################################
 sub present_editable_metadata {
-    my ($r,$uri) = @_;
+    my ($r,$uri, $file_type) = @_;
     # Construction Space Call
     # Header
     my $disuri=$uri;
@@ -915,13 +932,19 @@ ENDEDIT
         $r->print('<script language="JavaScript">'.
                   &Apache::loncommon::browser_and_searcher_javascript().
                   '</script>');
-        my %lt=&fieldnames();
+        my %lt=&fieldnames($file_type);
 	my $output;
-        foreach ('author','title','subject','keywords','abstract','notes',
+	my @fields;
+	if ($file_type eq 'portfolio') {
+	    @fields =  ('author','title','subject','keywords');
+	} else {
+	    @fields = ('author','title','subject','keywords','abstract','notes',
                  'copyright','customdistributionfile','language',
                  'standards',
                  'lowestgradelevel','highestgradelevel','sourceavail','sourcerights',
-                 'obsolete','obsoletereplacement') {
+                 'obsolete','obsoletereplacement');
+        }
+        foreach (@fields) {
             if (defined($ENV{'form.new_'.$_})) {
                 $Apache::lonpublisher::metadatafields{$_}=
                     $ENV{'form.new_'.$_};