--- loncom/interface/spreadsheet/studentcalc.pm 2003/09/18 20:12:40 1.22 +++ loncom/interface/spreadsheet/studentcalc.pm 2005/03/10 17:33:57 1.29 @@ -1,5 +1,5 @@ # -# $Id: studentcalc.pm,v 1.22 2003/09/18 20:12:40 matthew Exp $ +# $Id: studentcalc.pm,v 1.29 2005/03/10 17:33:57 matthew Exp $ # # Copyright Michigan State University Board of Trustees # @@ -58,17 +58,19 @@ use Apache::Spreadsheet(); use Apache::assesscalc(); use HTML::Entities(); use Time::HiRes; +use Apache::lonlocal; @Apache::studentcalc::ISA = ('Apache::Spreadsheet'); my @Sequences = (); +my $navmap; my %Exportrows = (); my $current_course; sub initialize { - &Apache::assesscalc::initialize(); &initialize_sequence_cache(); + &Apache::assesscalc::initialize($navmap); } sub initialize_package { @@ -89,14 +91,27 @@ sub initialize_sequence_cache { # # Set up the sequences and assessments undef(@Sequences); - my ($top,$sequences,$assessments) = - &Apache::loncoursedata::get_sequence_assessment_data(); - if (! defined($top) || ! ref($top)) { - # There has been an error, better report it - &Apache::lonnet::logthis('top is undefined (studentcalc.pm)'); - return; + undef($navmap); + $navmap = Apache::lonnavmaps::navmap->new(); + if (!defined($navmap)) { + &Apache::lonnet::logthis('student spreadsheet:Can not open Coursemap'); + } + my @all_sequences = $navmap->retrieveResources(undef, + sub { shift->is_map(); },1,0,1); + for my $sequence ($navmap->getById('0.0'), @all_sequences) { + if ($navmap->hasResource($sequence,sub { shift->is_problem(); }, 0)){ + push(@Sequences,$sequence); + } } - @Sequences = @{$sequences} if (ref($sequences) eq 'ARRAY'); +} + +sub get_resources { + my ($seq) = @_; + return () if (! defined($navmap) || ! ref($navmap)); + my @resources = $navmap->retrieveResources($seq, + sub { shift->is_problem(); }, + 0,0,0); + return @resources; } sub clear_package { @@ -110,15 +125,11 @@ sub get_title { my @title = (); # # Determine the students name - my %userenv = &Apache::loncoursedata::GetUserName($self->{'name'}, - $self->{'domain'}); - my $name = join(' ', - @userenv{'firstname','middlename','lastname','generation'}); - $name =~ s/\s+$//; - + my $name = &Apache::loncommon::plainname($self->{'name'}, + $self->{'domain'}); push (@title,$name); push (@title,$self->{'coursedesc'}); - push (@title,scalar(localtime(time))); + push (@title,&Apache::lonlocal::locallocaltime(time)); return @title; } @@ -140,24 +151,15 @@ sub get_html_title { sub parent_link { my $self = shift; - return '

Course level sheet

'."\n"; + return '

'.&mt('Course level sheet').'

'."\n"; } sub convenience_links { my $self = shift; my ($resource) = @_; - my $symb = &Apache::lonnet::escape($resource->{'symb'}); - my $result = <<"END"; - - - - - - - - - -END + my $result=&Apache::loncommon::submlink('',$self->{'name'},$self->{'domain'},$resource->symb,'LONcatInfo'); + $result .= &Apache::loncommon::pgrdlink('',$self->{'name'},$self->{'domain'},$resource->symb,'LONcatInfo'); + $result .= &Apache::loncommon::pprmlink('',$self->{'name'},$self->{'domain'},$resource->symb,'LONcatInfo'); return $result; } @@ -173,19 +175,30 @@ sub outsheet_html { my $editing_is_allowed = &Apache::lonnet::allowed('mgr', $ENV{'request.course.id'}); #################################### + # Report any calculation errors # + #################################### + $r->print($self->html_report_error()); + #################################### # Determine table structure # #################################### my $num_uneditable = 26; my $num_left = 52-$num_uneditable; + my %lt=&Apache::lonlocal::texthash( + 'st' => 'Student', + 'im' => 'Import', + 'ca' => 'Calculations', + 'as' => 'Assessment', + 'ro' => 'Row', + ); my $tableheader =<<"END";

- + + $lt{'im'} + $lt{'ca'} END my $label_num = 0; @@ -201,8 +214,8 @@ END $tableheader .="\n"; if ($self->blackout()) { $r->print('

'. - 'Some computations are not available at this time.
'. - 'There are problems whose status you are allowed to view.'. + &mt('Some computations are not available at this time.').'
'. + &mt('There are problems whose status you are not allowed to view.'). '

'."\n"); } else { $r->print($tableheader); @@ -215,7 +228,7 @@ END } # # Print out summary/export row - $r->print(''. + $r->print(''. $self->html_export_row($exportcolor)."\n"); } $r->print("
Student$lt{'st'} - Import - Calculations
Summary0
'.&mt('Summary').'0
\n"); @@ -225,13 +238,13 @@ END $tableheader =<<"END";

- + END } else { $tableheader =<<"END";

Row Assessment
$lt{'ro'} $lt{'as'}
- + END } foreach (split(//,'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz')){ @@ -249,12 +262,10 @@ END &initialize_sequence_cache(); } foreach my $Sequence (@Sequences) { - next if ($Sequence->{'num_assess'} < 1); - $r->print("

".$Sequence->{'title'}."

\n"); + $r->print("

".$Sequence->compTitle."

\n"); $r->print($tableheader); - foreach my $resource (@{$Sequence->{'contents'}}) { - next if ($resource->{'type'} ne 'assessment'); - my $rownum = $self->get_row_number_from_key($resource->{'symb'}); + foreach my $resource (&get_resources($Sequence)) { + my $rownum = $self->get_row_number_from_key($resource->symb); my $assess_filename = $self->{'row_source'}->{$rownum}; my $row_output = ''; if ($editing_is_allowed) { @@ -264,21 +275,21 @@ END ''.$resource->{'title'}.'
'; + '&usymb='.&Apache::lonnet::escape($resource->symb). + '">'.$resource->compTitle.'
'; $row_output .= &assess_file_selector($rownum, $assess_filename, \@AssessFileNames). ''; } else { - $row_output .= ''; + $row_output .= ''; } if ($self->blackout() && $self->{'blackout_rows'}->{$rownum}>0) { $row_output .= - ''."\n"; + ''."\n"; } else { $row_output .= $self->html_row($num_uneditable,$rownum, $exportcolor,$importcolor). @@ -351,22 +362,21 @@ sub csv_rows { # writes the meat of the spreadsheet to an excel worksheet. Called # by Spreadsheet::outsheet_excel; my $self = shift; - my ($filehandle) = @_; + my ($connection,$filehandle) = @_; # # Write a header row $self->csv_output_row($filehandle,undef, - ('Sequence or Folder','Assessment title')); + (&mt('Sequence or Folder'),&mt('Assessment title'))); # # Write each assessments row if (scalar(@Sequences)< 1) { &initialize_sequence_cache(); } foreach my $Sequence (@Sequences) { - next if ($Sequence->{'num_assess'} < 1); - foreach my $resource (@{$Sequence->{'contents'}}) { - my $rownum = $self->get_row_number_from_key($resource->{'symb'}); - my @assessdata = ($Sequence->{'title'}, - $resource->{'title'}); + foreach my $resource (&get_resources($Sequence)) { + my $rownum = $self->get_row_number_from_key($resource->symb); + my @assessdata = ($Sequence->compTitle, + $resource->compTitle); $self->csv_output_row($filehandle,$rownum,@assessdata); } } @@ -377,12 +387,12 @@ sub excel_rows { # writes the meat of the spreadsheet to an excel worksheet. Called # by Spreadsheet::outsheet_excel; my $self = shift; - my ($worksheet,$cols_output,$rows_output) = @_; + my ($connection,$worksheet,$cols_output,$rows_output,$format) = @_; # # Write a header row $cols_output = 0; foreach my $value ('Container','Assessment title') { - $worksheet->write($rows_output,$cols_output++,$value); + $worksheet->write($rows_output,$cols_output++,&mt($value),$format->{'h4'}); } $rows_output++; # @@ -391,11 +401,10 @@ sub excel_rows { &initialize_sequence_cache(); } foreach my $Sequence (@Sequences) { - next if ($Sequence->{'num_assess'} < 1); - foreach my $resource (@{$Sequence->{'contents'}}) { - my $rownum = $self->get_row_number_from_key($resource->{'symb'}); - my @assessdata = ($Sequence->{'title'}, - $resource->{'title'}); + foreach my $resource (&get_resources($Sequence)) { + my $rownum = $self->get_row_number_from_key($resource->symb); + my @assessdata = ($Sequence->compTitle, + $resource->compTitle); $self->excel_output_row($worksheet,$rownum,$rows_output++, @assessdata); } @@ -415,11 +424,10 @@ sub get_rows_in_sequence { my $self = shift(); my ($sequence) = @_; my @Rows; - foreach my $resource (@{$sequence->{'contents'}}) { - if ($resource->{'type'} eq 'assessment') { - my $rownum = $self->get_row_number_from_key($resource->{'symb'}); - push (@Rows,$rownum); - } + my @resources = &get_resources($sequence); + foreach my $resource (@resources) { + my $rownum = $self->get_row_number_from_key($resource->symb); + push (@Rows,$rownum); } return @Rows; } @@ -498,18 +506,17 @@ sub compute { &initialize_sequence_cache(); } $self->initialize_safe_space(); - &Apache::assesscalc::initialize_package($self->{'name'},$self->{'domain'}); + &Apache::assesscalc::initialize_package($self->{'name'},$self->{'domain'}, + $navmap); my %f = $self->formulas(); # # Process the formulas list - # the formula for the A column of a row is symb__&&__filename my %c = $self->constants(); foreach my $seq (@Sequences) { - next if ($seq->{'num_assess'}<1); - foreach my $resource (@{$seq->{'contents'}}) { + foreach my $resource (&get_resources($seq)) { if ($connection->aborted()) { $self->cleanup(); return; } - next if ($resource->{'type'} ne 'assessment'); - my $rownum = $self->get_row_number_from_key($resource->{'symb'}); + my $rownum = $self->get_row_number_from_key($resource->symb); my $cell = 'A'.$rownum; my $assess_filename = 'Default'; if (exists($self->{'row_source'}->{$rownum})) { @@ -517,13 +524,21 @@ sub compute { } else { $self->{'row_source'}->{$rownum} = $assess_filename; } - $f{$cell} = $resource->{'symb'}.'__&&&__'.$assess_filename; + $f{$cell} = $resource->symb.'__&&&__'.$assess_filename; if ($connection->aborted()) { $self->cleanup(); return; } - my $assessSheet = Apache::assesscalc->new($self->{'name'}, - $self->{'domain'}, - $assess_filename, - $resource->{'symb'}); + my $assessSheet; + $assessSheet = Apache::assesscalc->new($self->{'name'}, + $self->{'domain'}, + $assess_filename, + $resource->symb); my @exportdata = $assessSheet->export_data($r); + # + if ($assessSheet->badcalc()) { + $self->set_calcerror( + &mt('Error computing row for assessment "[_1]" (row [_2]):[_3]', + $assessSheet->get_title(),$rownum,$assessSheet->calcerror())); + } + # if ($connection->aborted()) { $self->cleanup(); return; } if ($assessSheet->blackout()) { $self->blackout(1); @@ -654,11 +669,16 @@ Writes the export data for this student ############################################# sub save_export_data { my $self = shift; - &Apache::assesscalc::save_cached_export_rows($self->{'name'}, - $self->{'domain'}); - return if ($self->temporary()); my $student = $self->{'name'}.':'.$self->{'domain'}; + return if ($self->temporary()); + if ($self->badcalc()){ + # do not save data away when calculations have not been done properly. + delete($Exportrows{$student}); + return; + } return if (! exists($Exportrows{$student})); + &Apache::assesscalc::save_cached_export_rows($self->{'name'}, + $self->{'domain'}); return if (! $self->is_default()); my $key = join(':',($self->{'name'},$self->{'domain'},'studentcalc')).':'; my $timekey = $key.'.time'; @@ -702,9 +722,16 @@ sub export_data { $self->compute($r); } if ($connection->aborted()) { $self->cleanup(); return; } - my @Data = @{$Exportrows{$student}->{'data'}}; - for (my $i=0; $i<=$#Data;$i++) { - $Data[$i]="'".$Data[$i]."'" if ($Data[$i]=~/\D/ && defined($Data[$i])); + my @Data; + if ($self->badcalc()) { + @Data = (); + } else { + @Data = @{$Exportrows{$student}->{'data'}}; + for (my $i=0; $i<=$#Data;$i++) { + if ($Data[$i]=~/\D/ && defined($Data[$i])) { + $Data[$i]="'".$Data[$i]."'"; + } + } } return @Data; }
 Assessment
 $lt{'as'}
src.'?symb='. + &Apache::lonnet::escape($resource->symb). '">Go To'; - $row_output .= ''.$resource->{'title'}.''.$resource->compTitle.'Unavailable at this time
'.&mt('Unavailable at this time').'