--- loncom/homework/optionresponse.pm 2011/09/16 02:27:59 1.178 +++ loncom/homework/optionresponse.pm 2011/11/29 13:24:38 1.183 @@ -1,7 +1,7 @@ # LearningOnline Network with CAPA # option list style responses # -# $Id: optionresponse.pm,v 1.178 2011/09/16 02:27:59 raeburn Exp $ +# $Id: optionresponse.pm,v 1.183 2011/11/29 13:24:38 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -32,6 +32,7 @@ package Apache::optionresponse; use strict; +use Apache::inputtags(); use Apache::response(); use Apache::lonlocal; use Apache::lonnet; @@ -155,13 +156,9 @@ ENDTABLE splice(@options,$delopt-1,1); $optchanged=1; } - if ($env{"form.$Apache::lonxml::curdepth.options"}) { + if ($env{"form.$Apache::lonxml::curdepth.options"} ne '') { my $newopt = $env{"form.$Apache::lonxml::curdepth.options"}; - if ($options[0]) { - push(@options,$newopt); - } else { - $options[0]=$newopt; - } + push(@options,$newopt); $optchanged=1; } my $rebuildtag = &Apache::edit::get_new_args($token,$parstack,$safeeval, @@ -213,7 +210,7 @@ sub end_foilgroup { my $randomize = &Apache::lonxml::get_param('randomize',$parstack, $safeeval,'-2'); if ($target eq 'web' || $target eq 'tex') { - $result.=&displayfoils($target,$max,$randomize,$TeXlayout,$checkboxvalue,$checkboxchoices,@opt); + $result.=&displayfoils($target,$max,$randomize,$TeXlayout,$checkboxvalue,$checkboxchoices,$tex_option_switch,@opt); $Apache::lonxml::post_evaluate=0; } elsif ( $target eq 'answer') { $result.=&displayanswers($max,$randomize,@opt); @@ -226,23 +223,6 @@ sub end_foilgroup { push (@{ $Apache::lonhomework::analyze{"$part_id.options"} },@opt); } elsif ( $target eq 'grade') { - my $nonlenient=0; - my $part=$Apache::inputtags::part; - my $lenientparm=&Apache::lonnet::EXT("resource.$part.lenient"); - - if ($Apache::lonhomework::scantronmode) { - $nonlenient=0; -# Grading an exam: we are grading lenient unless told not to - if ($lenientparm=~/^0|off|no$/i) { - $nonlenient=1; - } - } else { -# Web mode: we are non-lenient unless told to - $nonlenient=1; - if ($lenientparm=~/^1|on|yes$/i) { - $nonlenient=0; - } - } if ( &Apache::response::submitted()) { my @whichopt = &whichfoils($max,$randomize); @@ -303,6 +283,7 @@ sub end_foilgroup { } } my $part=$Apache::inputtags::part; + my $nonlenient=&Apache::inputtags::grading_is_nonlenient($part); my $id = $Apache::inputtags::response['-1']; my $responsestr=&Apache::lonnet::hash2str(%responsehash); my $gradestr =&Apache::lonnet::hash2str(%grade); @@ -485,7 +466,8 @@ sub check_for_invalid { } sub displayfoils { - my ($target,$max,$randomize,$TeXlayout,$checkboxvalue,$checkboxchoices,@opt)=@_; + my ($target,$max,$randomize,$TeXlayout,$checkboxvalue,$checkboxchoices, + $tex_option_switch,@opt)=@_; if (!defined(@{ $Apache::response::foilgroup{'names'} })) {return;} my @names = @{ $Apache::response::foilgroup{'names'} }; my @truelist; @@ -495,7 +477,9 @@ sub displayfoils { my $displayoptionintex=1; my @alphabet = ('A'..'Z'); my @whichopt = &whichfoils($max,$randomize); - &check_for_invalid(\@whichopt,\@opt); + unless (($target eq 'tex') && ($tex_option_switch eq 'nochoice')) { + &check_for_invalid(\@whichopt,\@opt); + } my $part=$Apache::inputtags::part; my $id=$Apache::inputtags::response[-1]; my $break; @@ -914,9 +898,6 @@ sub end_foil { &Apache::lonxml::warning(&mt('Foils without names exist. This can cause problems to malfunction.')); $name=$Apache::lonxml::curdepth; } - if ($name eq "0") { - &Apache::lonxml::error(&mt('Foil name [_1] is not supported. Please choose another name.',''.$name.'')); - } &Apache::lonxml::debug("Using a name of :$name:"); if (defined($Apache::response::foilnames{$name})) { &Apache::lonxml::error(&mt("Foil name [_1] appears more than once. Foil names need to be unique.",''.$name.''));