--- loncom/interface/lonquickgrades.pm 2011/03/19 02:22:18 1.76
+++ loncom/interface/lonquickgrades.pm 2011/03/26 20:04:15 1.81
@@ -1,7 +1,7 @@
# The LearningOnline Network with CAPA
# Quick Student Grades Display
#
-# $Id: lonquickgrades.pm,v 1.76 2011/03/19 02:22:18 www Exp $
+# $Id: lonquickgrades.pm,v 1.81 2011/03/26 20:04:15 www Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -475,10 +475,13 @@ sub process_category_edits {
unless ($cangrade) { return %categories; }
# First store everything
foreach my $id (split(/\,/,$categories{'order'})) {
+# Set names, types, and weight (there is only one of each per category)
%categories=&set_category_name($cangrade,$id,$env{'form.name_'.$id},%categories);
%categories=&set_category_total($cangrade,$id,$env{'form.totaltype_'.$id},$env{'form.total_'.$id},%categories);
%categories=&set_category_weight($cangrade,$id,$env{'form.weight_'.$id},%categories);
-# More changes here
+ %categories=&set_category_displayachieved($cangrade,$id,$env{'form.displayachieved_'.$id},%categories);
+# Set values for category rules (before names may change)
+ %categories=&set_category_rules($cangrade,$id,%categories);
}
# Now deal with commands
@@ -495,6 +498,10 @@ sub process_category_edits {
%categories=&add_category_content($1,$cangrade,$env{'form.addcont_'.$1.'_symb'},%categories);
} elsif ($cmd=~/^delcont\_(.+)\_\_\_\_\_\_(.+)$/) {
%categories=&del_category_content($1,$cangrade,$2,%categories);
+ } elsif ($cmd=~/^newrule\_(.+)$/) {
+ %categories=&add_calculation_rule($1,$cangrade,':',%categories);
+ } elsif ($cmd=~/^delrule\_(.+)\_\_\_\_\_\_(.*)$/) {
+ %categories=&del_calculation_rule($1,$cangrade,$2,%categories);
}
# Move to a new position
my $moveid=$env{'form.storemove'};
@@ -539,9 +546,10 @@ sub output_category_table_header {
}
$r->print('
'.&mt('Category').' | '.
''.&mt('Contents').' | '.
- ''.&mt('Calculation').' | '.
''.&mt('Total Points').' | '.
- ''.&mt('Relative Weight').' | ');
+ ''.&mt('Calculation').' | '.
+ ''.&mt('Relative Weight').' | '.
+ ''.&mt('Achieved').' | ');
$r->print(&Apache::loncommon::end_data_table_header_row());
}
@@ -606,23 +614,6 @@ ENDMOVE
&Apache::lonstathelpers::problem_selector('.',undef,1,1,'addcont_'.$id.'_',1,'this.form.cmd.value="addcont_'.$id.'";this.form.submit();'));
}
$r->print('');
-# Calculation
- $r->print('');
- foreach my $calcrule (split(/\,/,$categories{$id.'_calculations'})) {
- $r->print('- ');
- $r->print(&pretty_prt_rule($calcrule));
- if ($cangrade) {
- $r->print(' '.&mt('Delete').'');
- }
- $r->print('
');
- }
- $r->print(' ');
- if ($cangrade) {
- $r->print(' '.&mt('Add Calculation Rule').' '.
- &new_calc_rule_form('addcalc_'.$id.'_','this.form.cmd.value="addcalc_'.$id.'";this.form.submit();'));
- }
- $r->print(' | ');
-
# Total
if ($cangrade) {
@@ -636,6 +627,25 @@ ENDMOVE
} else {
$r->print(''.($categories{$id.'_totaltype'} eq 'default'?&mt('default'):$categories{$id.'_total'}).' | ');
}
+
+
+# Calculation
+ $r->print('');
+ foreach my $calcrule (split(/\,/,$categories{$id.'_calculations'})) {
+ $r->print('- ');
+ my ($code,$value)=split(/\:/,$calcrule);
+ $r->print(&pretty_prt_rule($cangrade,$id,$code,$value));
+ if ($cangrade) {
+ $r->print(' '.&mt('Delete').'');
+ }
+ $r->print('
');
+ }
+ $r->print(' ');
+ if ($cangrade) {
+ $r->print(' '.&new_calc_rule_form($id));
+ }
+ $r->print(' | ');
+
# Weight
if ($cangrade) {
$r->print(''.
@@ -644,6 +654,21 @@ ENDMOVE
} else {
$r->print(' | '.$categories{$id.'_weight'}.' | ');
}
+# Achieved
+ $r->print('');
+ if ($cangrade) {
+ $r->print('');
+ } else {
+ if ($categories{$id.'_displayachieved'} eq 'percent') {
+ $r->print(&mt('percent'));
+ } else {
+ $r->print(&mt('points'));
+ }
+ }
+ $r->print(' | ');
return ($value,$weight);
}
@@ -658,7 +683,7 @@ sub bottom_line_category {
if ($cangrade) {
$r->print(''.&mt('Create New Category').' | ');
}
- $r->print(''.&mt('Current:').$sum.' '.&mt('Total:').$total.'
| ');
+ $r->print(''.&mt('Current:').$sum.' '.&mt('Total:').$total.'
| ');
}
#
@@ -682,19 +707,42 @@ sub make_new_category {
}
$categories{$id.'_weight'}=0;
$categories{$id.'_totaltype'}='default';
+ $categories{$id.'_displayachieved'}='percent';
return %categories;
}
# === Calculation Rule Editing
+sub category_rule_codes {
+ return &Apache::lonlocal::texthash(
+ 'droplow' => 'Drop N lowest grade assignments',
+ 'drophigh' => 'Drop N highest grade assignments',
+ 'capabove' => 'Cap percentage above N percent',
+ 'capbelow' => 'Cap percentage below N percent');
+}
+
sub pretty_prt_rule {
- my ($rule)=@_;
- return $rule;
+ my ($cangrade,$id,$code,$value)=@_;
+ my $cid=$id.'_'.$code;
+ my %lt=&category_rule_codes();
+ my $ret='';
+ if ($cangrade) {
+ $ret.=' N=';
+ } else {
+ $ret.=$lt{$code}.'; N='.$value;
+ }
+ $ret.='';
+ return $ret;
}
sub new_calc_rule_form {
- return "New Rule";
+ my ($id)=@_;
+ return ''.&mt('New Calculation Rule').'';
}
#
@@ -721,7 +769,7 @@ sub del_calculation_rule {
unless ($cangrade) { return %categories; }
my @newcontent=();
foreach my $current (split(/\,/,$categories{$id.'_calculations'})) {
- unless ($current eq $delcontent) {
+ unless ($current=~/^\Q$delcontent\E\:/) {
push(@newcontent,$current);
}
}
@@ -729,6 +777,21 @@ sub del_calculation_rule {
return %categories;
}
+sub set_category_rules {
+ my ($cangrade,$id,%categories)=@_;
+ unless ($cangrade) { return %categories; }
+ my %lt=&category_rule_codes();
+ my @newrules=();
+ foreach my $code ('',(keys(%lt))) {
+ if ($env{'form.sel_'.$id.'_'.$code}) {
+ push(@newrules,$env{'form.sel_'.$id.'_'.$code}.':'.$env{'form.val_'.$id.'_'.$code});
+ }
+ }
+ $categories{$id.'_calculations'}=join(',',sort(@newrules));
+ return %categories;
+}
+
+
# === Category Editing
#
@@ -890,6 +953,15 @@ sub set_category_weight {
return %categories;
}
+sub set_category_displayachieved {
+ my ($cangrade,$id,$value,%categories)=@_;
+ unless ($cangrade) { return %categories; }
+ unless (($value eq 'percent') || ($value eq 'points')) { $value='percent'; }
+ $categories{$id.'_displayachieved'}=$value;
+ return %categories;
+}
+
+
#
# === end category-related
#