Diff for /loncom/interface/statistics/lonstudentsubmissions.pm between versions 1.59 and 1.60

version 1.59, 2010/04/12 16:28:22 version 1.60, 2010/08/04 01:56:07
Line 76  sub BuildStudentSubmissionsPage { Line 76  sub BuildStudentSubmissionsPage {
                                    '<div class="LC_info">'.&mt('Loading student data...').'</div>');                                     '<div class="LC_info">'.&mt('Loading student data...').'</div>');
     $r->rflush();      $r->rflush();
     #      #
       my %anoncounter =
            &Apache::lonnet::dump('nohist_anonsurveys',
                           $env{'course.'.$env{'request.course.id'}.'.domain'},
                           $env{'course.'.$env{'request.course.id'}.'.num'});
     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) {
Line 97  sub BuildStudentSubmissionsPage { Line 101  sub BuildStudentSubmissionsPage {
         # Determine which problems we are to analyze          # Determine which problems we are to analyze
         my @Symbs =           my @Symbs = 
             &Apache::lonstathelpers::get_selected_symbs('problemchoice');              &Apache::lonstathelpers::get_selected_symbs('problemchoice');
         foreach my $selected (@Symbs) {  
             $r->print('<input type="hidden" name="problemchoice" value="'.          # If there are multi-part problems with anonymous survey and named
                       $selected.'" />'.$/);          # parts check if named was picked for display.
           #
           my %mixed_named; 
           foreach my $envkey (%env) {
               if ($envkey =~ /^form\.mixed_(\d+:\d+)$/) {
                   my $item = $1; 
                   if ($env{$envkey} =~ /^symb_(.+)$/) {
                       my $symb = &unescape($1);
                       if (ref($mixed_named{$symb}) eq 'ARRAY') {
                           push(@{$mixed_named{$symb}},$item);
                       } else {
                           @{$mixed_named{$symb}} = ($item);
                       }
                   }
               }
         }          }
         #          #
         # Get resource objects          # Get resource objects
         my $navmap = Apache::lonnavmaps::navmap->new();          my $navmap = Apache::lonnavmaps::navmap->new();
         if (!defined($navmap)) {          if (!defined($navmap)) {
               foreach my $selected (@Symbs) {
                   $r->print('<input type="hidden" name="problemchoice" value="'.
                             &escape($selected).'" />'.$/);
                   if (ref($mixed_named{$selected}) eq 'ARRAY') {
                       foreach my $item (@{$mixed_named{$selected}}) {
                           $r->print('<input type="hidden" name="mixed_'.$item.'" value="'.&escape($selected).'" />'.$/);
                       }
                   }
               }
             $r->print('<div class="LC_error">'.&mt("Internal error").'</div>');              $r->print('<div class="LC_error">'.&mt("Internal error").'</div>');
             return;              return;
         }          }
         my %already_seen;          my %already_seen;
         my @Problems;          my (@Problems,@anonProbs,@namedProbs,$show_named);
         foreach my $symb (@Symbs) {          foreach my $symb (@Symbs) {
             my $resource = $navmap->getBySymb($symb);              my $resource = $navmap->getBySymb($symb);
             push(@Problems,$resource);              my ($hasanon,$hasnamed);
               if (ref($resource)) {
                   foreach my $partid (@{$resource->parts}) {
                       if (($anoncounter{$symb."\0".$partid}) || ($resource->is_anonsurvey($partid))) {
                           unless (exists($mixed_named{$symb})) {
                               $hasanon = 1;
                           }
                       } else {
                           $hasnamed = 1;
                       }
                   }
                   if ($hasanon) {
                       push(@anonProbs,$resource);
                   } elsif ($hasnamed) {
                       push(@namedProbs,$resource);
                   }
               }
         }          }
         #           if (@namedProbs > 0) {
               @Problems = @namedProbs;
               $show_named = 1;
           } elsif (@anonProbs > 0) {
               @Problems = @anonProbs;
           } 
           foreach my $selected (@Symbs) {
               $r->print('<input type="hidden" name="problemchoice" value="'.
                         &escape($selected).'" />'.$/);
               if (ref($mixed_named{$selected}) eq 'ARRAY') {
                   foreach my $item (@{$mixed_named{$selected}}) {
                       $r->print('<input type="hidden" name="mixed_'.$item.'" value="'.&escape($selected).'" />'.$/);
                   }
               }
           }
           # If these are to be anonymized, do a random shuffle of @Students. 
           unless ($show_named) {
               &array_shuffle(\@Students);
           }
           #
         $r->print('<h4>'.          $r->print('<h4>'.
                   &Apache::lonstatistics::section_and_enrollment_description().                    &Apache::lonstatistics::section_and_enrollment_description().
                   '</h4>');                    '</h4>');
Line 131  sub BuildStudentSubmissionsPage { Line 193  sub BuildStudentSubmissionsPage {
                 }                  }
             }              }
             if ($env{'form.output'} eq 'excel') {              if ($env{'form.output'} eq 'excel') {
                 &prepare_excel_output($r,\@Problems,\@Students);                  &prepare_excel_output($r,\@Problems,\@Students,\%anoncounter,$show_named);
             } elsif ($env{'form.output'} eq 'csv') {              } elsif ($env{'form.output'} eq 'csv') {
                 &prepare_csv_output($r,\@Problems,\@Students);                  &prepare_csv_output($r,\@Problems,\@Students,\%anoncounter,$show_named);
             } else {              } else {
                 &prepare_html_output($r,\@Problems,\@Students);                  &prepare_html_output($r,\@Problems,\@Students,\%anoncounter,$show_named);
             }              }
         }          }
         $r->print('<hr />');          $r->print('<hr />');
Line 150  sub BuildStudentSubmissionsPage { Line 212  sub BuildStudentSubmissionsPage {
                   &mt('Please select problems and use the [_1]Prepare Report[_2] button to continue.','<b>','</b>').                    &mt('Please select problems and use the [_1]Prepare Report[_2] button to continue.','<b>','</b>').
                   '</p>');                    '</p>');
         $r->print(&Apache::lonstathelpers::MultipleProblemSelector          $r->print(&Apache::lonstathelpers::MultipleProblemSelector
                   (undef,'problemchoice','Statistics'));                    (undef,'problemchoice','Statistics',\%anoncounter));
     }      }
 }  }
   
   sub array_shuffle {
       my $array = shift;
       return unless (ref($array) eq 'ARRAY');
       my $i = scalar(@$array);
       my $j;
       foreach my $item (@$array) {
           --$i;
           $j = int(rand($i+1));
           next if($i == $j);
           @$array [$i,$j] = @$array[$j,$i];
       }
       return @$array;
   }
   
 ##  ##
 ## get_extra_response_headers  ## get_extra_response_headers
 ##  ##
 sub get_extra_response_headers {  sub get_extra_response_headers {
       my ($show_named) = @_;
     my @extra_resp_headers;      my @extra_resp_headers;
     if ($env{'form.correctans'} eq 'true') {      if ($env{'form.correctans'} eq 'true') {
         push(@extra_resp_headers,'Correct');          push(@extra_resp_headers,'Correct');
     }       }
     if ($env{'form.prob_status'} eq 'true') {      if ($show_named) { 
         push(@extra_resp_headers,'Award Detail');           if ($env{'form.prob_status'} eq 'true') {
         push(@extra_resp_headers,'Time');              push(@extra_resp_headers,'Award Detail'); 
         push(@extra_resp_headers,'Attempt');              push(@extra_resp_headers,'Time');
         push(@extra_resp_headers,'Awarded');              push(@extra_resp_headers,'Attempt');
               push(@extra_resp_headers,'Awarded');
           }
     }      }
     return @extra_resp_headers;      return @extra_resp_headers;
 }  }
Line 211  sub get_headers { Line 290  sub get_headers {
 #########################################################  #########################################################
 #########################################################  #########################################################
 sub prepare_html_output {  sub prepare_html_output {
     my ($r,$problems,$students) = @_;      my ($r,$problems,$students,$anoncounter,$show_named) = @_;
     my $c = $r->connection();      my $c = $r->connection();
     #      #
     # Set a flag for the case when there is just one problem      # Set a flag for the case when there is just one problem
Line 222  sub prepare_html_output { Line 301  sub prepare_html_output {
     }      }
     #      #
     # Compute the number of columns per response      # Compute the number of columns per response
     my @extra_resp_headers = &get_extra_response_headers();      my @extra_resp_headers = &get_extra_response_headers($show_named);
     #      #
     # Create the table header      # Create the table header
     my @student_columns = @Apache::lonstatistics::SelectedStudentData;      my @student_columns;
     foreach (@student_columns) {      if ($show_named) {
         if($_ eq 'all') {          @student_columns = @Apache::lonstatistics::SelectedStudentData;
             @student_columns = ('fullname','username','domain','id','section','status','groups','comments');          if (grep(/^all$/,@student_columns)) {
               @student_columns = qw(fullname username domain id section status groups comments);
         }          }
       } else {
           @student_columns = ('username');
     }      }
     #      #
     my %headers;      my %headers;
Line 247  sub prepare_html_output { Line 329  sub prepare_html_output {
     foreach my $prob (@$problems) {      foreach my $prob (@$problems) {
         my %analysis = &Apache::lonstathelpers::get_problem_data($prob->src);          my %analysis = &Apache::lonstathelpers::get_problem_data($prob->src);
         $problem_analysis{$prob->src}=\%analysis;          $problem_analysis{$prob->src}=\%analysis;
           my $symb = $prob->symb();
         #          #
         my $prob_span = 0;          my $prob_span = 0;
         my $single_part = 0;          my $single_part = 0;
         if (scalar(@{$prob->parts}) == 1) {          if (scalar(@{$prob->parts}) == 1) {
             $single_part = 1;              $single_part = 1;
         }          }
           my $shown_parts = 0;
         foreach my $partid (@{$prob->parts}) {          foreach my $partid (@{$prob->parts}) {
               if (($prob->is_anonsurvey($partid)) || ($anoncounter->{$symb."\0".$partid})) {
                   next if ($show_named);
               } else {
                   next unless ($show_named);
               }
               $shown_parts ++;
             my $part_span = 0;              my $part_span = 0;
             my $responses = [$prob->responseIds($partid)];              my $responses = [$prob->responseIds($partid)];
             my $resptypes = [$prob->responseType($partid)];              my $resptypes = [$prob->responseType($partid)];
Line 294  sub prepare_html_output { Line 384  sub prepare_html_output {
             }              }
             $prob_span += $part_span;              $prob_span += $part_span;
         }          }
           next if (!$shown_parts);
         my $title = $prob->compTitle;          my $title = $prob->compTitle;
         if ($prob_span > 0) {          if ($prob_span > 0) {
             $headers{'problem'}.= qq{<th colspan="$prob_span">$title</th>};              $headers{'problem'}.= qq{<th colspan="$prob_span">$title</th>};
Line 330  sub prepare_html_output { Line 421  sub prepare_html_output {
             $count = 0;              $count = 0;
         }          }
         last if ($c->aborted());          last if ($c->aborted());
         foreach my $field (@student_columns) {          if ($show_named) {
             $student_row_data .= '<td valign="top">';              foreach my $field (@student_columns) {
             # handle comments like in lonstudentassessment.pm                  $student_row_data .= '<td valign="top">';
             if($field eq 'comments') {                  # handle comments like in lonstudentassessment.pm
                 $student_row_data .=                   if($field eq 'comments') {
                       $student_row_data .= 
                         '<a href="/adm/'.$student->{'domain'}.'/'.                          '<a href="/adm/'.$student->{'domain'}.'/'.
                         $student->{'username'}.'/'.'aboutme#coursecomment">'.&mt('Comments').'</a>';                          $student->{'username'}.'/'.'aboutme#coursecomment">'.&mt('Comments').'</a>';
             } else {                  } else {
                 $student_row_data .= $student->{$field};                      $student_row_data .= $student->{$field};
                   }
                   $student_row_data .= '</td>';
             }              }
             $student_row_data .= '</td>';          } else {
               $student_row_data = '<td valign="top" colspan="'.$student_column_count.'">'.&mt('Anonymized').'</td>';
         }          }
         #          #
         # Figure out what it is we need to output for this student          # Figure out what it is we need to output for this student
Line 348  sub prepare_html_output { Line 443  sub prepare_html_output {
         my %prob_data;          my %prob_data;
         my $maxrow;          my $maxrow;
         foreach my $prob (@$problems) {          foreach my $prob (@$problems) {
             $prob_data{$prob->symb}={};              my $symb = $prob->symb;
               $prob_data{$symb}={};
             foreach my $partid (@{$prob->parts}) {              foreach my $partid (@{$prob->parts}) {
                   if (($prob->is_anonsurvey($partid)) || ($anoncounter->{$symb."\0".$partid})) {
                       next if ($show_named);
                   } else {
                       next unless ($show_named);
                   }
                 my @responses = $prob->responseIds($partid);                  my @responses = $prob->responseIds($partid);
                 my @response_type = $prob->responseType($partid);                  my @response_type = $prob->responseType($partid);
                 for (my $i=0;$i<=$#responses;$i++) {                  for (my $i=0;$i<=$#responses;$i++) {
Line 639  sub html_non_essay_results { Line 740  sub html_non_essay_results {
 #########################################################  #########################################################
 #########################################################  #########################################################
 sub prepare_excel_output {  sub prepare_excel_output {
     my ($r,$Problems,$Students) = @_;      my ($r,$Problems,$Students,$anoncounter,$show_named) = @_;
     my $c = $r->connection();      my $c = $r->connection();
     #      #
     #      #
     # Determine the number of columns in the spreadsheet      # Determine the number of columns in the spreadsheet
     my $columncount = 3; # username, domain, id      my $columncount = 3; # username, domain, id
     my @extra_resp_headers = &get_extra_response_headers();      my @extra_resp_headers = &get_extra_response_headers($show_named);
     my $lastprob;      my $lastprob;
     my %problem_analysis;      my %problem_analysis;
     foreach my $prob (@$Problems) {      foreach my $prob (@$Problems) {
           my $symb = $prob->symb();
         my %analysis = &Apache::lonstathelpers::get_problem_data($prob->src);          my %analysis = &Apache::lonstathelpers::get_problem_data($prob->src);
         $problem_analysis{$prob->src}=\%analysis;          $problem_analysis{$prob->src}=\%analysis;
         foreach my $partid (@{$prob->parts}) {          foreach my $partid (@{$prob->parts}) {
               if (($prob->is_anonsurvey($partid)) || ($anoncounter->{$symb."\0".$partid})) {
                   next if ($show_named);
               } else {
                   next unless ($show_named);
               }
   
             my $responses = [$prob->responseIds($partid)];              my $responses = [$prob->responseIds($partid)];
             my $resptypes = [$prob->responseType($partid)];              my $resptypes = [$prob->responseType($partid)];
             for (my $i=0;$i<scalar(@$responses);$i++) {              for (my $i=0;$i<scalar(@$responses);$i++) {
Line 669  sub prepare_excel_output { Line 777  sub prepare_excel_output {
     if ($columncount > 255) {      if ($columncount > 255) {
         $r->print('<h1>'.&mt('Unable to complete request').'</h1>'.$/.          $r->print('<h1>'.&mt('Unable to complete request').'</h1>'.$/.
                   '<p>'.&mt('LON-CAPA is unable to produce your Excel spreadsheet because your selections will result in more than 255 columns.  Excel allows only 255 columns in a spreadsheet.').'</p>'.$/.                    '<p>'.&mt('LON-CAPA is unable to produce your Excel spreadsheet because your selections will result in more than 255 columns.  Excel allows only 255 columns in a spreadsheet.').'</p>'.$/.
                   '<p>'.&mt('Consider selecting fewer problems to generate reports on, or reducing the number of items per problem.  Or use HTML or CSV output.').'</p>'.$/.                    '<p>'.&mt('Consider selecting fewer problems to generate reports on, or reducing the number of items per problem.  Or use HTML or CSV output.').'</p>'.$/);
                   '<p>'.&mt('The last problem that will fit in the current spreadsheet is [_1].',$lastprob->compTitle).'</p>');          if (ref($lastprob)) {
               $r->print('<p>'.&mt('The last problem that will fit in the current spreadsheet is [_1].',$lastprob->compTitle).'</p>');
           }
         $r->rflush();          $r->rflush();
         return;          return;
     }      }
Line 709  sub prepare_excel_output { Line 819  sub prepare_excel_output {
     $worksheet->write($partid_row,0,'Part ID',$format->{'bold'});      $worksheet->write($partid_row,0,'Part ID',$format->{'bold'});
     $worksheet->write($respid_row,0,'Response ID',$format->{'bold'});      $worksheet->write($respid_row,0,'Response ID',$format->{'bold'});
     # Student headers      # Student headers
     my @StudentColumns = ('username','domain','id','section');      my @StudentColumns;
       if ($show_named) {
           @StudentColumns = qw(username domain id section);
       } else {
            @StudentColumns = qw(username);
       }
     foreach (@StudentColumns) {      foreach (@StudentColumns) {
         $worksheet->write($header_row,$cols_output++,ucfirst($_),          $worksheet->write($header_row,$cols_output++,ucfirst($_),
                           $format->{'bold'});                            $format->{'bold'});
Line 718  sub prepare_excel_output { Line 833  sub prepare_excel_output {
     my %start_col;      my %start_col;
     foreach my $prob (@$Problems) {      foreach my $prob (@$Problems) {
         my $title = $prob->compTitle;          my $title = $prob->compTitle;
           my $symb = $prob->symb();
         $worksheet->write($title_row,$cols_output,          $worksheet->write($title_row,$cols_output,
                           $title,$format->{'h3'});                            $title,$format->{'h3'});
         foreach my $partid (@{$prob->parts}) {          foreach my $partid (@{$prob->parts}) {
               if (($prob->is_anonsurvey($partid)) || ($anoncounter->{$symb."\0".$partid})) {
                   next if ($show_named);
               } else {
                   next unless ($show_named);
               }
             $worksheet->write($partid_row,$cols_output,              $worksheet->write($partid_row,$cols_output,
                               $prob->part_display($partid));                                $prob->part_display($partid));
             my $responses = [$prob->responseIds($partid)];              my $responses = [$prob->responseIds($partid)];
Line 757  sub prepare_excel_output { Line 878  sub prepare_excel_output {
         $cols_output = 0;          $cols_output = 0;
         my $student_row = $max_row;          my $student_row = $max_row;
         foreach my $field (@StudentColumns) {          foreach my $field (@StudentColumns) {
             $worksheet->write($student_row,$cols_output++,              if ($show_named) {
                               $student->{$field});                  $worksheet->write($student_row,$cols_output++,
                                     $student->{$field});
               } else {
                   $worksheet->write($student_row,$cols_output++,
                                         &mt('Anonymized'));
               }
         }          }
         my $last_student_col = $cols_output-1;          my $last_student_col = $cols_output-1;
         foreach my $prob (@$Problems) {          foreach my $prob (@$Problems) {
               my $symb = $prob->symb();
             foreach my $partid (@{$prob->parts}) {              foreach my $partid (@{$prob->parts}) {
                   if (($prob->is_anonsurvey($partid)) || ($anoncounter->{$symb."\0".$partid})) {
                       next if ($show_named);
                   } else {
                       next unless ($show_named);
                   }
                 my @Response = $prob->responseIds($partid);                  my @Response = $prob->responseIds($partid);
                 my @ResponseType = $prob->responseType($partid);                  my @ResponseType = $prob->responseType($partid);
                 for (my $i=0;$i<=$#Response;$i++) {                  for (my $i=0;$i<=$#Response;$i++) {
Line 813  sub prepare_excel_output { Line 945  sub prepare_excel_output {
         for (my $row = $student_row+1;$row<$max_row;$row++) {          for (my $row = $student_row+1;$row<$max_row;$row++) {
             my $cols = 0;              my $cols = 0;
             foreach my $field (@StudentColumns) {              foreach my $field (@StudentColumns) {
                 $worksheet->write($row,$cols++,                  if ($show_named) {
                                   $student->{$field});                      $worksheet->write($row,$cols++,
                                         $student->{$field});
                   } else {
                       $worksheet->write($row,$cols++,
                                         &mt('Anonymized'));
                   }
             }              }
         }          }
         &Apache::lonhtmlcommon::Increment_PrgWin($r,\%prog_state,          &Apache::lonhtmlcommon::Increment_PrgWin($r,\%prog_state,
Line 924  sub excel_format_item { Line 1061  sub excel_format_item {
 #########################################################  #########################################################
 #########################################################  #########################################################
 sub prepare_csv_output {  sub prepare_csv_output {
     my ($r,$problems,$students) = @_;      my ($r,$problems,$students,$anoncounter,$show_named) = @_;
     my $c = $r->connection();      my $c = $r->connection();
     #      #
     $r->print('<h2>'.      $r->print('<h2>'.
Line 956  sub prepare_csv_output { Line 1093  sub prepare_csv_output {
     }      }
     #      #
     # Compute the number of columns per response      # Compute the number of columns per response
     my @extra_resp_headers = &get_extra_response_headers();      my @extra_resp_headers = &get_extra_response_headers($show_named);
     #      #
     # Create the table header      # Create the table header
     my @student_columns = ('username','domain','id','section');      my @student_columns = ('username','domain','id','section');
       if ($show_named) {
           @student_columns = qw(username domain id section);
       } else {
           @student_columns = qw(username);
       }
       my $student_column_count = scalar(@student_columns);
     #      #
     my %headers;      my %headers;
     push(@{$headers{'student'}},@student_columns);      push(@{$headers{'student'}},@student_columns);
Line 973  sub prepare_csv_output { Line 1116  sub prepare_csv_output {
     my %start_col;      my %start_col;
     my $max_column = scalar(@student_columns);      my $max_column = scalar(@student_columns);
     foreach my $prob (@$problems) {      foreach my $prob (@$problems) {
           my $symb = $prob->symb();
         my %analysis = &Apache::lonstathelpers::get_problem_data($prob->src);          my %analysis = &Apache::lonstathelpers::get_problem_data($prob->src);
         $problem_analysis{$prob->src}=\%analysis;          $problem_analysis{$prob->src}=\%analysis;
         $headers{'problem'}->[$max_column] = $prob->compTitle;          $headers{'problem'}->[$max_column] = $prob->compTitle;
         foreach my $partid (@{$prob->parts}) {          foreach my $partid (@{$prob->parts}) {
               if (($prob->is_anonsurvey($partid)) || ($anoncounter->{$symb."\0".$partid})) {
                   next if ($show_named);
               } else {
                   next unless ($show_named);
               }
             $headers{'part'}->[$max_column] = $prob->part_display($partid);              $headers{'part'}->[$max_column] = $prob->part_display($partid);
             my $responses = [$prob->responseIds($partid)];              my $responses = [$prob->responseIds($partid)];
             my $resptypes = [$prob->responseType($partid)];              my $resptypes = [$prob->responseType($partid)];
Line 1010  sub prepare_csv_output { Line 1159  sub prepare_csv_output {
         last if ($c->aborted());          last if ($c->aborted());
         my @rows;          my @rows;
         foreach my $prob (@$problems) {          foreach my $prob (@$problems) {
               my $symb = $prob->symb;
             foreach my $partid (@{$prob->parts}) {              foreach my $partid (@{$prob->parts}) {
                   if (($prob->is_anonsurvey($partid)) || ($anoncounter->{$symb."\0".$partid})) {
                       next if ($show_named); 
                   } else {
                       next unless ($show_named);
                   }
                 my @responses = $prob->responseIds($partid);                  my @responses = $prob->responseIds($partid);
                 my @response_type = $prob->responseType($partid);                  my @response_type = $prob->responseType($partid);
                 for (my $i=0;$i<=$#responses;$i++) {                  for (my $i=0;$i<=$#responses;$i++) {
Line 1048  sub prepare_csv_output { Line 1203  sub prepare_csv_output {
             }              }
         }          }
         foreach my $row (@rows) {          foreach my $row (@rows) {
             print $outputfile '"'.join('","',              my $student_row_data = '';
                                        map { $student->{$_}; }              if ($show_named) {
                                        @student_columns).'"';                  $student_row_data = '"'.join('","',
             for (my $i=scalar(@student_columns);$i<$max_column;$i++) {                                          map { $student->{$_}; }
                                           @student_columns).'"';
               } else {
                   $student_row_data = '"'.&mt('Anonymized').'"';
               }
               print $outputfile $student_row_data;
               for (my $i=$student_column_count;$i<$max_column;$i++) {
                 my $value = &Apache::loncommon::csv_translate($row->[$i]);                  my $value = &Apache::loncommon::csv_translate($row->[$i]);
                 $value ||='';                  $value ||='';
                 print $outputfile ',"'.$value.'"';                  print $outputfile ',"'.$value.'"';

Removed from v.1.59  
changed lines
  Added in v.1.60


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