--- loncom/interface/spreadsheet/studentcalc.pm 2005/04/07 06:56:23 1.30 +++ loncom/interface/spreadsheet/studentcalc.pm 2005/05/15 03:06:59 1.32 @@ -1,5 +1,5 @@ # -# $Id: studentcalc.pm,v 1.30 2005/04/07 06:56:23 albertel Exp $ +# $Id: studentcalc.pm,v 1.32 2005/05/15 03:06:59 albertel Exp $ # # Copyright Michigan State University Board of Trustees # @@ -101,22 +101,29 @@ sub initialize_sequence_cache { for my $sequence ($navmap->getById('0.0'), @all_sequences) { if ($navmap->hasResource($sequence,sub { shift->is_problem(); }, 0)){ push(@Sequences,$sequence); + &get_resources($sequence); } } } +my %res_memoize; sub get_resources { my ($seq) = @_; + if (exists($res_memoize{$seq->symb()})) { + return @{$res_memoize{$seq->symb()}}; + } return () if (! defined($navmap) || ! ref($navmap)); my @resources = $navmap->retrieveResources($seq, sub { shift->is_problem(); }, 0,0,0); + $res_memoize{$seq->symb()}=\@resources; return @resources; } sub clear_package { undef(@Sequences); undef(%Exportrows); + undef(%res_memoize); &Apache::assesscalc::clear_package(); } @@ -496,8 +503,6 @@ END sub compute { my $self = shift; my ($r) = @_; - my $connection = $r->connection(); - if ($connection->aborted()) { $self->cleanup; return; } if (! defined($current_course) || $current_course ne $env{'request.course.id'} || ! @Sequences ) { @@ -515,7 +520,6 @@ sub compute { my %c = $self->constants(); foreach my $seq (@Sequences) { foreach my $resource (&get_resources($seq)) { - if ($connection->aborted()) { $self->cleanup(); return; } my $rownum = $self->get_row_number_from_key($resource->symb); my $cell = 'A'.$rownum; my $assess_filename = 'Default'; @@ -525,7 +529,6 @@ sub compute { $self->{'row_source'}->{$rownum} = $assess_filename; } $f{$cell} = $resource->symb.'__&&&__'.$assess_filename; - if ($connection->aborted()) { $self->cleanup(); return; } my $assessSheet; $assessSheet = Apache::assesscalc->new($self->{'name'}, $self->{'domain'}, @@ -539,7 +542,6 @@ sub compute { $assessSheet->get_title(),$rownum,$assessSheet->calcerror())); } # - if ($connection->aborted()) { $self->cleanup(); return; } if ($assessSheet->blackout()) { $self->blackout(1); $self->{'blackout_rows'}->{$rownum} = 1;