--- loncom/homework/matchresponse.pm 2003/03/26 18:43:15 1.14 +++ loncom/homework/matchresponse.pm 2003/09/23 01:52:57 1.20.2.1 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # Full matching style response # -# $Id: matchresponse.pm,v 1.14 2003/03/26 18:43:15 sakharuk Exp $ +# $Id: matchresponse.pm,v 1.20.2.1 2003/09/23 01:52:57 albertel Exp $ # # Copyright Michigan State University Board of Trustees # @@ -33,9 +33,6 @@ use HTML::Entities(); use Math::Random(); use Apache::optionresponse; -my $flag_for_exam_printing = 1; -my @Items = (); - BEGIN { &Apache::lonxml::register('Apache::matchresponse',('matchresponse')); } @@ -90,6 +87,7 @@ sub start_itemgroup { my $result; %Apache::response::itemgroup=(); %Apache::matchresponse::itemtable=(); + if ($target eq 'edit') { $result=&Apache::edit::tag_start($target,$token); $result.=&Apache::edit::select_arg('Randomize Order:','randomize', @@ -115,12 +113,13 @@ sub end_itemgroup { my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_; my $result; + if ($target eq 'edit') { $result=&Apache::edit::end_table(); } if (!defined(@{ $Apache::response::itemgroup{'names'} })) { return; } my @names=@{ $Apache::response::itemgroup{'names'} }; my $randomize =&Apache::lonxml::get_param('randomize',$parstack,$safeeval); if ($randomize ne 'no' ) { - @names=&whichorder($#names+1,$randomize,0, - \%Apache::response::itemgroup) + @names=&Apache::response::whichorder($#names+1,$randomize,0, + \%Apache::response::itemgroup); } my %letter_name_map; my %name_letter_map; @@ -129,10 +128,9 @@ sub end_itemgroup { foreach my $name (@names) { $letter_name_map{$alphabet[$i]}=$name; $name_letter_map{$name}=$alphabet[$i]; - $Items[$i] = $alphabet[$i]; $i++; } - $Apache::response::itemgroup{'letter_name_map'}=\%letter_name_map; + $Apache::response::itemgroup{'letter_name_map'}=\%letter_name_map; $Apache::response::itemgroup{'name_letter_map'}=\%name_letter_map; if ($target eq 'web') { my $table=''; @@ -146,18 +144,18 @@ sub end_itemgroup { $table.='
'; $Apache::matchresponse::itemtable{'display'}=$table; } elsif ($target eq 'tex') { - my $table=' \\\\\\\\ \begin{tabular}{ll} '; + my $table=' \\\\\\\\ \begin{description} '; my $i=0; foreach my $name (@names) { $Apache::response::itemgroup{$name.'.text'}=~s/\$\$/\$/g; - $table.=' '.$alphabet[$i].' & '. + $table.='\item['.$alphabet[$i].'] '. $Apache::response::itemgroup{$name.'.text'}. ' \\\\ '; $i++; } - $table.=' \end{tabular} \\\\ '; + $table.=' \end{description} \strut \\\\ '; $Apache::matchresponse::itemtable{'display'}=$table; - } elsif ($target eq 'edit') { $result=&Apache::edit::end_table(); } + } return $result; } @@ -250,6 +248,7 @@ sub end_foilgroup { } elsif ( $target eq 'grade') { &grade_response($max,$randomize); } + &Apache::lonxml::increment_counter(&getfoilcounts($max)); } elsif ($target eq 'edit') { $result=&Apache::edit::end_table(); } @@ -260,9 +259,10 @@ sub displayanswers { my ($max,$randomize,@opt)=@_; if (!defined(@{ $Apache::response::foilgroup{'names'} })) { return; } my @names = @{ $Apache::response::foilgroup{'names'} }; - my @whichfoils = &whichorder(&getfoilcounts($max),$randomize, - &Apache::response::showallfoils(), - \%Apache::response::foilgroup); + my @whichfoils = &Apache::response::whichorder(&getfoilcounts($max), + $randomize, + &Apache::response::showallfoils(), + \%Apache::response::foilgroup); my $result=&Apache::response::answer_header('matchresponse'); my %name_letter_map; if (defined(%{ $Apache::response::itemgroup{'name_letter_map'} })) { @@ -281,9 +281,10 @@ sub displayanswers { sub grade_response { my ($max,$randomize)=@_; - my (@whichfoils)=&whichorder(&getfoilcounts($max),$randomize, - &Apache::response::showallfoils(), - \%Apache::response::foilgroup); + my (@whichfoils)=&Apache::response::whichorder(&getfoilcounts($max), + $randomize, + &Apache::response::showallfoils(), + \%Apache::response::foilgroup); if (!defined($ENV{'form.submitted'})) { return; } my %responsehash; my %grade; @@ -352,9 +353,10 @@ sub displayfoils { my ($target,$max,$randomize)=@_; my $result; my $question; - my (@whichfoils)=&whichorder(&getfoilcounts($max),$randomize, - &Apache::response::showallfoils(), - \%Apache::response::foilgroup); + my (@whichfoils)=&Apache::response::whichorder(&getfoilcounts($max), + $randomize, + &Apache::response::showallfoils(), + \%Apache::response::foilgroup); my $part=$Apache::inputtags::part; my $solved=$Apache::lonhomework::history{"resource.$part.solved"}; my $status=$Apache::inputtags::status[-1]; @@ -368,7 +370,7 @@ sub displayfoils { %name_letter_map= %{ $Apache::response::itemgroup{'name_letter_map'} }; } - if (($solved =~ /^correct/) || ($status eq 'SHOW_ANSWER')) { + if ( &Apache::response::show_answer() ) { foreach my $name (@whichfoils) { my $text=$Apache::response::foilgroup{$name.'.text'}; my $value=$Apache::response::foilgroup{$name.'.value'}; @@ -387,7 +389,8 @@ sub displayfoils { my $lastresponse=$Apache::lonhomework::history{"resource.$part.$id.submission"}; my %lastresponse=&Apache::lonnet::str2hash($lastresponse); my @alphabet=('A'..'Z'); - my @Items_letter = splice @alphabet, 0, $#Items + 1; + my @used_letters=sort(keys(%letter_name_map)); + my $internal_counter=$Apache::lonxml::counter; foreach my $name (@whichfoils) { my $lastopt=$lastresponse{$name}; my $last_letter=$name_letter_map{$lastopt}; @@ -400,30 +403,35 @@ sub displayfoils { } } my $option; - foreach $option (sort(keys(%letter_name_map))) { + foreach $option (@used_letters) { if ($option eq $last_letter) { if ($target ne 'tex') {$optionlist.="\n";} } else { if ($target ne 'tex') {$optionlist.="\n";} } } - if ($target ne 'tex') { + if ($target ne 'tex' && $Apache::lonhomework::type ne 'exam') { $optionlist='\n"; } my $text=$Apache::response::foilgroup{$name.'.text'}; if ($target ne 'tex') { - $question.='
'.$optionlist.$text."\n"; + if ($Apache::lonhomework::type ne 'exam') { + $question.='
'.$optionlist.$text."\n"; + } else { + $question.='
'.$text."\n"; + } if ($Apache::lonhomework::type eq 'exam') { - $question.=&Apache::optionresponse::webbubbles(\@Items_letter,\@Items); + $question.=&Apache::optionresponse::webbubbles(\@used_letters,\@used_letters,$temp); } } else { if ($Apache::lonhomework::type eq 'exam') { $question.=' '.$optionlist.$text."\n"; my @emptyItems = (); - for (my $i=0;$i<=$#Items;$i++) {push @emptyItems, ' ';} - $question.='\vskip -2 mm\parbox{\textwidth}{\begin{enumerate}\item[\textbf{'.$Apache::lonxml::counter.'}.]\parbox{\textwidth - 5 mm}{'.&Apache::optionresponse::bubbles(\@Items_letter,\@emptyItems).'}\end{enumerate}} \vskip -10 mm \strut '; + for (my $i=0;$i<=$#used_letters;$i++) {push @emptyItems, ' ';} + $question.='\vskip -2 mm\parbox{\textwidth}{\begin{enumerate}\item[\textbf{'.$internal_counter.'}.]\parbox{\textwidth - 5 mm}{'.&Apache::optionresponse::bubbles(\@used_letters,\@emptyItems).'}\end{enumerate}} \vskip -10 mm \strut '; + $internal_counter++; } else { $question.=' \\\\ '.$optionlist.$text."\n"; } @@ -456,52 +464,6 @@ sub getfoilcounts { return $count; } -sub whichorder { - my ($max,$randomize,$showall,$hash)=@_; - #&Apache::lonxml::debug("man $max randomize $randomize"); - if (!defined(@{ $$hash{'names'} })) { return; } - my @names = @{ $$hash{'names'} }; - my @whichopt =(); - my (%top,@toplist,%bottom,@bottomlist); - if (!($showall || ($randomize eq 'no'))) { - my $current=0; - foreach my $name (@names) { - $current++; - if ($$hash{"$name.location"} eq 'top') { - $top{$name}=$current; - } elsif ($$hash{"$name.location"} eq 'bottom') { - $bottom{$name}=$current; - } - } - } - while ((($#whichopt+1) < $max) && ($#names > -1)) { - #&Apache::lonxml::debug("Have $#whichopt max is $max"); - my $aopt; - if ($showall || ($randomize eq 'no')) { - $aopt=0; - } else { - $aopt=int(&Math::Random::random_uniform() * ($#names+1)); - } - #&Apache::lonxml::debug("From $#whichopt $max $#names elms, picking $aopt"); - $aopt=splice(@names,$aopt,1); - #&Apache::lonxml::debug("Picked $aopt"); - if ($top{$aopt}) { - $toplist[$top{$aopt}]=$aopt; - } elsif ($bottom{$aopt}) { - $bottomlist[$bottom{$aopt}]=$aopt; - } else { - push (@whichopt,$aopt); - } - } - for (my $i=0;$i<=$#toplist;$i++) { - if ($toplist[$i]) { unshift(@whichopt,$toplist[$i]) } - } - for (my $i=0;$i<=$#bottomlist;$i++) { - if ($bottomlist[$i]) { push(@whichopt,$bottomlist[$i]) } - } - - return @whichopt; -} sub start_conceptgroup { my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_; @@ -612,7 +574,7 @@ sub end_foil { && !&Apache::response::showallfoils() ) { push @{ $Apache::response::conceptgroup{'names'} }, $name; $Apache::response::conceptgroup{"$name.value"} = $value; - if ($target eq 'tex') { + if ($target eq 'tex' && $Apache::lonhomework::type eq 'exam') { $Apache::response::conceptgroup{"$name.text"} = ' $\triangleright$ '.$text; } else { $Apache::response::conceptgroup{"$name.text"} = $text; @@ -621,8 +583,12 @@ sub end_foil { } else { push @{ $Apache::response::foilgroup{'names'} }, $name; $Apache::response::foilgroup{"$name.value"} = $value; - if ($Apache::lonhomework::type eq 'exam' and $flag_for_exam_printing) { - $Apache::response::foilgroup{"$name.text"} = '\vskip 5 mm $\triangleright$ '.$text; + if ($Apache::lonhomework::type eq 'exam') { + if ($target eq 'tex') { + $Apache::response::foilgroup{"$name.text"} = '\vskip 5 mm $\triangleright$ '.$text; + } else { + $Apache::response::foilgroup{"$name.text"} = $text; + } } else { if ($target eq 'tex') { $Apache::response::foilgroup{"$name.text"} = ' $\triangleright$ '.$text;