Diff for /loncom/homework/default_homework.lcpm between versions 1.146 and 1.152

version 1.146, 2009/12/27 19:40:42 version 1.152, 2011/02/26 17:50:13
Line 320  sub caparesponse_check_list { Line 320  sub caparesponse_check_list {
     }      }
     &LONCAPA_INTERNAL_DEBUG("Final final response :$responses->[0][-1]:$unit:");      &LONCAPA_INTERNAL_DEBUG("Final final response :$responses->[0][-1]:$unit:");
     $unit=~s/\s//;      $unit=~s/\s//;
       my $error;
     foreach my $response (@$responses) {      foreach my $response (@$responses) {
        foreach my $element (@$response) {         foreach my $element (@$response) {
           if (($type eq 'float') || (($type eq '') && ($unit ne ''))) {            if (($type eq 'float') || (($type eq '') && ($unit ne ''))) {
               $element =~ s/\s//g;                $element =~ s/\s//g;
           }            }
           my $appendunit=$unit;            my $appendunit=$unit;
   # Deal with percentages
   # unit is unit entered by student, answerunit is unit by author
   # Deprecated: divide answer by 100 if student entered percent,
   # but author did not. Too much confusion
 #          if (($unit=~/\%/) && ($answerunit ne '%'))  {  #          if (($unit=~/\%/) && ($answerunit ne '%'))  {
 #             $element=$element/100;  #             $element=$element/100;
 #             $appendunit=~s/\%//;  #             $appendunit=~s/\%//;
 #          }      #          }    
   # Author entered percent, student did not
             if (($unit!~/\%/) && ($answerunit=~/\%/)) {
                $element=$element*100;
                $appendunit='%'.$appendunit;
             }
   # Zero does not need a dimension
           if (($element==0) && ($unit!~/\w/) && ($answerunit=~/\w/)) {            if (($element==0) && ($unit!~/\w/) && ($answerunit=~/\w/)) {
              $appendunit=$answerunit;               $appendunit=$answerunit;
           }            }
           $element .= " $appendunit";            if ($appendunit ne '') {
                 $element .= " $appendunit";
             }  
           &LONCAPA_INTERNAL_DEBUG("Made response element :$element:");            &LONCAPA_INTERNAL_DEBUG("Made response element :$element:");
        }         }
     }      }
Line 344  sub caparesponse_check_list { Line 357  sub caparesponse_check_list {
  }   }
     }      }
   
       my $allow_control_char = 0;
       my $control_chars_removed = 0;
       my $ansstring;
       if ($type eq 'cs' || $type eq 'ci') {
           if (ref($LONCAPA::CAPAresponse_answer->{'answers'}) eq 'ARRAY') {
               foreach my $strans (@{$LONCAPA::CAPAresponse_answer->{'answers'}}) {
                   if (ref($strans) eq 'ARRAY') {
                       $ansstring = join("\0",@{$strans});
                       foreach my $item (@{$strans}) {  
                           if ($item =~ /[\000-\037]/) {
                               $allow_control_char = 1;
                           }
                       }
                   }
               }
           }
       }
   
 #    &LONCAPA_INTERNAL_DEBUG(&LONCAPA_INTERNAL_Dumper($responses));  #    &LONCAPA_INTERNAL_DEBUG(&LONCAPA_INTERNAL_Dumper($responses));
     my %memoized;      my %memoized;
Line 353  sub caparesponse_check_list { Line 383  sub caparesponse_check_list {
     my $response = $responses->[$i];      my $response = $responses->[$i];
     my $key = "$answer\0$response";      my $key = "$answer\0$response";
     my (@awards,@msgs);      my (@awards,@msgs);
     for (my $j=0; $j<scalar(@$response); $j++) {       for (my $j=0; $j<scalar(@$response); $j++) {
                   if ($type eq 'cs' || $type eq 'ci') {
                       unless ($allow_control_char) {
                           if ($response->[$j] =~ /[\000-\037]/) { 
                               $response->[$j] =~ s/[\000-\037]//g;
                               $control_chars_removed = 1;
                           }  
                       }
                   }
  my ($award,$msg) = &caparesponse_check($answer->[$j],   my ($award,$msg) = &caparesponse_check($answer->[$j],
        $response->[$j]);         $response->[$j]);
                   if ($type eq 'cs' || $type eq 'ci') {
                       $error = &verify_stringresponse($type,$award,$response->[$j],
                                                       $answer->[$j]);
                   }
  push(@awards,$award);   push(@awards,$award);
  push(@msgs,  $msg);   push(@msgs,  $msg);
     }      }
Line 380  sub caparesponse_check_list { Line 422  sub caparesponse_check_list {
  } else {   } else {
     my (@awards,@msgs);      my (@awards,@msgs);
     for (my $j=0; $j<scalar(@$response); $j++) {      for (my $j=0; $j<scalar(@$response); $j++) {
                           if ($type eq 'cs' || $type eq 'ci') {
                               unless ($allow_control_char) {
                                   if ($response->[$j] =~ /[\000-\037]/) {
                                       $response->[$j] =~ s/[\000-\037]//g;
                                       $control_chars_removed = 1;
                                   }
                               }
                           }
  my ($award,$msg) = &caparesponse_check($answer->[$j],   my ($award,$msg) = &caparesponse_check($answer->[$j],
        $response->[$j]);         $response->[$j]);
                           if ($type eq 'cs' || $type eq 'ci') {
                               $error = &verify_stringresponse($type,$award,$response->[$j],
                                                               $answer->[$j]);
                           }
  push(@awards,$award);   push(@awards,$award);
  push(@msgs,  $msg);   push(@msgs,  $msg);
     }      }
Line 428  sub caparesponse_check_list { Line 482  sub caparesponse_check_list {
     &LONCAPA_INTERNAL_DEBUG(" all final_awards ".join(':',@final_awards));      &LONCAPA_INTERNAL_DEBUG(" all final_awards ".join(':',@final_awards));
     my ($final_award,$final_msg) =       my ($final_award,$final_msg) = 
  &LONCAPA_INTERNAL_FINALIZEAWARDS(\@final_awards,\@final_msg,undef,1);   &LONCAPA_INTERNAL_FINALIZEAWARDS(\@final_awards,\@final_msg,undef,1);
     return ($final_award,$final_msg);      return ($final_award,$final_msg,$error,$control_chars_removed,$ansstring);
   }
   
   sub verify_stringresponse {
       my ($type,$award,$resp,$ans) = @_;
       return if ($award eq 'EXACT_ANS');
       my $error;
       if ($resp =~ /^\s|\s$/) {
           $resp =~ s{^\s+|\s+$}{}g;
       }
       if ($ans =~ /^\s|\s$/) {
           $ans =~ s{^\s+|\s+$}{}g;
       }
       if ($type eq 'ci') {
           $resp = lc($resp);
           $ans = lc($ans);
       }
       if ($resp eq $ans) {
           if ($award eq 'INCORRECT') {
               $error = 'MISGRADED';
           }
       }
       return $error;
 }  }
   
 sub cas {  sub cas {

Removed from v.1.146  
changed lines
  Added in v.1.152


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