--- loncom/xml/scripttag.pm	2004/04/15 20:24:34	1.112
+++ loncom/xml/scripttag.pm	2004/08/10 19:22:32	1.114
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # <script> definiton
 #
-# $Id: scripttag.pm,v 1.112 2004/04/15 20:24:34 albertel Exp $
+# $Id: scripttag.pm,v 1.114 2004/08/10 19:22:32 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -179,7 +179,10 @@ sub start_display {
       $result=&Apache::run::run($bodytext,$safeeval);
       if ($target eq 'grade' || $target eq 'answer' ||
 	  $target eq 'analyze') {
-	  $result=''; # grade should produce no output
+	  # grade/answer/analyxe should produce no output but if we
+	  # are redirecting, the redirecter should know what to do
+	  # with the output
+	  if (!$Apache::lonxml::redirection) { $result=''; }
       }
       $Apache::lonxml::post_evaluate=0;
   } elsif ($target eq "edit" ) {
@@ -483,7 +486,14 @@ sub end_comment {
 sub xmlparse {
   my ($string) = @_;
   &Apache::lonxml::debug("xmlparse recursion starting with $string");
+  # Apache::run::evaluate does an 'eval' on the name of the subroutine
+  # if it detects something that looks like a subroutine, this ends up calling
+  # things without any arguments and since perl is nice enough to pass
+  # along the default arguments when you don't explicitly say no arguments
+  # if you call &xmlparse, it gets &xmlparse passed as it argument.
+  # Same thing soccurs with &chemparse.
   if ($string eq '&xmlparse') { return '&xmlparse'; }
+  if ($string eq '&chemparse') { return '&chemparse'; }
   my ($target,$token,$tagstack,$parstack,$oldparser,$safeeval,$style)=
     @Apache::scripttag::parser_env;
   my @parser;