--- loncom/homework/radiobuttonresponse.pm	2001/01/08 21:52:04	1.2
+++ loncom/homework/radiobuttonresponse.pm	2001/01/11 22:17:30	1.4
@@ -7,13 +7,20 @@ package Apache::radiobuttonresponse;
 use strict;
 
 sub BEGIN {
-  &Apache::lonxml::register('Apache::radiobuttonresponse',('radiobuttonresponse','foilgroup','foil'));
+  &Apache::lonxml::register('Apache::radiobuttonresponse',('radiobuttonresponse'));
 }
 
 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={};
@@ -21,30 +28,76 @@ 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;
   my $result;
-  foreach $name ($Apache::response::foilgroup{'names'}) {
+  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') {
+      push (@truelist,$name);
+    } elsif ($Apache::response::foilgroup{$name.'.value'} eq 'false') {
+      push (@falselist,$name);
+    } elsif ($Apache::response::foilgroup{$name.'.value'} eq 'unused') {
+    } else {
+      &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 {
   my ($target,$token,$parstack,$parser,$safeeval,$style)=@_;
-  my $args ='';
-  if ( $#$parstack > -1 ) { $args=$$parstack[$#$parstack]; }
-  my $name = &Apache::run::run("{$args;".'return $name}',$safeeval);
-  push @{ $Apache::response::foilgroup{'names'} }, $name;
-  my $value = &Apache::run::run("{$args;".'return $value}',$safeeval);
-  $Apache::response::foilgroup{"$name.value"} = $value;
-  $Apache::response::foilgroup{"$name.text"} = $Apache::lonxml::outputstack;
-  $Apache::lonxml::redirection--;
+  if ($target eq 'web' || $target eq 'grade') {
+    my $args ='';
+    if ( $#$parstack > -1 ) { $args=$$parstack[$#$parstack]; }
+    my $name = &Apache::run::run("{$args;".'return $name}',$safeeval);
+    push @{ $Apache::response::foilgroup{'names'} }, $name;
+    my $value = &Apache::run::run("{$args;".'return $value}',$safeeval);
+    $Apache::response::foilgroup{"$name.value"} = $value;
+    $Apache::response::foilgroup{"$name.text"} = $Apache::lonxml::outputstack;
+  }
+  
+  $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 '';
 }