--- loncom/homework/optionresponse.pm 2012/10/12 12:45:46 1.187 +++ loncom/homework/optionresponse.pm 2012/12/18 16:30:23 1.189 @@ -1,7 +1,7 @@ # LearningOnline Network with CAPA # option list style responses # -# $Id: optionresponse.pm,v 1.187 2012/10/12 12:45:46 raeburn Exp $ +# $Id: optionresponse.pm,v 1.189 2012/12/18 16:30:23 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -108,7 +108,11 @@ sub start_foilgroup { my $optionlist="\n"; my $option; my @opt; - eval '@opt ='. &Apache::lonxml::get_param('options',$parstack,$safeeval); + my @raw_options = &Apache::lonxml::get_param('options', $parstack, $safeeval, 0, 0, 1); + + + eval '@opt ='. &Apache::lonxml::get_param('options',$parstack,$safeeval, 0, 0, 1); + my $count=1; foreach $option (@opt) { $optionlist.="\n"; @@ -148,7 +152,10 @@ ENDTABLE if ($target eq 'modified') { my @options; my $optchanged=0; - eval '@options ='.&Apache::lonxml::get_param('options',$parstack,$safeeval); + + + eval '@options ='.&Apache::lonxml::get_param('options',$parstack,$safeeval, 0, 0, 1); + if ($env{"form.$Apache::lonxml::curdepth.deleteopt"}) { my $delopt=$env{"form.$Apache::lonxml::curdepth.deleteopt"}; &Apache::lonxml::debug("Deleting :$delopt:"); @@ -492,8 +499,11 @@ sub check_for_invalid { sub displayfoils { my ($target,$max,$randomize,$TeXlayout,$checkboxvalue,$checkboxchoices, $tex_option_switch, $no_tfprompt, @opt)=@_; - if (!defined(@{ $Apache::response::foilgroup{'names'} })) {return;} - my @names = @{ $Apache::response::foilgroup{'names'} }; + my @names; + if (ref($Apache::response::foilgroup{'names'}) eq 'ARRAY') { + @names = @{ $Apache::response::foilgroup{'names'} }; + } + unless (@names > 0) { return;} my @truelist; my @falselist; my $result;