--- loncom/homework/caparesponse/caparesponse.pm	2004/10/21 02:43:34	1.156
+++ loncom/homework/caparesponse/caparesponse.pm	2005/01/10 18:42:21	1.160
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # caparesponse definition
 #
-# $Id: caparesponse.pm,v 1.156 2004/10/21 02:43:34 albertel Exp $
+# $Id: caparesponse.pm,v 1.160 2005/01/10 18:42:21 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -160,8 +160,8 @@ sub end_numericalresponse {
 		    my (@formats)=&Apache::lonxml::get_param_var('format',$parstack,$safeeval);
 		    my (@answers)=&Apache::lonxml::get_param_var('answer',$parstack,$safeeval);
 		    my (@incorrect)=&Apache::lonxml::get_param_var('incorrect',$parstack,$safeeval);
-		    my @values=&make_numerical_bubbles($number_of_bubbles,$target,$answers[0],$formats[0],\@incorrect,$safeeval);
-		    $response=$values[$response];
+		    my ($values,$display)=&make_numerical_bubbles($number_of_bubbles,$target,$answers[0],$formats[0],\@incorrect,$safeeval);
+		    $response=$values->[$response];
 		}
 		$Apache::lonhomework::results{"resource.$partid.$id.submission"}=$response;
 		my $args_ref= \%{$safeeval->varglob('LONCAPA::CAPAresponse_args')};
@@ -238,9 +238,10 @@ sub end_numericalresponse {
 	    my $unit=&Apache::lonxml::get_param_var('unit',$parstack,
 						    $safeeval);
 	    my (@incorrect)=&Apache::lonxml::get_param_var('incorrect',$parstack,$safeeval);
-	    my @bubble_values=&make_numerical_bubbles($number_of_bubbles,
-						      $target,$answers[0],
-						      $formats[0],\@incorrect,$safeeval);
+	    my ($bubble_values,$bubble_display)=
+		&make_numerical_bubbles($number_of_bubbles,
+					$target,$answers[0],
+					$formats[0],\@incorrect,$safeeval);
 	    my @alphabet=('A'..'Z');
 	    if ($target eq 'web') {
 		if ($tag eq 'numericalresponse') {
@@ -249,13 +250,13 @@ sub end_numericalresponse {
 		    my $previous=$Apache::lonhomework::history{"resource.$Apache::inputtags::part.$id.submission"};
 		    for (my $ind=0;$ind<$number_of_bubbles;$ind++) {
 			my $checked='';
-			if ($previous eq $bubble_values[$ind]) {
+			if ($previous eq $bubble_values->[$ind]) {
 			    $checked=" checked='on' ";
 			}
 			$result.='<td><input type="radio" name="HWVAL_'.$id.
-			    '" value="'.$bubble_values[$ind].'" '.$checked
+			    '" value="'.$bubble_values->[$ind].'" '.$checked
 			    .' /><b>'.$alphabet[$ind].'</b>: '.
-			    $bubble_values[$ind].'</td>';
+			    $bubble_display->[$ind].'</td>';
 		    }
 		    $result.='</tr></table>';
 		} elsif ($tag eq 'formularesponse') {
@@ -269,7 +270,7 @@ sub end_numericalresponse {
 		}
 		if ($tag eq 'numericalresponse') {
 		    my ($celllength,$number_of_tables,@table_range)=
-			&get_table_sizes($number_of_bubbles,\@bubble_values);
+			&get_table_sizes($number_of_bubbles,$bubble_display);
 		    my $j=0;
 		    my $cou=0;
 		    $result.='\vskip -1 mm \noindent \begin{enumerate}\item[\textbf{'.$Apache::lonxml::counter.'}.]';
@@ -280,7 +281,7 @@ sub end_numericalresponse {
 			}
 			$result.='}';
 			for (my $ind=$cou;$ind<$cou+$table_range[$j];$ind++) {
-			    $result.='\hskip -4 mm {\small \textbf{'.$alphabet[$ind].'}}$\bigcirc$ & \hskip -3 mm {\small '.$bubble_values[$ind].'} ';
+			    $result.='\hskip -4 mm {\small \textbf{'.$alphabet[$ind].'}}$\bigcirc$ & \hskip -3 mm {\small '.$bubble_display->[$ind].'} ';
 			    if ($ind != $cou+$table_range[$j]-1) {$result.=' & ';}
 			}
 			$cou += $table_range[$j];
@@ -400,7 +401,7 @@ sub get_table_sizes {
     my $cell_width=0;
     foreach my $member (@$rbubble_values) {
 	my $cell_width_real=0;
-	if ($member=~/(\+|-)?(\d*)\.?(\d*)\s*\$\\times\s*10\^{(\+|-)?(\d+)}\$/) {
+	if ($member=~/(\+|-)?(\d*)\.?(\d*)\s*\$?\\times\s*10\^{(\+|-)?(\d+)}\$?/) {
 	    $cell_width_real=(length($2)+length($3)+length($5)+7)*$scale;
 	} elsif ($member=~/(\d*)\.?(\d*)(E|e)(\+|-)?(\d*)/) {
 	    $cell_width_real=(length($1)+length($2)+length($5)+9)*$scale;
@@ -458,7 +459,7 @@ sub format_number {
 
 sub make_numerical_bubbles {
     my ($number_of_bubbles,$target,$answer,$format,$incorrect,$safeeval) =@_;
-    my @bubble_values = ();
+    my @bubble_values=();
     &Apache::lonxml::debug("answer is $answer incorrect is $incorrect");
     my @oldseed=&Math::Random::random_get_seed();
     if (defined($incorrect) && ref($incorrect)) {
@@ -473,11 +474,15 @@ sub make_numerical_bubbles {
 	    &Apache::lonxml::debug("Answer was :$answer: returning :".$#bubble_values.": whih are :".join(':',@bubble_values));
 	    &Math::Random::random_set_seed(@oldseed);
 	    if (defined($format) && $format ne '') {
+		my @bubble_display;
 		foreach my $value (@bubble_values) {
-		    $value=&format_number($value,$format,$target,$safeeval);
+		    push(@bubble_display,
+			 &format_number($value,$format,$target,$safeeval));
 		}
+		return (\@bubble_values,\@bubble_display);
+	    } else {
+		return (\@bubble_values,\@bubble_values);
 	    }
-	    return @bubble_values;
 	}
 	if (defined($$incorrect[0]) &&
 	    scalar(@$incorrect)+1 < $number_of_bubbles) {
@@ -491,14 +496,15 @@ sub make_numerical_bubbles {
     my $power = $powers[$ind];
     $ind=&Math::Random::random_uniform_integer(1,0,$#factors);
     my $factor = $factors[$ind];
+    my @bubble_display;
     for ($ind=0;$ind<$number_of_bubbles;$ind++) {
 	$bubble_values[$ind] = $answer*($factor**($power-$powers[$#powers-$ind]));
-	$bubble_values[$ind] = &format_number($bubble_values[$ind],
+	$bubble_display[$ind] = &format_number($bubble_values[$ind],
 					       $format,$target,$safeeval);
 
     }
     &Math::Random::random_set_seed(@oldseed);
-    return @bubble_values;
+    return (\@bubble_values,\@bubble_display);
 }
 
 sub get_tolrange {
@@ -609,6 +615,17 @@ sub end_stringresponse {
 	if ($Apache::lonhomework::type eq 'exam' ||
 	    $ENV{'form.submitted'} eq 'scantron') {
 	    $increment=&Apache::response::scored_response($part,$id);
+	} elsif ($Apache::lonhomework::type eq 'survey') {
+	    if ( !defined($ENV{'form.submitted'})) { return ''; }
+	    my $response = &Apache::response::getresponse();
+	    if ( $response =~ /[^\s]/) {
+		my %previous=&Apache::response::check_for_previous($response,
+								   $part,$id);
+		$Apache::lonhomework::results{"resource.$part.$id.submission"}=
+		    $response;
+		my $ad=$Apache::lonhomework::results{"resource.$part.$id.awarddetail"}='SUBMITTED';
+		&Apache::response::handle_previous(\%previous,$ad);
+	    }
 	} else {
 	    my $response = &Apache::response::getresponse();
 	    if ( $response =~ /[^\s]/) {