--- loncom/homework/radiobuttonresponse.pm	2001/01/12 01:07:13	1.5
+++ loncom/homework/radiobuttonresponse.pm	2001/02/19 20:36:49	1.12
@@ -1,8 +1,6 @@
 # The LearningOnline Network with CAPA
 # mutliple choice style responses
 
-# 11/23,11/24,11/28 Gerd Kortemeyer
-
 package Apache::radiobuttonresponse;
 use strict;
 
@@ -26,6 +24,7 @@ sub end_radiobuttonresponse {
 %Apache::response::foilgroup={};
 sub start_foilgroup {
   %Apache::response::foilgroup={};
+  return '';
 }
 
 sub setrandomnumber {
@@ -38,29 +37,33 @@ sub setrandomnumber {
   return '';
 }
 
-#FIXME needs to stablely do random picks
 sub end_foilgroup {
   my ($target,$token,$parstack,$parser,$safeeval,$style)=@_;
-  my $name;
+  
   my $result;
-  &setrandomnumber();
-  my $rrargs ='';
-  if ( $#$parstack > 0 ) { $rrargs=$$parstack['-2']; }
-  my $max = &Apache::run::run("{$rrargs;".'return $max}',$safeeval);
-  my $answer = int(rand $max);
-  if ($target eq 'web') {
-    $result=&displayfoils($max,$answer);
-  } elsif ( $target eq 'grade') {
-    if ( defined $ENV{'form.submit'}) {
-      my $response = $ENV{'form.HWVAL'.$Apache::inputtags::response['-1']};
-      if ( $response =~ /[^\s]/) {
-	my $id = $Apache::inputtags::response['-1'];
-	$Apache::lonhomework::results{"resource.$Apache::inputtags::part.$id.submission"}=$response;
-	&Apache::lonxml::debug("submitted a $response<br>\n");
-	if ($response == $answer) {
-	  $Apache::lonhomework::results{"resource.$Apache::inputtags::part.$id.awarddetail"}='EXACT_ANS';
-	} else {
-	  $Apache::lonhomework::results{"resource.$Apache::inputtags::part.$id.awarddetail"}='INCORRECT';
+  if ($target ne 'meta') {
+    my $name;
+    &setrandomnumber();
+    my ($truecnt,$falsecnt,$max) = &getfoilcounts($parstack,$safeeval);
+    my $count=0;
+    # we will add in 1 of the true statements
+    if (($falsecnt+1)>$max) { $count=$max } else { $count=$falsecnt+1; }
+    my $answer = int(rand ($count));
+    &Apache::lonxml::debug("Answer is $answer, $count from $max, $falsecnt");
+    if ($target eq 'web') {
+      $result=&displayfoils($max,$answer);
+    } elsif ( $target eq 'grade') {
+      if ( defined $ENV{'form.submitted'}) {
+	my $response = $ENV{'form.HWVAL'.$Apache::inputtags::response['-1']};
+	if ( $response =~ /[^\s]/) {
+	  my $id = $Apache::inputtags::response['-1'];
+	  $Apache::lonhomework::results{"resource.$Apache::inputtags::part.$id.submission"}=$response;
+	  &Apache::lonxml::debug("submitted a $response<br />\n");
+	  if ($response == $answer) {
+	    $Apache::lonhomework::results{"resource.$Apache::inputtags::part.$id.awarddetail"}='EXACT_ANS';
+	  } else {
+	    $Apache::lonhomework::results{"resource.$Apache::inputtags::part.$id.awarddetail"}='INCORRECT';
+	  }
 	}
       }
     }
@@ -68,6 +71,26 @@ sub end_foilgroup {
   return $result;
 }
 
+sub getfoilcounts {
+  my ($parstack,$safeeval)=@_;
+  my $rrargs ='';
+  if ( $#$parstack > 0 ) { $rrargs=$$parstack['-2']; }
+  my $max = &Apache::run::run("{$rrargs;".'return $max}',$safeeval);
+  my @names = @{ $Apache::response::foilgroup{'names'} };
+  my $truecnt=0;
+  my $falsecnt=0;
+  my $name;
+
+  foreach $name (@names) {
+    if ($Apache::response::foilgroup{$name.'.value'} eq 'true') {
+      $truecnt++;
+    } elsif ($Apache::response::foilgroup{$name.'.value'} eq 'false') {
+      $falsecnt++;
+    }
+  }
+  return ($truecnt,$falsecnt,$max);
+}
+
 sub displayfoils {
   my ($max,$answer)=@_;
   my @names = @{ $Apache::response::foilgroup{'names'} };
@@ -77,7 +100,7 @@ sub displayfoils {
   my $name;
 
   foreach $name (@names) {
-    #result.="<br><b>$name</b> is <i> $Apache::response::foilgroup{$name.'.value'} </i>";
+    #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') {
@@ -98,11 +121,23 @@ sub displayfoils {
     push (@whichfalse,$afalse);
   }
   splice(@whichfalse,$answer,0,$truelist[$whichtrue]);
-  my $temp=0;
   &Apache::lonxml::debug("the true statement is $answer");
-  foreach $name (@whichfalse) {
-    $result.="<br /><input type=\"radio\" name=\"HWVAL$Apache::inputtags::response['-1']\" value=\"$temp\">".$Apache::response::foilgroup{$name.'.text'}."</input>\n";
-    $temp++;
+  if ($Apache::lonhomework::history{"resource.$Apache::inputtags::part.solved"} =~ /^correct/ ) {
+    foreach $name (@whichfalse) {
+      $result.="<br />";
+      if ($Apache::response::foilgroup{$name.'.value'} eq 'true') { 
+	$result.='Correct';
+      } else {
+	$result.='Incorrect';
+      }
+      $result.=":".$Apache::response::foilgroup{$name.'.text'}."</input>\n";
+    }
+  } else {
+    my $temp=0;
+    foreach $name (@whichfalse) {
+      $result.="<br /><input type=\"radio\" name=\"HWVAL$Apache::inputtags::response['-1']\" value=\"$temp\">".$Apache::response::foilgroup{$name.'.text'}."</input>\n";
+      $temp++;
+    }
   }
   return $result."<br />";
 }