--- loncom/interface/lonstatistics.pm	2002/03/06 23:56:13	1.8
+++ loncom/interface/lonstatistics.pm	2002/05/13 16:41:46	1.17
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # (Publication Handler
 #
-# $Id: lonstatistics.pm,v 1.8 2002/03/06 23:56:13 minaeibi Exp $
+# $Id: lonstatistics.pm,v 1.17 2002/05/13 16:41:46 minaeibi Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -27,18 +27,20 @@
 #
 # (Navigate problems for statistical reports
 # YEAR=2001
-# 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
+# 5/5,7/9,7/25/1,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, 2/1, 2/6, 2/25, 3/2, 3/6 Behrouz Minaei
+# 1/22,2/1,2/6,2/25,3/2,3/6,3/17,3/21,3/22,3/26,4/7,5/6,5/12 Behrouz Minaei
+#
 ###
 
 package Apache::lonstatistics; 
 
-use strict;
+use strict; 
 use Apache::Constants qw(:common :http);
 use Apache::lonnet();
 use Apache::lonhomework;
+use Apache::loncommon;
 use HTML::TokeParser;
 use GDBM_File;
 
@@ -46,6 +48,7 @@ use GDBM_File;
 my %hash;
 my %CachData;
 my %GraphDat;
+my %OpResp;
 my %maps;
 my %mapsort;
 my %section;
@@ -65,8 +68,6 @@ my $Pos;
 my $r;
 my $OpSel1;
 my $OpSel2;
-my $OpSelDis1;
-my $OpSelDis2;
 my $OpSel3;
 my $OpSel4;
 my $GData;
@@ -76,9 +77,331 @@ my $lastres;
 my $DiscFlag;
 my $HWN;
 my $P_Order;
+my %foil_to_concept;
+my @Concepts;
+my %ConceptData;
 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,"D.F.1st",12,"D.F.2nd");
+my @shown;
+
+sub InitAnalysis {
+    my ($rid, $student)=@_;
+    my ($uname,$udom)=split(/\:/,$student);
+    $rid=~/(\d+)\.(\d+)/;
+    my $symb=&Apache::lonnet::declutter($hash{'map_id_'.$1}).'___'.$2.'___'.
+	     &Apache::lonnet::declutter($hash{'src_'.$rid});
+    my $URI = $hash{'src_'.$rid};
+    my $Answ=&Apache::lonnet::ssi($URI,('grade_target' => 'analyze',
+                                  'grade_username' => $uname,
+                                  'grade_domain' => $udom,
+                                  'grade_courseid' => $cid,
+                                  'grade_symb' => $symb));
+#    my $Answ=&Apache::lonnet::ssi($URI,('grade_target' => 'analyze'));
+
+    (my $garbage,$Answ)=split(/_HASH_REF__/,$Answ,2);
+    my %Answer=();
+    %Answer=&Apache::lonnet::str2hash($Answ);
+
+    my $parts='';
+    foreach my $elm (@{$Answer{"parts"}}) {
+	$parts.="$elm,";
+    }
+    chop($parts);
+    my $conc='';
+    foreach my $elm (@{$Answer{"$parts.concepts"}}) {
+	$conc.="$elm@";
+    }
+    chop($conc);
+
+    @Concepts=split(/\@/,$conc);
+    my $show='';
+    foreach my $elm (@{$Answer{"$parts.shown"}}) {
+	$show.="$elm@";
+    }
+    chop($show);
+    @shown=split(/\@/,$show);
+#    $r->print("<br> shown:".$show);#    $r->rflush();
+    foreach my $concept (@{$Answer{"$parts.concepts"}}) {
+	foreach my $foil (@{$Answer{"$parts.concept.$concept"}}) {
+	    $foil_to_concept{$foil} = $concept;
+	    $ConceptData{$foil} = $Answer{"$parts.foil.value.$foil"};
+	}
+    }
+    return $symb;
+}
+
+
+sub Interval {
+    my ($rid,$part,$symb)=@_;
+    my $Int=$ConceptData{"Interval"};
+    my $due = &Apache::lonnet::EXT('resource.$part.duedate',$symb)+1;
+    my $opn = &Apache::lonnet::EXT('resource.$part.opendate',$symb);
+    my $add=int(($due-$opn)/$Int);
+#$r->print("<br> $opn, to $due add=$add  int=$Int");#$r->rflush();
+    $ConceptData{"Int.0"}=$opn;
+    for (my $i=1;$i<$Int;$i++) {
+	$ConceptData{"Int.$i"}=$opn+$i*$add;
+    }
+    $ConceptData{"Int.$Int"}=$due;     
+    for (my $i=0;$i<$Int;$i++) {
+	for (my $n=0; $n<=$#Concepts; $n++ ) {
+	    my $tmp=$Concepts[$n];
+	    $ConceptData{"$tmp.$i.true"}=0;
+	    $ConceptData{"$tmp.$i.false"}=0;
+	}
+    }
+}
+
+
+sub ShowOpGraph {
+
+    my ($InpStr, $Int_No)=@_;
+
+    $r->print(<<ENDPOP);
+    <script language="JavaScript">
+
+    function display(name) {
+	document.forms.displayform.elements.dis.value=name;
+	window.status=name;
+    }
+    function cleardisplay() {
+	document.forms.displayform.elements.dis.value='';
+	window.status='No Concept in particular';
+    }
+
+    </script>
+ENDPOP
+
+    my ($rid,$part)=split(/\:/,substr($InpStr,8));
+    $ConceptData{"Interval"}=$Int_No;
+    my $symb=&InitAnalysis($rid,$students[0]);
+    &Interval($rid,$part,$symb);
+    my $URI = $hash{'src_'.$rid};
+    my $Src = $hash{'title_'.$rid};
+    $Src =~ s/\ /"_"/eg;
+    $r->print('<br><b>'.$URI.'</b>');
+    for (my $n=1; $n<=$#Concepts+1; $n++ ) {
+	my $tmp=$Concepts[$n-1];
+	$tmp =~ s/</" less than "/eg;
+	$r->print("<br><b>Concept $n</b>:$tmp");
+    }
+    $r->rflush();
+    
+    &Create_PrgWin();
+    &Update_PrgWin("Starting to analyze problem");
+    for (my $index=0;$index<=$#students;$index++) {
+	&Update_PrgWin($index);
+	&OpStatus($rid,$students[$index]);
+    }
+    &Close_PrgWin();
+
+    for (my $k=0; $k<$Int_No; $k++ ) {
+ 	my $data1=''; 
+	my $data2='';
+	&DrawGraph(&AdjustData($k).'+'.$Src.'+'.($k+1));
+    }
+#$Apache::lonxml::debug=1;
+#&Apache::lonhomework::showhash(%ConceptData);
+#$Apache::lonxml::debug=0;
+    my $Answ=&Apache::lonnet::ssi($URI);
+    $r->print("<br><b>Here you can see the Problem:</b><br>$Answ");
+}
+
+sub AdjustData {
+    my $k=shift;
+    my $Max=0;
+    my @data1;
+    my @data2;
+    for (my $n=0; $n<=$#Concepts; $n++ ) {
+	my $tmp=$Concepts[$n];
+	$data1[$n]=$ConceptData{"$tmp.$k.true"};
+	$data2[$n]=$ConceptData{"$tmp.$k.false"};
+	my $Sum=$data1[$n]+$data2[$n];
+	if ( $Max<$Sum ) {$Max=$Sum;}
+	$ConceptData{"$tmp.true"}+=$data1[$n];
+	$ConceptData{"$tmp.false"}+=$data2[$n];
+    }
+    for (my $n=0; $n<=$#Concepts; $n++ ) {
+	if ($data1[$n]+$data2[$n]<$Max) {
+	    $data2[$n]+=$Max-($data1[$n]+$data2[$n]);
+	}
+    }
+    return join(',',@data1).'+'.
+           join(',',@data2).'+'.
+           $Max.'+'.($#data1+1);
+}
+
+
+sub DrawGraph {
+    my $data=shift;
+    my($data1,$data2,$Max,$P_No,$Src,$k)=split(/\+/,$data);
+
+    my $Str="\n".'<table border=1>'.
+            "\n".'<tr>'.
+	    "\n".'<th> Correct Answers </th>'.
+	    "\n".'<th> Wrong Answers </th>'.
+	    "\n".'<th> From </th>'.
+	    "\n".'<th> To </th>'.
+	    "\n".'</tr>'.
+	    "\n"."<tr>".
+	    "\n"."<td> $data1 </td>".
+            "\n"."<td> $data2 </td>".
+            "\n"."<td> ".localtime($ConceptData{'Int.'.($k-1)})." </td>".
+            "\n"."<td> ".localtime($ConceptData{'Int.'.$k}-1)." </td>".
+            "\n"."</tr></table>";
+    $r->print($Str);
+#    $r->print('<br><b>Correct Answers:</b> '.$data1.
+#              '<br><b>Wrong Answers: </b>'.$data2); 
+#    $r->print('<br><b>From: </b>'.localtime($ConceptData{'Int.'.($k-1)}).
+#              '<br><b>To: </b>'.localtime($ConceptData{"Int.$k"})); 
+
+#   if ( $Max > 1 ) { 
+#	$Max += (10 - $Max % 10);
+#	$Max = int($Max);
+#   }
+#   else { $Max = 1; }
+    my $Titr=($ConceptData{'Interval'}>1) ? $Src.'_interval_'.$k : $Src;
+    $GData=$Titr.'&'.'Concepts'.'&'.'Answers'.'&'.$Max.'&'.$P_No.'&'.$data1.'&'.$data2;
+    if($ConceptData{'Interval'}>1){
+	$r->print('<br><IMG src="/cgi-bin/graph.gif?'.$GData.'" />');
+	return;
+    }
+    $r->print('<br>Move your mouse over a bar to find out the concept'); 
+    my $ptr='';
+    $ptr.="\n".'<form method="post" action="" name="displayform"> <input type=text name="dis" size=80> </form>';
+    $ptr.="\n".'<p><img src=/cgi-bin/graph.gif?'.$GData.
+          ' usemap="#Map" border=1>'.
+          "\n".'<map name="Map">';
+    my $gap=12;
+    my $Size=340;
+    my $barsize=($P_No) ? int($Size/$P_No) : 1;
+    for (my $i=0; $i<$P_No; $i++) {
+	my $x1=80+$gap*$i+($i*$barsize);
+	my $x2=$x1+$barsize;
+	my $y1=25;
+	my $y2=350;
+	my $j=$i+1;
+	$ptr.="\n".'<area shape="rect" coords='.$x1.','.$y1.','.$x2.','.$y2.' onMouseOver="display('."'Concept $j: $Concepts[$i]'".'); " href="javascript:alert('."'Concept $j: $Concepts[$i]'".');">';
+    } 
+    $ptr.="\n".'<area shape="default" onMouseOver="cleardisplay(); " href="javascript:alert('.'No Concept  in particular'.');">';
+    $ptr.="\n".'</map></p>';
+    $r->print('<br>'.$ptr.'<br>');
+}
+
+
+sub AnalyzeProblem {
+# -------------------------------- Selecting the number of intervals
+    my $OpSel='';
+    my $CurInt = $ENV{'form.interval'};
+    if ($CurInt eq '') {$CurMap = '1';}
+    my $Ptr = '<b>Select number of intervals</b>'."\n".
+       	      '<select name="interval">'."\n";                     	     	     
+    for (my $n=1;$n<=7;$n++) {	          
+	$Ptr .= '<option';
+        if ($CurInt eq $n) {$Ptr .= ' selected';}     
+	$Ptr .= '>'.$n."</option>"."\n";	     
+    }
+    $Ptr .= '</select>'."\n";
+    $r->print( $Ptr );
+
+    $r->print('<br><b> Option Response Problems in this course:</b><br><br>');
+    my $Str = "\n".'<table border=2>'.
+              "\n".'<tr>'.
+              "\n".'<th> # </th>'.
+	      "\n".'<th> Problem Title </th>'.
+	      "\n".'<th> Resouse </th>'.
+	      "\n".'<th> Address </th>'.
+	      "\n".'</tr>';
+
+     my $P_No=1;
+     foreach (sort keys %OpResp) {
+	 my ($rid,$part)=split(/\:/,$OpResp{$_});
+	 my $Temp = '<a href="'.$hash{'src_'.$rid}.
+                    '" target="_blank">'.$hash{'title_'.$rid}.'</a>';
+	 $Str .= "\n"."<tr>".
+	         "\n"."<td> $P_No </td>".
+                 "\n"."<td bgcolor=#DDFFDD> ".$Temp." </td>".
+                 "\n"."<td bgcolor=#EEFFCC> ".$hash{'src_'.$rid}." </td>".
+	         "\n"."<td> ".'<input type="submit" name="sort" value="'.'Analyze_'.$rid.'" />'.'</td>'.
+                 "\n"."</tr>";
+	 $P_No++;
+     }
+     $Str .= "\n".'</table>';
+     $Str .= "\n".'</form>';
+     $r->print($Str);
+     $r->rflush();	
+}
+
+
+sub Decide {
+    my ($type,$foil,$time)=@_; 
+    my $k=0;
+    while ($time>$ConceptData{'Int.'.($k+1)} && 
+           $k<$ConceptData{'Interval'}) {$k++;}
+    $ConceptData{"$foil_to_concept{$foil}.$k.$type"}++;
+}
+
+
+sub OpStatus {
+    my ($rid,$student)=@_;
+    my ($uname,$udom)=split(/\:/,$student);
+    my $code='U';
+    $rid=~/(\d+)\.(\d+)/;
+    my $symb=&Apache::lonnet::declutter($hash{'map_id_'.$1}).'___'.$2.'___'.
+	     &Apache::lonnet::declutter($hash{'src_'.$rid});
+    my %reshash=&Apache::lonnet::restore($symb,$cid,$udom,$uname);
+    my @True = ();
+    my @False = ();
+    my $flag=0;
+    @shown=();
+    if ($reshash{'version'}) {
+	for (my $version=1;$version<=$reshash{'version'};$version++) {
+	    my $time=$reshash{"$version:timestamp"};
+	    foreach (sort(split(/\:/,$reshash{$version.':keys'}))) {
+		if (($_=~/\.(\w+)\.(\w+)\.submission$/)) {
+		    #my $Id1=$1; my $Id2=$2;
+		    my $Resp = $reshash{$_};
+		    my %submission=&Apache::lonnet::str2hash($Resp);
+		    foreach (keys %submission) {
+			my $Ansr = $ConceptData{"$_"};
+			#my $Ansr = $Answer{"$Id1.$Id2.foil.value.$_"};
+			#$r->print("<br>shown: ".join(',',@shown)." size=$#shown");     
+			if ($submission{$_}) {
+			    #if($#shown==-1){
+				#my @str=split(/\&/,$Resp);
+				#my $garb;
+				#for(my $j=0;$j<=$#str;$j++){
+				#    ($shown[$j],$garb)=split(/\=/,$str[$j]);
+				#}
+				##$r->print("<br>shown: ".join(',',@shown)." size=$#shown");     
+			    #}
+			    if ($submission{$_} eq $Ansr) {
+				&Decide("true",$_,$time );
+			    }
+			    else {&Decide("false",$_,$time );}
+			}
+			#else {
+			#    if ($#shown==-1 && $flag==0) {
+			#	$flag++;
+			#	&InitAnalysis($rid,$student);
+			#    }
+			#    my @erl=split(/\:/,$_);
+			#    for (my $i=0;$i<=$#shown; $i++){
+			#	my $Ans=$Answer{"$Id1.$Id2.foil.value.$shown[$i]"};
+			#	if ($erl[$i] eq $Ans) {
+			#	    &Decide("true",$shown[$i],$time);
+			#	}
+			#	else {&Decide("false",$shown[$i],$time);}
+			#    }
+			#}
+		    }
+	        }	  
+	    }
+        }
+    }
+}
+
 
 #------- Processing upperlist and lowerlist according to each problem
 sub ProcessDisc {
@@ -109,7 +432,7 @@ sub ProcessDisc {
 	} while ( $Prb eq $Temp && $nIdx < $Count );
 #	$Proc{$Temp}=($Sum1/$nStud).':'.$nStud;
 	$Proc{$Temp}=($Sum1/$nStud).':'.($Sum2/$nStud);
-#        $r->print("$nIdx) $Temp --> ($nStud) $Proc{$Temp} <br>");
+#       $r->print("$nIdx) $Temp --> ($nStud) $Proc{$Temp} <br>");
 	$Sum1=0;
 	$Sum2=0;
 	$nStud=0;
@@ -155,17 +478,13 @@ sub NumericSort {
 
 # ------ Create different Student Report 
 sub StudentReport {
-
     my ($sname,$sdom)=@_;
-
     if ( $sname eq 'All Students' ) {
 	$r->print( '<h3><font color=blue>WARNING: 
                     Please select a student</font></h3>' );
 	return;
     }
-    my $shome=&Apache::lonnet::homeserver($sname,$sdom);          
-    my $reply=&Apache::lonnet::reply('dump:'.$sdom.':'.$sname.':'.$cid,$shome );
-    my %result = ();
+    my %result = &Apache::lonnet::dump($cid,$sdom,$sname);
     my $ResId;
     my $PrOrd;
     my $Code;
@@ -186,11 +505,8 @@ sub StudentReport {
 	      "\n".'<th> Results </th>'.
 	      "\n".'<th> Tries </th>'.
 	      "\n".'</tr>';
-    unless ($reply=~/^error\:/) {
-        foreach (split(/\&/,$reply)){
-            my ($name,$value)=split(/\=/,&Apache::lonnet::unescape($_));
-            $result{$name}=$value;
-        }
+    my ($temp)=keys(%result);
+    unless ($temp=~/^error\:/) {
         foreach my $CurCol (@cols) {
 	    if (!$CurCol){
 		my $Set=&Apache::lonnet::declutter($hash{'map_id_'.$1});
@@ -218,7 +534,7 @@ sub StudentReport {
 	    my $PartNo = 0;
 	    undef %TempHash;
 	    foreach (split(/\,/,&Apache::lonnet::metadata($meta,'keys'))){
-		if ($_=~/^stores\_(\d+)\_tries$/) {
+		if ($_=~/^stores\_(\w+)\_tries$/) {
                     my $Part=&Apache::lonnet::metadata($meta,$_.'.part');
 		    if ( $TempHash{"$Part"} eq '' ) { 
 			$TempHash{"$Part"} = $Part;
@@ -313,8 +629,6 @@ sub PreStatTable {
                   "_$ENV{'user.domain'}_$cid\_statistics.db";
     my $GraphDB = "/home/httpd/perl/tmp/$ENV{'user.name'}".
                   "_$ENV{'user.domain'}_$cid\_graph.db";
-    my $CachDisFac = "/home/httpd/perl/tmp/$ENV{'user.name'}".
-		     "_$ENV{'user.domain'}_$cid\_DiscFactor.db";
     my $OpSel11='';
     my $OpSel12='';
     my $OpSel13='';
@@ -360,10 +674,10 @@ sub PreStatTable {
 	'evaluating the problem according to a Criterion<br>'.
 	'<b>           [Applied Criterion in %27 Upper Students - '.
 	'Applied the same Criterion in %27 Lower Students]</b><br>'.
-    '<b>           1st </b>Criterion for Sorting the Students: '.
-	'Sum of Partial Credit Awarded / Total Number of Tries<br>'.
-    '<b>           2nd </b>Criterion for Sorting the Students: '.
-	'Total number of Correct Answers / Total Number of Tries'.	
+    '<b>           1st Criterion</b> for Sorting the Students: '.
+	'<b>Sum of Partial Credit Awarded / Total Number of Tries</b><br>'.
+    '<b>           2nd Criterion</b> for Sorting the Students: '.
+	'<b>Total number of Correct Answers / Total Number of Tries</b>'.	
             '</pre>';
 
     $r->print($Ptr);
@@ -371,7 +685,6 @@ sub PreStatTable {
 
     if ((-e "$CacheDB")&&($ENV{'form.sort'} ne 'Recalculate Statistics')) {
 	if (tie(%CachData,'GDBM_File',"$CacheDB",&GDBM_READER,0640)) {
-	    tie(%DiscFac,'GDBM_File',$CachDisFac,&GDBM_READER,0640);
 	    tie(%GraphDat,'GDBM_File',$GraphDB,&GDBM_WRCREAT,0640);
 	    &Cache_Statistics();
         }
@@ -381,7 +694,6 @@ sub PreStatTable {
     }
     else {
 	if (tie(%CachData,'GDBM_File',$CacheDB,&GDBM_WRCREAT,0640)) {
-	    tie(%DiscFac,'GDBM_File',$CachDisFac,&GDBM_WRCREAT,0640);
 	    tie(%GraphDat,'GDBM_File',$GraphDB,&GDBM_WRCREAT,0640);
 	    foreach (keys %DiscFac) {delete $CachData{$_};}
 	    foreach (keys %CachData) {delete $CachData{$_};}
@@ -392,11 +704,10 @@ sub PreStatTable {
 	    $r->print("Unable to tie hash to db file");
         }
     }
-    #$r->print('Total instances of the problems : '.($p_count*($#students+1)));
 
+#    $r->print('Total instances of the problems : '.($p_count*($#students+1)));
     untie(%CachData);
     untie(%GraphDat);
-    untie(%DiscFac);       
 }
 
 
@@ -438,13 +749,10 @@ sub usection {
 
 
 # ------ Dump the Student's DB file and handling the data for statistics table 
-
 sub ExtractStudentData {
-    my ($student,$coid)=@_;
+    my $student=shift;
     my ($sname,$sdom) = split( /\:/, $student );
-    my $shome=&Apache::lonnet::homeserver( $sname,$sdom );          
-    my $reply=&Apache::lonnet::reply('dump:'.$sdom.':'.$sname.':'.$coid,$shome );
-    my %result = ();
+    my %result = &Apache::lonnet::dump($cid,$sdom,$sname);
     my $ResId;
     my $PrOrd;
     my $Dis = '';
@@ -459,22 +767,20 @@ sub ExtractStudentData {
     my $TotParCr = 0;
     my $Wrongs;
     my %TempHash;
-    my $Version;
+    my $Version; 
     my $LatestVersion;
     my $SecLimit;
     my $MapLimit;
-    unless ($reply=~/^error\:/) {
-        foreach (split(/\&/,$reply)) {
-            my ($name,$value)=split(/\=/,&Apache::lonnet::unescape($_));
-            $result{$name}=$value;
-        }
+    my ($temp)=keys(%result);
+    unless ($temp=~/^error\:/) {
         foreach my $CurCol(@cols) {
 	    ($PrOrd,$ResId)=split(/\:/,$CurCol);
 	    if ( !$CurCol ) { next; }
             $ResId=~/(\d+)\.(\d+)/;
 	    my $MapId=$1;
 	    my $PrbId=$2;
-            my $Map = &Apache::lonnet::declutter( $hash{'map_id_'.$MapId} );
+            my $MapOrg = $hash{'map_id_'.$MapId};
+            my $Map = &Apache::lonnet::declutter($MapOrg);
             if ( $CurMap ne 'All Maps' ) {
 		my ( $ResMap, $NameMap ) = split(/\=/,$CurMap);
 		if ( $Map ne $ResMap ) { next; }
@@ -483,8 +789,9 @@ sub ExtractStudentData {
 	    my $PartNo = 0;
 	    $Dis .= ':';
 	    undef %TempHash;
-	    foreach (split(/\,/,&Apache::lonnet::metadata($meta,'keys'))) {
-		if ($_=~/^stores\_(\d+)\_tries$/) {
+
+	    foreach (split(/\,/,&Apache::lonnet::metadata($meta,'keys'))) { 
+		if ($_=~/^stores\_(\w+)\_tries$/) {
                     my $Part=&Apache::lonnet::metadata($meta,$_.'.part');
 		    if ( $TempHash{"$Part"} eq '' ) { 
 			$TempHash{"$Part"} = $Part;
@@ -493,22 +800,22 @@ sub ExtractStudentData {
 			$TempHash{"$Part.PrOrd"} = $PrOrd+$PartNo;
 			$PartNo++;
 		    }
+                    #my $Part=&Apache::lonnet::metadata($meta,$_.'.part');
 		}
             }
-
+            &Apache::lonnet::declutter( $hash{'src_'.$ResId} );
+	    my $URI = $hash{'src_'.$ResId};
             my $Prob = $Map.'___'.$PrbId.'___'.
-                       &Apache::lonnet::declutter( $hash{'src_'.$ResId} );
+                       &Apache::lonnet::declutter($URI);
             $Code='U';
             $Tries = 0;
 	    $ParCr = 0;
             $Wrongs = 0;
-  	    $LatestVersion = $result{"version:$Prob"};       
-
+  	    $LatestVersion = $result{"version:$Prob"};        
 	    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;
@@ -520,7 +827,6 @@ sub ExtractStudentData {
 			    $TempHash{"$Part.ParCr"} = ($ParCr) ? $ParCr : 0;        
 			    $TotalTries += $TempHash{"$Part.Tries"};
 			    $TotParCr += $TempHash{"$Part.ParCr"};
-#$r->print($Version.'---'.$Prob.'==='.$Time.'<br>');
 			    my $Val = $result{"$Version:$Prob:resource.$Part.solved"};
 			    if ( $Val eq 'correct_by_student' )
                                { $Wrongs = $Tries - 1; $Code = 'C'; } 
@@ -542,11 +848,11 @@ sub ExtractStudentData {
 		       {$ProbSolved++;$Yes=1;}		
 
  #		    my $ptr = "$hash{'title_'.$ResId}";
-		    my $ptr = $TempHash{$part.'.PrOrd'}.':'.$ResId;
+		    my $ptr = $TempHash{$part.'.PrOrd'}.'&'.$ResId;
 
 		    if ( $PartNo > 1 ) {                
 			$ptr .= "*(part $part)";
-			$Dis .= ':';
+			$Dis .= '&';
 		    }
 		    my $Fac = ($TempHash{"$part.Tries"}) ? 
                               ($TempHash{"$part.ParCr"}/$TempHash{"$part.Tries"}) : 0;
@@ -555,14 +861,12 @@ sub ExtractStudentData {
 			$DisF = sprintf( "%.4f", $Fac );
 		    }
 		    else {$DisF = $Fac;}
-#		    $DisF .= '+'.$TempHash{"$part.Time"};
+#		    $DisF .= '+'.$TempHash{"$part.Time"};33333333
 		    $TimeTot += $TempHash{"$part.Time"};
 		    $Dis .= $TempHash{$part.'.PrOrd'}.'='.$DisF.'+'.$Yes;
-
-            #$r->print('<br>'.$DisFactor,'###',$DiscFac{$DisFactor});
-		    $ptr .= ":$TempHash{$part.'.Tries'}".
-		            ":$TempHash{$part.'.Wrongs'}".
-                            ":$TempHash{$part.'.Code'}";
+		    $ptr .= "&$TempHash{$part.'.Tries'}".
+		            "&$TempHash{$part.'.Wrongs'}".
+                            "&$TempHash{$part.'.Code'}";
 		    push (@list, $ptr);
 		    $TotalOpend++;
 		    $ProbTot++;
@@ -570,7 +874,7 @@ sub ExtractStudentData {
             }
 	    #else { 
 		#for(my $n=0; $n<$PartNo; $n++) {
-		#    push (@list, "$hash{'title_'.$ResId}*$ResId:0:0:U");
+		#    push (@list, "$TempHash{'0'.'.PrOrd'}.':'.$ResId:0:0:U");
 		#    $ProbTot++; 
 		#}
 	    #}
@@ -579,7 +883,6 @@ sub ExtractStudentData {
 	    my $DisFac = ( $TotalTries ) ? ($TotParCr/$TotalTries) : 0;
 	    my $DisFactor = sprintf( "%.4f", $DisFac );
 	    $DiscFac{$DisFactor}=$Dis;
-            #$r->print('<br>'.$DisFactor,'###',$DiscFac{$DisFactor});
 	    #my $time;
 	    #if ($ProbSolved){
 		#$time = int(($TimeTot/$ProbSolved)-10000000);
@@ -595,6 +898,7 @@ sub ExtractStudentData {
 # ------------------------------------------------------------ Build page table
 sub tracetable {
     my ($rid,$beenhere)=@_;
+    my $IsMap=0;
     $rid=~/(\d+)\.(\d+)/;
     $maps{&Apache::lonnet::declutter($hash{'map_id_'.$1})}='';#$hash{'title_'.$rid}; 
     #$maps{$HWN}=$hash{'title_'.$rid}; 
@@ -606,7 +910,9 @@ sub tracetable {
                $cols[$#cols+1]=0;
 	       $P_Order++;
 	       $HWN=$P_Order;
-               $mapsort{$HWN} = $rid.':'; 
+               $mapsort{$HWN} = $rid.':';
+	       $IsMap=1;
+ 
                #$maps{&Apache::lonnet::declutter($hash{'src_'.$rid})}= 
                #      $hash{'title_'.$rid}; 
            }
@@ -621,36 +927,53 @@ sub tracetable {
       
 	       my $meta=$hash{'src_'.$frid};
 	       my $PartNo = 0;
+	       my $Part;
+	#       if ($IsMap==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');
+			   if ($_=~/^stores\_(\w+)\_tries$/) {
+			       $Part=&Apache::lonnet::metadata($meta,$_.'.part');
 			       $P_Order++;
 			       $mapsort{$HWN} .= '&'.$P_Order;
 			       $PartNo++;
-			       $r->print('<br>'.$PartNo.'---'.$P_Order);
+			       #$r->print('<br>'.$PartNo.'---'.$P_Order);
+			   }
+			   foreach my $K(split(/\,/,&Apache::lonnet::metadata($meta,'packages'))) {
+			       if ($K=~/^optionresponse\_($Part)\_(\w+)$/) {
+				   #$r->print('<br>'.$_.'...'.$P_Order.'---'.$Part);
+				   $OpResp{$P_Order}="$frid:$Part";
+			       } 
 			   }
 		       }
 		   }
                }
 	   }
+	 #  }
        } else {
 	   $cols[$#cols+1]=($P_Order+1).':'.$rid;
 	   my $meta=$hash{'src_'.$rid};
 	   my $PartNo = 0;
-           if ($meta) {
+	   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');
+		   foreach my $Key(split(/\,/,&Apache::lonnet::metadata($meta,'keys'))) {
+		       if ($Key=~/^stores\_(\w+)\_tries$/) {
+			   my $Part=&Apache::lonnet::metadata($meta,$Key.'.part');
 			   $P_Order++;
 			   $mapsort{$HWN} .= '&'.$P_Order;
 			   $PartNo++;
+			   foreach (split(/\,/,&Apache::lonnet::metadata($meta,'packages'))) {
+			       if ($_=~/^optionresponse\_($Part)\_(\w+)$/) {
+				   #$r->print('<br>'.$_.'...'.$P_Order.'---'.$Part);
+				   $OpResp{$P_Order}="$rid:$Part";;
+			       } 
+		   
+			   }
+
 		       }
 		   }
 	       }
-	   }
+	   }       
        }
        if (defined($hash{'to_'.$rid})) {
           foreach (split(/\,/,$hash{'to_'.$rid})){
@@ -671,7 +994,8 @@ sub MySort {
     }
 }
 
-sub Build_Statistics {
+sub Create_PrgWin {
+#----------- Create progress
     $r->print(<<ENDPOP);
     <script>
     popwin=open('','popwin','width=400,height=100');
@@ -686,16 +1010,30 @@ sub Build_Statistics {
 ENDPOP
 
     $r->rflush();
-# ---------------------------- Gathering the Data of students' tries
-    my $index;
-    for ($index=0;$index<=$#students;$index++) {
+}
+
+
+sub Update_PrgWin {
 #----------- update progress
-        $r->print('<script>popwin.document.popremain.remaining.value="'.
-                  'Computing '.($index+1).'/'.($#students+1).': '.
-                  $students[$index].'";</script>');
-        $r->rflush();
+    my $index = shift;
+    $r->print('<script>popwin.document.popremain.remaining.value="'.
+              'Computing '.($index+1).'/'.($#students+1).': '.
+              $students[$index].'";</script>');
+    $r->rflush();
+}
 
-        &ExtractStudentData($students[$index],$cid);
+sub Close_PrgWin {
+#--------------------- close Progress Line
+    $r->print('<script>popwin.close()</script>');
+    $r->rflush(); 
+}
+
+sub Build_Statistics {
+    &Create_PrgWin();
+# ---------------------------- Gathering the Data of students' tries
+    for (my $index=0;$index<=$#students;$index++) {
+	&Update_PrgWin($index);
+        &ExtractStudentData($students[$index]);
     }
 
 # -------------------- sorting the Data
@@ -723,7 +1061,7 @@ ENDPOP
 	&CreateTable(1,$Hid);
 	for (my $i=1; $i<=$#lpr; $i++) {
 	    my %storestats=();
-	    my ($PrOrd,$Prob,$Tries,$Wrongs,$Code)=split(/\:/,$list[$nIdx]);
+	    my ($PrOrd,$Prob,$Tries,$Wrongs,$Code)=split(/\&/,$list[$nIdx]);
 	    my $Temp = $Prob;
 	    my $MxTries = 0;
 	    my $TotalTries = 0;
@@ -743,7 +1081,7 @@ ENDPOP
 		elsif( $Code eq 'I' ) { $Incorrect++; }
 		elsif( $Code eq 'O' ) { $Override++; }
 		elsif( $Code eq 'U' ) { $StdNo--; }
-		($PrOrd,$Prob,$Tries,$Wrongs,$Code)=split(/\:/,$list[$nIdx]);
+		($PrOrd,$Prob,$Tries,$Wrongs,$Code)=split(/\&/,$list[$nIdx]);
 	    }	
 
 	    $p_count++;
@@ -752,7 +1090,7 @@ ENDPOP
 
 	    $Temp = '<a href="'.$hash{'src_'.$ResId}.
                 '" target="_blank">'.$hash{'title_'.$ResId}.$Dummy.'</a>';
-    
+
 	    my $res = &Apache::lonnet::declutter($hash{'src_'.$ResId});
 	    my $urlres=$res;
 
@@ -802,7 +1140,7 @@ ENDPOP
 		    $Skewness += $Dif*$Dif*$Dif;
 		} 
 		$Skewness /= $StdNo;
-	    $Skewness /= $StdDev*$StdDev*$StdDev;
+		$Skewness /= $StdDev*$StdDev*$StdDev;
 	    }
 
 #--------------------- Compute the Discrimination Factors
@@ -819,10 +1157,11 @@ ENDPOP
 	    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.':'.$_D1.':'.$_D2.':'.$Prob;
+	    my $join = $lpr[$i].'&'.$Temp.'&'.$StdNo.'&'.
+                       $TotalTries.'&'.$MxTries.'&'.$Avg.'&'.
+                       $YES.'&'.$Override.'&'.$Wrng.'&'.$DoD.'&'.
+		       $SD.'&'.$Sk.'&'.$_D1.'&'.$_D2.'&'.
+                       $Prob;
 	    $CachData{($p_count-1)}=$join;
 
 	    $urlres=~/^(\w+)\/(\w+)/;
@@ -836,9 +1175,7 @@ ENDPOP
 	}
 	&CloseTable();
     }
-#--------------------- close Progress Line
-    $r->print('<script>popwin.close()</script>');
-    $r->rflush(); 
+    &Close_PrgWin();
 }
 
 sub Cache_Statistics {
@@ -850,14 +1187,14 @@ sub Cache_Statistics {
     if ($Pos > 0) {$Pos++;}
     $p_count = 0;
     foreach my $key( keys %CachData) { 
-	my @Temp=split(/\:/,$CachData{$key});
+	my @Temp=split(/\&/,$CachData{$key});
 	if ( $Pos == 0 ) {
 	    ($UnUseful,$Useful)=split(/\>/,$Temp[$Pos]);
 	}
 	else {
 	    $Useful = $Temp[$Pos];
 	}   
-	$list[$p_count]=$Useful.'&'.$CachData{$key};
+	$list[$p_count]=$Useful.'@'.$CachData{$key};
         $p_count++;
     }
 
@@ -871,7 +1208,8 @@ sub Cache_Statistics {
 	    &CreateTable(1,$Hid);
 	    my @lpr=split(/\&/,$pr);
 	    for (my $i=1; $i<=$#lpr; $i++) {
-		my($Pre, $Post) = split(/\&/,$list[$nIdx]); 
+		my($Pre, $Post) = split(/\@/,$list[$nIdx]); 
+		#$r->print('<br>'.$Pre.'---'.$Post);
 		&TableRow($Post,$i,$nIdx);
 		$nIdx++;
 	    }
@@ -881,7 +1219,7 @@ sub Cache_Statistics {
     else {
 	&CreateTable(0);
 	for ( my $nIdx = 0; $nIdx < $p_count; $nIdx++ ) {
-	    my($Pre, $Post) = split(/\&/,$list[$nIdx]); 
+	    my($Pre, $Post) = split(/\@/,$list[$nIdx]); 
 	    &TableRow($Post,$nIdx,$nIdx);
 	} 
 	&CloseTable();
@@ -890,10 +1228,11 @@ sub Cache_Statistics {
 
 sub TableRow {
     my ($Str,$Idx,$RealIdx)=@_;
-    my($PrOrd,$Temp,$StdNo,$TotalTries,$MxTries,$Avg,$YES,
-       $Override,$Wrng,$DoD,$SD,$Sk,$_D1,$_D2,$Prob)=split(/\:/,$Str);	
-    $r->print( "\n".'<tr>'.
+    my($PrOrd,$Temp,$StdNo,$TotalTries,$MxTries,$Avg,$YES,$Override,
+       $Wrng,$DoD,$SD,$Sk,$_D1,$_D2,$Prob)=split(/\&/,$Str);	
+    my $Ptr =  "\n".'<tr>'.
                "\n".'<td>'.($RealIdx+1).'</td>'.
+          #     "\n".'<td>'.$PrOrd.$Temp.'</td>'.
                "\n".'<td>'.$Temp.'</td>'.
                "\n".'<td bgcolor="#EEFFCC"> '.$StdNo.'</td>'.
                "\n".'<td bgcolor="#EEFFCC">'.$TotalTries.'</td>'.
@@ -906,15 +1245,15 @@ sub TableRow {
                "\n".'<td bgcolor="#DDFFDD"> '.$SD.'</td>'.
                "\n".'<td bgcolor="#DDFFDD"> '.$Sk.'</td>'.
                "\n".'<td bgcolor="#FFDDFF"> '.$_D1.'</td>'.
-               "\n".'<td bgcolor="#FFDDFF"> '.$_D2.'</td>'.
-               "\n".'</tr>' );
+	       "\n".'<td bgcolor="#FFDDFF"> '.$_D2.'</td>';
+    $r->print("\n".$Ptr.'</tr>' );
     $GraphDat{$RealIdx}=$DoD.':'.$Wrng;
 }
 
 # ------------------------------------------- Prepare data for Graphical chart
 
 sub GetGraphData {
-    my $Tag = shift;
+    my $ylab = shift;
     my $Col;
     my $data='';
     my $count = 0;
@@ -925,12 +1264,12 @@ sub GetGraphData {
     foreach (keys %GraphDat) {delete $GraphDat{$_};}
     if (-e "$GraphDB") {
 	if (tie(%GraphDat,'GDBM_File',"$GraphDB",&GDBM_READER,0640)) {
-	    if ( $Tag eq 'DoDiff Graph' ) {
-		$Tag = 'Degree-of-Difficulty';
+	    if ( $ylab eq 'DoDiff Graph' ) {
+		$ylab = 'Degree-of-Difficulty';
 		$Col = 0;
 	    }
 	    else {
-		$Tag = 'Wrong-Percentage';
+		$ylab = 'Wrong-Percentage';
 		$Col = 1;
 	    }
 	    foreach (sort NumericSort keys %GraphDat) { 
@@ -940,11 +1279,15 @@ sub GetGraphData {
 		$data .= $inf.',';
 		$count++;
 	    }
+	    if ( $Max > 1 ) { 
+		$Max += (10 - $Max % 10);
+		$Max = int($Max);
+	    }
+	    else { $Max = 1; }
             untie(%GraphDat);
 	    my $Course = $ENV{'course.'.$cid.'.description'};
 	    $Course =~ s/\ /"_"/eg;
-	    $GData=$Course.'&'.$Tag.'&'.$Max.'&'.$count.'&'.$data;
-
+	    $GData=$Course.'&'.'Problems'.'&'.$ylab.'&'.$Max.'&'.$count.'&'.$data;
 	}
 	else {
 	    $r->print("Unable to tie hash to db file");
@@ -964,6 +1307,8 @@ sub initial {
   undef %CachData;
   undef %GraphDat;
   undef %DiscFac;
+  undef %OpResp;
+  undef %ConceptData;
   undef $CurMap;
   undef $CurSec;
   undef $CurStu;
@@ -1005,21 +1350,23 @@ sub ClassList {
                      ( ($Status eq 'Active') ) ) { $active=0; }
 		if ( ($Status eq 'Expired') && 
                      ($end == 0 || $now < $end) ) { $active=0; }
+#//new
+#		$Status = 'Any';
+#end new
 		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;}
+#		    if ($ssec==-1 ) {next;}
+		    if ($ssec==-1 || $ssec eq 'adm' ) {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;
@@ -1036,17 +1383,16 @@ sub ClassList {
         $r->rflush();
 # --------------- Find all assessments and put them into some linear-like order
 	&tracetable($firstres,'&'.$lastres.'&');
-
 #    my $c=0;
 #    foreach (sort keys %mapsort) {
 #	$r->print('<br>'.$c.')'.$_.' ... '.$mapsort{$_});
 #	$c++;
 #    }
-#foreach(@cols) {
-#    $c++;
-#    $r->print('<br>'.$cols[$c]);
-#}
-#$r->print('<br> Count = '.$c);
+#	my $c=1;
+#	foreach (sort keys %OpResp) {
+#	    $r->print('<br>'.$c.')'.$_.' ... '.$OpResp{$_}.' ... '.$hash{'src_'.$OpResp{$_}});
+#	    $c++;
+#	}
 
     }
 
@@ -1054,62 +1400,72 @@ sub ClassList {
     else {
 	$r->print('<h3>Undefined course sequence</h3>');
     }
-    &MapSecOptions();
+}
+
+
+sub Title {
+    $r->print('<html><head><title>LON-CAPA Statistics</title></head>');
+    $r->print('<body bgcolor="#FFFFFF">'.
+              '<script>window.focus(); window.width=500;window.height=500;</script>'.
+              '<img align=right src=/adm/lonIcons/lonlogos.gif>');
+# ---------------------------------------------------------------- Course title
+    $r->print('<h1> Course : "'.
+              $ENV{'course.'.$ENV{'request.course.id'}.
+              '.description'}.'"</h1><h2>'.localtime().'</h2>');
+# ------------------------------- This is going to take a while, produce output
+    $r->rflush();	
+}
+
+sub CreateForm {
+    $r->print("\n".'<form name=stat method=post action="/adm/statistics" >');
+    my $content = $ENV{'form.sort'};
+    if ($content eq '' || $content eq 'Return to Menu') {
+	my $Ptr = '<h3>';
+	$Ptr .= '<input type=submit name=sort value="Problem Stats"/>';
+	$Ptr .= '<br><br>';
+	$Ptr .= '<input type=submit name=sort value="Problem Analysis"/>';
+	$Ptr .= '<br><br>';
+	$Ptr .= '<input type=submit name=sort value="Student Assessment"/>';
+	$Ptr .= '</h3>';
+	$r->print( $Ptr );
+    }
+    else {
+	if ( $content eq 'Student Assessment' || 
+	     $content eq 'Create Student Report' ) {
+	    &MapSecOptions();
+	    &StudentOptions();
+	    &StudentReport($CurStu,$StuBox{"$CurStu"});
+	}
+	elsif ( $content eq 'Problem Analysis' ) {
+	    &AnalyzeProblem();
+	}
+	else {
+	    &MapSecOptions();
+	    &PreStatTable();
+	}
+    }
 }
 
 
 sub Menu {
+    &initial();
+    &Title();
+    &ClassList();
     my $InpStr = $ENV{'form.sort'};
-    if ( $InpStr eq 'DoDiff Graph' || $InpStr eq '%Wrong Graph' ) {      
+    if ($InpStr=~/^Analyze\_/) {
+	&ShowOpGraph($InpStr,$ENV{'form.interval'});
+    }
+    elsif ( $InpStr eq 'DoDiff Graph' || $InpStr eq '%Wrong Graph' ) {      
 	&GetGraphData($InpStr);
     	$r->print('<IMG src="/cgi-bin/graph.gif?'.$GData.'" />');
     }
     else {
-	$r->print('<html><head><title>LON-CAPA Statistics</title></head>');
-
-
-        $r->print('<body bgcolor="#FFFFFF">'.
-                  '<script>window.focus(); window.width=500;window.height=500; </script>'.
-                  '<img align=right src=/adm/lonIcons/lonlogos.gif>');
-# ---------------------------------------------------------------- Course title
-        $r->print('<h1> Course : "'.
-                  $ENV{'course.'.$ENV{'request.course.id'}.
-                  '.description'}.'"</h1><h2>'.localtime().'</h2>');
-# ------------------------------- This is going to take a while, produce output
-        $r->rflush();	
-
-	$r->print("\n".'<form name=stat method=post action="/adm/statistics" >');
-
-	my $content = $ENV{'form.sort'};
-	if ($content eq '' || $content eq 'Return to Menu') {
-	    my $Ptr = '<h3>';
-	    $Ptr .= '<input type=submit name=sort value="Problem Evaluation"/>';
-	    $Ptr .= '<br><br>';
-	    $Ptr .= '<input type=submit name=sort value="Student Assessment"/>';
-	    $Ptr .= '</h3>';
-	    $r->print( $Ptr );
-      	}
-	else {
-	    &initial();
-	    &ClassList();
-	    if ( $content eq 'Discrimination' || #'Problem Evaluation' ||
-                 $content eq 'Recalculate Discrimintion Factor' ) {
-		&CreateDiscFac();
-	    }
-	    elsif ( $content eq 'Student Assessment' || 
-                    $content eq 'Create Student Report' ) {
-		&StudentOptions();
-		&StudentReport($CurStu,$StuBox{"$CurStu"});
-	    }
-	    else {
-		&PreStatTable();
-	    }
-	}
-	$r->print("\n".'</form>'.
-                  "\n".'</body>'.
-                  "\n".'</html>');
-	$r->rflush();
+	&CreateForm();
+	$r->print("\n".'</form>');
     }
+    $r->print("\n".'</body>'.
+	      "\n".'</html>');
+    $r->rflush();
 }
 
 sub StudentOptions {
@@ -1135,6 +1491,7 @@ sub StudentOptions {
     $r->rflush();	
 }
 
+
 sub GetStatus {
     $OpSel1='';
     $OpSel2='';
@@ -1256,6 +1613,3 @@ sub handler {
 }
 1;
 __END__
-
-
-