--- loncom/homework/radiobuttonresponse.pm	2001/01/08 22:06:48	1.3
+++ loncom/homework/radiobuttonresponse.pm	2001/01/11 22:17:30	1.4
@@ -11,11 +11,16 @@ sub BEGIN {
 }
 
 sub start_radiobuttonresponse {
+  my ($target,$token,$parstack,$parser,$safeeval,$style)=@_;
   #when in a radiobutton response use these
   &Apache::lonxml::register('Apache::radiobuttonresponse',('foilgroup','foil'));
+  my $id = &Apache::response::start_response($parstack,$safeeval);
+  return '';
 }
 
 sub end_radiobuttonresponse {
+  &Apache::response::end_response;
+  return '';
 }
 
 %Apache::response::foilgroup={};
@@ -23,6 +28,7 @@ sub start_foilgroup {
   %Apache::response::foilgroup={};
 }
 
+#FIXME needs to stablely do random picks
 sub end_foilgroup {
   my ($target,$token,$parstack,$parser,$safeeval,$style)=@_;
   my $name;
@@ -30,6 +36,11 @@ sub end_foilgroup {
   my @names = @{ $Apache::response::foilgroup{'names'} };
   my @truelist;
   my @falselist;
+  my $rndseed=&Apache::lonnet::rndseed();
+  $rndseed=$rndseed
+    +&Apache::lonnet::numval($Apache::inputtags::part)
+      +&Apache::lonnet::numval($Apache::inputtags::response['-1']);
+  srand($rndseed*10);
   foreach $name (@names) {
     $result.="<br><b>$name</b> is <i> $Apache::response::foilgroup{$name.'.value'} </i>";
     if ($Apache::response::foilgroup{$name.'.value'} eq 'true') {
@@ -41,12 +52,32 @@ sub end_foilgroup {
       &Apache::lonxml::error("Unknow state $Apache::response::foilgroup{$name.'.value'} for $name in <foilgroup>");
     }
   }
+  my $rrargs ='';
+  if ( $#$parstack > 0 ) { $rrargs=$$parstack['-2']; }
+  my $max = &Apache::run::run("{$rrargs;".'return $max}',$safeeval);
   my $whichtrue = rand $#truelist;
+  &Apache::lonxml::debug("Max is $max, From $#truelist elms, picking $whichtrue");
+  my @whichfalse =();
+  my $temp=0;
+  while ((($#whichfalse+2) < $max) && ($#falselist > -1)) {
+    my $afalse=rand $#falselist;
+    &Apache::lonxml::debug("From $#falselist elms, picking $afalse");
+    $afalse=splice(@falselist,$afalse,1);
+    &Apache::lonxml::debug("Picked $afalse");
+    push (@whichfalse,$afalse);
+  }
+  my $where = rand $max;
+  splice(@whichfalse,$where,0,$truelist[$whichtrue]);
+  my $temp;
+  foreach $temp (@whichfalse) {
+    $result.='<input type="radio" name="bob">'.$Apache::response::foilgroup{$temp.'.text'}."</input><br />\n";
+  }
   return $result;
 }
 
 sub start_foil {
-  $Apache::lonxml::redirection++;
+  &Apache::lonxml::debug("redirection be  $Apache::lonxml::redirection");
+  $Apache::lonxml::redirection--;
 }
 
 sub end_foil {
@@ -61,7 +92,12 @@ sub end_foil {
     $Apache::response::foilgroup{"$name.text"} = $Apache::lonxml::outputstack;
   }
   
-  $Apache::lonxml::redirection--;
+  $Apache::lonxml::redirection++;
+  &Apache::lonxml::debug("redirection is  $Apache::lonxml::redirection");
+  if ($Apache::lonxml::redirection == 1) { 
+    &Apache::lonxml::debug("reset $Apache::lonxml::outputstack");
+    $Apache::lonxml::outputstack=''; 
+  }
   return '';
 }