--- loncom/interface/lonparmset.pm 2005/10/06 22:05:56 1.258 +++ loncom/interface/lonparmset.pm 2005/10/14 04:25:56 1.259 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # Handler to set parameters for assessments # -# $Id: lonparmset.pm,v 1.258 2005/10/06 22:05:56 albertel Exp $ +# $Id: lonparmset.pm,v 1.259 2005/10/14 04:25:56 banghart Exp $ # # Copyright Michigan State University Board of Trustees # @@ -2703,7 +2703,7 @@ ENDMAINFORMHEAD permission => $parm_permission, }, { text => 'Set Portfolio Metadata', - action => 'setinstmeta', + action => 'setrestrictmeta', permission => $parm_permission, }, { text => 'Set/Modify Resource Parameters - Helper Mode', @@ -2753,8 +2753,8 @@ ENDMAINFORMHEAD sub output_row { my ($r, $field_name, $field_text) = @_; my $output; - my $options=$env{'course.'.$env{'request.course.id'}.'.metadata.'.$field_name.'.options'}; - my $values=$env{'course.'.$env{'request.course.id'}.'.metadata.'.$field_name.'.values'}; + 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'; @@ -2784,7 +2784,8 @@ sub output_row { $output.=''; return ($output); } -sub setinstmeta { + +sub setrestrictmeta { my ($r)=@_; my $next_meta; my $output; @@ -2799,9 +2800,9 @@ sub setinstmeta { 'Set Portfolio 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.'}; + my $key_base = $env{'course.'.$env{'request.course.id'}.'.'}; my $save_field = ''; - if ($env{'form.instmeta'}) { + if ($env{'form.restrictmeta'}) { foreach my $field (sort(keys(%env))) { if ($field=~m/^form.(.+)_(.+)$/) { my $options; @@ -2818,10 +2819,10 @@ sub setinstmeta { if ($env{'form.'.$meta_field.'_active'}) { $options.='active,'; } - my $name = 'metadata.'.$save_field; + my $name = $save_field; $put_result = &Apache::lonnet::put('environment', - {'metadata.'.$meta_field.'.options'=>$options, - 'metadata.'.$meta_field.'.values'=>$env{'form.'.$meta_field.'_values'}, + {$meta_field.'.options'=>$options, + $meta_field.'.values'=>$env{'form.'.$meta_field.'_values'}, },$dom,$crs); } } @@ -2835,14 +2836,14 @@ sub setinstmeta { $output.= &output_row($r, $field, $metadata_fields{$field}); } $r->print(< +

$output
ActiveMetadata ItemValuesOnly One ChoiceStudent Can Add Choices
- +

ENDenv @@ -3024,6 +3025,7 @@ Main handler. Calls &assessparms and &c ################################################## # use Data::Dumper; + sub handler { my $r=shift; @@ -3084,10 +3086,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=>"Portfolio 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"});