Diff for /loncom/interface/loncommon.pm between versions 1.1171 and 1.1173

version 1.1171, 2014/01/21 14:38:51 version 1.1173, 2014/02/11 14:29:04
Line 3846  sub get_previous_attempt { Line 3846  sub get_previous_attempt {
                         if (($data eq 'award') || ($data eq 'awarddetail')) {                          if (($data eq 'award') || ($data eq 'awarddetail')) {
                             my $value = &format_previous_attempt_value($key,                              my $value = &format_previous_attempt_value($key,
                                              $returnhash{$version.':'.$key});                                               $returnhash{$version.':'.$key});
                             $prevattempts.='<td>'.&HTML::Entities::encode($value, '"<>&').'&nbsp;</td>';                              $prevattempts.='<td>'.$value.'&nbsp;</td>';
                         } else {                          } else {
                             $prevattempts.='<td>&nbsp;</td>';                              $prevattempts.='<td>&nbsp;</td>';
                         }                          }
Line 3854  sub get_previous_attempt { Line 3854  sub get_previous_attempt {
                         if ($key =~ /\./) {                          if ($key =~ /\./) {
                             my $value = &format_previous_attempt_value($key,                              my $value = &format_previous_attempt_value($key,
                                               $returnhash{$version.':'.$key});                                                $returnhash{$version.':'.$key});
                             $prevattempts.='<td>'.&HTML::Entities::encode($value, '"<>&').'&nbsp;</td>';                              $prevattempts.='<td>'.$value.'&nbsp;</td>';
                         } else {                          } else {
                             $prevattempts.='<td>&nbsp;</td>';                              $prevattempts.='<td>&nbsp;</td>';
                         }                          }
Line 3865  sub get_previous_attempt { Line 3865  sub get_previous_attempt {
                     next if ($key =~ /\.foilorder$/);                      next if ($key =~ /\.foilorder$/);
     my $value = &format_previous_attempt_value($key,      my $value = &format_previous_attempt_value($key,
             $returnhash{$version.':'.$key});              $returnhash{$version.':'.$key});
     $prevattempts.='<td>'.&HTML::Entities::encode($value, '"<>&').'&nbsp;</td>';      $prevattempts.='<td>'.$value.'&nbsp;</td>';
         }          }
             }              }
     $prevattempts.=&end_data_table_row();      $prevattempts.=&end_data_table_row();
Line 3890  sub get_previous_attempt { Line 3890  sub get_previous_attempt {
                       if ($key =~/$regexp$/ && (defined &$gradesub)) {                        if ($key =~/$regexp$/ && (defined &$gradesub)) {
                           $value = &$gradesub($value);                            $value = &$gradesub($value);
                       }                        }
                       $prevattempts.='<td>'. &HTML::Entities::encode($value, '"<>&').'&nbsp;</td>';                        $prevattempts.='<td>'. $value.'&nbsp;</td>';
                   } else {                    } else {
                       $prevattempts.='<td>&nbsp;</td>';                        $prevattempts.='<td>&nbsp;</td>';
                   }                    }
Line 3899  sub get_previous_attempt { Line 3899  sub get_previous_attempt {
                   if ($key =~/$regexp$/ && (defined &$gradesub)) {                    if ($key =~/$regexp$/ && (defined &$gradesub)) {
                       $value = &$gradesub($value);                        $value = &$gradesub($value);
                   }                    }
                   $prevattempts.='<td>'.&HTML::Entities::encode($value, '"<>&').'&nbsp;</td>';                    $prevattempts.='<td>'.$value.'&nbsp;</td>';
               }                }
           } else {            } else {
       my $value = &format_previous_attempt_value($key,$lasthash{$key});        my $value = &format_previous_attempt_value($key,$lasthash{$key});
       if ($key =~/$regexp$/ && (defined &$gradesub)) {        if ($key =~/$regexp$/ && (defined &$gradesub)) {
                   $value = &$gradesub($value);                    $value = &$gradesub($value);
               }                }
       $prevattempts.='<td>'.&HTML::Entities::encode($value, '"<>&').'&nbsp;</td>';       $prevattempts.='<td>'.$value.'&nbsp;</td>';
           }            }
       }        }
       $prevattempts.= &end_data_table_row().&end_data_table();        $prevattempts.= &end_data_table_row().&end_data_table();
Line 3927  sub get_previous_attempt { Line 3927  sub get_previous_attempt {
 sub format_previous_attempt_value {  sub format_previous_attempt_value {
     my ($key,$value) = @_;      my ($key,$value) = @_;
     if (($key =~ /timestamp/) || ($key=~/duedate/)) {      if (($key =~ /timestamp/) || ($key=~/duedate/)) {
  $value = &Apache::lonlocal::locallocaltime($value);          $value = &Apache::lonlocal::locallocaltime($value);
     } elsif (ref($value) eq 'ARRAY') {      } elsif (ref($value) eq 'ARRAY') {
  $value = '('.join(', ', @{ $value }).')';          $value = &HTML::Entities::encode('('.join(', ', @{ $value }).')','"<>&');
     } elsif ($key =~ /answerstring$/) {      } elsif ($key =~ /answerstring$/) {
         my %answers = &Apache::lonnet::str2hash($value);          my %answers = &Apache::lonnet::str2hash($value);
           my @answer = %answers;
           %answers = map {&HTML::Entities::encode($_, '"<>&')} @answer;
         my @anskeys = sort(keys(%answers));          my @anskeys = sort(keys(%answers));
         if (@anskeys == 1) {          if (@anskeys == 1) {
             my $answer = $answers{$anskeys[0]};              my $answer = $answers{$anskeys[0]};
Line 3954  sub format_previous_attempt_value { Line 3956  sub format_previous_attempt_value {
             }               } 
         }          }
     } else {      } else {
  $value = &unescape($value);          $value = &HTML::Entities::encode(&unescape($value), '"<>&');
     }      }
     return $value;      return $value;
 }  }
Line 14888  sub captcha_display { Line 14890  sub captcha_display {
     if ($captcha eq 'original') {      if ($captcha eq 'original') {
         $output = &create_captcha();          $output = &create_captcha();
         unless ($output) {          unless ($output) {
             $error = 'captcha';               $error = 'captcha';
         }          }
     } elsif ($captcha eq 'recaptcha') {      } elsif ($captcha eq 'recaptcha') {
         $output = &create_recaptcha($pubkey);          $output = &create_recaptcha($pubkey);
         unless ($output) {          unless ($output) {
             $error = 'recaptcha';               $error = 'recaptcha';
         }          }
     }      }
     return ($output,$error);      return ($output,$error);
Line 14973  sub create_captcha { Line 14975  sub create_captcha {
             $output = '<input type="hidden" name="crypt" value="'.$md5sum.'" />'."\n".              $output = '<input type="hidden" name="crypt" value="'.$md5sum.'" />'."\n".
                       &mt('Type in the letters/numbers shown below').'&nbsp;'.                        &mt('Type in the letters/numbers shown below').'&nbsp;'.
                      '<input type="text" size="5" name="code" value="" /><br />'.                       '<input type="text" size="5" name="code" value="" /><br />'.
                      '<img src="'.$captcha_params{'www_output_dir'}.'/'.$md5sum.'.png" />';                       '<img src="'.$captcha_params{'www_output_dir'}.'/'.$md5sum.'.png" alt="captcha" />';
             last;              last;
         }          }
     }      }

Removed from v.1.1171  
changed lines
  Added in v.1.1173


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