--- loncom/interface/statistics/lonstudentassessment.pm 2003/09/03 18:23:10 1.66 +++ loncom/interface/statistics/lonstudentassessment.pm 2003/10/02 17:07:41 1.69 @@ -1,6 +1,6 @@ # The LearningOnline Network with CAPA # -# $Id: lonstudentassessment.pm,v 1.66 2003/09/03 18:23:10 matthew Exp $ +# $Id: lonstudentassessment.pm,v 1.69 2003/10/02 17:07:41 matthew Exp $ # # Copyright Michigan State University Board of Trustees # @@ -200,6 +200,9 @@ sub BuildStudentAssessmentPage { # are immediately reflected in the chart. &Apache::lonnet::clear_EXT_cache_status(); # + # Clean out loncoursedata's package data, just to be safe. + &Apache::loncoursedata::clear_internal_caches(); + # # Call the initialize routine selected above $initialize->($r); foreach my $student (@Students) { @@ -931,9 +934,7 @@ END # # Add a worksheet my $sheetname = $ENV{'course.'.$ENV{'request.course.id'}.'.description'}; - if (length($sheetname) > 31) { - $sheetname = substr($sheetname,0,31); - } + $sheetname = &Apache::loncommon::clean_excel_name($sheetname); $excel_sheet = $excel_workbook->addworksheet($sheetname); # # Put the course description in the header @@ -1407,7 +1408,8 @@ sub StudentTriesOnSequence { } elsif ($status eq 'excused') { $symbol = 'x'; $max--; - } elsif ($status eq 'correct_by_student' && + } elsif (($status eq 'correct_by_scantron' || + $status eq 'correct_by_student') && exists($resource_data->{'resource.'.$partnum.'.tries'})){ $tries = $resource_data->{'resource.'.$partnum.'.tries'}; if ($tries > 9) { @@ -1514,6 +1516,7 @@ sub StudentPerformanceOnSequence { my $awarded = 0; if (exists($resource_data->{'resource.'.$part.'.awarded'})) { $awarded = $resource_data->{'resource.'.$part.'.awarded'}; + $awarded = 0 if (! $awarded); } # $partscore = $weight*$awarded;