--- loncom/interface/lonwhatsnew.pm 2005/04/18 20:35:07 1.11 +++ loncom/interface/lonwhatsnew.pm 2005/04/29 14:54:18 1.12 @@ -1,5 +1,5 @@ # -# $Id: lonwhatsnew.pm,v 1.11 2005/04/18 20:35:07 raeburn Exp $ +# $Id: lonwhatsnew.pm,v 1.12 2005/04/29 14:54:18 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -525,20 +525,20 @@ sub getitems { } # Compile maxtries and degree of difficulty for problem parts my @parts = @{$resource->parts()}; - my %stats = (); + my %stats; my $warning = 0; my $rowColor; - foreach (@parts) { - %{$stats{$_}} = (); + foreach my $part (@parts) { + %{$stats{$part}} = (); my ($attempts,$users,$corrects,$degdiff,$av_attempts); - if (exists($resourcetracker{$symb.'_'.$_.'_attempts'})) { - $attempts = $resourcetracker{$symb.'_'.$_.'_attempts'}; + if (exists($resourcetracker{$symb."\0".$part."\0attempts"})) { + $attempts = $resourcetracker{$symb."\0".$part."\0attempts"}; } - if (exists($resourcetracker{$symb.'_'.$_.'_users'})) { - $users = $resourcetracker{$symb.'_'.$_.'_users'}; + if (exists($resourcetracker{$symb."\0".$part."\0users"})) { + $users = $resourcetracker{$symb."\0".$part."\0users"}; } - if (exists($resourcetracker{$symb.'_'.$_.'_correct'})) { - $corrects = $resourcetracker{$symb.'_'.$_.'_correct'}; + if (exists($resourcetracker{$symb."\0".$part."\0correct"})) { + $corrects = $resourcetracker{$symb."\0".$part."\0correct"}; } if ($attempts > 0) { $degdiff = 1 - ($corrects/$attempts); @@ -548,9 +548,9 @@ sub getitems { $av_attempts = $attempts/$users; } if (($degdiff ne '' && $degdiff >= $diffcheck) || ($av_attempts ne '' && $av_attempts >= $triescheck)) { - $stats{$_}{degdiff} = $degdiff; - $stats{$_}{attempts} = $av_attempts; - $stats{$_}{users} = $users; + $stats{$part}{degdiff} = $degdiff; + $stats{$part}{attempts} = $av_attempts; + $stats{$part}{users} = $users; $warning = 1; } } @@ -561,22 +561,22 @@ sub getitems { $rowColor = $rowColor2; } $$triggered{$symb}{title} = $resource->title; - foreach (@parts) { - if (exists($stats{$_}{users})) { + foreach my $part (@parts) { + if (exists($stats{$part}{users})) { if ($$triggered{$symb}{numparts}) { $$triggered{$symb}{text} .= ''."\n"; } if (@parts > 1) { $$triggered{$symb}{text} .= ' - part - '.$_.''; + part - '.$part.''; } else { $$triggered{$symb}{text} .= ' single part'; } $$triggered{$symb}{text} .= ' - '.$stats{$_}{users}.' - '.$stats{$_}{attempts}.' - '.$stats{$_}{degdiff}.' + '.$stats{$part}{users}.' + '.$stats{$part}{attempts}.' + '.$stats{$part}{degdiff}.' '; $$triggered{$symb}{numparts} ++; }