--- loncom/xml/lonxml.pm	2000/06/29 13:58:23	1.6
+++ loncom/xml/lonxml.pm	2000/06/29 18:52:54	1.7
@@ -8,10 +8,22 @@ package Apache::lonxml;
 use strict;
 use HTML::TokeParser;
 use Safe;
+
+sub register {
+  my $space;
+  my @taglist;
+  my $temptag;
+  ($space,@taglist) = @_;
+  foreach $temptag (@taglist) {
+    $Apache::lonxml::alltags{$temptag}=$space;
+  }
+}
+
 use Apache::style;
 use Apache::lontexconvert;
-use Apache::londefdef;
 use Apache::run;
+use Apache::londefdef;
+use Apache::scripttag;
 #==================================================   Main subroutine: xmlparse  
 
 sub xmlparse {
@@ -22,7 +34,6 @@ sub xmlparse {
  my $finaloutput = ''; 
  my $newarg = '';
  my $tempostring = '';
- my $tempocont = '';
  my $safeeval = new Safe;
  $safeeval->permit("entereval");
 #-------------------- Redefinition of the target in the case of compound target
@@ -40,97 +51,57 @@ sub xmlparse {
  
  while ($token = $pars->get_token) {
    if ($token->[0] eq 'T') {
-     $finaloutput .= $token->[1];
-     $tempocont .= $token->[1];
+     $finaloutput .= &Apache::run::evaluate($token->[1],$safeeval,'');
    } elsif ($token->[0] eq 'S') {
-#------------------------------------------------------------- add tag to stack 	    
+     # add tag to stack 	    
      push (@stack,$token->[1]);
-#----------------------------------------- add parameters list to another stack
-     map {$tempostring .= "$_=$token->[2]->{$_},"} @{$token->[3]};
+     # add parameters list to another stack
+     map {$tempostring .= "my \$$_=\"$token->[2]->{$_}\";"} @{$token->[3]};
      push (@parstack,$tempostring);
      $tempostring = '';
      
      if (exists $style_for_target{$token->[1]}) {
-#       print "Style for $token->[1] is " .$style_for_target{$token->[1]}."\n";
-#---------------------------------------------------- use style file definition
+       #print "Style for $token->[1] is " .$style_for_target{$token->[1]}."\n";
+       # use style file definition
 
        $newarg = $style_for_target{$token->[1]};
        
-       if (index($newarg,'script') != -1 ) {
-	 my $pat = HTML::TokeParser->new(\$newarg);
-	 my $tokenpat = '';
-	 my $partstring = '';
-	 my $oustring = '';
-	 my $outputstring;
-	 
-	 while  ($tokenpat = $pat->get_token) {
-	   if ($tokenpat->[0] eq 'T') {
-#	     print "evaluating $tokenpat->[4]\n";
-	     $oustring .= &Apache::run::evaluate($tokenpat->[1],$safeeval);
-	   } elsif ($tokenpat->[0] eq 'S') {
-	     if ($tokenpat->[1] eq 'script') {
-	       while  ($tokenpat = $pat->get_token and $tokenpat->[1] ne 'script') {
-		 if ($tokenpat->[0] eq 'S')  {
-		   $partstring .=  $tokenpat->[4];
-		 } elsif ($tokenpat->[0] eq 'T') {
-		   $partstring .=  $tokenpat->[1];
-		 } elsif ($tokenpat->[0] eq 'E') {
-		   $partstring .=  $tokenpat->[2];
-		 }
-	       }
-	       
-	       map {$partstring =~ s/\$$_/$token->[2]->{$_}/g; } @{$token->[3]};
-#	       print "want to use run\n";	       
-	       &Apache::run::run($partstring,$safeeval);
-	       
-	       $partstring = '';
-	     } else {
-#	       print "evaluating $tokenpat->[4]\n";
-	       $oustring .= &Apache::run::evaluate($tokenpat->[4],$safeeval);
-	     }
-	   } elsif ($tokenpat->[0] eq 'E' and $tokenpat->[1] ne 'evaluate') {
-#	     print "hereish\n";
-	     $oustring .= $tokenpat->[1];    
-	   }
+       my $pat = HTML::TokeParser->new(\$newarg);
+       my $tokenpat = '';
+       my $partstring = '';
+       my $oustring = '';
+       my $outputstring;
+       
+       while  ($tokenpat = $pat->get_token) {
+	 if ($tokenpat->[0] eq 'T') {
+	   $partstring = $tokenpat->[1];
+	 } elsif ($tokenpat->[0] eq 'S') {
+	   my $sub="start_$tokenpat->[1]";
+	   $partstring = &callsub($sub, $target, $tokenpat, \@parstack)
+	 } elsif ($tokenpat->[0] eq 'E') {
+	   my $sub="end_$tokenpat->[1]";
+	   $partstring = &callsub($sub, $target, $tokenpat, \@parstack)
 	 }
-	 $newarg =  $oustring;
-       } else {
-	 map {$newarg =~ s/\$$_/$token->[2]->{$_}/g; } @{$token->[3]};
+        # generate the my mechanism
+	# map {$partstring =~ s/\$$_/$token->[2]->{$_}/g; } @{$token->[3]};
+	 print "Temp: $parstack[$#parstack]\n";
+	 $oustring .= &Apache::run::evaluate($partstring,$safeeval,$parstack[$#parstack]);
        }
-       $finaloutput .= $newarg;
+       $finaloutput .= $oustring;
      } else {
-       # use default definition of tag
        my $sub="start_$token->[1]";
-       {
-	 no strict 'refs';
-	 if (defined (&$sub)) {
-	   $currentstring = &$sub($target,$token,\@parstack);
-	   $finaloutput .= $currentstring;
-	   $currentstring = '';
-	 } else {
-	   $finaloutput .= $token->[4];
-	 }
-	 use strict 'refs';    
-       }
+       #print "use default definition of tag $sub\n";
+       my $result = &callsub($sub, $target, $token, \@parstack);
+       $finaloutput .= &Apache::run::evaluate($result,$safeeval,$parstack[$#parstack]);
      }              
    } elsif ($token->[0] eq 'E')  {
      # Put here check for correct final tag (to avoid existence of 
      # starting tag only)
-        
+     
      pop @stack; 
      unless (exists $style_for_target{$token->[1]}) {
        my $sub="end_$token->[1]";
-       {
-	 no strict 'refs';
-	 if (defined(&$sub)) {
-	   $currentstring = &$sub($target,$token,\@parstack);
-	   $finaloutput .= $currentstring;
-	   $currentstring = '';
-	 } else {
-	   $finaloutput .= $token->[4];
-	 }
-	 use strict 'refs';
-       }
+       $finaloutput .= callsub($sub, $target, $token, \@parstack);
      }
      #---- end tag from the style file
      if (exists $style_for_target{'/'."$token->[1]"}) {
@@ -149,6 +120,7 @@ sub xmlparse {
              if ($tokenpat->[1] eq 'script') {
                while  ($tokenpat = $pat->get_token and $tokenpat->[1] ne 'script') {
 		 if ($tokenpat->[0] eq 'S')  {
+		   
 		   $partstring .=  $tokenpat->[4];
 		 } elsif ($tokenpat->[0] eq 'T') {
 		   $partstring .=  $tokenpat->[1];
@@ -191,5 +163,27 @@ sub xmlparse {
  return $finaloutput;
 }
 
+sub callsub {
+  my ($sub,$target,$token,@parstack)=@_;
+  my $currentstring='';
+  {
+    no strict 'refs';
+    if (my $space=$Apache::lonxml::alltags{$token->[1]}) {
+      #print "Calling sub $sub in $space \n";
+      $sub="$space\:\:$sub";
+      $currentstring = &$sub($target,$token,\@parstack);
+    } else {
+      #print "NOT Calling sub $sub\n";
+      if (defined($token->[4])) {
+	$currentstring = $token->[4];
+      } else {
+	$currentstring = $token->[2];
+      }
+    }
+    use strict 'refs';
+  }
+  return $currentstring;
+}
+
 1;
 __END__