[LON-CAPA-cvs] cvs: loncom /homework radiobuttonresponse.pm

albertel lon-capa-cvs@mail.lon-capa.org
Fri, 03 May 2002 19:44:13 -0000


albertel		Fri May  3 15:44:13 2002 EDT

  Modified files:              
    /loncom/homework	radiobuttonresponse.pm 
  Log:
  - radiobuttonresponse was showing max+1 foils
  
  
Index: loncom/homework/radiobuttonresponse.pm
diff -u loncom/homework/radiobuttonresponse.pm:1.39 loncom/homework/radiobuttonresponse.pm:1.40
--- loncom/homework/radiobuttonresponse.pm:1.39	Wed Apr  3 14:32:36 2002
+++ loncom/homework/radiobuttonresponse.pm	Fri May  3 15:44:13 2002
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # mutliple choice style responses
 #
-# $Id: radiobuttonresponse.pm,v 1.39 2002/04/03 19:32:36 albertel Exp $
+# $Id: radiobuttonresponse.pm,v 1.40 2002/05/03 19:44:13 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -213,13 +213,13 @@
       push (@falselist,$name);
     } elsif ($Apache::response::foilgroup{$name.'.value'} eq 'unused') {
     } else {
-      &Apache::lonxml::error("Unknown state $Apache::response::foilgroup{$name.'.value'} for $name in <foilgroup>");
+      &Apache::lonxml::error(&HTML::Entites::encode("No valid value assigned ($Apache::response::foilgroup{$name.'.value'}) for foil $name in <foilgroup>"));
     }
   }
   my $whichtrue = int(rand($#truelist+1));
   &Apache::lonxml::debug("Max is $max, From $#truelist elms, picking $whichtrue");
   my @whichfalse =();
-  while ((($#whichfalse+1) < $max) && ($#falselist > -1)) {
+  while ((($#whichfalse+1) < $max-1) && ($#falselist > -1)) {
     &Apache::lonxml::debug("Have $#whichfalse max is $max");
     my $afalse=int(rand($#falselist+1));
     &Apache::lonxml::debug("From $#falselist elms, picking $afalse");