--- loncom/homework/hint.pm	2002/10/28 23:31:48	1.30
+++ loncom/homework/hint.pm	2004/03/12 18:41:43	1.48
@@ -1,13 +1,13 @@
 # The LearningOnline Network with CAPA 
 # implements the tags that control the hints
 #
-# $Id: hint.pm,v 1.30 2002/10/28 23:31:48 albertel Exp $
+# $Id: hint.pm,v 1.48 2004/03/12 18:41:43 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
 # This file is part of the LearningOnline Network with CAPA (LON-CAPA).
 #
-# LON-CAPA is free software; you can redistribute it and/or modify
+# LON-CAPA me&aree software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
 # the Free Software Foundation; either version 2 of the License, or
 # (at your option) any later version.
@@ -33,202 +33,214 @@ use Apache::lonnet;
 use capa;
 
 BEGIN {
-  &Apache::lonxml::register('Apache::hinttags',('hintgroup','hintpart','numericalhint','stringhint','formulahint','optionhint','radiobuttonhint'));
+    &Apache::lonxml::register('Apache::hinttags',('hintgroup','hintpart','numericalhint','stringhint','formulahint','optionhint','radiobuttonhint'));
 }
 
 
 @Apache::hint::which=();
 sub start_hintgroup {
-  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  my $skiptoend='0';
-  my $result;
-
-  if ($target eq 'web') {
-    my $id=$Apache::inputtags::part;
-    my $numtries=$Apache::lonhomework::history{"resource.$id.tries"};
-    if ( $numtries eq '') { $numtries = 0; }
-    my $hinttries=&Apache::lonnet::EXT("resource.$id.hinttries");
-    if ( $hinttries eq '') { $hinttries = 1; }
-    &Apache::lonxml::debug("found :$id:$numtries:$hinttries:");
-    if ( $numtries < $hinttries ) {
-      $skiptoend='1';
-    } else {
-      if ($target eq 'web') {$result='<table bgcolor="#dddddd"><tr><td>';}
-    }
-    if ($skiptoend) {
-      &Apache::lonxml::get_all_text("/hintgroup",$$parser[$#$parser]);
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+    my $skiptoend='0';
+    my $result;
+
+    if ($target eq 'web') {
+	my $id=$Apache::inputtags::part;
+	my $numtries=$Apache::lonhomework::history{"resource.$id.tries"};
+	if ( $numtries eq '') { $numtries = 0; }
+	my $hinttries=&Apache::lonnet::EXT("resource.$id.hinttries");
+	if ($ENV{'request.state'} eq 'construct' &&
+	    defined($Apache::inputtags::params{'hinttries'})) {
+	    $hinttries=$Apache::inputtags::params{'hinttries'};
+	}
+
+	if ( $hinttries eq '') { $hinttries = 1; }
+	&Apache::lonxml::debug("found :$id:$numtries:$hinttries:");
+	my $gradestatus=$Apache::lonhomework::history{"resource.$id.solved"};
+	if ( $numtries < $hinttries || $gradestatus =~ /^correct/) {
+	    &Apache::lonxml::get_all_text("/hintgroup",$parser);
+	}
+	&Apache::lonxml::startredirection;
+    } elsif ($target eq 'tex') {
+	$result .= '\keephidden{';
     }
-  } elsif ($target eq 'tex') {
-    $result .= '\keephidden{'; 
-  }
-  @Apache::hint::which=();
-  return $result;
+    @Apache::hint::which=();
+    return $result;
 }
 
 sub end_hintgroup {
-  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  my $result;
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+    my $result;
 
-  if ($target eq 'web') {
-    my $id=$Apache::inputtags::part;
-    my $numtries=$Apache::lonhomework::history{"resource.$id.tries"};
-    if ( $numtries eq '') { $numtries = 0; }
-    my $hinttries=&Apache::lonnet::EXT("resource.$id.hinttries");
-    if ( $hinttries eq '') { $hinttries = 1; }
-    &Apache::lonxml::debug("found :$id:$numtries:$hinttries:");
-    if ( $numtries >= $hinttries ) {
-      $result='</td></tr></table>';
+    if ($target eq 'web') {
+	my $id=$Apache::inputtags::part;
+	my $numtries=$Apache::lonhomework::history{"resource.$id.tries"};
+	if ( $numtries eq '') { $numtries = 0; }
+	my $hinttries=&Apache::lonnet::EXT("resource.$id.hinttries");
+	if ( $hinttries eq '') { $hinttries = 1; }
+	&Apache::lonxml::debug("found :$id:$numtries:$hinttries:");
+	my $hinttext=&Apache::lonxml::endredirection;
+	if ($Apache::lonhomework::type ne 'exam' &&
+	    $numtries >= $hinttries && $hinttext =~/\S/) {
+	    $result='<table bgcolor="#dddddd"><tr><td>'.
+		$hinttext.'</td></tr></table>';
+	}
+    } elsif ($target eq 'edit') {
+	$result.=&Apache::edit::end_table();
+    } elsif ($target eq 'tex') {
+	$result .= '}';
     }
-  } elsif ($target eq 'edit') {
-    $result.=&Apache::edit::end_table();
-  } elsif ($target eq 'tex') {
-    $result .= '}'; 
-  }
-  @Apache::hint::which=();
-  return $result;
+    @Apache::hint::which=();
+    return $result;
 }
 
 sub start_numericalhint {
-  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  #do everything in end, so intervening <responseparams> work
-  &Apache::response::start_hintresponse($parstack,$safeeval);
-  my $result;
-  if ($target eq 'edit') {
-    $result.=&Apache::edit::tag_start($target,$token);
-    $result.=&Apache::edit::text_arg('Name:','name',$token);
-    $result.=&Apache::edit::text_arg('Answer:','answer',$token);
-    if ($token->[1] eq 'numericalhint') {
-      $result.=&Apache::edit::text_arg('Unit:','unit',$token,5);
-      $result.=&Apache::edit::text_arg('Format:','format',$token,4);
-    } elsif ($token->[1] eq 'stringhint') {
-      $result.=&Apache::edit::select_arg('Type:','type',['cs','ci','mc'],$token);
-    } elsif ($token->[1] eq 'formulahint') {
-      $result.=&Apache::edit::text_arg('Sample Points:','samples',$token,40);
-    }
-    $result.=&Apache::edit::end_row();
-    $result.=&Apache::edit::start_spanning_row();
-  } elsif ($target eq 'modified') {
-    my $constructtag;
-    if ($token->[1] eq 'numericalhint') {
-      $constructtag=&Apache::edit::get_new_args($token,$parstack,$safeeval,
-						'name','answer','unit','format');
-    } elsif ($token->[1] eq 'stringhint') {
-      $constructtag=&Apache::edit::get_new_args($token,$parstack,$safeeval,
-						'name','answer','type');
-    } elsif ($token->[1] eq 'formulahint') {
-      $constructtag=&Apache::edit::get_new_args($token,$parstack,$safeeval,
-						'name','answer','samples');
-    }
-    if ($constructtag) {
-      $result  = &Apache::edit::rebuild_tag($token);
-      $result .= &Apache::edit::handle_insert();
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+    #do everything in end, so intervening <responseparams> work
+    &Apache::response::start_hintresponse($parstack,$safeeval);
+    my $result;
+    if ($target eq 'edit') {
+	$result.=&Apache::edit::tag_start($target,$token);
+	$result.=&Apache::edit::text_arg('Name:','name',$token);
+	$result.=&Apache::edit::text_arg('Answer:','answer',$token);
+	if ($token->[1] eq 'numericalhint') {
+	    $result.=&Apache::edit::text_arg('Unit:','unit',$token,5).
+		&Apache::loncommon::help_open_topic('Physical_Units');
+	    $result.=&Apache::edit::text_arg('Format:','format',$token,4).
+		&Apache::loncommon::help_open_topic('Numerical_Response_Format');
+	} elsif ($token->[1] eq 'stringhint') {
+	    $result.=&Apache::edit::select_arg('Type:','type',
+			    [['cs','Case Sensitive'],['ci','Case Insensitive'],
+			     ['mc','Case Insensitive, Any Order']],$token);
+	} elsif ($token->[1] eq 'formulahint') {
+	    $result.=&Apache::edit::text_arg('Sample Points:','samples',$token,40);
+	}
+	$result.=&Apache::edit::end_row();
+	$result.=&Apache::edit::start_spanning_row();
+    } elsif ($target eq 'modified') {
+	my $constructtag;
+	if ($token->[1] eq 'numericalhint') {
+	    $constructtag=&Apache::edit::get_new_args($token,$parstack,
+						     $safeeval,'name',
+						     'answer','unit','format');
+	} elsif ($token->[1] eq 'stringhint') {
+	    $constructtag=&Apache::edit::get_new_args($token,$parstack,
+						     $safeeval,'name','answer',
+						     'type');
+	} elsif ($token->[1] eq 'formulahint') {
+	    $constructtag=&Apache::edit::get_new_args($token,$parstack,
+						     $safeeval,'name','answer',
+						     'samples');
+	}
+	if ($constructtag) {
+	    $result  = &Apache::edit::rebuild_tag($token);
+	    $result .= &Apache::edit::handle_insert();
+	}
+    } elsif ($target eq 'web') {
+	&Apache::response::reset_params();
     }
-  }
-  return $result;
+    return $result;
 }
 
 sub end_numericalhint {
-  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  my $result;
-  if ($target eq 'web') {
-    $safeeval->share_from('capa',['&caparesponse_capa_check_answer']);
-    my $name= &Apache::lonxml::get_param('name',$parstack,$safeeval);
-    &Apache::response::setup_params('numericalhint');
-    my $partid=$Apache::inputtags::part;
-    my $id=$Apache::inputtags::response['-1'];
-    #id submissions occured under
-    my $submitid=$Apache::inputtags::response['-2'];
-    my $response = $Apache::lonhomework::history{
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+    my $result;
+    if ($target eq 'web') {
+	if (!$Apache::lonxml::default_homework_loaded) {
+	    &Apache::lonxml::default_homework_load($safeeval);
+	}
+	$safeeval->share_from('capa',['&caparesponse_capa_check_answer']);
+	my $name= &Apache::lonxml::get_param('name',$parstack,$safeeval);
+	&Apache::response::setup_params('numericalhint',$safeeval);
+	my $partid=$Apache::inputtags::part;
+	my $id=$Apache::inputtags::response['-1'];
+	#id submissions occured under
+	my $submitid=$Apache::inputtags::response['-2'];
+	my $response = $Apache::lonhomework::history{
 			    "resource.$partid.$submitid.submission"};
-    &Apache::lonxml::debug("hintgroup is using $response<br />\n");
-    #build safe space expression
-    my $expression="&caparesponse_check_list('".$response."','".
-      $$parstack[$#$parstack];
-    #need to get all possible parms
-    foreach my $key (keys(%Apache::inputtags::params)) {
-      $expression.= ';my $'. #'
-	$key.'="'.$Apache::inputtags::params{$key}.'"';
+	&Apache::lonxml::debug("hintgroup is using $response<br />\n");
+	#build safe space expression
+	my $expression="&caparesponse_check_list('".$response."','".
+	    $$parstack[$#$parstack];
+	#need to get all possible parms
+	foreach my $key (keys(%Apache::inputtags::params)) {
+	    $expression.= ';my $'. #'
+		$key.'="'.$Apache::inputtags::params{$key}.'"';
+	}
+	if ($$tagstack[-1] eq 'formulahint') {
+	    $expression.=';my $type="fml";';
+	} elsif ($$tagstack[-1] eq 'numericalhint') {
+	    $expression.=';my $type="float";';
+	}
+	$expression.="');";
+	my @answer=&Apache::lonxml::get_param_var('answer',$parstack,$safeeval);
+	&Apache::lonxml::debug('answer is'.join(':',@answer));
+	@{$safeeval->varglob('CAPARESPONSE_CHECK_LIST_answer')}=@answer;
+		
+	$result = &Apache::run::run($expression,$safeeval);
+	&Apache::lonxml::debug("$expression:result:$result:$Apache::lonxml::curdepth");
+	my ($awards) = split /:/ , $result;
+	my ($ad) = &Apache::inputtags::finalizeawards(split /,/ , $awards);
+	if ($ad eq 'EXACT_ANS' || $ad eq 'APPROX_ANS') { push (@Apache::hint::which,$name); }
+	$result='';
+    } elsif ($target eq 'meta') {
+	$result=&Apache::response::meta_package_write($token->[1]);
+    } elsif ($target eq 'edit') {
+	$result.='</td></tr>'.&Apache::edit::end_table;
     }
-    $expression.="');";
-    $result = &Apache::run::run($expression,$safeeval);
-    &Apache::lonxml::debug("$expression:result:$result:$Apache::lonxml::curdepth");
-    my ($awards) = split /:/ , $result;
-    my ($ad) = &Apache::inputtags::finalizeawards(split /,/ , $awards);
-    if ($ad eq 'EXACT_ANS' || $ad eq 'APPROX_ANS') { push (@Apache::hint::which,$name); }
-    $result='';
-  } elsif ($target eq 'meta') {
-    $result=&Apache::response::meta_package_write('numericalhint');
-  } elsif ($target eq 'edit') {
-    $result.='</td></tr>'.&Apache::edit::end_table;
-  }
-  &Apache::response::end_hintresponse();
-  return $result;
+    &Apache::response::end_hintresponse();
+    return $result;
 }
 
 sub start_stringhint {
-  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  my $result;
-  if ($target eq 'meta') {
-    $result=&Apache::response::meta_package_write('stringhint');
-  } else {
-    $result.=&start_numericalhint(@_);
-  }
-  return $result;
+    return &start_numericalhint(@_);
 }
 
 sub end_stringhint {
-  return end_numericalhint(@_);
+    return &end_numericalhint(@_);
 }
 
 sub start_formulahint {
-  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
-  my $result;
-  if ($target eq 'meta') {
-    $result=&Apache::response::meta_package_write('formulahint');
-  } else {
-    $result.=&start_numericalhint(@_);
-  }
-  return $result;
+    return &start_numericalhint(@_);
 }
 
 sub end_formulahint {
-  return end_numericalhint(@_);
+    return end_numericalhint(@_);
 }
 
 # a part shows if it is on, if no specific parts are on, then default shows
 sub start_hintpart {
-  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
 
-  my $show ='0';
-  my $result = '';
-  if ($target eq 'web') {
-    my $on= &Apache::lonxml::get_param('on',$parstack,$safeeval);
-    &Apache::lonxml::debug("hintpart sees $on and ,$#Apache::hint::which");
-    if ( $on eq 'default' && $#Apache::hint::which == '-1') {
-      $show=1;
-    } else {
-      my $which;
-      foreach $which (@Apache::hint::which) { if ($which eq $on) { $show = 1; last } }
-    }
-    if (!$show) {
-      &Apache::lonxml::get_all_text("/hintpart",$$parser[$#$parser]);
-    }
-  } elsif ($target eq 'grade') {
-    &Apache::lonxml::get_all_text("/hintpart",$$parser[$#$parser]);
-  } elsif ($target eq 'edit') {
-    $result.= &Apache::edit::tag_start($target,$token);
-    $result.= &Apache::edit::text_arg('On:','on',$token);
-    $result.= &Apache::edit::end_row();
-    $result.= &Apache::edit::start_spanning_row();
-  } elsif ($target eq 'modified') {
-    my $constructtag=&Apache::edit::get_new_args($token,$parstack,$safeeval,
-						 'on');
-    if ($constructtag) {
-      $result = &Apache::edit::rebuild_tag($token);
-      $result.=&Apache::edit::handle_insert();
+    my $show ='0';
+    my $result = '';
+    if ($target eq 'web') {
+	my $on= &Apache::lonxml::get_param('on',$parstack,$safeeval);
+	&Apache::lonxml::debug("hintpart sees $on and ,$#Apache::hint::which");
+	if ( $on eq 'default' && $#Apache::hint::which == '-1') {
+	    $show=1;
+	} else {
+	    my $which;
+	    foreach $which (@Apache::hint::which) { if ($which eq $on) { $show = 1; last } }
+	}
+	if (!$show) {
+	    &Apache::lonxml::get_all_text("/hintpart",$parser);
+	}
+    } elsif ($target eq 'grade') {
+	&Apache::lonxml::get_all_text("/hintpart",$parser);
+    } elsif ($target eq 'edit') {
+	$result.= &Apache::edit::tag_start($target,$token);
+	$result.= &Apache::edit::text_arg('On:','on',$token);
+	$result.= &Apache::edit::end_row();
+	$result.= &Apache::edit::start_spanning_row();
+    } elsif ($target eq 'modified') {
+	my $constructtag=&Apache::edit::get_new_args($token,$parstack,
+						     $safeeval,'on');
+	if ($constructtag) {
+	    $result = &Apache::edit::rebuild_tag($token);
+	    $result.=&Apache::edit::handle_insert();
+	}
     }
-  }
-  return $result;
+    return $result;
 }
 
 sub end_hintpart {
@@ -338,7 +350,7 @@ sub start_radiobuttonhint {
     return $result;
 }
 
-sp;&end_radiobuttonhint {
+sub end_radiobuttonhint {
     my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
     my $result;
     if ($target eq 'web') {
@@ -353,13 +365,25 @@ sp;&end_radiobuttonhint {
 	my $response = $Apache::lonhomework::history{
 			    "resource.$part_id.submission"};
 	($response)=&Apache::lonnet::str2hash($response);
-	if ($answer[0] eq 'foil' && $response eq $answer[1]) {
-	    push (@Apache::hint::which,$name);
+	&Apache::lonxml::debug("response is $response");
+	
+	if ($answer[0] eq 'foil') {
+	    shift(@answer);
+	    foreach my $answer (@answer) {
+		if ($response eq $answer) {
+		    push (@Apache::hint::which,$name);
+		    last;
+		}
+	    }
 	} elsif ($answer[0] eq 'concept') {
-	    if (ref($Apache::hint::radiobutton{"$part_id.concept.".$answer[1]})) {
-		my @names=@{ $Apache::hint::radiobutton{"$part_id.concept.".$answer[1]} };
-		if (grep(/^\Q$response\E$/,@names)) {
-		    push(@Apache::hint::which,$name);
+	    shift(@answer);
+	    foreach my $answer (@answer) {
+		if (ref($Apache::hint::radiobutton{"$part_id.concept.".$answer})) {
+		    my @names=@{ $Apache::hint::radiobutton{"$part_id.concept.".$answer} };
+		    if (grep(/^\Q$response\E$/,@names)) {
+			push(@Apache::hint::which,$name);
+			last;
+		    }
 		}
 	    }
 	}