--- loncom/interface/spreadsheet/assesscalc.pm 2003/10/03 15:39:46 1.17.2.2 +++ loncom/interface/spreadsheet/assesscalc.pm 2003/09/05 01:06:45 1.18 @@ -1,5 +1,5 @@ # -# $Id: assesscalc.pm,v 1.17.2.2 2003/10/03 15:39:46 albertel Exp $ +# $Id: assesscalc.pm,v 1.18 2003/09/05 01:06:45 matthew Exp $ # # Copyright Michigan State University Board of Trustees # @@ -44,11 +44,12 @@ assesscalc package Apache::assesscalc; use strict; +use warnings FATAL=>'all'; +no warnings 'uninitialized'; use Apache::Constants qw(:common :http); use Apache::lonnet; use Apache::loncommon; use Apache::Spreadsheet; -use Apache::loncoursedata(); use HTML::Entities(); use Spreadsheet::WriteExcel; use GDBM_File; @@ -126,7 +127,6 @@ sub clear_package { sub initialize { &clear_package(); - &Apache::loncoursedata::clear_internal_caches(); } ######################################################## @@ -150,7 +150,6 @@ sub initialize_package { } &load_cached_export_rows(); &load_parameter_caches(); - &Apache::loncoursedata::clear_internal_caches(); } ######################################################## @@ -170,7 +169,6 @@ sub load_parameter_caches { # # Course Parameters Cache if (! %courseopt) { - &Apache::lonnet::logthis("loading course options"); $current_course = $ENV{'request.course.id'}; undef(%courseopt); if (! defined($current_name) || ! defined($current_domain)) { @@ -597,10 +595,7 @@ sub compute { while (my ($parm,$value) = each(%parameters)) { last if ($self->blackout()); next if ($parm !~ /^(parameter_.*)_problemstatus$/); - if ($parameters{$1.'_answerdate'} ne '' && - $parameters{$1.'_answerdate'} < time) { - next; - } + next if ($parameters{$1.'_answerdate'}blackout(1); @@ -695,7 +690,7 @@ 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'}, $current_domain,$current_name,undef); @@ -741,7 +736,9 @@ sub export_data { ! exists($Exportrows{$symb}) || ! defined($Exportrows{$symb}) || ! $self->check_expiration_time($Exportrows{$symb}->{'time'}) || ! exists($Exportrows{$symb}->{$self->{'filename'}}) || - ! defined($Exportrows{$symb}->{$self->{'filename'}})) { + ! defined($Exportrows{$symb}->{$self->{'filename'}}) || + ! ref($Exportrows{$symb}->{$self->{'filename'}}) + ) { $self->compute(); } my @Data = @{$Exportrows{$symb}->{$self->{'filename'}}};