[LON-CAPA-cvs] cvs: loncom /interface/statistics lonproblemanalysis.pm

matthew lon-capa-cvs@mail.lon-capa.org
Mon, 19 Jan 2004 18:55:10 -0000


matthew		Mon Jan 19 13:55:10 2004 EDT

  Modified files:              
    /loncom/interface/statistics	lonproblemanalysis.pm 
  Log:
  1. Removed debugging code.
  2. Unescape the foilid.  This fixes a bug with spaces in foilids causing
  data to be missing in the plots.
  
  
Index: loncom/interface/statistics/lonproblemanalysis.pm
diff -u loncom/interface/statistics/lonproblemanalysis.pm:1.57 loncom/interface/statistics/lonproblemanalysis.pm:1.58
--- loncom/interface/statistics/lonproblemanalysis.pm:1.57	Mon Jan 19 11:33:16 2004
+++ loncom/interface/statistics/lonproblemanalysis.pm	Mon Jan 19 13:55:10 2004
@@ -1,6 +1,6 @@
 # The LearningOnline Network with CAPA
 #
-# $Id: lonproblemanalysis.pm,v 1.57 2004/01/19 16:33:16 matthew Exp $
+# $Id: lonproblemanalysis.pm,v 1.58 2004/01/19 18:55:10 matthew Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -1251,8 +1251,6 @@
     my @Foils = sort(keys(%Foildata));
     my %Concepts;
     foreach my $foilid (@Foils) {
-        &Apache::lonnet::logthis('foilid = '.$foilid);
-        &Apache::lonnet::logthis('_Concept = '.$Foildata{$foilid}->{'_Concept'});
         push(@{$Concepts{$Foildata{$foilid}->{'_Concept'}}},
              $foilid);
     }
@@ -1289,7 +1287,6 @@
     foreach my $concept (sort $sortfunction (keys(%Concepts))) {
         if (! defined($Concepts{$concept})) {
             $Concepts{$concept}=[];
-            &Apache::lonnet::logthis('concept error: '.$concept.' does not have a value');
 #            next;
         }
         push(@Concepts,{ name => $concept,
@@ -1720,6 +1717,7 @@
     my @Foilsubs   = split('&',$submission);
     for (my $j=0;$j<=$#Foilgrades;$j++) {
         my ($foilid,$correct)  = split('=',$Foilgrades[$j]);
+        $foilid = &Apache::lonnet::unescape($foilid);
         my (undef,$submission) = split('=',$Foilsubs[$j]);
         if ($correct) {
             $RowData{$foilid}->{'_correct'}++;