--- loncom/interface/lonquickgrades.pm 2003/03/10 06:42:32 1.11
+++ loncom/interface/lonquickgrades.pm 2003/06/18 19:31:38 1.15
@@ -107,6 +107,8 @@ HEADER
# the counts to the parent map.)
my $totalParts = 0; my $totalPossible = 0; my $totalRight = 0;
+ my $now = time();
+ my $topLevelParts = 0; my $topLevelRight = 0;
# Pre-run: Count parts correct
while ( $depth > 0 ) {
@@ -117,25 +119,32 @@ HEADER
{
# Get number of correct, incorrect parts
my $parts = $curRes->parts();
- if (scalar(@{$parts}) > 1) { shift @{$parts}; } # lose the first "0"
- $totalParts += scalar(@{$parts});
+ $totalParts += $curRes->countParts();
my $partsRight = 0;
my $stack = $iterator->getStack();
for my $part (@{$parts}) {
my $status = $curRes->getCompletionStatus($part);
+ my $thisright = 0;
if ($status == $curRes->CORRECT || $status == $curRes->CORRECT_BY_OVERRIDE
- || $status == $curRes->EXCUSED) {
+ || $status == $curRes->EXCUSED || $status == $curRes->ANSWER_SUBMITTED) {
$partsRight++;
$totalRight++;
+ $thisright = 1;
}
my $dateStatus = $curRes->getDateStatus($part);
- if ($status != $curRes->OPEN_LATER) {
+ if ($curRes->opendate($part) < $now) {
$totalPossible++;
}
}
-
+
+ if ($depth == 1) { # in top-level only
+ $topLevelParts += scalar(@{$parts});
+ $topLevelRight += $partsRight;
+ $r->print($curRes->compTitle() . '
');
+ }
+
# Crawl down stack and record parts correct and total
for my $res (@{$stack}) {
if (ref($res) && $res->is_map()) {
@@ -162,9 +171,6 @@ HEADER
my $indentString = ' ';
- my $unaccountedTotal = $totalParts;
- my $unaccountedCorrect = $totalRight;
-
# Second pass: Print the maps.
$r->print('
Sequence | '); $r->print('Done / Total |
"); - for (my $i = 1; $i < $depth; $i++) { $r->print($indentString); } + my $thisIndent = ''; + for (my $i = 1; $i < $depth; $i++) { $thisIndent .= $indentString; } - $r->print("$title | "); - $r->print("$correct / $total | $thisIndent | \n");
}
}
@@ -205,12 +202,12 @@ HEADER
}
# If there were any problems at the top level, print an extra "catchall"
- if ($unaccountedTotal > 0) {
- my $ratio = $unaccountedCorrect / $unaccountedTotal;
+ if ($topLevelParts > 0) {
+ my $ratio = $topLevelRight / $topLevelParts;
my $color = mixColors(\@start, \@end, $ratio);
$r->print("
"); - $r->print("Problems Not Contained In Sequences | "); - $r->print("$unaccountedCorrect / $unaccountedTotal | "); + $r->print("$topLevelRight / $topLevelParts | "); } my $maxHelpLink = Apache::loncommon::help_open_topic("Quick_Grades_Possibly_Correct");