--- loncom/homework/caparesponse/caparesponse.pm	2007/06/05 22:30:05	1.215
+++ loncom/homework/caparesponse/caparesponse.pm	2007/08/29 21:57:47	1.219
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # caparesponse definition
 #
-# $Id: caparesponse.pm,v 1.215 2007/06/05 22:30:05 albertel Exp $
+# $Id: caparesponse.pm,v 1.219 2007/08/29 21:57:47 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -431,6 +431,10 @@ sub check_submission {
     
     &add_in_tag_answer($parstack,$safeeval);
 
+    if (!%answer) {
+	&Apache::lonxml::error("No answers are defined");
+    }
+
     my (@final_awards,@final_msgs,@names);
     foreach my $name (keys(%answer)) {
 	&Apache::lonxml::debug(" doing $name with ".join(':',@{ $answer{$name}{'answers'} }));
@@ -605,6 +609,7 @@ sub end_numericalresponse {
 	my $part_id="$partid.$id";
 	if ($target eq 'analyze') {
 	    push (@{ $Apache::lonhomework::analyze{"parts"} },$part_id);
+	    push (@{ $Apache::lonhomework::analyze{"$part_id.bubble_lines"} }, 1);
 	    $Apache::lonhomework::analyze{"$part_id.type"} = $tag;
 	    my (@incorrect)=&Apache::lonxml::get_param_var('incorrect',$parstack,$safeeval);
 	    if ($#incorrect eq 0) { @incorrect=(split(/,/,$incorrect[0])); }
@@ -859,7 +864,13 @@ sub get_table_sizes {
     my $bubbles_per_line=int($textwidth/$cell_width);
     if ($bubbles_per_line > $number_of_bubbles) {
 	$bubbles_per_line=$number_of_bubbles;
-    }elsif (($bubbles_per_line > $number_of_bubbles/2) && ($number_of_bubbles % 2==0)) {$bubbles_per_line=$number_of_bubbles/2;}
+    } elsif (($bubbles_per_line > $number_of_bubbles/2) 
+	     && ($number_of_bubbles % 2==0)) {
+	$bubbles_per_line=$number_of_bubbles/2;
+    }
+    if ($bubbles_per_line < 1) {
+	$bubbles_per_line=1;
+    }
     my $number_of_tables = int($number_of_bubbles/$bubbles_per_line);
     my @table_range = ();
     for (my $i=0;$i<$number_of_tables;$i++) {push @table_range,$bubbles_per_line;}
@@ -999,7 +1010,8 @@ sub get_sigrange {
     my ($sig)=@_;
     #&Apache::lonxml::debug("Got a sig of :$sig:");
     my $courseid=$env{'request.course.id'};
-    if (lc($env{"course.$courseid.disablesigfigs"}) eq 'yes') {
+    if ($env{'request.state'} ne 'construct'
+	&& lc($env{"course.$courseid.disablesigfigs"}) eq 'yes') {
 	return (15,0);
     }
     my $sig_lbound;