--- loncom/interface/lonstatistics.pm	2002/02/06 16:59:01	1.4
+++ loncom/interface/lonstatistics.pm	2002/03/02 23:22:32	1.6
@@ -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.6 2002/03/02 23:22:32 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, 3/2 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 Sets Order",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'}".
@@ -219,9 +244,22 @@ sub PreStatTable {
                   "_$ENV{'user.domain'}_$cid\_graph.db";
     my $CachDisFac = "/home/httpd/perl/tmp/$ENV{'user.name'}".
 		     "_$ENV{'user.domain'}_$cid\_DiscFactor.db";
-    $r->print('<br><input type="submit" name="sort" value="Recalculate Statistics" />');
+    my $OpSel11='';
+    my $OpSel12='';
+    my $OpSel13='';
+    my $Status = $ENV{'form.status'};
+    if ( $Status eq 'Any' ) { $OpSel13='selected'; }
+    elsif ($Status eq 'Expired' ) { $OpSel12 = 'selected'; }
+    else { $OpSel11 = 'selected'; }
 
     my $Ptr = '';
+    $Ptr .= '<br><b> Student Status: &nbsp; </b>'."\n".
+            '<select name="status">'. 
+            '<option '.$OpSel11.' >Active</option>'."\n".
+            '<option '.$OpSel12.' >Expired</option>'."\n".
+	    '<option '.$OpSel13.' >Any</option> </select> '."\n";
+    $Ptr .= '&nbsp;&nbsp;&nbsp;';
+    $Ptr .= '<input type=submit name=sort value="Recalculate Statistics"/>'."\n";
 
     $Ptr .= '<br><b> Sorting Type: &nbsp; </b>'."\n".
             '<select name="order"> <option '.$OpSel1.' >Ascending</option>'."\n".
@@ -235,33 +273,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,34 +318,44 @@ sub PreStatTable {
 
     untie(%CachData);
     untie(%GraphDat);
-    untie(%DiscFac);
-
-    $r->print("\n".'</table>'."\n");
-    $r->rflush();        
+    untie(%DiscFac);       
 }
 
 
 # ------------------------------------- Find the section of student in a course
 
 sub usection {
-    my ($udom,$unam,$courseid)=@_;
+    my ($udom,$unam,$courseid,$ActiveFlag)=@_;
     $courseid=~s/\_/\//g;
     $courseid=~s/^(\w)/\/$1/;
-    map {
+    foreach (split(/\&/,&Apache::lonnet::reply('dump:'.
+             $udom.':'.$unam.':roles',
+             &Apache::lonnet::homeserver($unam,$udom)))){
         my ($key,$value)=split(/\=/,$_);
         $key=&Apache::lonnet::unescape($key);
         if ($key=~/^$courseid(?:\/)*(\w+)*\_st$/) {
             my $section=$1;
             if ($key eq $courseid.'_st') { $section=''; }
 	    my ($dummy,$end,$start)=split(/\_/,&Apache::lonnet::unescape($value));
-            $section=($section) ? $section : '(none)';
-#            $section=(int($section)) ? int($section) : $section;
-#            $r->print($unam.'...'.$section.'<br>');
+	    if ( $ActiveFlag ne 'Any' ) {
+		my $now=time;
+		my $notactive=0;
+		if ($start) {
+		    if ($now<$start) { $notactive=1; }
+		}
+		if ($end) {
+		    if ($now>$end) { $notactive=1; }
+		}
+		if ((($ActiveFlag eq 'Expired') && $notactive == 1) || 
+                    (($ActiveFlag eq 'Active') && $notactive == 0 ) ) {
+		    return $section;
+		}
+		else { return '-1'; } 
+	    }
 	    return $section;
         }
-    } split(/\&/,&Apache::lonnet::reply('dump:'.$udom.':'.$unam.':roles',
-                        &Apache::lonnet::homeserver($unam,$udom)));
-    return '';
+    }
+    return '-1';
 }
 
 
@@ -329,6 +368,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 +386,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 +405,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 +462,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 +478,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 +499,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,51 +515,71 @@ 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});
+          }
        }
     }
 }
 
 sub MySort {          
-    if ( $Pos > 0 && $Pos < 11 ) {
+    if ( $Pos > 0 ) {
 	if ($ENV{'form.order'} eq 'Descending') {$b <=> $a;}
 	else { $a <=> $b; }
     }
@@ -528,16 +591,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="#88CC88">'+
       '<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 +619,8 @@ ENDPOP
     $r->print('<script>popwin.close()</script>');
     $r->rflush(); 
 # -------------------- sorting the Data
-    @list = sort(@list);
+    @list = sort (@list);
+
     $OpSel2='';
     $OpSel1='selected';
  		   
@@ -567,140 +631,129 @@ 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++;
+    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,$ResId)=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.':'.$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 ) {
+	if ( $Pos == 0 ) {
 	    ($UnUseful,$Useful)=split(/\>/,$Temp[$Pos]);
 	}
 	else {
@@ -712,27 +765,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,$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 +871,9 @@ sub initial {
   undef $p_count;
   undef $Pos;
   undef $GData;
+  $DiscFlag=0; 
+  $P_Order=100000;
+  $HWN=$P_Order;
 }
 
 
@@ -812,23 +892,42 @@ sub ClassList {
 	my $classlst=&Apache::lonnet::reply
                  ('dump:'.$cdom.':'.$cnum.':classlist',$chome);
 	my $StudNo = 0;
+	my $now=time;
 	unless ($classlst=~/^error\:/) {
 	    foreach (sort split(/\&/,$classlst)) {
 		my ($name,$value)=split(/\=/,$_);
 		my ($end,$start)=split(/\:/,&Apache::lonnet::unescape($value));
 		$name=&Apache::lonnet::unescape($name);
 		my ($sname,$sdom)=split(/\:/,$name);
-		my $ssec=&Apache::lonnet::usection($sdom,$sname,$cid);
-                if ($ssec==-1) {next;}
-		$ssec=($ssec) ? $ssec : '(none)';
-		#$ssec=(int($ssec)) ? int($ssec) : $ssec;
-		#$r->print($sname.'...'.$ssec.'<br>');
-		$section{$ssec}=$ssec;
-		if ($CurSec eq 'All Sections' || $ssec eq $CurSec) {
-		    $students[$StudNo]=$name;
-		    $StuBox{$sname}=$ssec;
-		}
-		$StudNo++;
+		my $active=1;
+		my $Status=$ENV{'form.status'};
+		$Status = ($Status) ? $Status : 'Active';
+		if ( ( ($end) && $now > $end ) && 
+                     ( ($Status eq 'Active') ) ) { $active=0; }
+		if ( ($Status eq 'Expired') && 
+                     ($end == 0 || $now < $end) ) { $active=0; }
+		if ($active) {
+		    my $thisindex=$#students+1;
+		    $name=&Apache::lonnet::unescape($name);
+		    $students[$thisindex]=$name;
+		    my ($sname,$sdom)=split(/\:/,$name);
+
+		    #my %reply=&Apache::lonnet::idrget($sdom,$sname);
+		    #my $reply=&Apache::lonnet::reply('get:'.$sdom.':'.$sname.
+		    #         ':environment:lastname&generation&firstname&middlename',
+		    #         &Apache::lonnet::homeserver($sname,$sdom));
+		    my $ssec=&usection($sdom,$sname,$cid,$Status);
+		    if ($ssec==-1) {next;}
+		    $ssec=($ssec) ? $ssec : '(none)';
+		    #$ssec=(int($ssec)) ? int($ssec) : $ssec;
+		    #$r->print($sname.'...'.$ssec.'<br>');
+		    $section{$ssec}=$ssec;
+		    if ($CurSec eq 'All Sections' || $ssec eq $CurSec) {
+			$students[$StudNo]=$name;
+			$StuBox{$sname}=$ssec;
+		    }
+		    $StudNo++;
+	        }
 	    }
 	}
 	else {
@@ -839,10 +938,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 +1055,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';