Diff for /loncom/interface/statistics/lonstudentassessment.pm between versions 1.4 and 1.18

version 1.4, 2002/07/26 19:49:26 version 1.18, 2002/11/08 22:16:47
Line 6 Line 6
 # Copyright Michigan State University Board of Trustees  # Copyright Michigan State University Board of Trustees
 #  #
 # This file is part of the LearningOnline Network with CAPA (LON-CAPA).  # This file is part of the LearningOnline Network with CAPA (LON-CAPA).
 #  
 # LON-CAPA is free software; you can redistribute it and/or modify  # LON-CAPA is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by  # it under the terms of the GNU General Public License as published by
 # the Free Software Foundation; either version 2 of the License, or  # the Free Software Foundation; either version 2 of the License, or
Line 49  sub BuildStudentAssessmentPage { Line 48  sub BuildStudentAssessmentPage {
         $studentInformation,$r,$c)=@_;          $studentInformation,$r,$c)=@_;
 #    $jr = $r;  #    $jr = $r;
     my %cache;      my %cache;
     unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER,0640)) {      unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
         $r->print('<html><body>Unable to tie database.</body></html>');          $r->print('<html><body>Unable to tie database.</body></html>');
         return;          return;
     }      }
Line 86  sub BuildStudentAssessmentPage { Line 85  sub BuildStudentAssessmentPage {
                                             $students);                                              $students);
     $r->print(&CreateInterface(\%cache, $selectedName, $students, $formName,      $r->print(&CreateInterface(\%cache, $selectedName, $students, $formName,
                                $doNotShow));                                 $doNotShow));
       $r->rflush();
   
     my $Str = '';      my $Str = '';
     if($selectedName eq 'No Student Selected') {      if($selectedName eq 'No Student Selected') {
Line 98  sub BuildStudentAssessmentPage { Line 98  sub BuildStudentAssessmentPage {
     $r->print(&CreateTableHeadings(\%cache, $spacing, $infoKeys, $infoHeadings,      $r->print(&CreateTableHeadings(\%cache, $spacing, $infoKeys, $infoHeadings,
                                    $sequenceKeys, $sequenceHeadings));                                     $sequenceKeys, $sequenceHeadings));
     untie(%cache);      untie(%cache);
       if($c->aborted()) {  return $Str; }
   
     my $selected=0;      my $selected=0;
     $r->print('<pre>'."\n");      $r->print('<pre>'."\n");
     foreach (@$students) {      foreach (@$students) {
         next if ($_ ne $selectedName &&           if($c->aborted()) { return $Str; }
           next if ($_ ne $selectedName &&
                  $selectedName ne 'All Students');                   $selectedName ne 'All Students');
         $selected = 1;          $selected = 1;
         my $courseData;   
         if(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER,0640)) {          my @who = ($_);
             if($cache{$_.':lastDownloadTime'} eq 'Not downloaded') {          next if(&Apache::loncoursedata::DownloadStudentCourseData(\@who, 'true',
                 untie(%cache);                                                               $cacheDB, 'true',
                 $courseData =                                                                'false', $courseID,
                     &Apache::loncoursedata::DownloadCourseInformation($_,                                                                $r, $c) ne 'OK');
                                                                     $courseID);          next if($c->aborted());
                 if(tie(%cache,'GDBM_File',$cacheDB,&GDBM_WRCREAT,0640)) {  
                     &Apache::loncoursedata::ProcessStudentData(\%cache,           if(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
                                                              $courseData, $_);              my @before=();
                     untie(%cache);              my @after=();
                 } else {              my @updateColumn=();
                     last if($c->aborted());              my $foundUpdate = 0;
               foreach(@$infoKeys) {
                   if(/updateTime/) {
                       $foundUpdate=1;
                       push(@updateColumn, $_);
                     next;                      next;
                 }                  }
             } else {                  if($foundUpdate) {
                 untie(%cache);                      push(@after, $_);
                   } else {
                       push(@before, $_);
                   }
             }              }
         } else {              my $displayString = 'DISPLAYDATA'.$spacing;
             last if($c->aborted());              $r->print(&Apache::lonhtmlcommon::FormatStudentInformation(
             next;                                                           \%cache, $_,
         }                                                           \@before,
                                                            $displayString,
                                                            'preformatted'));
   
         last if ($c->aborted());              if($foundUpdate) {
                   $displayString = '';
                   $displayString .= '<a href="/adm/statistics?reportSelected=';
                   $displayString .= &Apache::lonnet::escape('Student Assessment');
                   $displayString .= '&download='.$_.'">';
                   $displayString .= 'DISPLAYDATA</a>'.$spacing;
                   $r->print(&Apache::lonhtmlcommon::FormatStudentInformation(
                                                                      \%cache, $_,
                                                                      \@updateColumn,
                                                                      $displayString,
                                                                      'preformatted'));
               }
   
         if(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER,0640)) {              $displayString = 'DISPLAYDATA'.$spacing;
             my $displayString = 'DISPLAYDATA'.$spacing;  
             $r->print(&Apache::lonhtmlcommon::FormatStudentInformation(              $r->print(&Apache::lonhtmlcommon::FormatStudentInformation(
                                                          \%cache, $_,                                                           \%cache, $_,
                                                          $infoKeys,                                                           \@after,
                                                          $displayString,                                                           $displayString,
                                                          'preformatted'));                                                           'preformatted'));
             $r->print(&StudentReport(\%cache, $_, $spacing, $sequenceKeys));              $r->print(&StudentReport(\%cache, $_, $spacing, $sequenceKeys));
             $r->print("\n");              $r->print("\n");
               $r->rflush();
             untie(%cache);              untie(%cache);
         }          }
     }      }
   
   
   #    $r->print("AverageTtal");
       $r->print(&StudentAverageTotal(\%cache, $spacing, $sequenceKeys));
       $r->print("\n");
       $r->rflush();
       untie(%cache);
   
   
     $r->print('</pre>'."\n");      $r->print('</pre>'."\n");
     if($selected == 0) {      if($selected == 0) {
  $Str .= '<h3><font color=blue>WARNING: ';   $Str .= '<h3><font color=blue>WARNING: ';
Line 163  sub CreateInterface { Line 194  sub CreateInterface {
     $Str .= '<input type="submit" name="PreviousStudent" ';      $Str .= '<input type="submit" name="PreviousStudent" ';
     $Str .= 'value="Previous Student" />'."\n";      $Str .= 'value="Previous Student" />'."\n";
     $Str .= '&nbsp&nbsp&nbsp'."\n";      $Str .= '&nbsp&nbsp&nbsp'."\n";
     $Str .= &Apache::lonhtmlcommon::StudentOptions($cache, $students,       $Str .= &Apache::lonhtmlcommon::StudentOptions($cache, $students,
                                                    $selectedName,                                                      $selectedName, 
                                                    'StudentAssessment',                                                      'StudentAssessment', 
                                                    $formName);                                                     $formName);
Line 188  sub CreateInterface { Line 219  sub CreateInterface {
     $Str .= '</td><td>'."\n";      $Str .= '</td><td>'."\n";
     $Str .= '<input type="submit" name="DefaultColumns" ';      $Str .= '<input type="submit" name="DefaultColumns" ';
     $Str .= 'value="Default Column Display" />'."\n";      $Str .= 'value="Default Column Display" />'."\n";
       $Str .= '</td><td>'."\n";
       $Str .= '<input type="submit" name="displaymode" ';
       if (! exists($ENV{'form.displaymode'}) ||
           lc($ENV{'form.displaymode'}) eq 'display with links') {
           $Str .= 'value="Display without links" />';
           # Set the current value, in case it is undefined
           $ENV{'form.displaymode'} = 'Display with links'; 
       } else {
           $Str .= 'value="Display with links" />';
       }
       $Str .= "\n";
     $Str .= '</td></tr></table>'."\n";      $Str .= '</td></tr></table>'."\n";
   
     return $Str;      return $Str;
Line 209  sub CreateTableHeadings { Line 251  sub CreateTableHeadings {
     my $displayString = '<td align="left"><pre><a href="/adm/statistics?';      my $displayString = '<td align="left"><pre><a href="/adm/statistics?';
     $displayString .= 'sort=LINKDATA">DISPLAYDATA</a>FORMATTING';      $displayString .= 'sort=LINKDATA">DISPLAYDATA</a>FORMATTING';
     $displayString .= $spacing.'</pre></td>'."\n";      $displayString .= $spacing.'</pre></td>'."\n";
     $Str .= &Apache::lonhtmlcommon::CreateHeadings($cache,       $Str .= &Apache::lonhtmlcommon::CreateHeadings($cache,
                                                    $infoKeys,                                                     $infoKeys,
                                                    $infoHeadings,                                                     $infoHeadings,
                                                    $displayString,                                                     $displayString,
                                                    'preformatted');                                                     'preformatted');
   
     $displayString  = '<td align="left"><pre>DISPLAYDATA'.$spacing;      $displayString  = '<td align="left"><pre>DISPLAYDATAFORMATTING'.$spacing;
     $displayString .= '</pre></td>'."\n";      $displayString .= '</pre></td>'."\n";
     $Str .= &Apache::lonhtmlcommon::CreateHeadings($cache,      $Str .= &Apache::lonhtmlcommon::CreateHeadings($cache,
                                                    $sequenceKeys,                                                     $sequenceKeys,
Line 234  sub CreateTableHeadings { Line 276  sub CreateTableHeadings {
 =item &FormatStudentData()  =item &FormatStudentData()
   
 First, FormatStudentInformation is called and prefixes the course information.  First, FormatStudentInformation is called and prefixes the course information.
 This function produces a formatted string of the student's course information.  This function produces a formatted string of the student\'s course information.
 Each column of data represents all the problems for a given sequence.  For  Each column of data represents all the problems for a given sequence.  For
 valid grade data, a link is created for that problem to a submission record  valid grade data, a link is created for that problem to a submission record
 for that problem.  for that problem.
Line 266  sub StudentReport { Line 308  sub StudentReport {
     my ($username,$domain)=split(':',$name);      my ($username,$domain)=split(':',$name);
   
     my $Str = '';      my $Str = '';
       if(defined($cache->{$name.':error'})) {
           return $Str;
       }
     if($cache->{$name.':error'} =~ /course/) {      if($cache->{$name.':error'} =~ /course/) {
         $Str .= '<b><font color="blue">No course data for student </font>';          $Str .= '<b><font color="blue">No course data for student </font>';
         $Str .= '<font color="red">'.$username.'.</font></b><br>';          $Str .= '<font color="red">'.$username.'.</font></b><br>';
         return $Str;          return $Str;
     }      }
   
     my $Version;      my $hasVersion = 'false';
     my $problemsCorrect = 0;      my $hasFinalData = 'false';
     my $totalProblems   = 0;  
     my $problemsSolved  = 0;  
     my $numberOfParts   = 0;  
 #    foreach my $sequence (split(':', $cache->{'orderedSequences'})) {  
     foreach my $sequence (@$showSequences) {      foreach my $sequence (@$showSequences) {
           my $hasData = 'false';
         my $characterCount=0;          my $characterCount=0;
         foreach my $problemID (split(':', $cache->{$sequence.':problems'})) {          foreach my $problemID (split(':', $cache->{$sequence.':problems'})) {
             my $problem = $cache->{$problemID.':problem'};              my $problem = $cache->{$problemID.':problem'};
             my $LatestVersion = $cache->{$name.':version:'.$problem};              # All grades (except for versionless parts) are displayed as links
               # to their submission record.  Loop through all the parts for the
             # Output dashes for all the parts of this problem if there              # current problem in the correct order and prepare the output links
             # is no version information about the current problem.              foreach(split(/\:/,$cache->{$sequence.':'.$problemID.
             if(!$LatestVersion) {                                          ':parts'})) {
                 foreach my $part (split(/\:/,$cache->{$sequence.':'.                  if($cache->{$name.':'.$problemID.':NoVersion'} eq 'true' ||
                                                       $problemID.                     $cache->{$name.':'.$problemID.':'.$_.':code'} eq ' ' ||
                                                       ':parts'})) {                     $cache->{$name.':'.$problemID.':'.$_.':code'} eq '') {
                     $Str .= ' ';                      $Str .= ' ';
                     $totalProblems++;  
                     $characterCount++;                      $characterCount++;
                       next;
                 }                  }
                 next;                  $hasVersion = 'true';
                   $hasData = 'true';
                   if (lc($ENV{'form.displaymode'}) ne 'display without links') {
                       $Str .= '<a href="/adm/grades?symb=';
                       $Str .= &Apache::lonnet::escape($problem);
                       $Str .= '&student='.$username.'&domain='.$domain;
                       $Str .= '&command=submission">'; 
                   }
                   my $code = $cache->{$name.':'.$problemID.':'.$_.':code'};
                   my $tries = $cache->{$name.':'.$problemID.':'.$_.':tries'};
                   if($code eq '*' && $tries < 10 && $tries ne '') {
                       $code = $tries;
                   }
                   $Str .= $code;
                   if (lc($ENV{'form.displaymode'}) ne 'display without links') {
                       $Str .= '</a>';
                   }
                   $characterCount++;
             }              }
           }
   
             my %partData=undef;          # Output the number of correct answers for the current sequence.
             # Initialize part data, display skips correctly          # This part takes up 6 character slots, but is formated right
             # Skip refers to when a student made no submissions on that          # justified.
             # part/problem.          my $spacesNeeded=$cache->{$sequence.':columnWidth'}-$characterCount;
             foreach my $part (split(/\:/,$cache->{$sequence.':'.          $spacesNeeded -= 3;
                                                   $problemID.          $Str .= (' 'x$spacesNeeded);
                                                   ':parts'})) {  
                 $partData{$part.':tries'}=0;  
                 $partData{$part.':code'}=' ';  
             }  
   
             # Looping through all the versions of each part, starting with the  #        my $outputProblemsCorrect = sprintf("%3d", $cache->{$name.':'.$sequence.
             # oldest version.  Basically, it gets the most recent   #    ':problemsCorrect'});
             # set of grade data for each part.  
     for(my $Version=1; $Version<=$LatestVersion; $Version++) {   my $outputProblemsCorrect = sprintf("%2d/%2d", $cache->{$name.':'.$sequence.
                 foreach my $part (split(/\:/,$cache->{$sequence.':'.                                              ':problemsCorrect'},
                                                       $problemID.                                              $characterCount);
                                                       ':parts'})) {          if($hasData eq 'true') {
               $Str .= '<font color="#007700">'.$outputProblemsCorrect.'</font>';
                     if(!defined($cache->{$name.":$Version:$problem".              $hasFinalData = 'true';
                                                ":resource.$part.solved"})) {          } else {
                         # No grade for this submission, so skip              $Str .= '<font color="#007700">     </font>';
                         next;          }
                     }          $Str .= $spacing;
       }
                     my $tries=0;  
                     my $code=' ';      # Output the total correct problems over the total number of problems.
       # I don't like this type of formatting, but it is a solution.  Need
                     $tries = $cache->{$name.':'.$Version.':'.$problem.      # a way to dynamically determine the space requirements.
                                       ':resource.'.$part.'.tries'};      my $outputProblemsSolved = sprintf("%4d", $cache->{$name.':problemsSolved'});
                     $partData{$part.':tries'}=($tries) ? $tries : 0;      my $outputTotalProblems  = sprintf("%4d", $cache->{$name.':totalProblems'});
       if($hasFinalData eq 'true') {
                     my $val = $cache->{$name.':'.$Version.':'.$problem.          $Str .= '<font color="#000088">'.$outputProblemsSolved.
                                        ':resource.'.$part.'.solved'};      ' / '.$outputTotalProblems.'</font>';
                     if    ($val eq 'correct_by_student')   {$code = '*';}       } else {
                     elsif ($val eq 'correct_by_override')  {$code = '+';}          $Str .= '<font color="#000088">           </font>';
                     elsif ($val eq 'incorrect_attempted')  {$code = '.';}       }
                     elsif ($val eq 'incorrect_by_override'){$code = '-';}  
                     elsif ($val eq 'excused')              {$code = 'x';}      if($hasVersion eq 'false') {
                     elsif ($val eq 'ungraded_attempted')   {$code = '#';}          $Str = '<b><font color="blue">No course data.</font></b>';
                     else                                   {$code = ' ';}      }
                     $partData{$part.':code'}=$code;  
                 }      return $Str;
             }  }
   
   
   sub StudentAverageTotal {
       
       return "";
   
       my ($cache,$name,$spacing,$showSequences)=@_;
       my $username = $name;
       my $Str = '';
       my $hasVersion = 'false';
       my $hasFinalData = 'false';
       foreach my $sequence (@$showSequences) {
           my $hasData = 'false';
           my $characterCount=0;
           foreach my $problemID (split(':', $cache->{$sequence.':problems'})) {
               my $problem = $cache->{$problemID.':problem'};
             # All grades (except for versionless parts) are displayed as links              # All grades (except for versionless parts) are displayed as links
             # to their submission record.  Loop through all the parts for the              # to their submission record.  Loop through all the parts for the
             # current problem in the correct order and prepare the output links              # current problem in the correct order and prepare the output links
             $Str .= '<a href="/adm/grades?symb=';  
             $Str .= &Apache::lonnet::escape($problem);  
             $Str .= '&student='.$username.'&domain='.$domain;  
             $Str .= '&command=submission">';   
             foreach(split(/\:/,$cache->{$sequence.':'.$problemID.              foreach(split(/\:/,$cache->{$sequence.':'.$problemID.
                                         ':parts'})) {                                          ':parts'})) {
                 if($partData{$_.':code'} eq '*') {                  if($cache->{$name.':'.$problemID.':NoVersion'} eq 'true' ||
                     $problemsCorrect++;                     $cache->{$name.':'.$problemID.':'.$_.':code'} eq ' ' ||
                     if (($partData{$_.':tries'}<10) &&                     $cache->{$name.':'.$problemID.':'.$_.':code'} eq '') {
                         ($partData{$_.':tries'} ne '')) {                      $Str .= ' ';
                         $partData{$_.':code'}=$partData{$_.':tries'};                      $characterCount++;
                     }                      next;
                 } elsif($partData{$_.':code'} eq '+') {  
                     $problemsCorrect++;  
                 }                  }
                   $hasVersion = 'true';
                 $Str .= $partData{$_.':code'};                  $hasData = 'true';
                 $characterCount++;                  $Str .= '<a href="/adm/grades?symb=';
                   $Str .= &Apache::lonnet::escape($problem);
                 if($partData{$_.':code'} ne 'x') {                  $Str .= '&student= Average &domain= Total';
                     $totalProblems++;                  $Str .= '&command=submission">';
                   my $code = $cache->{$name.':'.$problemID.':'.$_.':code'};
                   my $tries = $cache->{$name.':'.$problemID.':'.$_.':tries'};
                   if($code eq '*' && $tries < 10 && $tries ne '') {
                       $code = $tries;
                 }                  }
                   $Str .= $code;
                   $Str .= '</a>';
                   $characterCount++;
             }              }
             $Str.='</a>';  
         }          }
   
         # Output the number of correct answers for the current sequence.          # Output the number of correct answers for the current sequence.
         # This part takes up 6 character slots, but is formated right           # This part takes up 6 character slots, but is formated right
         # justified.          # justified.
         my $spacesNeeded=$cache->{$sequence.':columnWidth'}-$characterCount;          my $spacesNeeded=$cache->{$sequence.':columnWidth'}-$characterCount;
         $spacesNeeded -= 3;          $spacesNeeded -= 3;
         $Str .= (' 'x$spacesNeeded);          $Str .= (' 'x$spacesNeeded);
   
  my $outputProblemsCorrect = sprintf( "%3d", $problemsCorrect );  #        my $outputProblemsCorrect = sprintf("%3d", $cache->{$name.':'.$sequence.
  $Str .= '<font color="#007700">'.$outputProblemsCorrect.'</font>';  #    ':problemsCorrect'});
  $problemsSolved += $problemsCorrect;  
  $problemsCorrect=0;  
   
    my $outputProblemsCorrect = sprintf("%2d/%2d", $cache->{$name.':'.$sequence.
                                               ':problemsCorrect'},
                                               $characterCount);
           if($hasData eq 'true') {
               $Str .= '<font color="#007700">'.$outputProblemsCorrect.'</font>';
               $hasFinalData = 'true';
           } else {
               $Str .= '<font color="#007700">     </font>';
           }
         $Str .= $spacing;          $Str .= $spacing;
     }      }
   
     # Output the total correct problems over the total number of problems.      # Output the total correct problems over the total number of problems.
     # I don't like this type of formatting, but it is a solution.  Need      # I don't like this type of formatting, but it is a solution.  Need
     # a way to dynamically determine the space requirements.      # a way to dynamically determine the space requirements.
     my $outputProblemsSolved = sprintf( "%4d", $problemsSolved );      my $outputProblemsSolved = sprintf("%4d", $cache->{$name.':problemsSolved'});
     my $outputTotalProblems  = sprintf( "%4d", $totalProblems );      my $outputTotalProblems  = sprintf("%4d", $cache->{$name.':totalProblems'});
     $Str .= '<font color="#000088">'.$outputProblemsSolved.      if($hasFinalData eq 'true') {
           $Str .= '<font color="#000088">'.$outputProblemsSolved.
     ' / '.$outputTotalProblems.'</font>';      ' / '.$outputTotalProblems.'</font>';
       } else {
           $Str .= '<font color="#000088">           </font>';
       }
   
       if($hasVersion eq 'false') {
           $Str = '<b><font color="blue">No course data.</font></b>';
       }
   
     return $Str;      return $Str;
 }  }
   
   
   
 =pod  =pod
   
 =item &CreateLegend()  =item &CreateLegend()
Line 409  problems. Line 497  problems.
   
 sub CreateLegend {  sub CreateLegend {
     my $Str = "<p><pre>".      my $Str = "<p><pre>".
               "1..9: correct by student in 1..9 tries\n".                "   1  correct by student in 1 try\n".
               "   *: correct by student in more than 9 tries\n".                "   7  correct by student in 7 tries\n".
       "   +: correct by override\n".                "   *  correct by student in more than 9 tries\n".
               "   -: incorrect by override\n".        "   +  correct by override\n".
       "   .: incorrect attempted\n".                "   -  incorrect by override\n".
       "   #: ungraded attempted\n".        "   .  incorrect attempted\n".
               "    : not attempted\n".        "   #  ungraded attempted\n".
       "   x: excused".                "      not attempted (blank field)\n".
               "</pre><p>";         "   x  excused".
                 "</pre><p>";
     return $Str;      return $Str;
 }  }
   
Line 426  sub CreateLegend { Line 515  sub CreateLegend {
 =item &CreateColumnSelectionBox()  =item &CreateColumnSelectionBox()
   
 If there are columns not being displayed then this selection box is created  If there are columns not being displayed then this selection box is created
 with a list of those columns.  When selections are made and the page   with a list of those columns.  When selections are made and the page
 refreshed, the columns will be removed from this box and the column is  refreshed, the columns will be removed from this box and the column is
 put back in the chart.  If there is no columns to select, no row is added  put back in the chart.  If there is no columns to select, no row is added
 to the interface table.  to the interface table.
Line 437  Input: $CacheData, $headings Line 526  Input: $CacheData, $headings
   
 $CacheData: A pointer to a hash tied to the cached data  $CacheData: A pointer to a hash tied to the cached data
   
 $headings:  An array of the names of the columns for the student information.    $headings:  An array of the names of the columns for the student information.
 They are used for displaying which columns are missing.  They are used for displaying which columns are missing.
   
 Output: $notThere  Output: $notThere
   
 $notThere: The string contains one row of a table.  The first column has the   $notThere: The string contains one row of a table.  The first column has the
 name of the selection box.  The second contains the selection box   name of the selection box.  The second contains the selection box
 which has a size of four.  which has a size of four.
   
 =back  =back
Line 473  sub CreateColumnSelectionBox { Line 562  sub CreateColumnSelectionBox {
   
 =item &CreateColumnSelectors()  =item &CreateColumnSelectors()
   
 This function generates the checkboxes above the column headings.  The   This function generates the checkboxes above the column headings.  The
 column will be removed if the checkbox is unchecked.  column will be removed if the checkbox is unchecked.
   
 =over 4  =over 4
Line 523  sub CreateColumnSelectors { Line 612  sub CreateColumnSelectors {
 sub FindSelectedStudent {  sub FindSelectedStudent {
     my($cache, $selectedName, $students)=@_;      my($cache, $selectedName, $students)=@_;
   
     if($selectedName eq 'All Students' ||       if($selectedName eq 'All Students' ||
        $selectedName eq 'No Student Selected') {         $selectedName eq 'No Student Selected') {
         return $selectedName;          return $selectedName;
     }      }

Removed from v.1.4  
changed lines
  Added in v.1.18


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