Diff for /loncom/interface/statistics/lonproblemanalysis.pm between versions 1.119 and 1.127

version 1.119, 2005/03/15 00:51:43 version 1.127, 2006/08/17 21:19:47
Line 27 Line 27
 package Apache::lonproblemanalysis;  package Apache::lonproblemanalysis;
   
 use strict;  use strict;
 use Apache::lonnet();  use Apache::lonnet;
 use Apache::loncommon();  use Apache::loncommon();
 use Apache::lonhtmlcommon();  use Apache::lonhtmlcommon();
 use Apache::loncoursedata();  use Apache::loncoursedata();
Line 38  use Apache::lonstudentsubmissions(); Line 38  use Apache::lonstudentsubmissions();
 use HTML::Entities();  use HTML::Entities();
 use Time::Local();  use Time::Local();
 use capa;  use capa;
   use lib '/home/httpd/lib/perl/';
   use LONCAPA;
     
   
 my $plotcolors = ['#33ff00',   my $plotcolors = ['#33ff00', 
                   '#0033cc', '#990000', '#aaaa66', '#663399', '#ff9933',                    '#0033cc', '#990000', '#aaaa66', '#663399', '#ff9933',
Line 59  sub BuildProblemAnalysisPage { Line 62  sub BuildProblemAnalysisPage {
     #      #
     my %Saveable_Parameters = ('Status' => 'scalar',      my %Saveable_Parameters = ('Status' => 'scalar',
                                'Section' => 'array',                                 'Section' => 'array',
                                  'Groups' => 'array',
                                'NumPlots' => 'scalar',                                 'NumPlots' => 'scalar',
                                'AnalyzeOver' => 'scalar',                                 'AnalyzeOver' => 'scalar',
                                );                                 );
Line 73  sub BuildProblemAnalysisPage { Line 77  sub BuildProblemAnalysisPage {
     #      #
     my @Students = @Apache::lonstatistics::Students;      my @Students = @Apache::lonstatistics::Students;
     #      #
     if (@Students < 1 && exists($ENV{'form.firstrun'})) {      if (@Students < 1 && exists($env{'form.firstrun'})) {
         $r->print('<h2>There are no students in the sections selected</h2>');          $r->print('<h2>There are no students in the sections/groups selected</h2>');
     }      }
     #      #
     my @CacheButtonHTML =       my @CacheButtonHTML = 
Line 82  sub BuildProblemAnalysisPage { Line 86  sub BuildProblemAnalysisPage {
     $r->rflush();      $r->rflush();
     #      #
     my $problem_types = '(option|radiobutton|numerical)';      my $problem_types = '(option|radiobutton|numerical)';
     if (exists($ENV{'form.problemchoice'}) &&       if (exists($env{'form.problemchoice'}) && 
         ! exists($ENV{'form.SelectAnother'})) {          ! exists($env{'form.SelectAnother'})) {
         foreach my $button (@SubmitButtons) {          foreach my $button (@SubmitButtons) {
             if ($button->{'name'} eq 'break') {              if ($button->{'name'} eq 'break') {
                 $r->print("<br />\n");                  $r->print("<br />\n");
Line 97  sub BuildProblemAnalysisPage { Line 101  sub BuildProblemAnalysisPage {
             $r->print($html.('&nbsp;'x5));              $r->print($html.('&nbsp;'x5));
         }          }
         #          #
         $r->print(&Apache::lonstathelpers::submission_report_form  
                                                       ('problem_analysis'));          # This is commented out pending actual implementation of 
           # CSV and Excel output.
           #$r->print(&Apache::lonstathelpers::submission_report_form
           #                                              ('problem_analysis'));
         #          #
         $r->print('<hr />');          $r->print('<hr />');
         $r->rflush();          $r->rflush();
         #          #
         # Determine which problem we are to analyze          # Determine which problem we are to analyze
         my $current_problem = &Apache::lonstathelpers::get_target_from_id          my $current_problem = &Apache::lonstathelpers::get_target_from_id
             ($ENV{'form.problemchoice'});              ($env{'form.problemchoice'});
         #          #
         my ($navmap,$prev,$curr,$next) =           my ($navmap,$prev,$curr,$next) = 
             &Apache::lonstathelpers::get_prev_curr_next($current_problem,              &Apache::lonstathelpers::get_prev_curr_next($current_problem,
                                                         $problem_types,                                                          $problem_types,
                                                         'response',                                                          'response',
                                                         );                                                          );
         if (exists($ENV{'form.PrevProblemAnalysis'}) && defined($prev)) {          if (exists($env{'form.PrevProblemAnalysis'}) && defined($prev)) {
             $current_problem = $prev;              $current_problem = $prev;
         } elsif (exists($ENV{'form.NextProblemAnalysis'}) && defined($next)) {          } elsif (exists($env{'form.NextProblemAnalysis'}) && defined($next)) {
             $current_problem = $next;              $current_problem = $next;
         } else {          } else {
             $current_problem = $curr;              $current_problem = $curr;
         }          }
         #          #
         # Store the current problem choice and send it out in the form          # Store the current problem choice and send it out in the form
         $ENV{'form.problemchoice'} =           $env{'form.problemchoice'} = 
             &Apache::lonstathelpers::make_target_id($current_problem);              &Apache::lonstathelpers::make_target_id($current_problem);
         $r->print('<input type="hidden" name="problemchoice" value="'.          $r->print('<input type="hidden" name="problemchoice" value="'.
                   $ENV{'form.problemchoice'}.'" />');                    $env{'form.problemchoice'}.'" />');
         #          #
         if (! defined($current_problem->{'resource'})) {          if (! defined($current_problem->{'resource'})) {
             $r->print('resource is undefined');              $r->print('resource is undefined');
Line 133  sub BuildProblemAnalysisPage { Line 140  sub BuildProblemAnalysisPage {
             $r->print('<h1>'.$resource->compTitle.'</h1>');              $r->print('<h1>'.$resource->compTitle.'</h1>');
             $r->print('<h3>'.$resource->src.'</h3>');              $r->print('<h3>'.$resource->src.'</h3>');
             $r->print('<h4>'.&Apache::lonstatistics::section_and_enrollment_description().'</h4>');              $r->print('<h4>'.&Apache::lonstatistics::section_and_enrollment_description().'</h4>');
             if ($ENV{'form.show_prob'} eq 'true') {              if ($env{'form.show_prob'} eq 'true') {
                 $r->print(&Apache::lonstathelpers::render_resource($resource));                  $r->print(&Apache::lonstathelpers::render_resource($resource));
             }              }
             $r->rflush();              $r->rflush();
Line 159  sub BuildProblemAnalysisPage { Line 166  sub BuildProblemAnalysisPage {
         }          }
         $r->print('<hr />');          $r->print('<hr />');
     } else {      } else {
         $r->print('<input type="submit" name="ProblemAnalysis" value="'.          my $submit_button = '<input type="submit" '.
                   &mt('Analyze Problem').'" />');              'name="ProblemAnalysis" value="'.
               &mt('Analyze Problem').'" />';
           $r->print($submit_button);
         $r->print('&nbsp;'x5);          $r->print('&nbsp;'x5);
         $r->print('<h3>'.&mt('Please select a problem to analyze').'</h3>');          $r->print('<h3>'.&mt('Please select a problem to analyze').'</h3>');
         $r->print(&Apache::lonstathelpers::problem_selector($problem_types));          $r->print(&Apache::lonstathelpers::problem_selector($problem_types,
                                                               $submit_button));
     }      }
 }  }
   
Line 178  sub numerical_response_analysis { Line 188  sub numerical_response_analysis {
     my ($r,$problem,$problem_analysis,$students) = @_;      my ($r,$problem,$problem_analysis,$students) = @_;
     my $c = $r->connection();      my $c = $r->connection();
     #      #
     if ($ENV{'form.AnalyzeOver'} !~ /^(tries|time)$/) {      if ($env{'form.AnalyzeOver'} !~ /^(tries|time)$/) {
         $r->print('Bad request');          $r->print('Bad request');
     }      }
     #      #
Line 188  sub numerical_response_analysis { Line 198  sub numerical_response_analysis {
     # Gather student data      # Gather student data
     my $response_data = &Apache::loncoursedata::get_response_data      my $response_data = &Apache::loncoursedata::get_response_data
         ([&Apache::lonstatistics::get_selected_sections()],          ([&Apache::lonstatistics::get_selected_sections()],
            [&Apache::lonstatistics::get_selected_groups()],
          $Apache::lonstatistics::enrollment_status,           $Apache::lonstatistics::enrollment_status,
          $resource->symb,$respid);           $resource->symb,$respid);
     #      #
Line 207  sub numerical_response_analysis { Line 218  sub numerical_response_analysis {
         return;          return;
     }      }
     my $analysis_html = '<table>';      my $analysis_html = '<table>';
     for (my $plot_num = 1;$plot_num<=$ENV{'form.NumPlots'};$plot_num++) {      for (my $plot_num = 1;$plot_num<=$env{'form.NumPlots'};$plot_num++) {
         my $restriction_function;          my $restriction_function;
         my $header_message;          my $header_message;
         my $stats_message;          my $stats_message;
         my $post_message; # passed through &mt sooner rather than later          my $post_message; # passed through &mt sooner rather than later
         my $no_data_message;          my $no_data_message;
         my @extra_data;          my @extra_data;
         if ($ENV{'form.AnalyzeOver'} eq 'tries') {          if ($env{'form.AnalyzeOver'} eq 'tries') {
             $restriction_function = sub {($_[0]->{'tries'} == $plot_num?1:0)};              $restriction_function = sub {($_[0]->{'tries'} == $plot_num?1:0)};
             $header_message = 'Attempt [_1]';              $header_message = 'Attempt [_1]';
             $stats_message =               $stats_message = 
Line 338  sub numerical_plot_percent { Line 349  sub numerical_plot_percent {
     while (my ($ans,$submissions) = each(%$responses)) {      while (my ($ans,$submissions) = each(%$responses)) {
         while (my ($submission,$counts) = each(%$submissions)) {          while (my ($submission,$counts) = each(%$submissions)) {
             my ($correct_count,$incorrect_count) = @$counts;              my ($correct_count,$incorrect_count) = @$counts;
             my $scaled_value = 100*($submission-$ans)/abs($ans);      my $scaled_value = 
    ($ans) ? 100*($submission-$ans)/abs($ans)
          : 0;
             if ($scaled_value < $bins[0]) {              if ($scaled_value < $bins[0]) {
                 $bins[0]=$scaled_value -1;                  $bins[0]=$scaled_value -1;
             }              }
Line 393  sub numerical_plot_differences { Line 406  sub numerical_plot_differences {
     } elsif ($low_bin < 0 && $high_bin < -$low_bin) {      } elsif ($low_bin < 0 && $high_bin < -$low_bin) {
         $high_bin = -$low_bin;          $high_bin = -$low_bin;
     }      }
     if (($high_bin -$low_bin)/$min_bin_size * 2 > $max_bins) {      if (!$min_bin_size ||
           ($high_bin -$low_bin)/$min_bin_size * 2 > $max_bins) {
         $min_bin_size = abs($high_bin - $low_bin) / $max_bins * 2;          $min_bin_size = abs($high_bin - $low_bin) / $max_bins * 2;
     }      }
     my @bins;      my @bins;
Line 743  sub circle { Line 757  sub circle {
 sub radio_response_analysis {  sub radio_response_analysis {
     my ($r,$problem,$problem_analysis,$students) = @_;      my ($r,$problem,$problem_analysis,$students) = @_;
     #      #
     if ($ENV{'form.AnalyzeOver'} !~ /^(tries|time)$/) {      if ($env{'form.AnalyzeOver'} !~ /^(tries|time)$/) {
         $r->print('Bad request');          $r->print('Bad request');
     }      }
     #      #
Line 773  sub radio_response_analysis { Line 787  sub radio_response_analysis {
     # Gather student data      # Gather student data
     my $response_data = &Apache::loncoursedata::get_response_data      my $response_data = &Apache::loncoursedata::get_response_data
         ([&Apache::lonstatistics::get_selected_sections()],          ([&Apache::lonstatistics::get_selected_sections()],
            [&Apache::lonstatistics::get_selected_groups()],
          $Apache::lonstatistics::enrollment_status,           $Apache::lonstatistics::enrollment_status,
          $resource->symb,$respid);           $resource->symb,$respid);
     my $correct;   # either a hash reference or a scalar      my $correct;   # either a hash reference or a scalar
Line 785  sub radio_response_analysis { Line 800  sub radio_response_analysis {
             my ($idx,@remainder) = split('&',$student->{'answer'});              my ($idx,@remainder) = split('&',$student->{'answer'});
             my ($answer) = ($remainder[$idx]=~/^(.*)=([^=]*)$/);              my ($answer) = ($remainder[$idx]=~/^(.*)=([^=]*)$/);
             $correct->{$student->{'username'}.':'.$student->{'domain'}}=              $correct->{$student->{'username'}.':'.$student->{'domain'}}=
                 &Apache::lonnet::unescape($answer);                  &unescape($answer);
         }          }
     } else {      } else {
         foreach my $foil (keys(%$foildata)) {          foreach my $foil (keys(%$foildata)) {
Line 804  sub radio_response_analysis { Line 819  sub radio_response_analysis {
     }      }
     #      #
     $analysis_html.='<table>';      $analysis_html.='<table>';
     for (my $plot_num = 1;$plot_num<=$ENV{'form.NumPlots'};$plot_num++) {      for (my $plot_num = 1;$plot_num<=$env{'form.NumPlots'};$plot_num++) {
         # classify data ->correct foil -> selected foil          # classify data ->correct foil -> selected foil
         my ($restriction_function,          my ($restriction_function,
             $correct_foil_title,$incorrect_foil_title,              $correct_foil_title,$incorrect_foil_title,
             $pre_graph_text,$post_graph_text,              $pre_graph_text,$post_graph_text,
             $no_data_text,@extra_data);              $no_data_text,@extra_data);
         if ($ENV{'form.AnalyzeOver'} eq 'tries') {          if ($env{'form.AnalyzeOver'} eq 'tries') {
             $restriction_function = sub {($_[0]->{'tries'} == $plot_num?1:0)};              $restriction_function = sub {($_[0]->{'tries'} == $plot_num?1:0)};
             $correct_foil_title = 'Attempt '.$plot_num;              $correct_foil_title = 'Attempt '.$plot_num;
             $incorrect_foil_title = 'Attempt '.$plot_num;              $incorrect_foil_title = 'Attempt '.$plot_num;
Line 818  sub radio_response_analysis { Line 833  sub radio_response_analysis {
                 'Attempt [_1], [_2] submissions, [_3] correct, [_4] incorrect';                  'Attempt [_1], [_2] submissions, [_3] correct, [_4] incorrect';
             $post_graph_text = '';              $post_graph_text = '';
             $no_data_text = 'No data exists for attempt [_1]';              $no_data_text = 'No data exists for attempt [_1]';
         } elsif ($ENV{'form.AnalyzeOver'} eq 'time') {          } elsif ($env{'form.AnalyzeOver'} eq 'time') {
             my $starttime = &Apache::lonhtmlcommon::get_date_from_form              my $starttime = &Apache::lonhtmlcommon::get_date_from_form
                 ('startdate_'.$plot_num);                  ('startdate_'.$plot_num);
             my $endtime = &Apache::lonhtmlcommon::get_date_from_form              my $endtime = &Apache::lonhtmlcommon::get_date_from_form
Line 940  sub ensure_start_end_times { Line 955  sub ensure_start_end_times {
             (localtime($last - $sec_in_day*($plot_num-1)))[3..5];              (localtime($last - $sec_in_day*($plot_num-1)))[3..5];
         $start = &Time::Local::timelocal(0,0,0,$sday,$smon,$syear);          $start = &Time::Local::timelocal(0,0,0,$sday,$smon,$syear);
         $end   = $start + $sec_in_day;          $end   = $start + $sec_in_day;
         if ($plot_num == $ENV{'form.NumPlots'}) {          if ($plot_num == $env{'form.NumPlots'}) {
             $start = $first;              $start = $first;
         }          }
     }      }
Line 1178  sub OptionResponseAnalysis { Line 1193  sub OptionResponseAnalysis {
     # Note: part data is not needed.      # Note: part data is not needed.
     my $PerformanceData = &Apache::loncoursedata::get_response_data      my $PerformanceData = &Apache::loncoursedata::get_response_data
         ([&Apache::lonstatistics::get_selected_sections()],          ([&Apache::lonstatistics::get_selected_sections()],
            [&Apache::lonstatistics::get_selected_groups()],
          $Apache::lonstatistics::enrollment_status,           $Apache::lonstatistics::enrollment_status,
          $resource->symb,$respid);           $resource->symb,$respid);
     if (! defined($PerformanceData) ||       if (! defined($PerformanceData) || 
Line 1187  sub OptionResponseAnalysis { Line 1203  sub OptionResponseAnalysis {
                   '</h2>');                    '</h2>');
     }  else {      }  else {
         $r->rflush();          $r->rflush();
         if ($ENV{'form.AnalyzeOver'} eq 'tries') {          if ($env{'form.AnalyzeOver'} eq 'tries') {
             my $analysis_html = &OR_tries_analysis($r,              my $analysis_html = &OR_tries_analysis($r,
                                                    $PerformanceData,                                                     $PerformanceData,
                                                    $problem_data);                                                     $problem_data);
             $r->print($analysis_html);              $r->print($analysis_html);
             $r->rflush();              $r->rflush();
         } elsif ($ENV{'form.AnalyzeOver'} eq 'time') {          } elsif ($env{'form.AnalyzeOver'} eq 'time') {
             my $analysis_html = &OR_time_analysis($PerformanceData,              my $analysis_html = &OR_time_analysis($PerformanceData,
                                                   $problem_data);                                                    $problem_data);
             $r->print($analysis_html);              $r->print($analysis_html);
Line 1215  sub OptionResponseAnalysis { Line 1231  sub OptionResponseAnalysis {
 sub OR_tries_analysis {  sub OR_tries_analysis {
     my ($r,$PerformanceData,$ORdata) = @_;      my ($r,$PerformanceData,$ORdata) = @_;
     my $mintries = 1;      my $mintries = 1;
     my $maxtries = $ENV{'form.NumPlots'};      my $maxtries = $env{'form.NumPlots'};
     my ($table,$Foils,$Concepts) = &build_foil_index($ORdata);      my ($table,$Foils,$Concepts) = &build_foil_index($ORdata);
     if (! defined($Concepts)) {      if (! defined($Concepts)) {
         $Concepts = [];          $Concepts = [];
Line 1425  sub OR_time_analysis { Line 1441  sub OR_time_analysis {
                 '</h3>'.$table;                  '</h3>'.$table;
     }      }
     #      #
     my $num_plots = $ENV{'form.NumPlots'};      my $num_plots = $env{'form.NumPlots'};
     my $num_data = scalar(@$performance_data)-1;      my $num_data = scalar(@$performance_data)-1;
     #      #
     my $current_index;      my $current_index;
Line 1808  sub build_foil_key { Line 1824  sub build_foil_key {
 sub CreateInterface {  sub CreateInterface {
     ##      ##
     ## Environment variable initialization      ## Environment variable initialization
     if (! exists$ENV{'form.AnalyzeOver'}) {      if (! exists($env{'form.AnalyzeOver'})) {
         $ENV{'form.AnalyzeOver'} = 'tries';          $env{'form.AnalyzeOver'} = 'tries';
     }      }
     ##      ##
     ## Build the menu      ## Build the menu
     my $Str = '';      my $Str = '';
     $Str .= &Apache::lonhtmlcommon::breadcrumbs      $Str .= &Apache::lonhtmlcommon::breadcrumbs('Detailed Problem Analysis');
         (undef,'Detailed Problem Analysis');  
     $Str .= '<table cellspacing="5">'."\n";      $Str .= '<table cellspacing="5">'."\n";
     $Str .= '<tr>';      $Str .= '<tr>';
     $Str .= '<td align="center"><b>'.&mt('Sections').'</b></td>';      $Str .= '<td align="center"><b>'.&mt('Sections').'</b></td>';
       $Str .= '<td align="center"><b>'.&mt('Groups').'</b></td>';
     $Str .= '<td align="center"><b>'.&mt('Enrollment Status').'</b></td>';      $Str .= '<td align="center"><b>'.&mt('Enrollment Status').'</b></td>';
     $Str .= '<td align="center">&nbsp;</td>';      $Str .= '<td align="center">&nbsp;</td>';
     $Str .= '</tr>'."\n";      $Str .= '</tr>'."\n";
Line 1828  sub CreateInterface { Line 1844  sub CreateInterface {
     $Str .= &Apache::lonstatistics::SectionSelect('Section','multiple',5);      $Str .= &Apache::lonstatistics::SectionSelect('Section','multiple',5);
     $Str .= '</td>';      $Str .= '</td>';
     #      #
       $Str .= '<td align="center">'."\n";
       $Str .= &Apache::lonstatistics::GroupSelect('Group','multiple',5);
       $Str .= '</td>';
       #
     $Str .= '<td align="center">';      $Str .= '<td align="center">';
     $Str .= &Apache::lonhtmlcommon::StatusOptions(undef,undef,5);      $Str .= &Apache::lonhtmlcommon::StatusOptions(undef,undef,5);
     $Str .= '</td>';      $Str .= '</td>';
Line 1838  sub CreateInterface { Line 1858  sub CreateInterface {
     ##      ##
     my $showprob_checkbox =       my $showprob_checkbox = 
         '<input type="checkbox" name="show_prob" value="true" ';          '<input type="checkbox" name="show_prob" value="true" ';
     if ($ENV{'form.show_prob'} eq 'true') {      if ($env{'form.show_prob'} eq 'true') {
         $showprob_checkbox .= 'checked ';          $showprob_checkbox .= 'checked ';
     }      }
     $showprob_checkbox.= ' />';      $showprob_checkbox.= ' />';
Line 1848  sub CreateInterface { Line 1868  sub CreateInterface {
     ##      ##
     my $analyze_selector = '<select name="AnalyzeOver" >';      my $analyze_selector = '<select name="AnalyzeOver" >';
     $analyze_selector .= '<option value="tries" ';      $analyze_selector .= '<option value="tries" ';
     if (! exists($ENV{'form.AnalyzeOver'}) ||       if (! exists($env{'form.AnalyzeOver'}) || 
         $ENV{'form.AnalyzeOver'} eq 'tries'){          $env{'form.AnalyzeOver'} eq 'tries'){
         # Default to tries          # Default to tries
         $analyze_selector .= ' selected ';          $analyze_selector .= ' selected ';
     }      }
     $analyze_selector .= '>'.&mt('Tries').'</option>';      $analyze_selector .= '>'.&mt('Tries').'</option>';
     $analyze_selector .= '<option value="time" ';      $analyze_selector .= '<option value="time" ';
     $analyze_selector .= ' selected ' if ($ENV{'form.AnalyzeOver'} eq 'time');      $analyze_selector .= ' selected ' if ($env{'form.AnalyzeOver'} eq 'time');
     $analyze_selector .= '>'.&mt('Time').'</option>';      $analyze_selector .= '>'.&mt('Time').'</option>';
     $analyze_selector .= '</select>';      $analyze_selector .= '</select>';
     $Str .= '<nobr><label>'.      $Str .= '<nobr><label>'.
Line 1865  sub CreateInterface { Line 1885  sub CreateInterface {
             '</label></nobr><br />'.$/;              '</label></nobr><br />'.$/;
     ##      ##
     my $numplots_selector = '<select name="NumPlots">';      my $numplots_selector = '<select name="NumPlots">';
     if (! exists($ENV{'form.NumPlots'})       if (! exists($env{'form.NumPlots'}) 
         || $ENV{'form.NumPlots'} < 1           || $env{'form.NumPlots'} < 1 
         || $ENV{'form.NumPlots'} > 20) {          || $env{'form.NumPlots'} > 20) {
         $ENV{'form.NumPlots'} = 5;          $env{'form.NumPlots'} = 5;
     }      }
     foreach my $i (1,2,3,4,5,6,7,8,10,15,20) {      foreach my $i (1,2,3,4,5,6,7,8,10,15,20) {
         $numplots_selector .= '<option value="'.$i.'" ';          $numplots_selector .= '<option value="'.$i.'" ';
         if ($ENV{'form.NumPlots'} == $i) { $numplots_selector.=' selected '; }          if ($env{'form.NumPlots'} == $i) { $numplots_selector.=' selected '; }
         $numplots_selector .= '>'.$i.'</option>';          $numplots_selector .= '>'.$i.'</option>';
     }      }
     $numplots_selector .= '</select></nobr><br />';      $numplots_selector .= '</select></nobr><br />';
Line 1920  sub hashify_attempt { Line 1940  sub hashify_attempt {
     my %attempt;      my %attempt;
     $attempt{'student'}    = $row->[&Apache::loncoursedata::RD_sname()];      $attempt{'student'}    = $row->[&Apache::loncoursedata::RD_sname()];
     $attempt{'tries'}      = $row->[&Apache::loncoursedata::RD_tries()];      $attempt{'tries'}      = $row->[&Apache::loncoursedata::RD_tries()];
     $attempt{'submission'} = &Apache::lonnet::unescape($row->[&Apache::loncoursedata::RD_submission()]);      $attempt{'submission'} = &unescape($row->[&Apache::loncoursedata::RD_submission()]);
     $attempt{'award'}      = $row->[&Apache::loncoursedata::RD_awarddetail()];      $attempt{'award'}      = $row->[&Apache::loncoursedata::RD_awarddetail()];
     $attempt{'timestamp'}  = $row->[&Apache::loncoursedata::RD_timestamp()];      $attempt{'timestamp'}  = $row->[&Apache::loncoursedata::RD_timestamp()];
     return %attempt;      return %attempt;
Line 1944  sub Process_OR_Row { Line 1964  sub Process_OR_Row {
     my @Foilsubs   = split('&',$submission);      my @Foilsubs   = split('&',$submission);
     for (my $j=0;$j<=$#Foilgrades;$j++) {      for (my $j=0;$j<=$#Foilgrades;$j++) {
         my ($foilid,$correct)  = split('=',$Foilgrades[$j]);          my ($foilid,$correct)  = split('=',$Foilgrades[$j]);
         $foilid = &Apache::lonnet::unescape($foilid);          $foilid = &unescape($foilid);
         my (undef,$submission) = split('=',$Foilsubs[$j]);          my (undef,$submission) = split('=',$Foilsubs[$j]);
         if ($correct) {          if ($correct) {
             $RowData{$foilid}->{'_correct'}++;              $RowData{$foilid}->{'_correct'}++;
         } else {          } else {
             $submission = &Apache::lonnet::unescape($submission);              $submission = &unescape($submission);
             $RowData{$foilid}->{$submission}++;              $RowData{$foilid}->{$submission}++;
         }          }
         $RowData{$foilid}->{'_total'}++;          $RowData{$foilid}->{'_total'}++;

Removed from v.1.119  
changed lines
  Added in v.1.127


FreeBSD-CVSweb <freebsd-cvsweb@FreeBSD.org>