--- loncom/interface/lonstatistics.pm	2002/02/06 16:59:01	1.4
+++ loncom/interface/lonstatistics.pm	2002/03/02 01:42:42	1.5
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # (Publication Handler
 #
-# $Id: lonstatistics.pm,v 1.4 2002/02/06 16:59:01 minaeibi Exp $
+# $Id: lonstatistics.pm,v 1.5 2002/03/02 01:42:42 minaeibi Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -27,12 +27,10 @@
 #
 # (Navigate problems for statistical reports
 # YEAR=2001
-# 5/05/01, 7/09/01, 7/25/01, 8/11/01,9/13/01, 9/26/01 Behrouz Minaei
-# 10/5/01, 10/9/01, 10/22/01, 10/26/01 Behrouz Minaei
-# 11/1/01, 11/4/01, 11/16/01 Behrouz Minaei
-# 12/14/01, 12/16/01, 12/18/01,12/20/01,12/31/01 Behrouz Minaei
+# 5/05,7/09,7/25/01,8/11,9/13,9/26,10/5,10/9,10/22,10/26 Behrouz Minaei
+# 11/1, 11/4, 11/16, 12/14, 12/16, 12/18,12/20,12/31 Behrouz Minaei
 # YEAR=2002
-# 1/22/02,2/1/02 Behrouz Minaei
+# 1/22, 2/1, 2/6, 2/25 Behrouz Minaei
 ###
 
 package Apache::lonstatistics; 
@@ -50,7 +48,7 @@ my %hash;
 my %CachData;
 my %GraphDat;
 my %maps;
-my @mapsort;
+my %mapsort;
 my %section;
 my %StuBox;
 my %DiscFac;
@@ -74,13 +72,13 @@ my $GData;
 my $cid;
 my $firstres;
 my $lastres;
-my $DiscFlag=0;
-my $HWN=0;
-
-my %Header = (0,"Problem Title",1,"#Stdnts",2,"Tries",3,"Mod",
-              4,"Mean",5,"#YES",6,"#yes",7,"%Wrng",8,"S.D.",
-              9,"Skew.",10,"DoDiff",11,"Map");
-#             9,"Skew.",10,"DoDiff",11,"Dis.F.",12,"Resourse URL");
+my $DiscFlag;
+my $HWN;
+my $P_Order;
+my %Header = (0,"Homework Problems",1,"#Stdnts",2,"Tries",3,"Mod",
+              4,"Mean",5,"#YES",6,"#yes",7,"%Wrng",8,"DoDiff",
+              9,"S.D.",10,"Skew.",11,"Map");
+#             11,"Dis.F.",12,"Resourse URL");
 
 sub NumericSort {          
     $a <=> $b;
@@ -98,6 +96,7 @@ sub StudentReport {
     my $reply=&Apache::lonnet::reply('dump:'.$sdom.':'.$sname.':'.$cid,$shome );
     my %result = ();
     my $ResId;
+    my $PrOrd;
     my $Code;
     my $Tries;
     my $TotalTries = 0;
@@ -117,12 +116,12 @@ sub StudentReport {
 	      "\n".'<th> Tries </th>'.
 	      "\n".'</tr>';
     unless ($reply=~/^error\:/) {
-        map {
+        foreach (split(/\&/,$reply)){
             my ($name,$value)=split(/\=/,&Apache::lonnet::unescape($_));
             $result{$name}=$value;
-        } split(/\&/,$reply);;
-        foreach $ResId (@cols) {
-	    if ( !$ResId ) {
+        }
+        foreach my $CurCol (@cols) {
+	    if (!$CurCol){
 		my $Set=&Apache::lonnet::declutter($hash{'map_id_'.$1});
 		if ( $Set ) {
 		    $SetNo++;
@@ -137,6 +136,7 @@ sub StudentReport {
 		$PtrCod='';
 		next; 
 	    }
+	    ($PrOrd,$ResId)=split(/\:/,$CurCol);
             $ResId=~/(\d+)\.(\d+)/;
             my $Map = &Apache::lonnet::declutter( $hash{'map_id_'.$1} );
             if ( $CurMap ne 'All Maps' ) {
@@ -146,17 +146,18 @@ sub StudentReport {
 	    my $meta=$hash{'src_'.$ResId};
 	    my $PartNo = 0;
 	    undef %TempHash;
-	    map {
+	    foreach (split(/\,/,&Apache::lonnet::metadata($meta,'keys'))){
 		if ($_=~/^stores\_(\d+)\_tries$/) {
                     my $Part=&Apache::lonnet::metadata($meta,$_.'.part');
 		    if ( $TempHash{"$Part"} eq '' ) { 
 			$TempHash{"$Part"} = $Part;
 			$TempHash{$PartNo}=$Part;
 			$TempHash{"$Part.Code"} = '-';  
+			$TempHash{"$Part.PrOrd"} = $PrOrd+$PartNo;  
 			$PartNo++;
 		    }
 		}
-            } split(/\,/,&Apache::lonnet::metadata($meta,'keys'));
+            }
 
             my $Prob = $Map.'___'.$2.'___'.
                        &Apache::lonnet::declutter( $hash{'src_'.$ResId} );
@@ -210,7 +211,31 @@ sub StudentReport {
     $r->rflush();
 }
 
+sub CreateTable {
+    my ($Hd, $Hid)=@_;
+    if ( $Hd == 1 ) {
+#	$Hid=~/(\d+)\.(\d+)/;
+	$r->print('<br><a href="'.$hash{'src_'.$Hid}.
+                  '" target="_blank">'.$hash{'title_'.$Hid}.'</a>'
+#                  .' ('.&Apache::lonnet::declutter($hash{'map_id_'.$1}).')'
+                 );
+    }
+    my $Result = "\n".'<table border=2>';
+    $Result .= '<tr><th>P#</th>'."\n";
+    for ( my $nIdx=0; $nIdx < 11; $nIdx++ ) { 
+	$Result .= '<th>'.'<input type="submit" name="sort" value="'.
+                   $Header{$nIdx}.'" />'.'</th>'."\n";
+    }
+    $Result .= "\n".'</tr>'."\n";    
+    $r->print( $Result );
+    $r->rflush();
+}
 
+sub CloseTable {
+    $r->print("\n".'</table>'."\n");
+    $r->rflush();
+}
+ 
 # ------------------------------------------- Prepare Statistics Table
 sub PreStatTable {
     my $CacheDB = "/home/httpd/perl/tmp/$ENV{'user.name'}".
@@ -235,33 +260,24 @@ sub PreStatTable {
     '  #Stdnts: Total Number of Students opened the problem.<br>'. 
     '  Tries  : Total Number of Tries for solving the problem.<br>'. 
     '  Mod    : Maximunm Number of Tries for solving the problem.<br>'. 
-    '  Mean   : Average Number of the tries. [ Tries / #Stdns ]<br>'.
+    '  Mean   : Average Number of the tries. [ Tries / #Stdnts ]<br>'.
     '  #YES   : Number of students solved the problem correctly.<br>'. 
     '  #yes   : Number of students solved the problem by override.<br>'.
     '  %Wrng  : Percentage of students tried to solve the problem but'.
     ' still incorrect. [ 100*((#Stdnts-(#YES+#yes))/#Stdnts) ]<br>'.
+#    '  DoDiff : Degree of Difficulty of the problem. [ Tries/(#YES+#yes+0.1) ]<br>'.
+    '  DoDiff : Degree of Difficulty of the problem. [ 1 - ((#YES+#yes) / Tries) ]<br>'.
     '  S.D.   : Standard Deviation of the tries.'.
     '[ sqrt(sum((Xi - Mean)^2)) / (#Stdnts-1)'.
-    ' where Xi is every student\'s tries ]<br>'.
+    ' where Xi denotes every student\'s tries ]<br>'.
     '  Skew.  : Skewness of the students tries.'.
     ' [ (sqrt( sum((Xi - Mean)^3) / #Stdnts)) / (S.D.^3) ]<br>'.
-#    '  DoDiff : Degree of Difficulty of the problem. [ Tries/(#YES+#yes+0.1) ]<br>'.
-    '  DoDiff : Degree of Difficulty of the problem. [ 1 - ((#YES+#yes) / Tries) ]<br>'.
+
 #    '  Dis.F. : Discrimination Factor. [ Sum of Partial Credits Awarded / Total Number of Tries in %27 upper and lower students]</b>'.	
             '</pre>';
 
     $r->print($Ptr);
     $r->rflush();	
-		      
-    my $Result = "\n".'<table border=2>';
-    $Result .= '<tr><th>P#</th>'."\n";
-    for ( my $nIdx=0; $nIdx < 12; $nIdx++ ) { 
-	$Result .= '<th>'.'<input type="submit" name="sort" value="'.
-                   $Header{$nIdx}.'" />'.'</th>'."\n";
-    }
-    $Result .= "\n".'</tr>'."\n";    
-    $r->print( $Result );
-    $r->rflush();
 
     if ((-e "$CacheDB")&&($ENV{'form.sort'} ne 'Recalculate Statistics')) {
 	if (tie(%CachData,'GDBM_File',"$CacheDB",&GDBM_READER,0640)) {
@@ -289,10 +305,7 @@ sub PreStatTable {
 
     untie(%CachData);
     untie(%GraphDat);
-    untie(%DiscFac);
-
-    $r->print("\n".'</table>'."\n");
-    $r->rflush();        
+    untie(%DiscFac);       
 }
 
 
@@ -329,6 +342,7 @@ sub ExtractStudentData {
     my $reply=&Apache::lonnet::reply('dump:'.$sdom.':'.$sname.':'.$coid,$shome );
     my %result = ();
     my $ResId;
+    my $PrOrd;
     my $Dis = '';
     my $Code;
     my $Tries;
@@ -346,15 +360,17 @@ sub ExtractStudentData {
     my $SecLimit;
     my $MapLimit;
     unless ($reply=~/^error\:/) {
-        map {
+        foreach (split(/\&/,$reply)) {
             my ($name,$value)=split(/\=/,&Apache::lonnet::unescape($_));
             $result{$name}=$value;
-#$r->print($name.'='.$value.'<br>');
-        } split(/\&/,$reply);
-        foreach $ResId (@cols) {
-	    if ( !$ResId ) { next; }
+        }
+        foreach my $CurCol(@cols) {
+	    ($PrOrd,$ResId)=split(/\:/,$CurCol);
+	    if ( !$CurCol ) { next; }
             $ResId=~/(\d+)\.(\d+)/;
-            my $Map = &Apache::lonnet::declutter( $hash{'map_id_'.$1} );
+	    my $MapId=$1;
+	    my $PrbId=$2;
+            my $Map = &Apache::lonnet::declutter( $hash{'map_id_'.$MapId} );
             if ( $CurMap ne 'All Maps' ) {
 		my ( $ResMap, $NameMap ) = split(/\=/,$CurMap);
 		if ( $Map ne $ResMap ) { next; }
@@ -363,19 +379,20 @@ sub ExtractStudentData {
 	    my $PartNo = 0;
 	    $Dis .= ':';
 	    undef %TempHash;
-	    map {
+	    foreach (split(/\,/,&Apache::lonnet::metadata($meta,'keys'))) {
 		if ($_=~/^stores\_(\d+)\_tries$/) {
                     my $Part=&Apache::lonnet::metadata($meta,$_.'.part');
 		    if ( $TempHash{"$Part"} eq '' ) { 
 			$TempHash{"$Part"} = $Part;
 			$TempHash{$PartNo}=$Part;
 			$TempHash{"$Part.Code"} = 'U';  
+			$TempHash{"$Part.PrOrd"} = $PrOrd+$PartNo;
 			$PartNo++;
 		    }
 		}
-            } split(/\,/,&Apache::lonnet::metadata($meta,'keys'));
+            }
 
-            my $Prob = $Map.'___'.$2.'___'.
+            my $Prob = $Map.'___'.$PrbId.'___'.
                        &Apache::lonnet::declutter( $hash{'src_'.$ResId} );
             $Code='U';
             $Tries = 0;
@@ -419,9 +436,10 @@ sub ExtractStudentData {
                     if ( $TempHash{$part.'.Code'} eq 'C' ||
                          $TempHash{$part.'.Code'} eq 'O'  ) 
 		       {$ProbSolved++;$Yes=1;}		    
-		    my $ptr = "$hash{'title_'.$ResId}";
+#		    my $ptr = "$hash{'title_'.$ResId}";
+		    my $ptr = $TempHash{$part.'.PrOrd'}.':'.$ResId;
 		    if ( $PartNo > 1 ) {                
-			$ptr .= " (part $part)";
+			$ptr .= "*(part $part)";
 			$Dis .= ':';
 		    }
 		    my $Fac = ($TempHash{"$part.Tries"}) ? 
@@ -434,21 +452,20 @@ sub ExtractStudentData {
 #		    $DisF .= '+'.$TempHash{"$part.Time"};
 		    $TimeTot += $TempHash{"$part.Time"};
 		    $Dis .= $ptr.'*'.$ResId.'='.$DisF.'+'.$Yes;
-		    $ptr .= "*$ResId:$TempHash{$part.'.Tries'}".
+		    $ptr .= ":$TempHash{$part.'.Tries'}".
 		            ":$TempHash{$part.'.Wrongs'}".
-                            ":$TempHash{$part.'.Code'}";		    
-#$r->print($sname.' -- '.$ptr.'--- timestamp='.$TempHash{"$part.Time"}.'<br>');
+                            ":$TempHash{$part.'.Code'}";
 		    push (@list, $ptr);
 		    $TotalOpend++;
 		    $ProbTot++;
 		}
             }
-	    else { 
-		for(my $n=0; $n<$PartNo; $n++) {
-		    push (@list, "$hash{'title_'.$ResId}*$ResId:0:0:U");
-		    $ProbTot++; 
-		}
-	    }
+	    #else { 
+		#for(my $n=0; $n<$PartNo; $n++) {
+		#    push (@list, "$hash{'title_'.$ResId}*$ResId:0:0:U");
+		#    $ProbTot++; 
+		#}
+	    #}
         }
 	if ( $TotalTries ) {
 	    my $DisFac = ( $TotalTries ) ? ($TotParCr/$TotalTries) : 0;
@@ -456,7 +473,7 @@ sub ExtractStudentData {
 	    my $DisFactor = sprintf( "%.4f", $DisFac );
 	    my $time;
 	    if ($ProbSolved){
-		$time = int(($TimeTot/$ProbSolved)/10000000);
+		$time = int(($TimeTot/$ProbSolved)-10000000);
 	    }
 	    $DiscFac{($DisFactor.':'.$sname.':'.$ProbTot.':'.$TotalOpend.':'.
                       $TotalTries.':'.$ProbSolved.':'.$time)}=$Dis;
@@ -472,45 +489,65 @@ sub tracetable {
     my ($rid,$beenhere)=@_;
     $rid=~/(\d+)\.(\d+)/;
     $maps{&Apache::lonnet::declutter($hash{'map_id_'.$1})}='';#$hash{'title_'.$rid}; 
+    #$maps{$HWN}=$hash{'title_'.$rid}; 
     unless ($beenhere=~/\&$rid\&/) {
        $beenhere.=$rid.'&'; 
        if (defined($hash{'is_map_'.$rid})) {
 	   my $cmap=$hash{'map_type_'.$hash{'map_pc_'.$hash{'src_'.$rid}}};
            if ( $cmap eq 'sequence' || $cmap eq 'page' ) { 
                $cols[$#cols+1]=0;
-	       $HWN++;
-               $mapsort[$HWN]=$rid.$hash{'title_'.$rid}; 
+	       $P_Order++;
+	       $HWN=$P_Order;
+               $mapsort{$HWN} = $rid.':'; 
                #$maps{&Apache::lonnet::declutter($hash{'src_'.$rid})}= 
                #      $hash{'title_'.$rid}; 
            }
            if ((defined($hash{'map_start_'.$hash{'src_'.$rid}})) &&
                (defined($hash{'map_finish_'.$hash{'src_'.$rid}}))) {
-              my $frid=$hash{'map_finish_'.$hash{'src_'.$rid}};
+	       my $frid=$hash{'map_finish_'.$hash{'src_'.$rid}};
 
-                &tracetable($hash{'map_start_'.$hash{'src_'.$rid}},
-                '&'.$frid.'&');
+               &tracetable($hash{'map_start_'.$hash{'src_'.$rid}},
+			   '&'.$frid.'&');
 
-              if ($hash{'src_'.$frid}) {
-                 if ($hash{'src_'.$frid}=~
-                                 /\.(problem|exam|quiz|assess|survey|form)$/) {
-		     $cols[$#cols+1]=$frid;
-		     $mapsort[$HWN] .= '&'.$frid;
-                 }
-	      }
+	       $cols[$#cols+1]=($P_Order+1).':'.$frid;
+      
+	       my $meta=$hash{'src_'.$frid};
+	       my $PartNo = 0;
+               if ($meta) {
+		   if ($meta=~/\.(problem|exam|quiz|assess|survey|form)$/) {
+		       foreach (split(/\,/,&Apache::lonnet::metadata($meta,'keys'))) {
+			   if ($_=~/^stores\_(\d+)\_tries$/) {
+			       &Apache::lonnet::metadata($meta,$_.'.part');
+			       $P_Order++;
+			       $mapsort{$HWN} .= '&'.$P_Order;
+			       $PartNo++;
+			       $r->print('<br>'.$PartNo.'---'.$P_Order);
+			   }
+		       }
+		   }
+               }
 	   }
        } else {
-          if ($hash{'src_'.$rid}) {
-             if ($hash{'src_'.$rid}=~
-                                 /\.(problem|exam|quiz|assess|survey|form)$/) {
-	         $cols[$#cols+1]=$rid;
-		 $mapsort[$HWN] .= '&'.$rid;
-             }
-          }
+	   $cols[$#cols+1]=($P_Order+1).':'.$rid;
+	   my $meta=$hash{'src_'.$rid};
+	   my $PartNo = 0;
+           if ($meta) {
+	       if ($meta=~/\.(problem|exam|quiz|assess|survey|form)$/) {
+		   foreach (split(/\,/,&Apache::lonnet::metadata($meta,'keys'))) {
+		       if ($_=~/^stores\_(\d+)\_tries$/) {
+			   &Apache::lonnet::metadata($meta,$_.'.part');
+			   $P_Order++;
+			   $mapsort{$HWN} .= '&'.$P_Order;
+			   $PartNo++;
+		       }
+		   }
+	       }
+	   }
        }
        if (defined($hash{'to_'.$rid})) {
-          map {
+          foreach (split(/\,/,$hash{'to_'.$rid})){
               &tracetable($hash{'goesto_'.$_},$beenhere);
-          } split(/\,/,$hash{'to_'.$rid});
+          }
        }
     }
 }
@@ -528,16 +565,16 @@ sub MySort {
 
 sub Build_Statistics {
     $r->print(<<ENDPOP);
-<script>
+    <script>
     popwin=open('','popwin','width=400,height=100');
-    popwin.document.writeln('<html><body bgcolor="#8cee8c">'+
+    popwin.document.writeln('<html><body bgcolor="#7777EE">'+
       '<title>LON-CAPA Statistics</title>'+
       '<h4>Computation Progress</h4>'+
       '<form name=popremain>'+
       '<input type=text size=35 name=remaining value=Starting></form>'+
       '</body></html>');
     popwin.document.close();
-</script>
+    </script>
 ENDPOP
 
     $r->rflush();
@@ -556,7 +593,8 @@ ENDPOP
     $r->print('<script>popwin.close()</script>');
     $r->rflush(); 
 # -------------------- sorting the Data
-    @list = sort(@list);
+    @list = sort (@list);
+
     $OpSel2='';
     $OpSel1='selected';
  		   
@@ -567,137 +605,126 @@ ENDPOP
     my $ResId;
     my $NoElements = $#list + 1;
 #-------------------------------- loop for data representation
-    while ( $nIdx < $NoElements ) {
-        my %storestats=();
-	my ($Prob,$Tries,$Wrongs,$Code)=split(/\:/,$list[$nIdx]);
-	my $Temp = $Prob;
-	my $MxTries = 0;
-	my $TotalTries = 0;
-	my $YES = 0;
-	my $Incorrect = 0;
-	my $Override = 0;
-	my $StdNo = 0;
-	my @StdLst; 
-	do {
-	    $nIdx++;
-	    $StdNo++;
-	    $StdLst[ $StdNo ] = $Tries;
-	    $TotalTries += $Tries;
-	    if ( $MxTries < $Tries ) { $MxTries = $Tries; } 
-	    if ( $Code eq 'C' ){ $YES++; }
-	    elsif( $Code eq 'I' ) { $Incorrect++; }
-	    elsif( $Code eq 'O' ) { $Override++; }
-	    elsif( $Code eq 'U' ) { $StdNo--; }
-	    ($Prob,$Tries,$Wrongs,$Code)=split(/\:/,$list[$nIdx]);
-	} while ( $Prob eq $Temp && $nIdx < $NoElements );	
-
-	$p_count++;
-
-	($Temp,$ResId)=split(/\*/,$Temp);
+    foreach (sort keys %mapsort) {
+	my ($Hid,$pr)=split(/\:/,$mapsort{$_});
+	my @lpr=split(/\&/,$pr);
+	&CreateTable(1,$Hid);
+	for (my $i=1; $i<=$#lpr; $i++) {
+	    my %storestats=();
+	    my ($PrOrd,$Prob,$Tries,$Wrongs,$Code)=split(/\:/,$list[$nIdx]);
+	    my $Temp = $Prob;
+	    my $MxTries = 0;
+	    my $TotalTries = 0;
+	    my $YES = 0;
+	    my $Incorrect = 0;
+	    my $Override = 0;
+	    my $StdNo = 0;
+	    my @StdLst;
+	    while ( $PrOrd == $lpr[$i] ) 
+	    {
+		$nIdx++;
+		$StdNo++;
+		$StdLst[ $StdNo ] = $Tries;
+		$TotalTries += $Tries;
+		if ( $MxTries < $Tries ) { $MxTries = $Tries; } 
+		if ( $Code eq 'C' ){ $YES++; }
+		elsif( $Code eq 'I' ) { $Incorrect++; }
+		elsif( $Code eq 'O' ) { $Override++; }
+		elsif( $Code eq 'U' ) { $StdNo--; }
+		($PrOrd,$Prob,$Tries,$Wrongs,$Code)=split(/\:/,$list[$nIdx]);
+	    }	
+
+	    $p_count++;
+	    my $Dummy;
+	    ($ResId,$Dummy)=split(/\*/,$Temp);
 
-        $Temp = '<a href="'.$hash{'src_'.$ResId}.'" target="_blank">'.$Temp.'</a>';
+	    $Temp = '<a href="'.$hash{'src_'.$ResId}.
+                '" target="_blank">'.$hash{'title_'.$ResId}.$Dummy.'</a>';
     
-	my $res = &Apache::lonnet::declutter($hash{'src_'.$ResId});
-        my $urlres=$res;
+	    my $res = &Apache::lonnet::declutter($hash{'src_'.$ResId});
+	    my $urlres=$res;
 
-        $ResId=~/(\d+)\.(\d+)/;
-        my $Map = &Apache::lonnet::declutter( $hash{'map_id_'.$1} );
-        $urlres=$Map;
+	    $ResId=~/(\d+)\.(\d+)/;
+	    my $Map = &Apache::lonnet::declutter( $hash{'map_id_'.$1} );
+	    $urlres=$Map;
  
-
-        $res = '<a href="'.$hash{'src_'.$ResId}.'">'.$res.'</a>';
-
-        #$Map = '<a href="'.$Map.'">'.$res.'</a>';
+	    $res = '<a href="'.$hash{'src_'.$ResId}.'">'.$res.'</a>';
+	    #$Map = '<a href="'.$Map.'">'.$res.'</a>';
 
 #------------------------ Compute the Average of Tries about one problem
-	my $Average = ($StdNo) ? $TotalTries/$StdNo : 0;
+	    my $Average = ($StdNo) ? $TotalTries/$StdNo : 0;
 
-        $storestats{$ENV{'request.course.id'}.'___'.$urlres.'___timestamp'}=time;       
-        $storestats{$ENV{'request.course.id'}.'___'.$urlres.'___stdno'}=$StdNo;
-        $storestats{$ENV{'request.course.id'}.'___'.$urlres.'___avetries'}=$Average;
+	    $storestats{$ENV{'request.course.id'}.'___'.$urlres.'___timestamp'}=time;       
+	    $storestats{$ENV{'request.course.id'}.'___'.$urlres.'___stdno'}=$StdNo;
+	    $storestats{$ENV{'request.course.id'}.'___'.$urlres.'___avetries'}=$Average;
    
 #-------------------------------- Compute percentage of Wrong tries
-	my $Wrong = ( $StdNo ) ? 100 * ( $Incorrect / $StdNo ) : 0;
+	    my $Wrong = ( $StdNo ) ? 100 * ( $Incorrect / $StdNo ) : 0;
 
 #-------------------------------- Compute Standard Deviation
-	my $StdDev = 0; 
-	if ( $StdNo > 1 ) {
-	    for ( my $n = 0; $n < $StdNo; $n++ ) {
-                my $Dif = $StdLst[ $n ]-$Average;
-		$StdDev += $Dif*$Dif;
-	    } 
-	    $StdDev /= ( $StdNo - 1 );
-	    $StdDev = sqrt( $StdDev );
-	}
+	    my $StdDev = 0; 
+	    if ( $StdNo > 1 ) {
+		for ( my $n = 0; $n < $StdNo; $n++ ) {
+		    my $Dif = $StdLst[ $n ]-$Average;
+		    $StdDev += $Dif*$Dif;
+		} 
+		$StdDev /= ( $StdNo - 1 );
+		$StdDev = sqrt( $StdDev );
+	    }
 
 #-------------------------------- Compute Degree of Difficulty
-	my $DoDiff = 0;
-	if( $TotalTries > 0 ) {
-	    $DoDiff = 1 - ( ( $YES + $Override ) / $TotalTries );
+	    my $DoDiff = 0;
+	    if( $TotalTries > 0 ) {
+		$DoDiff = 1 - ( ( $YES + $Override ) / $TotalTries );
 #	    $DoDiff =  ($TotalTries)/($YES + $Override+ 0.1);	    
-	}
+	    }
        
-        $storestats{$ENV{'request.course.id'}.'___'.$urlres.'___difficulty'}=$DoDiff;
+	    $storestats{$ENV{'request.course.id'}.'___'.$urlres.'___difficulty'}=$DoDiff;
 
 #-------------------------------- Compute the Skewness
-	my $Skewness = 0;
-        my $Sum = 0; 
-	if ( $StdNo > 0 && $StdDev > 0 ) {
-	    for ( my $n = 0; $n < $StdNo; $n++ ) {
-                my $Dif = $StdLst[ $n ]-$Average;
-		$Skewness += $Dif*$Dif*$Dif;
-	    } 
-	    $Skewness /= $StdNo;
+	    my $Skewness = 0;
+	    my $Sum = 0; 
+	    if ( $StdNo > 0 && $StdDev > 0 ) {
+		for ( my $n = 0; $n < $StdNo; $n++ ) {
+		    my $Dif = $StdLst[ $n ]-$Average;
+		    $Skewness += $Dif*$Dif*$Dif;
+		} 
+		$Skewness /= $StdNo;
 	    $Skewness /= $StdDev*$StdDev*$StdDev;
-	}
+	    }
 #-----------------  Some restition in presenting the float numbers
-	my $Avg = sprintf( "%.2f", $Average );
-	my $Wrng = sprintf( "%.1f", $Wrong );
-	my $SD = sprintf( "%.1f", $StdDev );
-	my $DoD = sprintf( "%.2f", $DoDiff );
-	my $Sk = sprintf( "%.1f", $Skewness );
-    
-	$CachData{($p_count-1)}=$Temp.':'.$StdNo.':'.$TotalTries.':'.
-                                $MxTries.':'.$Avg.':'.$YES.':'.
-                                $Override.':'.$Wrng.':'.$SD.':'.
-                                $Sk.':'.$DoD.':'.$Map.':'.$Prob;
-
-        $urlres=~/^(\w+)\/(\w+)/;
-        if ($StdNo) { 
-           &Apache::lonnet::put('resevaldata',\%storestats,$1,$2); 
-        }
-
+	    my $Avg = sprintf( "%.2f", $Average );
+	    my $Wrng = sprintf( "%.1f", $Wrong );
+	    my $SD = sprintf( "%.1f", $StdDev );
+	    my $DoD = sprintf( "%.2f", $DoDiff );
+	    my $Sk = sprintf( "%.1f", $Skewness );
+	    my $join = $PrOrd.':'.$Temp.':'.$StdNo.':'.
+                       $TotalTries.':'.$MxTries.':'.$Avg.':'.
+                       $YES.':'.$Override.':'.$Wrng.':'.$DoD.':'.
+		       $SD.':'.$Sk.':'.$Map.':'.$Prob;
+	    $CachData{($p_count-1)}=$join;
+
+	    $urlres=~/^(\w+)\/(\w+)/;
+	    if ($StdNo) { 
+		&Apache::lonnet::put('resevaldata',\%storestats,$1,$2); 
+	    }
 #-------------------------------- Row of statistical table
-        if ( $DiscFlag == 0 ) {
-	    $r->print( "\n".'<tr>'.
-               "\n".'<td>'.$p_count.'</td>'.
-               "\n".'<td>'.$Temp.'</td>'.
-               "\n".'<td>'.$StdNo.'</td>'.
-               "\n".'<td>'.$TotalTries.'</td>'.
-               "\n".'<td>'.$MxTries.'</td>'.
-               "\n".'<td>'.$Avg.'</td>'.
-               "\n".'<td>'.$YES.'</td>'.
-               "\n".'<td>'.$Override.'</td>'.
-               "\n".'<td>'.$Wrng.'</td>'.
-               "\n".'<td>'.$SD.'</td>'.
-               "\n".'<td>'.$Sk.'</td>'.
-               "\n".'<td>'.$DoD.'</td>'.
-               "\n".'<td>'.$Map.'</td>'.
-                       "\n".'</tr>' );
-	    $GraphDat{$nIdx}=$DoD.':'.$Wrng;
-	} 
+	    if ( $DiscFlag == 0 ) {
+		&TableRow($join,$i,($p_count-1));
+	    } 
+	}
+	&CloseTable();
     }
 }
 
-
 sub Cache_Statistics {
     my @list = ();
     my $Useful;
     my $UnUseful;
     my %myHeader = reverse( %Header );
     $Pos = $myHeader{$ENV{'form.sort'}};
+    if ($Pos > 0) {$Pos++;}
     $p_count = 0;
-
     foreach my $key( keys %CachData) { 
 	my @Temp=split(/\:/,$CachData{$key});
 	if ( $Pos == 0 || $Pos == 11 ) {
@@ -712,27 +739,51 @@ sub Cache_Statistics {
 
     @list = sort MySort (@list);
 
-    for ( my $nIdx = 0; $nIdx < $p_count; $nIdx++ ) {
-	my( $Pre, $Post ) = split(/\&/,$list[$nIdx]); 
-	my ($Temp,$StdNo,$TotalTries,$MxTries,$Avg,$YES,
-            $Override,$Wrng,$SD,$Sk,$DoD,$res,$Prob)=split(/\:/,$Post);	
-	$r->print( "\n".'<tr>'.
-               "\n".'<td>'.($nIdx+1).'</td>'.
-               "\n".'<td  bgcolor="#FFFFFF">'.$Temp.'</td>'.
-               "\n".'<td>'.$StdNo.'</td>'.
-               "\n".'<td>'.$TotalTries.'</td>'.
-               "\n".'<td>'.$MxTries.'</td>'.
-               "\n".'<td>'.$Avg.'</td>'.
-               "\n".'<td>'.$YES.'</td>'.
-               "\n".'<td>'.$Override.'</td>'.
-               "\n".'<td>'.$Wrng.'</td>'.
-               "\n".'<td>'.$SD.'</td>'.
-               "\n".'<td>'.$Sk.'</td>'.
-               "\n".'<td>'.$DoD.'</td>'.
-               "\n".'<td>'.$res.'</td>'.
+    my $nIdx=0;
+
+    if ( $Pos == 0 ) {
+	foreach (sort keys %mapsort) {
+	    my ($Hid,$pr)=split(/\:/,$mapsort{$_});
+	    &CreateTable(1,$Hid);
+	    my @lpr=split(/\&/,$pr);
+	    for (my $i=1; $i<=$#lpr; $i++) {
+		my($Pre, $Post) = split(/\&/,$list[$nIdx]); 
+		&TableRow($Post,$i,$nIdx);
+		$nIdx++;
+	    }
+	    &CloseTable();
+	}
+    }
+    else {
+	&CreateTable(0);
+	for ( my $nIdx = 0; $nIdx < $p_count; $nIdx++ ) {
+	    my($Pre, $Post) = split(/\&/,$list[$nIdx]); 
+	    &TableRow($Post,$nIdx,$nIdx);
+	} 
+	&CloseTable();
+    }
+}
+
+sub TableRow {
+    my ($Str,$Idx,$RealIdx)=@_;
+    my($PrOrd,$Temp,$StdNo,$TotalTries,$MxTries,$Avg,$YES,
+       $Override,$Wrng,$DoD,$SD,$Sk,$res,$Prob)=split(/\:/,$Str);	
+    $r->print( "\n".'<tr>'.
+               "\n".'<td>'.($RealIdx+1).'</td>'.
+               "\n".'<td bgcolor="#FFFFFF">'.$Temp.'</td>'.
+               "\n".'<td bgcolor="#EEFFDD"> '.$StdNo.'</td>'.
+               "\n".'<td bgcolor="#EEFFDD">'.$TotalTries.'</td>'.
+               "\n".'<td bgcolor="#EEFFDD">'.$MxTries.'</td>'.
+               "\n".'<td bgcolor="#DDFFFF">'.$Avg.'</td>'.
+               "\n".'<td bgcolor="#DDFFFF"> '.$YES.'</td>'.
+               "\n".'<td bgcolor="#DDFFFF"> '.$Override.'</td>'.
+               "\n".'<td bgcolor="#FFDDDD"> '.$Wrng.'</td>'.
+               "\n".'<td bgcolor="#FFDDDD">'.$DoD.'</td>'.
+               "\n".'<td bgcolor="#DDFFDD"> '.$SD.'</td>'.
+               "\n".'<td bgcolor="#DDFFDD"> '.$Sk.'</td>'.
+#               "\n".'<td>'.$res.'</td>'.
                "\n".'</tr>' );
-	$GraphDat{$nIdx}=$DoD.':'.$Wrng;
-    } 
+    $GraphDat{$RealIdx}=$DoD.':'.$Wrng;
 }
 
 # ------------------------------------------- Prepare data for Graphical chart
@@ -794,6 +845,9 @@ sub initial {
   undef $p_count;
   undef $Pos;
   undef $GData;
+  $DiscFlag=0; 
+  $P_Order=100000;
+  $HWN=$P_Order;
 }
 
 
@@ -839,10 +893,14 @@ sub ClassList {
 # --------------- Find all assessments and put them into some linear-like order
 	&tracetable($firstres,'&'.$lastres.'&');
 
-#my $c=0;
-#foreach(@mapsort) {
+#    my $c=0;
+#    foreach (sort keys %mapsort) {
+#	$r->print('<br>'.$c.')'.$_.' ... '.$mapsort{$_});
+#	$c++;
+#    }
+#foreach(@cols) {
 #    $c++;
-#    $r->print('<br>'.$mapsort[$c]);
+#    $r->print('<br>'.$cols[$c]);
 #}
 #$r->print('<br> Count = '.$c);
 
@@ -952,6 +1010,14 @@ sub GetStatus {
 
     if ( $ENV{'form.order'} eq 'Descending' ) { $OpSel2='selected'; }
     else { $OpSel1 = 'selected'; }
+
+    my %myHeader = reverse( %Header );
+    $Pos = $myHeader{$ENV{'form.sort'}};
+    if ($Pos == 0) {
+	$OpSel1 = 'selected';
+	$ENV{'form.order'}='Ascendig';
+    }
+
     $CurMap = $ENV{'form.maps'};
     if ( $CurMap eq '' ) { 
 	$CurMap = 'All Maps';