- | Assessment |
- |
+ $lt{'as'} |
+ |
- Calculations |
+ $lt{'ca'}
END
my $label_num = 0;
foreach (split(//,'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz')){
if ($label_num<$num_uneditable) {
- $tableheader .= '| ';
+ $tableheader .= ' | ';
} else {
$tableheader .= ' | ';
}
@@ -383,21 +488,22 @@ END
$r->print($tableheader);
#
# Print out template row
- $r->print(' |
| Template | | '.
- $self->html_template_row($num_uneditable)."
\n");
+ $r->print('| '.&mt('Template').' | | '.
+ $self->html_template_row($num_uneditable,$importcolor).
+ "
\n");
#
# Print out summary/export row
- $r->print('| Export | 0 | '.
- $self->html_export_row()."
\n");
+ $r->print('| '.&mt('Export').' | 0 | '.
+ $self->html_export_row($exportcolor)."
\n");
#
# Prepare to output rows
- $tableheader =<<"END";
-
-| row | Item |
-END
+ $tableheader =
+ ''.
+ '| '.&mt('Row').' | '.
+ ''.&mt('Item').' | ';
foreach (split(//,'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz')){
if ($label_num<$num_uneditable) {
- $tableheader.='';
+ $tableheader.=' | ';
} else {
$tableheader.=' | ';
}
@@ -405,12 +511,17 @@ END
}
#
my $num_output = 0;
- foreach my $rownum ($self->rows()) {
+ foreach my $rownum (sort {$a <=> $b} ($self->rows())) {
+ if (! $self->parameter_part_is_valid(
+ $self->{'formulas'}->{'A'.$rownum}
+ )) {
+ next;
+ }
if ($num_output++ % 50 == 0) {
$r->print(" |
|---|
\n".$tableheader);
}
$r->print('
|---|
| '.$rownum.' | '.
- $self->assess_html_row($num_uneditable,$rownum)."
\n");
+ $self->assess_html_row($rownum,$importcolor)."\n");
}
$r->print("
\n");
return;
@@ -418,89 +529,241 @@ END
sub assess_html_row {
my $self = shift();
- my ($num_uneditable,$row) = @_;
- my $requester_is_student = ($ENV{'request.role'} =~ /^st\./);
+ my ($row,$importcolor) = @_;
my $parameter_name = $self->{'formulas'}->{'A'.$row};
my @rowdata = $self->get_row($row);
my $num_cols_output = 0;
my $row_html;
- if (exists($nice_parameter_name{$parameter_name})) {
- my $name = $nice_parameter_name{$parameter_name};
+ my $name=$self->get_parm_name($parameter_name);
+ if ($name ne '') {
$name =~ s/ /\ /g;
$row_html .= ''.$name.'
'.$parameter_name.' | ';
} else {
$row_html .= ''.$parameter_name.' | ';
}
foreach my $cell (@rowdata) {
- if ($requester_is_student ||
- $num_cols_output++ < $num_uneditable) {
- $row_html .= '';
- $row_html .= &Apache::Spreadsheet::html_uneditable_cell($cell,'#FFDDDD');
- } else {
- $row_html .= ' | ';
- $row_html .= &Apache::Spreadsheet::html_editable_cell($cell,'#E0FFDD');
- }
+ if ($num_cols_output < 1) {
+ $row_html .= ' | ';
+ $row_html .= &Apache::Spreadsheet::html_uneditable_cell($cell,
+ '#FFDDDD');
+ } else {
+ $row_html .= ' | ';
+ $row_html .= &Apache::Spreadsheet::html_editable_cell($cell,
+ '#E0FFDD',1);
+ }
$row_html .= ' | ';
+ $num_cols_output++;
}
return $row_html;
}
-sub outsheet_csv {
- my $self = shift;
- my ($r)=@_;
-}
-
-sub outsheet_excel {
- my $self = shift;
- my ($r)=@_;
+sub csv_rows {
+ # writes the meat of the spreadsheet to an excel worksheet. Called
+ # by Spreadsheet::outsheet_excel;
+ my $self = shift;
+ my ($connection,$filehandle) = @_;
+ #
+ # Write a header row
+ $self->csv_output_row($filehandle,undef,
+ (&mt('Parameter'),&mt('Description'),&mt('Value')));
+ #
+ # Write each row
+ foreach my $rownum (sort {$a <=> $b} ($self->rows())) {
+ my $parameter_name = $self->{'formulas'}->{'A'.$rownum};
+ my $description = $self->get_parm_name($parameter_name);
+ $self->csv_output_row($filehandle,$rownum,
+ $parameter_name,$description);
+ }
+ return;
}
-sub display {
- my $self = shift;
- my ($r) = @_;
- $self->compute();
- $self->outsheet_html($r);
+sub excel_rows {
+ # writes the meat of the spreadsheet to an excel worksheet. Called
+ # by Spreadsheet::outsheet_excel;
+ my $self = shift;
+ my ($connection,$worksheet,$cols_output,$rows_output,$format) = @_;
+ return if (! ref($worksheet));
+ #
+ # Write a header row
+ $cols_output = 0;
+ foreach my $value ('Parameter','Description','Value') {
+ $worksheet->write($rows_output,$cols_output++,$value,$format->{'h4'});
+ }
+ $rows_output++;
+ #
+ # Write each row
+ foreach my $rownum (sort {$a <=> $b} ($self->rows())) {
+ my $parameter_name = $self->{'formulas'}->{'A'.$rownum};
+ my $description = $self->get_parm_name($parameter_name);
+ $self->excel_output_row($worksheet,$rownum,$rows_output++,
+ $parameter_name,$description);
+ }
+ return;
}
-sub compute {
+##
+## Routines to support assesscalc::compute
+##
+sub get_parm {
my $self = shift;
- $self->logthis('computing');
- $self->initialize_safe_space();
- #
- # Definitions
- undef(%nice_parameter_name);
- my %parameters; # holds underscored parameters by name
+ my @Mandatory_parameters = @_;
+ my %parameters;
#
- # Get the metadata fields and determine their proper names
- my ($symap,$syid,$srcf)=split(/___/,$self->{'symb'});
+ my ($symap,$syid,$srcf) = &Apache::lonnet::decode_symb($self->{'symb'});
my @Metadata = split(/\,/,&Apache::lonnet::metadata($srcf,'keys'));
- foreach my $parm (@Metadata) {
+ foreach my $parm (@Mandatory_parameters,@Metadata) {
next if ($parm !~ /^(resource\.|stores|parameter)_/);
my $cleaned_name = $parm;
$cleaned_name =~ s/^resource\./stores_/;
$cleaned_name =~ s/\./_/g;
- my $display = &Apache::lonnet::metadata($srcf,
- $cleaned_name.'.display');
- if (! $display) {
- $display .= &Apache::lonnet::metadata($srcf,$cleaned_name.'.name');
- }
- $parameters{$cleaned_name}++;
- $nice_parameter_name{$cleaned_name} = $display;
+ $parameters{$cleaned_name}=1;
+ }
+ return (keys(%parameters));
+}
+
+sub get_parm_name {
+ my $self = shift;
+ my $parm = shift;
+ my ($symap,$syid,$srcf) = &Apache::lonnet::decode_symb($self->{'symb'});
+ my $display = &Apache::lonnet::metadata($srcf,$parm.'.display');
+ if (! $display) {
+ $display .= &Apache::lonnet::metadata($srcf,$parm.'.name');
+ }
+ return $display;
+}
+
+sub get_parameter_values {
+ my $self = shift();
+ my @Parameters;
+ my ($parameters) = @_;
+ if (!ref($parameters)) {
+ @Parameters = @_;
+ } elsif (ref($parameters) eq 'ARRAY') {
+ @Parameters = @$parameters;
+ } elsif (ref($parameters) eq 'HASH') {
+ @Parameters = keys(%$parameters);
}
#
- # Get the values of the metadata fields
- $self->ensure_current_parameter_caches();
+ my %parameters;
+ #
my $filename = $self->{'coursefilename'}.'_parms.db';
if (tie(%parmhash,'GDBM_File',
$self->{'coursefilename'}.'_parms.db',&GDBM_READER(),0640)) {
- foreach my $parmname (keys(%parameters)) {
- my $value = $self->parmval($parmname);
+ my ($mapname,$id,$fn)=&Apache::lonnet::decode_symb($self->{'symb'});
+ $mapname = &Apache::lonnet::deversion($mapname);
+ foreach my $parmname (@Parameters) {
+ my $value = $self->parmval($parmname,$self->{'symb'},
+ $self->{'name'},$self->{'domain'},
+ $self->{'section'},undef,
+ $mapname,$id,$fn,$self->{'groups'});
$parameters{$parmname} =$value;
}
untie(%parmhash);
} else {
$self->logthis('unable to tie '.$filename);
}
+ return %parameters;
+}
+
+sub deal_with_export_row {
+ my $self = shift();
+ my @exportarray = @_;
+ $Exportrows{$self->{'symb'}}->{'time'} = time;
+ $Exportrows{$self->{'symb'}}->{$self->{'filename'}} = \@exportarray;
+ #
+ # Save the export data
+ $self->save_export_data();
+ return;
+}
+
+sub get_problem_state {
+ my $self = shift;
+ my %student_parameters;
+ if (exists($userdata{$self->{'symb'}}) &&
+ ref($userdata{$self->{'symb'}}) eq 'HASH') {
+ %student_parameters = %{$userdata{$self->{'symb'}}};
+ }
+ return %student_parameters;
+}
+
+sub determine_parts {
+ my $self = shift;
+ my $check_hidden = shift;
+ if (exists($self->{'Parts'}) && ref($self->{'Parts'}) eq 'HASH') {
+ return;
+ }
+ my (undef,undef,$url) = &Apache::lonnet::decode_symb($self->{'symb'});
+ my $src = &Apache::lonnet::clutter($url);
+ return if (! defined($src));
+ my %Parts;
+ my $metadata = &Apache::lonnet::metadata($src,'packages');
+ foreach (split(',',$metadata)) {
+ my ($part) = (/^part_(.*)$/);
+ if (!defined($part)) { next; }
+ if (!$check_hidden) { $Parts{$part}++; next; }
+ if (!&Apache::loncommon::check_if_partid_hidden
+ ($part,$self->{'symb'},$self->{'name'},$self->{'domain'})
+ ) {
+ $Parts{$part}++;
+ }
+ }
+ # Make sure part 0 is defined.
+ $Parts{'0'}++;
+ $self->{'Parts'} = \%Parts;
+ return;
+}
+
+sub parameter_part_is_valid {
+ my $self = shift;
+ my ($parameter) = @_;
+ return 1 if ($parameter eq 'timestamp');
+ if (! defined($self->{'Parts'}) ||
+ ! ref ($self->{'Parts'}) ||
+ ref($self->{'Parts'}) ne 'HASH') {
+ return 1;
+ }
+ #
+ my ($start,@pieces)=split('_',$parameter);
+ if ( $start !~ m/^(resource|stores|parameter)$/) { return 0; }
+ while (@pieces) {
+ pop(@pieces);
+ my $testpart=join('_',@pieces);
+ if (exists($self->{'Parts'}->{$testpart}) &&
+ $self->{'Parts'}->{$testpart} ) {
+ return 1;
+ }
+ }
+ return 0;
+}
+
+sub compute {
+ my $self = shift;
+ my ($r) = @_;
+ $self->initialize_safe_space();
+ #########################################
+ #########################################
+ ### ###
+ ### Retrieve the problem parameters ###
+ ### ###
+ #########################################
+ #########################################
+ my @Mandatory_parameters = ("stores_0_solved",
+ "stores_0_awarddetail",
+ "stores_0_awarded",
+ "timestamp",
+ "stores_0_tries",
+ "stores_0_award");
+ #
+ # Definitions
+ undef(%nice_parameter_name);
+ my %parameters; # holds underscored parameters by name
+ #
+ # Get the metadata fields and determine their proper names
+ my @parameters=$self->get_parm(@Mandatory_parameters);
+ #
+ # Get the values of the metadata fields
+ $self->ensure_current_caches();
+ %parameters = $self->get_parameter_values(@parameters);
#
# Clean out unnecessary parameters
foreach (keys(%parameters)) {
@@ -508,15 +771,14 @@ sub compute {
}
#
# Get the students performance data
- my %student_parameters =
- &Apache::loncoursedata::get_current_state($self->{'name'},
- $self->{'domain'},
- $self->{'symb'},
- $self->{'cid'});
+ $self->determine_parts(($parameters{'parameter_0_hiddenparts'} ne ''));
+ my %student_parameters = $self->get_problem_state();
while (my ($parm,$value) = each(%student_parameters)) {
$parm =~ s/^resource\./stores_/;
$parm =~ s/\./_/g;
- $parameters{$parm} = $value;
+ # Clean out any bad parameters
+ next if (! $self->parameter_part_is_valid($parm));
+ $parameters{$parm} = $value;
}
#
# Set up the formulas and parameter values
@@ -524,11 +786,14 @@ sub compute {
my %c;
#
# Check for blackout requirements
- if ((!exists($ENV{'request.role.adv'}) || !$ENV{'request.role.adv'})) {
+ if ((!exists($env{'request.role.adv'}) || !$env{'request.role.adv'})) {
while (my ($parm,$value) = each(%parameters)) {
last if ($self->blackout());
next if ($parm !~ /^(parameter_.*)_problemstatus$/);
- next if ($parameters{$1.'_answerdate'}blackout(1);
@@ -540,22 +805,26 @@ sub compute {
while (my ($parm,$value) = each(%parameters)) {
my $cell = 'A'.$self->get_row_number_from_key($parm);
$f{$cell} = $parm;
- $c{$parm} = '';
- if ($value ne '') {
- $c{$parm} = '"'.$value.'"';
+ if ($parm =~ /_submission$/ && $value =~ /(\{|\})/) {
+ $value = 'witheld';
}
+ $value = 'q{'.$value.'}' if ($value =~/([^\d\.]|\.\.)/);
+ $c{$parm} = $value;
}
- $self->formulas(%f);
- $self->constants(%c);
+ foreach my $cell (grep(/^A/,keys(%f))) {
+ # Clean out any bad formulas
+ next if (exists($c{$f{$cell}}));
+ next if ($cell eq 'A0');
+ delete($f{$cell});
+ }
+ $self->formulas(\%f);
+ $self->constants(\%c);
$self->calcsheet();
#
# Store export row in cache
my @exportarray = $self->exportrow();
- $Exportrows{$self->{'symb'}}->{'time'} = time;
- $Exportrows{$self->{'symb'}}->{$self->{'filename'}} = \@exportarray;
- #
- # Save the export data
- $self->save_export_data();
+ $self->deal_with_export_row(@exportarray);
+ $self->save() if ($self->need_to_save());
return;
}
@@ -565,6 +834,7 @@ sub compute {
sub sett {
my $self = shift;
my %t=();
+ undef(%Apache::Spreadsheet::sheet_values);
#
# Deal with the template row by copying the template formulas into each
# row.
@@ -582,7 +852,7 @@ sub sett {
# Replace 'A0' with the value from 'A0'
$t{$cell}=~s/(^|[^\"\'])([A-Za-z]\d+)/$1\$sheet_values\{\'$2\'\}/g;
# Replace parameters
- $t{$cell}=~s/(^|[^\"\'])\[([^\]]+)\]/$1.$self->expandnamed($2)/ge;
+ $t{$cell}=~s/(^|[^\"\'])\[([^\]]+)\]/$1.$self->expandnamed($2)/sge;
}
}
#
@@ -590,14 +860,18 @@ sub sett {
while (my ($cell,$formula) = each(%{$self->{'formulas'}})) {
next if ($cell =~ /template_/);
if ($cell =~ /^A/ && $cell ne 'A0') {
- if ($formula !~ /^\!/) {
- $t{$cell}=$self->{'constants'}->{$formula};
+ if ($formula !~ /^\!/
+ && exists($self->{'constants'}->{$formula})
+ && $self->{'constants'}->{$formula} ne ''
+ ) {
+ $Apache::Spreadsheet::sheet_values{$cell}=
+ eval($self->{'constants'}->{$formula});
}
} else {
$t{$cell}=$formula;
$t{$cell}=~s/\.\.+/\,/g;
$t{$cell}=~s/(^|[^\"\'])([A-Za-z]\d+)/$1\$sheet_values\{\'$2\'\}/g;
- $t{$cell}=~s/(^|[^\"\'])\[([^\]]+)\]/$1.$self->expandnamed($2)/ge;
+ $t{$cell}=~s/(^|[^\"\'])\[([^\]]+)\]/$1.$self->expandnamed($2)/sge;
}
}
# Put %t into the safe space
@@ -624,17 +898,21 @@ These rows are saved in the students dir
########################################################
########################################################
sub load_cached_export_rows {
- %Exportrows = undef;
+ undef(%Exportrows);
my @tmp = &Apache::lonnet::dump('nohist_calculatedsheets_'.
- $ENV{'request.course.id'},
+ $env{'request.course.id'},
$current_domain,$current_name,undef);
if ($tmp[0]!~/^error/) {
my %tmp = @tmp;
- my $default_filename = $ENV{'course.'.$ENV{'request.course.id'}.
+ my $default_filename = $env{'course.'.$env{'request.course.id'}.
'.spreadsheet_default_assesscalc'};
# We only got one key, so we will access it directly.
while (my ($key,$sheetdata) = each(%tmp)) {
my ($sname,$sdom,$sheettype,$symb) = split(':',$key);
+ if (! defined($sname) || $sname eq '' ||
+ ! defined($sdom) || $sdom eq '' ) {
+ next;
+ }
if ($symb =~ /\.time$/) {
$symb =~ s/\.time$//;
$Exportrows{$symb}->{'time'} = $sheetdata;
@@ -665,21 +943,31 @@ spreadsheet only if necessary.
#############################################
sub export_data {
my $self = shift;
+ my ($r) = @_;
my $symb = $self->{'symb'};
- if (! exists($ENV{'request.role.adv'}) || ! $ENV{'request.role.adv'} ||
+ if (! exists($env{'request.role.adv'}) || ! $env{'request.role.adv'} ||
! exists($Exportrows{$symb}) || ! defined($Exportrows{$symb}) ||
! $self->check_expiration_time($Exportrows{$symb}->{'time'}) ||
! exists($Exportrows{$symb}->{$self->{'filename'}}) ||
- ! defined($Exportrows{$symb}->{$self->{'filename'}})) {
- $self->compute();
- }
- my @Data = @{$Exportrows{$symb}->{$self->{'filename'}}};
- if ($Data[0] =~ /^(.*)___=___/) {
- $self->{'sheetname'} = $1;
- $Data[0] =~ s/^(.*)___=___//;
- }
- for (my $i=0;$i<$#Data;$i++) {
- $Data[$i]="'".$Data[$i]."'" if ($Data[$i]=~/\D/ && defined($Data[$i]));
+ ! defined($Exportrows{$symb}->{$self->{'filename'}}) ||
+ ! ref($Exportrows{$symb}->{$self->{'filename'}})
+ ) {
+ $self->compute($r);
+ }
+ my @Data;
+ if ($self->badcalc()) {
+ @Data = ();
+ } else {
+ @Data = @{$Exportrows{$symb}->{$self->{'filename'}}};
+ if ($Data[0] =~ /^(.*)___=___/) {
+ $self->{'sheetname'} = $1;
+ $Data[0] =~ s/^(.*)___=___//;
+ }
+ for (my $i=0;$i<$#Data;$i++) {
+ if ($Data[$i]=~/\D/ && defined($Data[$i])) {
+ $Data[$i]="'".$Data[$i]."'";
+ }
+ }
}
return @Data;
}
@@ -699,23 +987,26 @@ Writes the export data for this spreadsh
#############################################
sub save_export_data {
my $self = shift;
+ return if ($self->temporary());
my $student = $self->{'name'}.':'.$self->{'domain'};
my $symb = $self->{'symb'};
+ if ($self->badcalc()){
+ # do not save data away when calculations have not been done properly.
+ delete($Exportrows{$symb});
+ return;
+ }
if (! exists($Exportrows{$symb}) ||
! exists($Exportrows{$symb}->{$self->{'filename'}})) {
return;
}
my $key = join(':',($self->{'name'},$self->{'domain'},'assesscalc',$symb));
my $timekey = $key.'.time';
- my $newstore= join('___;___',@{$Exportrows{$symb}->{$self->{'filename'}}});
+ my $newstore= join('___;___',
+ map {s/[^[:print:]]//g;$_;} # strip out unprintable
+ @{$Exportrows{$symb}->{$self->{'filename'}}});
$newstore = $self->{'filename'}.'___=___'.$newstore;
- my $result = &Apache::lonnet::put
- ('nohist_calculatedsheets_'.$ENV{'request.course.id'},
- { $key => $newstore,
- $timekey => $Exportrows{$symb}->{'time'} },
- $self->{'domain'},
- $self->{'name'});
-
+ $newExportrows{$student}->{$key} = $newstore;
+ $newExportrows{$student}->{$timekey} = $Exportrows{$symb}->{'time'};
return;
}