--- loncom/homework/optionresponse.pm 2011/09/16 02:27:59 1.178
+++ loncom/homework/optionresponse.pm 2015/04/28 13:12:52 1.200
@@ -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.200 2015/04/28 13:12:52 raeburn Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -51,7 +51,8 @@ sub start_optionresponse {
undef(%Apache::response::foilnames);
if ($target eq 'edit') {
$result.=&Apache::edit::start_table($token)
- .'<tr><td><span class="LC_nobreak">'.&mt('Multiple Option Response Question').'</span> '
+ .'<tr><td><span class="LC_nobreak">'.&Apache::loncommon::insert_folding_button().' '
+ .&mt('Multiple Option Response Question').'</span> '
.&Apache::loncommon::help_open_topic('Option_Response_Problems')."</td>"
.'<td><span class="LC_nobreak">'.&mt('Delete?').' '
.&Apache::edit::deletelist($target,$token)
@@ -64,7 +65,7 @@ sub start_optionresponse {
$token,'4').
&Apache::edit::select_arg('Randomize Foil Order:','randomize',
['yes','no'],$token).
- &Apache::edit::select_arg(&mt('Display of options when printed'),'TeXlayout',
+ &Apache::edit::select_arg('Display of options when printed','TeXlayout',
[['horizontal','Normal list'],
['vertical','Listed in a vertical column']],$token).
&Apache::edit::end_row().&Apache::edit::start_spanning_row();
@@ -108,7 +109,11 @@ sub start_foilgroup {
my $optionlist="<option></option>\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.="<option value=\"$count\">$option</option>\n";
@@ -116,17 +121,24 @@ sub start_foilgroup {
}
my $insertlist=&Apache::edit::insertlist($target,$token);
$result.=&Apache::edit::start_table($token);
- $result.= (<<ENDTABLE);
- <tr><td>Select Options</td>
+ my %lt = &Apache::lonlocal::texthash(
+ sel => 'Select Options',
+ add => 'Add new Option:',
+ del => 'Delete an Option:',
+ maxcheck => 'Enter maximum number students may check (e.g., 3)',
+ mincheck => 'Enter minimum number students need to check (e.g., 1)',
+ );
+ $result.= sprintf(<<ENDTABLE, &Apache::loncommon::insert_folding_button());
+ <tr><td>%s $lt{'sel'}</td>
<td>
- Add new Option: <input type="text" name="$Apache::lonxml::curdepth.options" />
+ $lt{'add'} <input type="text" name="$Apache::lonxml::curdepth.options" />
</td>
- <td>Delete an Option:
+ <td>$lt{'del'}
<select name="$Apache::lonxml::curdepth.deleteopt">$optionlist</select>
</td>
ENDTABLE
$result.= '<td>'.&Apache::edit::checked_arg('Print options:','texoptions',
- [ ['nochoice','Don\'t show option list'] ]
+ [ ['nochoice',"Don't show option list"] ]
,$token).'</td>';
$result.= '<td><label>'.&mt('Two-option checkboxes for:').
'<select name="checkboxvalue_'.$Apache::lonxml::curdepth.'">';
@@ -137,10 +149,38 @@ ENDTABLE
}
$result.='>'.$option.'</option>';
}
- $result.='</select></label>';
+ $result.='</select></label> ';
+ my %checknumtxt = (
+ maxcheck => 'Max. boxes permitted to check',
+ mincheck => 'Min. boxes required to check',
+ );
+ my (%onclicknum,%checknum);
+ foreach my $item ('maxcheck','mincheck') {
+ $onclicknum{$item} = 'onclick="'."updateNumber('$item','$Apache::lonxml::curdepth','check','$lt{$item}');".'"';
+ $checknum{$item} = &Apache::lonxml::get_param($item,$parstack,$safeeval);
+ }
$result.=&Apache::edit::checked_arg('Checkbox options:','checkboxoptions',
- [ ['nochoice','Don\'t show option list'] ]
- ,$token).'</td>';
+ [ ['nochoice',"Don't show option list"] ]
+ ,$token).' '.
+ &Apache::edit::checked_arg('','noprompt',
+ [ ['nochoice','Omit "Select all that are ... "' ] ]
+ ,$token);
+ foreach my $item ('maxcheck','mincheck') {
+ my $currnum = $checknum{$item};
+ unless ($currnum =~ /^\d+$/) {
+ $currnum = 'nochoice';
+ }
+ $result.= ' <span class="LC_nobreak">'.
+ &Apache::edit::checked_arg('',$item,[ [$currnum,$checknumtxt{$item}] ],
+ $token,$onclicknum{$item},1).' '.
+ '<span id="'.$item.'_numtext_'.$Apache::lonxml::curdepth.'">';
+ if ($checknum{$item} ne '') {
+ $result .= ': <a href="javascript:updateNumber('."'$item','$Apache::lonxml::curdepth','link','$lt{$item}'".')">'.
+ $checknum{$item}.'</a>';
+ }
+ $result .= '</span></span>';
+ }
+ $result.= '</td>';
$result.= &Apache::edit::end_row();
$result.= &Apache::edit::start_spanning_row();
$result.= $insertlist.'<br />';
@@ -148,24 +188,39 @@ 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:");
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,
- 'texoptions','checkboxvalue','checkboxoptions');
+ 'texoptions','checkboxvalue','checkboxoptions',
+ 'noprompt','maxcheck','mincheck');
+ if ($rebuildtag) {
+ my $maxcheck = $token->[2]->{'maxcheck'};
+ my $mincheck = $token->[2]->{'mincheck'};
+ if ($mincheck =~ /D/) {
+ $token->[2]->{'mincheck'}='';
+ }
+ if ($maxcheck =~ /D/) {
+ $token->[2]->{'maxcheck'}='';
+ }
+ if (($mincheck ne '') && ($maxcheck ne '')) {
+ if ($mincheck > $maxcheck) {
+ $token->[2]->{'mincheck'}=$maxcheck;
+ }
+ }
+ }
if ($optchanged || $rebuildtag ) {
$result = "<foilgroup options=\"(";
foreach my $option (@options) {
@@ -177,7 +232,10 @@ ENDTABLE
$result.=')" ';
$result .= 'texoptions="'.$token->[2]{'texoptions'}.'" ';
$result .= 'checkboxoptions="'.$token->[2]{'checkboxoptions'}.'" ';
- $result .= 'checkboxvalue="'.$token->[2]{'checkboxvalue'}.'"';
+ $result .= 'checkboxvalue="'.$token->[2]{'checkboxvalue'}.'" ';
+ $result .= 'noprompt="'.$token->[2]{'noprompt'}.'" ';
+ $result .= 'maxcheck="'.$token->[2]{'maxcheck'}.'" ';
+ $result .= 'mincheck="'.$token->[2]{'mincheck'}.'"';
$result .= '>';
} # else nothing changed so just use the default mechanism
}
@@ -207,13 +265,19 @@ sub end_foilgroup {
-2,0);
my $checkboxvalue=&Apache::lonxml::get_param('checkboxvalue',$parstack,$safeeval);
my $checkboxchoices=(&Apache::lonxml::get_param('checkboxoptions',$parstack,$safeeval) ne 'nochoice');
+ my $maxcheck=&Apache::lonxml::get_param('maxcheck', $parstack, $safeeval);
+ $maxcheck =~ s/\D+//g;
+ my $mincheck=&Apache::lonxml::get_param('mincheck', $parstack, $safeeval);
+ $mincheck =~ s/\D+//g;
+ my $noprompt=&Apache::lonxml::get_param('noprompt', $parstack, $safeeval);
+
if ($target eq 'tex' && $tex_option_switch eq 'nochoice') {@opt=();}
&Apache::lonxml::debug("Options are $#opt");
my $max = &Apache::lonxml::get_param('max',$parstack,$safeeval,'-2');
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, $noprompt,$maxcheck,$mincheck, @opt);
$Apache::lonxml::post_evaluate=0;
} elsif ( $target eq 'answer') {
$result.=&displayanswers($max,$randomize,@opt);
@@ -226,23 +290,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);
@@ -252,6 +299,11 @@ sub end_foilgroup {
my $right=0;
my $wrong=0;
my $ignored=0;
+ my $checked=0;
+ my $corrchkd=0;
+ my $corrunchkd=0;
+ my $incorrchkd=0;
+ my $incorrunchkd=0;
my ($numrows,$bubbles_per_row);
if ($Apache::lonhomework::scantronmode) {
my $numitems = scalar(@opt);
@@ -261,6 +313,30 @@ sub end_foilgroup {
if ($numrows < 1) {
$numrows = 1;
}
+ my $checkboxopt;
+ if ($checkboxvalue) {
+ $checkboxopt = &check_box_opt($target,$checkboxvalue,@opt);
+ if (($checkboxopt) && ($Apache::lonhomework::type !~ /survey/)) {
+ my $corrnumcheck = 0;
+ if (($maxcheck ne '') || ($mincheck ne '')) {
+ foreach $name (@whichopt) {
+ if ($Apache::response::foilgroup{$name.'.value'} eq $checkboxopt) {
+ $corrnumcheck ++;
+ }
+ }
+ if (($maxcheck ne '') && ($corrnumcheck > $maxcheck)) {
+ $maxcheck = $corrnumcheck;
+ }
+ if (($mincheck ne '') && ($corrnumcheck < $mincheck)) {
+ $mincheck = $corrnumcheck;
+ }
+ }
+ }
+ }
+ my $part=$Apache::inputtags::part;
+ my $id = $Apache::inputtags::response['-1'];
+ my @relweights;
+ my $nonlenient=&grading_is_nonlenient($part,$id,\@relweights);
foreach $name (@whichopt) {
my $response;
if ($env{'form.submitted'} eq 'scantron') {
@@ -298,12 +374,26 @@ sub end_foilgroup {
} else {
$grade{$name}='0'; $wrong++;
}
+ if ($checkboxopt) {
+ if ($response eq $checkboxopt) {
+ $checked++;
+ if ($grade{$name}) {
+ $corrchkd ++;
+ } else {
+ $incorrchkd ++;
+ }
+ } else {
+ if ($grade{$name}) {
+ $corrunchkd ++;
+ } else {
+ $incorrunchkd ++;
+ }
+ }
+ }
} else {
$ignored++;
}
}
- my $part=$Apache::inputtags::part;
- my $id = $Apache::inputtags::response['-1'];
my $responsestr=&Apache::lonnet::hash2str(%responsehash);
my $gradestr =&Apache::lonnet::hash2str(%grade);
my %previous=&Apache::response::check_for_previous($responsestr,
@@ -315,65 +405,90 @@ sub end_foilgroup {
if ($Apache::lonhomework::type eq 'randomizetry') {
$Apache::lonhomework::results{"resource.$part.$id.foilorder"} = &Apache::lonnet::array2str(@whichopt);
}
- if (($Apache::lonhomework::type eq 'survey') ||
- ($Apache::lonhomework::type eq 'surveycred') ||
- ($Apache::lonhomework::type eq 'anonsurvey') ||
- ($Apache::lonhomework::type eq 'anonsurveycred')) {
- if ($ignored == 0) {
- my $ad;
- if ($Apache::lonhomework::type eq 'anonsurveycred') {
- $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='ANONYMOUS_CREDIT';
- } elsif ($Apache::lonhomework::type eq 'anonsurvey') {
- $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='ANONYMOUS';
- } elsif ($Apache::lonhomework::type eq 'surveycred') {
- $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='SUBMITTED_CREDIT';
- } else {
- $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='SUBMITTED';
- }
- &Apache::response::handle_previous(\%previous,$ad);
- } elsif ($wrong==0 && $right==0) {
- } else {
- my $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='MISSING_ANSWER';
- &Apache::response::handle_previous(\%previous,$ad);
- }
- } elsif ($nonlenient) {
+ my $skipaward;
+ if ($checkboxvalue) {
+ if ((($maxcheck ne '') && ($checked > $maxcheck)) ||
+ (($mincheck ne '') && ($checked < $mincheck))) {
+ my $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='WRONG_NUMBOXESCHECKED';
+ &Apache::response::handle_previous(\%previous,$ad);
+ $skipaward = 1;
+ }
+ }
+ unless ($skipaward) {
+ if (($Apache::lonhomework::type eq 'survey') ||
+ ($Apache::lonhomework::type eq 'surveycred') ||
+ ($Apache::lonhomework::type eq 'anonsurvey') ||
+ ($Apache::lonhomework::type eq 'anonsurveycred')) {
+ if ($ignored == 0) {
+ my $ad;
+ if ($Apache::lonhomework::type eq 'anonsurveycred') {
+ $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='ANONYMOUS_CREDIT';
+ } elsif ($Apache::lonhomework::type eq 'anonsurvey') {
+ $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='ANONYMOUS';
+ } elsif ($Apache::lonhomework::type eq 'surveycred') {
+ $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='SUBMITTED_CREDIT';
+ } else {
+ $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='SUBMITTED';
+ }
+ &Apache::response::handle_previous(\%previous,$ad);
+ } elsif ($wrong==0 && $right==0) {
+ } else {
+ my $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='MISSING_ANSWER';
+ &Apache::response::handle_previous(\%previous,$ad);
+ }
+ } elsif ($nonlenient) {
#
# Non-lenient mode. All right or all wrong
#
- my $ad;
- if ($wrong==0 && $ignored==0) {
- $ad='EXACT_ANS';
- } elsif ($wrong==0 && $right==0) {
- #nothing submitted
- } else {
- if ($ignored==0) {
- $ad='INCORRECT';
- } else {
- $ad='MISSING_ANSWER';
- }
- }
- $Apache::lonhomework::results{"resource.$part.$id.awarddetail"}=$ad;
- &Apache::response::handle_previous(\%previous,$ad);
- } else {
+ my $ad;
+ if ($wrong==0 && $ignored==0) {
+ $ad='EXACT_ANS';
+ } elsif ($wrong==0 && $right==0) {
+ #nothing submitted
+ } else {
+ if ($ignored==0) {
+ $ad='INCORRECT';
+ } else {
+ $ad='MISSING_ANSWER';
+ }
+ }
+ $Apache::lonhomework::results{"resource.$part.$id.awarddetail"}=$ad;
+ &Apache::response::handle_previous(\%previous,$ad);
+ } else {
#
# This is lenient mode
#
- my $ad;
- if ($wrong==0 && $right==0) {
- #nothing submitted only assign a score if we
- #need to override a previous grade
- if (defined($Apache::lonhomework::history{"resource.$part.$id.awarddetail"})) {
+ my $ad;
+ if ($wrong==0 && $right==0) {
+ #nothing submitted only assign a score if we
+ #need to override a previous grade
+ if (defined($Apache::lonhomework::history{"resource.$part.$id.awarddetail"})) {
+ $ad='ASSIGNED_SCORE';
+ }
+ } else {
$ad='ASSIGNED_SCORE';
- }
- } else {
- $ad='ASSIGNED_SCORE';
- }
- $Apache::lonhomework::results{"resource.$part.$id.awarddetail"}=$ad;
- $Apache::lonhomework::results{"resource.$part.$id.awarded"}=
- $right/(scalar(@whichopt));
- $Apache::lonhomework::results{"resource.$part.$id.numfoils"}=
- scalar(@whichopt);
- }
+ }
+ $Apache::lonhomework::results{"resource.$part.$id.awarddetail"}=$ad;
+ my $awarded;
+ my $totalfoils = $#{ $Apache::response::foilgroup{'names'} }+1;
+ if (($checkboxopt) && ($max >= $totalfoils) && (@relweights > 0)) {
+ if ($right == scalar(@whichopt)) {
+ $awarded = 1;
+ } else {
+ $awarded = ($corrchkd*$relweights[0] + $corrunchkd*$relweights[1] +
+ $incorrchkd*$relweights[2] + $incorrunchkd*$relweights[3])/(scalar(@whichopt));
+ $awarded = 1 if ($awarded > 1);
+ $awarded = 0 if ($awarded < 0);
+ }
+ } else {
+ $awarded = $right/(scalar(@whichopt));
+ }
+ $Apache::lonhomework::results{"resource.$part.$id.awarded"}=
+ $awarded;
+ $Apache::lonhomework::results{"resource.$part.$id.numfoils"}=
+ scalar(@whichopt);
+ }
+ }
}
}
my $part_id = $Apache::inputtags::part;
@@ -406,6 +521,42 @@ sub end_foilgroup {
return $result;
}
+sub grading_is_nonlenient {
+ my ($part,$id,$relweights) = @_;
+# Web mode: we are non-lenient unless told otherwise
+ my $defaultparm = 'off';
+ my $nonlenient = 1;
+# Grading a bubblesheet exam: we are grading lenient unless told otherwise
+ if ($Apache::lonhomework::scantronmode) {
+ $defaultparm = 'on';
+ $nonlenient = 0;
+ }
+ my $lenientparm =
+ &Apache::response::get_response_param($part,'lenient',$defaultparm);
+ if ($lenientparm eq 'default') {
+ $lenientparm = $defaultparm;
+ }
+ $lenientparm =
+ &Apache::response::get_response_param($part.'_'.$id,'lenient',$lenientparm);
+ if ($lenientparm eq 'default') {
+ $lenientparm = $defaultparm;
+ }
+ if ($lenientparm=~/^(?:0|off|no)$/i) {
+ $nonlenient = 1;
+ } elsif ($lenientparm=~/^(?:1|on|yes)$/i) {
+ $nonlenient = 0;
+ } elsif ($lenientparm =~ /^\-?[.\d]+,\-?[.\d]+,\-?[.\d]+,\-?[.\d]+$/) {
+ if (ref($relweights) eq 'ARRAY') {
+ @{$relweights} = split(/,/,$lenientparm);
+ }
+ $nonlenient = 0;
+ }
+ if (!$nonlenient) {
+ $Apache::inputtags::leniency=1;
+ }
+ return $nonlenient;
+}
+
sub getfoilcounts {
my ($max)=@_;
# +1 since instructors will count from 1
@@ -425,8 +576,11 @@ sub whichfoils {
sub displayanswers {
my ($max,$randomize,@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'} };
+ }
+ return if (!@names);
my @whichopt = &whichfoils($max,$randomize);
my $result;
if ($Apache::lonhomework::type eq 'exam') {
@@ -459,7 +613,7 @@ sub check_box_opt {
# Check if we are in checkbox mode: checkboxvalue specified, on web, only two options.
# If so, return "checked" value
if ($#opt!=1) { return ''; }
- if ($target ne 'web') { return ''; }
+ unless (($target eq 'web') || ($target eq 'grade')) { return ''; }
return $checkboxvalue;
}
@@ -485,9 +639,13 @@ sub check_for_invalid {
}
sub displayfoils {
- my ($target,$max,$randomize,$TeXlayout,$checkboxvalue,$checkboxchoices,@opt)=@_;
- if (!defined(@{ $Apache::response::foilgroup{'names'} })) {return;}
- my @names = @{ $Apache::response::foilgroup{'names'} };
+ my ($target,$max,$randomize,$TeXlayout,$checkboxvalue,$checkboxchoices,
+ $tex_option_switch, $no_tfprompt, $maxcheck, $mincheck, @opt)=@_;
+ 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;
@@ -495,7 +653,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;
@@ -505,8 +665,15 @@ sub displayfoils {
foreach $name (@whichopt) {
my $text=$Apache::response::foilgroup{$name.'.text'};
my $lastresp;
- unless ((($Apache::lonhomework::history{"resource.$part.type"} eq 'anonsurvey') || ($Apache::lonhomework::history{"resource.$part.type"} eq 'anonsurveycred')) && (defined($env{'form.grade_symb'}))) {
+ if ((($env{'form.grade_username'} eq '') && ($env{'form.grade_domain'} eq '')) ||
+ (($env{'form.grade_username'} eq $env{'user.name'}) &&
+ ($env{'form.grade_domain'} eq $env{'user.domain'}))) {
$lastresp = $Apache::lonhomework::history{"resource.$part.$id.submission"};
+ } else {
+ unless (($Apache::lonhomework::history{"resource.$part.type"} eq 'anonsurvey') ||
+ ($Apache::lonhomework::history{"resource.$part.type"} eq 'anonsurveycred')) {
+ $lastresp = $Apache::lonhomework::history{"resource.$part.$id.submission"};
+ }
}
my %lastresponse=&Apache::lonnet::str2hash($lastresp);
my $lastopt=$lastresponse{$name};
@@ -545,14 +712,28 @@ sub displayfoils {
}
}
unless ($newvariation) {
- %lastresponse=&Apache::lonnet::str2hash($Apache::lonhomework::history{"resource.$part.$id.submission"});
+ if ((($env{'form.grade_username'} eq '') && ($env{'form.grade_domain'} eq '')) ||
+ (($env{'form.grade_username'} eq $env{'user.name'}) &&
+ ($env{'form.grade_domain'} eq $env{'user.domain'}))) {
+ %lastresponse =
+ &Apache::lonnet::str2hash($Apache::lonhomework::history{"resource.$part.$id.submission"});
+ } else {
+ unless (($Apache::lonhomework::history{"resource.$part.type"} eq 'anonsurvey') ||
+ ($Apache::lonhomework::history{"resource.$part.type"} eq 'anonsurveycred')) {
+ %lastresponse =
+ &Apache::lonnet::str2hash($Apache::lonhomework::history{"resource.$part.$id.submission"});
+ }
+ }
}
my $internal_counter=$Apache::lonxml::counter;
- my $checkboxopt=&check_box_opt($target,$checkboxvalue,@opt);
- if ($checkboxopt) {
+ my $checkboxopt;
+ if ($target eq 'web') {
+ $checkboxopt=&check_box_opt($target,$checkboxvalue,@opt);
+ }
+ if ($checkboxopt && (!$no_tfprompt)) {
$result.='<br />'.
($checkboxchoices?&mt('Choices: ').'<b>'.$opt[0].','.$opt[1].'</b>. ':'').
- &mt('Select all that are <b>[_1]</b>.',$checkboxopt);
+ &mt('Select all that are [_1].','<b>'.$checkboxopt.'</b>');
}
foreach $name (@whichopt) {
my $text=$Apache::response::foilgroup{$name.'.text'};
@@ -803,7 +984,7 @@ sub bubbles {
if ($response eq 'rankresponse') {$opt[$ind]='Rank '.$opt[$ind];}
if ($ind==0) {$leftmargin=6;} else {$leftmargin=10;}
- $current_length += (length($opt[$ind])+length($item)+4)*2;
+ $current_length += (length($opt[$ind])+length($item)+5)*2;
if ($current_length<($textwidth-$leftmargin) and $ind!=$number_of_bubbles) {
@@ -914,9 +1095,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.','<b><tt>'.$name.'</tt></b>'));
- }
&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.",'<b><tt>'.$name.'</tt></b>'));