--- loncom/interface/lonparmset.pm 2005/09/29 21:10:32 1.254
+++ loncom/interface/lonparmset.pm 2005/11/11 20:37:47 1.264
@@ -1,7 +1,7 @@
# The LearningOnline Network with CAPA
# Handler to set parameters for assessments
#
-# $Id: lonparmset.pm,v 1.254 2005/09/29 21:10:32 banghart Exp $
+# $Id: lonparmset.pm,v 1.264 2005/11/11 20:37:47 albertel Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -2702,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',
@@ -2749,36 +2749,32 @@ ENDMAINFORMHEAD
$r->print($menu_html);
return;
}
-### Set instructor defined metadata
+### Set portfolio metadata
sub output_row {
- my ($r, $field_name) = @_;
+ 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 $checked;
unless (defined($options)) {
$options = 'active,stuadd';
- $values = 'Type a list of values separated by commas.';
+ $values = '';
}
- $output.='
'.$field_name.' | ';
- $output.=' | ';
- if ($options =~ m/onlyone/) {
- $checked = ' CHECKED ';
- } else {
- $checked = ' ';
- }
- $output.=' | ';
- if ($options =~ m/stuadd/) {
- $checked = ' CHECKED ';
- } else {
- $checked = ' ';
+ $output.=''.$field_text.':';
+ $output.='
';
+
+ my @options= ( ['active', 'Show to student'],
+ ['onlyone','Student may select only one choice'],
+ ['stuadd', 'Student may type choices']);
+ foreach my $opt (@options) {
+ my $checked = ($options =~ m/$opt->[0]/) ? ' checked="checked" ' : '' ;
+ $output.=(' 'x5).'
';
}
- $output.=' | ';
-
- $output.='
';
return ($output);
}
-sub setinstmeta {
+
+sub setrestrictmeta {
my ($r)=@_;
my $next_meta;
my $output;
@@ -2786,16 +2782,16 @@ sub setinstmeta {
my $put_result;
$r->print(&Apache::lonxml::xmlbegin());
$r->print('
- LON-CAPA Set Instructor Metadata
+ LON-CAPA Restrict Metadata
');
- $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.'};
+ 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;
@@ -2812,7 +2808,7 @@ 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'},
@@ -2820,26 +2816,19 @@ sub setinstmeta {
}
}
}
-
}
&Apache::lonnet::coursedescription($env{'request.course.id'});
- $output.= &output_row($r, "Title");
- $output.= &output_row($r, "Subject");
- $output.= &output_row($r, "Keywords");
+ 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
-
-
$r->print('