--- loncom/homework/structuretags.pm 2005/04/18 21:10:41 1.285 +++ loncom/homework/structuretags.pm 2005/04/18 22:39:35 1.288 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # definition of tags that give a structure to a document # -# $Id: structuretags.pm,v 1.285 2005/04/18 21:10:41 raeburn Exp $ +# $Id: structuretags.pm,v 1.288 2005/04/18 22:39:35 albertel Exp $ # # Copyright Michigan State University Board of Trustees # @@ -347,24 +347,29 @@ sub finalize_storage { # and corrects for each part for an instance of a problem, as appropriate. sub store_aggregates { my ($symb,$courseid) = @_; - my %aggregate = (); - my @parts = (); - my ($cdomain,$cname) = split/_/,$courseid; - foreach my $key (keys %Apache::lonhomework::results) { - if ($key =~ /resource\.(\d+)\.tries/) { - push @parts, $1; + my %aggregate; + my @parts; + my $cdomain = $env{'course.'.$env{'request.course.id'}.'.domain'}; + my $cname = $env{'course.'.$env{'request.course.id'}.'.num'}; + foreach my $key (keys(%Apache::lonhomework::results)) { + if ($key =~ /resource\.([^\.]+)\.tries/) { + push(@parts, $1); } } - foreach (@parts) { - if ($Apache::lonhomework::results{'resource.'.$_.'.award'} eq 'APPROX_ANS' || $Apache::lonhomework::results{'resource.'.$_.'.award'} eq 'EXACT_ANS') { - $aggregate{$symb.'_'.$_.'_correct'} = 1; + foreach my $part (@parts) { + if ($Apache::lonhomework::results{'resource.'.$part.'.award'} + eq 'APPROX_ANS' || + $Apache::lonhomework::results{'resource.'.$part.'.award'} + eq 'EXACT_ANS') { + $aggregate{$symb."\0".$part."\0correct"} = 1; } - if ($Apache::lonhomework::results{'resource.'.$_.'.tries'} == 1) { - $aggregate{$symb.'_'.$_.'_users'} = 1; + if ($Apache::lonhomework::results{'resource.'.$part.'.tries'} == 1) { + $aggregate{$symb."\0".$part."\0users"} = 1; } - $aggregate{$symb.'_'.$_.'_attempts'} = 1; + $aggregate{$symb."\0".$part."\0attempts"} = 1; } - &Apache::lonnet::cinc('nohist_resourcetracker',\%aggregate,$cdomain,$cname); + &Apache::lonnet::cinc('nohist_resourcetracker',\%aggregate, + $cdomain,$cname); } sub checkout_msg {