--- loncom/homework/hint.pm	2001/03/31 00:35:17	1.9
+++ loncom/homework/hint.pm	2001/08/11 17:06:52	1.17
@@ -11,19 +11,19 @@ sub BEGIN {
 
 @Apache::hint::which=();
 sub start_hintgroup {
-  my ($target,$token,$parstack,$parser,$safeeval,$style)=@_;
+  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
   my $skiptoend='0';
   my $result;
   
-  if ($target eq 'web' || $target eq 'grade') {
+  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'; 
+    if ( $numtries < $hinttries ) {
+      $skiptoend='1';
     } else {
       if ($target eq 'web') {$result='<table bgcolor="#dddddd"><tr><td>';}
     }
@@ -36,17 +36,17 @@ sub start_hintgroup {
 }
 
 sub end_hintgroup {
-  my ($target,$token,$parstack,$parser,$safeeval,$style)=@_;
+  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
   my $result;
 
-  if ($target ne 'meta' || $target ne 'grade') {
+  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 ) {
+    if ( $numtries => $hinttries ) {
       if ($target eq 'web') {$result='</td></tr></table>';}
     }	
   }
@@ -55,17 +55,18 @@ sub end_hintgroup {
 }
 
 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);
+  return '';
 }
 
 sub end_numericalhint {
-  my ($target,$token,$parstack,$parser,$safeeval,$style)=@_;
+  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
   my $result;
-  if ($target ne 'meta' && $target ne 'grade') {
-    my $args ='';
+  if ($target eq 'web') {
     $safeeval->share_from('capa',['&caparesponse_capa_check_answer']);
-    if ( $#$parstack > -1 ) { $args=$$parstack[$#$parstack]; }
-    my $name = &Apache::run::run("{$args;".'return $name}',$safeeval);
+    my $name= &Apache::lonxml::get_param('name',$parstack,$safeeval);
     my $response = $Apache::lonhomework::history{"resource.$Apache::inputtags::part.$Apache::inputtags::response['-1'].submission"};
     &Apache::lonxml::debug("hintgroup is using $response<br />\n");
     my $expression="&caparesponse_check_list('".$response."','".
@@ -79,20 +80,21 @@ sub end_numericalhint {
     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');
   }
+  &Apache::response::end_hintresponse();
   return $result;
 }
 
 # a part shows if it is on, if no specific parts are on, then default shows
 sub start_hintpart {
-  my ($target,$token,$parstack,$parser,$safeeval,$style)=@_;
+  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
 
   my $show ='0';
 
   if ($target eq 'web') {
-    my $args ='';
-    if ( $#$parstack > -1 ) { $args=$$parstack[$#$parstack]; }
-    my $on = &Apache::run::run("{$args;".'return $on}',$safeeval);
+    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;
@@ -110,6 +112,7 @@ sub start_hintpart {
 }
 
 sub end_hintpart {
+  return '';
 }
 
 1;