--- loncom/homework/optionresponse.pm	2007/04/05 10:29:58	1.140
+++ loncom/homework/optionresponse.pm	2008/08/08 16:37:13	1.151
@@ -1,7 +1,7 @@
 # LearningOnline Network with CAPA
 # option list style responses
 #
-# $Id: optionresponse.pm,v 1.140 2007/04/05 10:29:58 foxr Exp $
+# $Id: optionresponse.pm,v 1.151 2008/08/08 16:37:13 bisitz Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -70,7 +70,10 @@ sub start_optionresponse {
     $result=&Apache::response::meta_package_write('optionresponse');
   } elsif ($target eq 'analyze') {
     my $part_id="$Apache::inputtags::part.$id";
+    $Apache::lonhomework::analyze{"$part_id.type"} = 'optionresponse';
     push (@{ $Apache::lonhomework::analyze{"parts"} },$part_id);
+    push (@{ $Apache::lonhomework::analyze{"$part_id.bubble_lines"} }, 1);
+
   }
   return $result;
 }
@@ -275,7 +278,14 @@ sub end_foilgroup {
 	}
       }
     }
-    &Apache::lonxml::increment_counter(&getfoilcounts($max));
+    my $part_id     = $Apache::inputtags::part;
+    my $response_id = $Apache::inputtags::response[-1];
+    &Apache::lonxml::increment_counter(&getfoilcounts($max),
+				       "$part_id.$response_id");
+    if ($target eq 'analyze') {
+	&Apache::lonhomework::set_bubble_lines();
+    }
+
   } elsif ($target eq 'edit') {
     $result.=&Apache::edit::end_table();
   }
@@ -355,27 +365,6 @@ sub check_for_invalid {
     }
 }
 
-sub setup_prior_tries_hash {
-    my ($whichopt) = @_;
-    my $part=$Apache::inputtags::part;
-    my $id=$Apache::inputtags::response[-1];	
-    foreach my $i (1..$Apache::lonhomework::history{'version'}) {
-	my $key = "$i:resource.$part.$id.submission";
-	next if (!exists($Apache::lonhomework::history{"$key"}));
-	my %lastresponse=
-	    &Apache::lonnet::str2hash($Apache::lonhomework::history{$key});
-	my $output;
-	foreach my $name (@$whichopt) {
-	    next if (!defined($lastresponse{$name}));
-	    $output .= '<td>'.$lastresponse{$name}.'</td>';
-	}
-	next if (!defined($output));
-	$output =
-	    '<table class="LC_optres_prior"><tr>'.$output.'</tr></table>';
-	$Apache::inputtags::submission_display{$key} = $output;
-    }
-}
-
 sub displayfoils {
   my ($target,$max,$randomize,$TeXlayout,@opt)=@_;
   if (!defined(@{ $Apache::response::foilgroup{'names'} })) {return;}
@@ -499,7 +488,9 @@ sub displayfoils {
   }
 
   if ($target eq 'web') {
-      &setup_prior_tries_hash(\@whichopt);
+      &Apache::response::setup_prior_tries_hash(\&Apache::rankresponse::format_prior_answer,
+						[\@whichopt,
+						 'submissiongrading']);
   }
 
   if ($target ne 'tex') {
@@ -512,7 +503,7 @@ sub displayfoils {
 
 sub optionlist_correction {
     my ($TeXlayout,@options) = @_;
-    my $texoptionlist='\\item [] Choices: ';
+    my $texoptionlist='\\item [] '.&mt('Choices:').' ';
     if ($TeXlayout eq 'vertical') {$texoptionlist='\\item []';}
     if (scalar(@options) > 0) {
 	foreach my $option (@options) {
@@ -567,20 +558,19 @@ sub bubbles {
     my $number_of_bubbles = $#opt + 1;
     my $current_length = 0;
     my $textwidth;
-    if ($max_width) {
+    if (defined($max_width)) {
 	$textwidth=$max_width;
-	&Apache::lonnet::logthis("Max width passed in: $max_width");
-    }
-    elsif ($env{'form.textwidth'} ne '') {
+	&Apache::lonxml::debug("Max width passed in: $max_width");
+    } elsif ($env{'form.textwidth'} ne '') {
 	$env{'form.textwidth'}=~/(\d+)/;
 	$textwidth=$1;
-	&Apache::lonnet::logthis("Max width from form: $textwidth");
+	&Apache::lonxml::debug("Max width from form: $textwidth");
     } else {
 	$env{'form.textwidth'}=~/(\d*)\.?(\d*)/;
 	$textwidth=$1.'.'.$2;
-	&Apache::lonnet::logthis("Max width defaults? $textwidth");
+	&Apache::lonxml::debug("Max width defaults? $textwidth");
     }
-    &Apache::lonnet::logthis("Final maxwidth: $textwidth");
+    &Apache::lonxml::debug("Final maxwidth: $textwidth");
     for (my $ind=0;$ind<=$number_of_bubbles;$ind++) {
 	my $leftmargin;
 	$opt[$ind]=&Apache::lonxml::latex_special_symbols($opt[$ind]);
@@ -694,12 +684,12 @@ sub end_foil {
       my $name = &Apache::lonxml::get_param('name',$parstack,$safeeval);
       &Apache::lonxml::debug("Got a name of :$name:");
       if ($name eq "") {
-	  &Apache::lonxml::warning("Foils without names exist. This can cause problems to malfunction.");
+	  &Apache::lonxml::warning(&mt('Foils without names exist. This can cause problems to malfunction.'));
 	  $name=$Apache::lonxml::curdepth;
       }
       &Apache::lonxml::debug("Using a name of :$name:");
       if (defined($Apache::response::foilnames{$name})) {
-	  &Apache::lonxml::error(&mt("Foil name <b><tt>[_1]</tt></b> appears more than once. Foil names need to be unique.",$name));
+	  &Apache::lonxml::error(&mt("Foil name [_1] appears more than once. Foil names need to be unique.",'<b><tt>'.$name.'</tt></b>'));
       }
       $Apache::response::foilnames{$name}++;
       my $location =&Apache::lonxml::get_param('location',$parstack,$safeeval);
@@ -725,25 +715,29 @@ sub end_foil {
 
 sub start_drawoptionlist {
     my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-    if (($target ne 'meta')  && ($target ne 'answer')) {
+    if ($target !~ /^(?:meta|answer|modified|edit)$/) {
 	return $token->[4];
     }
 }
 
 sub end_drawoptionlist {
     my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-    if (($target ne 'meta') && ($target ne 'answer')) {
+    if ($target !~ /^(?:meta|answer|modified|edit)$/) {
 	return $token->[2];
     }
 }
 
 sub insert_foil {
-  return '
+    return '
 <foil name="" value="unused">
 <startouttext />
 <endouttext />
 </foil>';
 }
+
+sub insert_drawoptionlist {
+    return '<drawoptionlist />';
+}
 1;
 __END__