--- loncom/interface/Attic/lonchart.pm	2001/10/19 22:03:05	1.8
+++ loncom/interface/Attic/lonchart.pm	2001/11/22 17:25:30	1.12
@@ -18,6 +18,7 @@
 # 7/10/01 Behrouz Minaei
 # 9/8 Gerd Kortemeyer
 # 10/18/01, 10/19/01 Behrouz Minaei
+# 11/17/01, 11/22/01 Behrouz Minaei
 
 package Apache::lonchart;
 
@@ -32,6 +33,7 @@ my %hash;
 my @cols;
 my @rowlabels;
 my @students;
+my $r;
 
 # ------------------------------------------------------------- Find out status
 
@@ -47,44 +49,61 @@ sub ExtractStudentData {
     my $Wrongs;
     my %TempHash;
     my $Version;
+    my $ProbNo;
+    my $PrTotal;
     my $LatestVersion;            
                 
     my $Str=substr($students[$index].
             '                                                        ',0,14).' ! '.
             substr($rowlabels[$index].
             '                                                        ',0,45).' ! ';
-
     unless ($reply=~/^error\:/) {
         map {
             my ($name,$value)=split(/\=/,&Apache::lonnet::unescape($_));
             $result{$name}=$value;
         } split(/\&/,$reply);
+	$ProbNo = 0;
+	$PrTotal = 0;
+	my $IterationNo = 0;
         foreach $ResId (@cols) {
-	    if ( !$ResId ) { $Str .= ' ! '; next; }
+	    if ($IterationNo == 0) {$IterationNo++; next;}
+	    if (!$ResId) { 
+		my $PrNo = sprintf( "%3d", $ProbNo );
+		$Str .= ' '.'<font color="#007700">'.$PrNo.'</font> ';
+		$PrTotal += $ProbNo;
+		$ProbNo=0;
+		next; 
+	    }
             $ResId=~/(\d+)\.(\d+)/;
+	    my $meta=$hash{'src_'.$ResId};
+	    my $PartNo = 0;
+	    undef %TempHash;
+	    map {
+		if ($_=~/^stores\_(\d+)\_tries$/) {
+                    my $Part=&Apache::lonnet::metadata($meta,$_.'.part');
+		    if ( $TempHash{"$Part"} eq '' ) { 
+			$TempHash{"$Part"} = $Part;
+			$TempHash{$PartNo}=$Part;
+			$TempHash{"$Part.Code"} = ' ';  
+			$PartNo++;
+		    }
+		}
+            } split(/\,/,&Apache::lonnet::metadata($meta,'keys'));
+
             my $Prob = &Apache::lonnet::declutter( $hash{'map_id_'.$1} ).
                        '___'.$2.'___'.
                        &Apache::lonnet::declutter( $hash{'src_'.$ResId} );
             $Code=' ';
             $Tries = 0;
   	    $LatestVersion = $result{"version:$Prob"};
-
-	    undef %TempHash;
-	    my $PartNo = 0;
             
             if ( $LatestVersion ) {
-
 		for ( my $Version=1; $Version<=$LatestVersion; $Version++ ) {
 		    my $vkeys = $result{"$Version:keys:$Prob"};
 		    my @keys = split(/\:/,$vkeys);		  
 		    foreach my $Key (@keys) {		  
 			if (($Key=~/\.(\w+)\.solved$/) && ($Key!~/^\d+\:/)) {
 			    my $Part = $1;
-			    if ( $TempHash{"$Part"} eq '' ) { 
-				$TempHash{"$Part"} = $Part;
-				$TempHash{$PartNo}=$Part;  
-				$PartNo++;
-			    }
 			    $TempHash{"$Part.Tries"} = $result{"$Version:$Prob:resource.$Part.tries"};
 			    $Tries = $result{"$Version:$Prob:resource.$Part.tries"};
 			    my $Val = $result{"$Version:$Prob:$Key"};
@@ -100,17 +119,22 @@ sub ExtractStudentData {
 
 		for ( my $n = 0; $n < $PartNo; $n++ ) {		  
 		    my $part = $TempHash{$n};
-                    if (($TempHash{$part.'.Code'} eq '*')&&
-                        ($TempHash{$part.'.Tries'}<10)) {
-                        $TempHash{$part.'.Code'}=$Tries;
+                    if ($TempHash{$part.'.Code'} eq '*') {
+			$ProbNo++;
+                        if ($TempHash{$part.'.Tries'}<10) {
+			    $TempHash{$part.'.Code'}=$Tries;
+			}
                     }
-		    $Str .= $TempHash{$part.'.Code'}; 
+		    $Str .= $TempHash{$part.'.Code'};
 		}
             }   
-	    else { $Str .= ' '; }
+	    else {for(my $n=0; $n<$PartNo; $n++) {$Str.=' ';}}
         } 
     }
-    return $Str;
+    my $PrTot = sprintf( "%5d", $PrTotal );
+    $Str .= ' '.'<font color="#000088">'.$PrTot.'</font> ';
+
+    return $Str ;
 }
 
 
@@ -160,7 +184,7 @@ sub tracetable {
 # ================================================================ Main Handler
 
 sub handler {
-  my $r=shift;
+  $r=shift;
 
   if (&Apache::lonnet::allowed('vgr',$ENV{'request.course.id'})) {
 # ------------------------------------------- Set document type for header only