--- loncom/interface/lonparmset.pm	2005/08/30 00:40:12	1.245
+++ loncom/interface/lonparmset.pm	2005/10/14 17:05:44	1.260
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to set parameters for assessments
 #
-# $Id: lonparmset.pm,v 1.245 2005/08/30 00:40:12 banghart Exp $
+# $Id: lonparmset.pm,v 1.260 2005/10/14 17:05:44 banghart Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -1921,6 +1921,9 @@ sub crsenv {
 	     'disablesigfigs'
 	         => '<b>'.&mt('Disable checking of Significant Figures').'</b><br />'.
                     ' ('.&mt('"[_1]" to disable, anything else if not','<tt>yes</tt>').')',
+	     'disableexampointprint'
+	         => '<b>'.&mt('Disable automatically printing point values onto exams.').'</b><br />'.
+                    ' ('.&mt('"[_1]" to disable, anything else if not','<tt>yes</tt>').')',
 	     'tthoptions'
 	         => '<b>'.&mt('Default set of options to pass to tth/m when converting tex').'</b>'
              ); 
@@ -1947,7 +1950,8 @@ sub crsenv {
                              'default_enrollment_start_date',
                              'default_enrollment_end_date',
 			     'tthoptions',
-			     'disablesigfigs'
+			     'disablesigfigs',
+			     'disableexampointprint'
                              );
 	foreach my $parameter (sort(keys(%values))) {
             unless (($parameter =~ m/^internal\./)||($parameter =~ m/^metadata\./)) {
@@ -2189,20 +2193,41 @@ sub listdata {
     my $pointer=0;
     $tableopen=0;
     my $foundkeys=0;
+    my %keyorder=&standardkeyorder();
     foreach my $thiskey (sort {
 	if ($sortorder eq 'realmstudent') {
-	    my ($astudent,$arealm)=($a=~/^$env{'request.course.id'}\.([^\.]+)\.(.+)\.[^\.]+$/);
-	    my ($bstudent,$brealm)=($b=~/^$env{'request.course.id'}\.([^\.]+)\.(.+)\.[^\.]+$/);
-	    if (!defined($astudent) && !defined($bstudent) &&
-		!defined($arealm) && !defined($brealm)) {
-		($arealm)=($a=~/^$env{'request.course.id'}\.(.+)\.[^\.]+$/);
-		($brealm)=($b=~/^$env{'request.course.id'}\.(.+)\.[^\.]+$/);
+	    my ($astudent,$arealm)=($a=~/^\Q$env{'request.course.id'}\E\.\[([^\.]+)\]\.(.+)\.[^\.]+$/);
+	    my ($bstudent,$brealm)=($b=~/^\Q$env{'request.course.id'}\E\.\[([^\.]+)\]\.(.+)\.[^\.]+$/);
+	    if (!defined($astudent)) {
+		($arealm)=($a=~/^\Q$env{'request.course.id'}\E\.(.+)$/);
+	    }
+	    if (!defined($bstudent)) {
+		($brealm)=($b=~/^\Q$env{'request.course.id'}\E\.(.+)$/);
+	    }
+	    $arealm=~s/\.type//;
+	    my ($ares, $aparm) = ($arealm=~/^(.*)\.(.*)$/);
+	    $aparm=$keyorder{'parameter_0_'.$aparm};
+	    $brealm=~s/\.type//;
+	    my ($bres, $bparm) = ($brealm=~/^(.*)\.(.*)$/);
+	    $bparm=$keyorder{'parameter_0_'.$bparm};	   
+	    if ($ares eq $bres) {
+		if (defined($aparm) && defined($bparm)) {
+		    ($aparm <=> $bparm);
+		} elsif (defined($aparm)) {
+		    -1;
+		} elsif (defined($bparm)) {
+		    1;
+		} else {
+		    ($arealm cmp $brealm) || ($astudent cmp $bstudent);
+		}
+	    } else {
+		($arealm cmp $brealm) || ($astudent cmp $bstudent);
 	    }
-            ($arealm cmp $brealm) || ($astudent cmp $bstudent);
 	} else {
 	    $a cmp $b;
 	}
     } keys %{$listdata}) {
+	 
 	if ($$listdata{$thiskey.'.type'}) {
             my $thistype=$$listdata{$thiskey.'.type'};
             if ($$resourcedata{$thiskey.'.type'}) {
@@ -2677,8 +2702,8 @@ ENDMAINFORMHEAD
 	    action => 'crsenv',
             permission => $parm_permission,
             },
-          { text => 'Set Instructor Defined Metadata',
-	    action => 'setinstmeta',
+          { text => 'Set Portfolio Metadata',
+	    action => 'setrestrictmeta',
             permission => $parm_permission,
             },
           { text => 'Set/Modify Resource Parameters - Helper Mode',
@@ -2724,95 +2749,98 @@ ENDMAINFORMHEAD
     $r->print($menu_html);
     return;
 }
-### Set instructor defined metadata
-sub setinstmeta {
+### Set portfolio metadata
+sub output_row {
+    my ($r, $field_name, $field_text) = @_;
+    my $output;
+    my $options=$env{'course.'.$env{'request.course.id'}.'.'.$field_name.'.options'};
+    my $values=$env{'course.'.$env{'request.course.id'}.'.'.$field_name.'.values'};
+    my $checked;
+    unless (defined($options)) {
+        $options = 'active,stuadd';
+        $values = 'Type a list of values separated by commas.';
+    }
+    $output.='<strong>'.$field_text.':</strong>';
+    $output.='<input name="'.$field_name.'_values" type="text" value="'.$values.'" /><br />';
+    if ($options =~ m/active/) {
+        $checked = ' checked="checked" ';
+    } else {
+        $checked = '  ';
+    }
+    $output.='&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="'.$field_name.'_active" value="yes"'.$checked.' />';
+    $output.='Active <br />';
+    if ($options =~ m/onlyone/) {
+        $checked = ' checked="checked" ';
+    } else {
+        $checked = '  ';
+    }
+    $output.='&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="'.$field_name.'_onlyone" value="yes"'.$checked.' />';
+    $output.='Student may select only one choice <br />';
+    if ($options =~ m/stuadd/) {
+        $checked = ' checked="checked" ';
+    } else {
+        $checked = ' ';
+    }
+    $output.='&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="'.$field_name.'_stuadd" value="yes"'.$checked.' />';
+    $output.='Student may type choices <br />';
+    return ($output);
+}
+
+sub setrestrictmeta {
     my ($r)=@_;
     my $next_meta;
     my $output;
     my $item_num;
-    if ($env{'form.serial'}) {
-        $next_meta = $env{'form.serial'};
-    } else {
-        $next_meta = 1;
-    }
+    my $put_result;
     $r->print(&Apache::lonxml::xmlbegin());
     $r->print('<head>
-            <title>LON-CAPA Set Instructor Metadata</title>
+            <title>LON-CAPA Restrict Metadata</title>
             </head>');
-    $r->print(&Apache::loncommon::bodytag('Set Instructor Metadata'));
+    $r->print(&Apache::loncommon::bodytag('Restrict Metadata'));
     $r->print(&Apache::lonhtmlcommon::breadcrumbs(undef,
-						    'Set Instructor Metadata'));
+						    'Restrict Metadata'));
     my $dom = $env{'course.'.$env{'request.course.id'}.'.domain'};
     my $crs = $env{'course.'.$env{'request.course.id'}.'.num'};
-    my $key_base = $env{'course.'.$env{'request.course.id'}.'.metadata.'};
-    if ($env{'form.state'}eq 'add') {
-        $r->print('<strong>Compose a new metadata item:</strong><br />');
-        $r->print('<form method="post" action="/adm/parmset">');
-        $r->print ('Title of item: '.&Apache::lonhtmlcommon::textbox("title",undef,undef,undef).'<br />');
-        $r->print ('List if items (comma separated): '.&Apache::lonhtmlcommon::textbox("values",undef,undef,undef).'<br />');
-        $r->print ('Select one: <input type="radio" name="type" value="one"><br />');
-        $r->print ('Select many: <input type="radio" name="type" value="many"><br />');
-        $r->print('<input type="hidden" name="serial" value="'.$next_meta.'" />');
-        $r->print('<input type="hidden" name="action" value="setinstmeta" />');
-        $r->print('<input type="hidden" name="state" value="save" />');
-        $r->print('<input type="submit" value="Save Metadata">');
-        $r->print('</form>');
-    } elsif ($env{'form.state'}eq 'save') {
-        $r->print('Here we save the metadata');
-        $key_base.=$env{'form.serial'}.'.';
-        $r->print('The title: '.$env{'form.title'}.'<br>');
-        $r->print('The values: '.$env{'form.values'}.'<br>');
-        $r->print('The type: '.$env{'form.type'}.'<br>');
-        $r->print('The number: '.$env{'form.serial'}.'<br>');
-        my $name = 'metadata.'.$env{'form.serial'}.'.title';
-        my $value = $env{'form.title'};
-        my $put_result = &Apache::lonnet::put('environment',
-                                                  {$name=>$value},$dom,$crs);
-        $r->print('the put result was: '.$put_result.'<br />');
-        $name = 'metadata.'.$env{'form.serial'}.'.type';
-        $value = $env{'form.type'};
-        $put_result = &Apache::lonnet::put('environment',
-                                                  {$name=>$value},$dom,$crs);
-        $name = 'metadata.'.$env{'form.serial'}.'.values';
-        $value = $env{'form.values'};
-        $put_result = &Apache::lonnet::put('environment',
-                                                  {$name=>$value},$dom,$crs);
-        &Apache::lonnet::coursedescription($env{'request.course.id'});        
-    } else {
-        
-        foreach my $key (sort keys %env) {
-            if ($key=~m/^course\.$env{'request.course.id'}\.metadata\.\d+\.title$/) {
-                $key=~m/^course\.$env{'request.course.id'}\.metadata\.(\d+)\.title/;
-                $item_num = $1;
-                if ($1 ge $next_meta) {
-                    $next_meta = $1 + 1;
+    my $key_base = $env{'course.'.$env{'request.course.id'}.'.'};
+    my $save_field = '';
+    if ($env{'form.restrictmeta'}) {
+        foreach my $field (sort(keys(%env))) {
+            if ($field=~m/^form.(.+)_(.+)$/) {
+                my $options;
+                my $meta_field = $1;
+                my $meta_key = $2;
+                if ($save_field ne $meta_field) {
+                    $save_field = $meta_field;
+            	    if ($env{'form.'.$meta_field.'_stuadd'}) {
+            	        $options.='stuadd,';
+            	    } 
+            	    if ($env{'form.'.$meta_field.'_onlyone'}) {
+            	        $options.='onlyone,';
+            	    } 
+            	    if ($env{'form.'.$meta_field.'_active'}) {
+            	        $options.='active,';
+            	    }
+                    my $name = $save_field;
+                     $put_result = &Apache::lonnet::put('environment',
+                                                  {$meta_field.'.options'=>$options,
+                                                   $meta_field.'.values'=>$env{'form.'.$meta_field.'_values'},
+                                                   },$dom,$crs);
                 }
-                $output.='<tr><td>'.$env{$key}.'</td>';
-                $output.='<td>Select one: <input type="radio" name="type'.$item_num.'" value="one" /><br />';
-                $output.='Select many: <input type="radio" name="type'.$item_num.'" value="many" /></td>';
-                $output.='<td><input name="values'.$item_num.'" type="text" value="'.$env{$key.'.values'}.'" /></td></tr>';
             }
         }
-    $output.='<tr><td><i>Create new instructor metadata item</i><br />';
-    $output.='<input type="text" name="newitem" /></td>';
-    $output.='<td>Select one: <input type="radio" name="typenew" value="one" /><br />';
-    $output.='Select many: <input type="radio" name="typenew" value="many" /></td>';
-    $output.='<td><input name="valuesnew" type="text" value="" /></td></tr>';
-    $output.='<td><input type="hidden" name="serial" value="'.$next_meta.'" /></td></tr>';
+    }
+    &Apache::lonnet::coursedescription($env{'request.course.id'});
+    my %metadata_fields = &Apache::lonmeta::fieldnames('portfolio');
+    foreach my $field (sort(keys(%metadata_fields))) {
+	$output.= &output_row($r, $field, $metadata_fields{$field});
+    }
     $r->print(<<ENDenv);       
-        <form method="post" action="/adm/parmset?action=setinstmeta" name="form">
-        
+        <form method="post" action="/adm/parmset?action=setrestrictmeta" name="form">
         <p>
-        <table border=2>
-        <tr><th>Title</th><th>Options</th><th>Choices</th></tr>
         $output
-        </table>
-        <input type="submit" name="instmeta" value="Update Instructor Metadata">
+        <input type="submit" name="restrictmeta" value="Update Metadata Restrictions">
         </form>
-
 ENDenv
-        
-    }
     $r->print('</body>
                 </html>');
     return 'ok';
@@ -2989,6 +3017,7 @@ Main handler.  Calls &assessparms and &c
 ##################################################
 #    use Data::Dumper;
 
+
 sub handler {
     my $r=shift;
 
@@ -3049,10 +3078,10 @@ sub handler {
             &Apache::lonhtmlcommon::add_breadcrumb({href=>'/adm/parmset?action=setoverview',
 						    text=>"Overview Mode"});
 	    &overview($r);
-        } elsif ($env{'form.action'} eq 'setinstmeta' && $parm_permission) {
-            &Apache::lonhtmlcommon::add_breadcrumb({href=>'/adm/parmset?action=setinstmeta',
-						    text=>"Instructor Metadata"});
-	    &setinstmeta($r);
+        } elsif ($env{'form.action'} eq 'setrestrictmeta' && $parm_permission) {
+            &Apache::lonhtmlcommon::add_breadcrumb({href=>'/adm/parmset?action=setrestrictmeta',
+						    text=>"Restrict Metadata"});
+	    &setrestrictmeta($r);
         } elsif ($env{'form.action'} eq 'newoverview' && $parm_permission) {
             &Apache::lonhtmlcommon::add_breadcrumb({href=>'/adm/parmset?action=setoverview',
 						    text=>"Overview Mode"});