--- loncom/xml/lonxml.pm	2000/08/02 16:47:53	1.14
+++ loncom/xml/lonxml.pm	2000/08/03 18:24:24	1.15
@@ -60,41 +60,9 @@ sub xmlparse {
      push (@parstack,&parstring($token));
      
      if (exists $style_for_target{$token->[1]}) {
-       #basically recurse, but we never got more than one level down so just 
-       #create the new context here
-       my @innerstack = (); 
-       my @innerparstack = ();
-       # use style file definition
-       $newarg = $style_for_target{$token->[1]};       
-       my $pat = HTML::TokeParser->new(\$newarg);
-       my $tokenpat = '';
-       my $partstring = '';
-
-       while  ($tokenpat = $pat->get_token) {
-	 if ($tokenpat->[0] eq 'T') {
-	   $partstring = $tokenpat->[1];
-	 } elsif ($tokenpat->[0] eq 'S') {
-	   push (@innerstack,$tokenpat->[1]);
-	   push (@innerparstack,&parstring($tokenpat));
-	   $partstring = &callsub("start_$tokenpat->[1]", 
-				  $target, $tokenpat, \@innerparstack,
-				  $pat, $safeeval, \%style_for_target);
-	 } elsif ($tokenpat->[0] eq 'E') {
-	   #clear out any tags that didn't end
-	   while ($tokenpat->[1] ne $innerstack[$#innerstack] 
-		  && ($#innerstack > 0)) {pop @innerstack;pop @innerparstack;}
-	   $partstring = &callsub("end_$tokenpat->[1]",
-				  $target, $tokenpat, \@innerparstack,
-				  $pat, $safeeval, \%style_for_target);
-	 }
-	 #pass both the variable to the style tag, and the tag we 
-	 #are processing inside the <definedtag>
-	 if ($partstring ne "" ) {
-	   $finaloutput .= &Apache::run::evaluate($partstring,$safeeval,
-		$parstack[$#parstack].$innerparstack[$#innerparstack]);
-	 }
-	 if ($tokenpat->[0] eq 'E') { pop @innerstack;pop @innerparstack; }
-       }
+       $finaloutput .= &recurse($style_for_target{$token->[1]},
+				$target,$safeeval,\%style_for_target,
+				@parstack);
      } else {
        my $result = &callsub("start_$token->[1]", $target, $token,\@parstack,
 			     $pars, $safeeval, \%style_for_target);
@@ -109,38 +77,9 @@ sub xmlparse {
 	    && ($#stack > 0)) {pop @stack;pop @parstack;}
 
      if (exists $style_for_target{'/'."$token->[1]"}) {
-       my @innerstack = (); 
-       my @innerparstack = ();
-       $newarg = $style_for_target{'/'."$token->[1]"};
-       my $pat = HTML::TokeParser->new(\$newarg);
-       my $tokenpat;
-       my $partstring = '';
-       
-       while  ($tokenpat = $pat->get_token) {
-	 if ($tokenpat->[0] eq 'T') {
-	   $partstring = $tokenpat->[1];
-	 } elsif ($tokenpat->[0] eq 'S') {
-	   push (@innerstack,$tokenpat->[1]);
-	   push (@innerparstack,&parstring($tokenpat));
-	   $partstring = &callsub("start_$tokenpat->[1]", 
-				  $target, $tokenpat, \@innerparstack,
-				  $pat, $safeeval, \%style_for_target);
-	 } elsif ($tokenpat->[0] eq 'E') {
-	   #clear out any tags that didn't end
-	   while ($tokenpat->[1] ne $innerstack[$#innerstack] 
-		  && ($#innerstack > 0)) {pop @innerstack;pop @innerparstack;}
-	   $partstring = &callsub("end_$tokenpat->[1]",
-				  $target, $tokenpat, \@innerparstack,
-				  $pat, $safeeval, \%style_for_target);
-	 }
-	 #pass both the variable to the style tag, and the tag we 
-	 #are processing inside the <definedtag>
-	 if ( $partstring ne "" ) {
-	   $finaloutput .= &Apache::run::evaluate($partstring,$safeeval,
-		$parstack[$#parstack].$innerparstack[$#innerparstack]);
-	 }
-	 if ($tokenpat->[0] eq 'E') { pop @innerstack;pop @innerparstack; }
-       }
+       $finaloutput .= &recurse($style_for_target{'/'."$token->[1]"},
+			       $target,$safeeval,\%style_for_target,
+				@parstack);
      } else {
        my $result = &callsub("end_$token->[1]", $target, $token, \@parstack,
 			     $pars,$safeeval, \%style_for_target);
@@ -156,6 +95,43 @@ sub xmlparse {
  return $finaloutput;
 }
 
+sub recurse {
+  
+  my @innerstack = (); 
+  my @innerparstack = ();
+  my ($newarg,$target,$safeeval,$style_for_target,@parstack) = @_;
+  my $pat = HTML::TokeParser->new(\$newarg);
+  my $tokenpat;
+  my $partstring = '';
+  my $output='';
+  while  ($tokenpat = $pat->get_token) {
+    if ($tokenpat->[0] eq 'T') {
+      $partstring = $tokenpat->[1];
+    } elsif ($tokenpat->[0] eq 'S') {
+      push (@innerstack,$tokenpat->[1]);
+      push (@innerparstack,&parstring($tokenpat));
+      $partstring = &callsub("start_$tokenpat->[1]", 
+			     $target, $tokenpat, \@innerparstack,
+			     $pat, $safeeval, $style_for_target);
+    } elsif ($tokenpat->[0] eq 'E') {
+      #clear out any tags that didn't end
+      while ($tokenpat->[1] ne $innerstack[$#innerstack] 
+	     && ($#innerstack > 0)) {pop @innerstack;pop @innerparstack;}
+      $partstring = &callsub("end_$tokenpat->[1]",
+			     $target, $tokenpat, \@innerparstack,
+			     $pat, $safeeval, $style_for_target);
+    }
+    #pass both the variable to the style tag, and the tag we 
+    #are processing inside the <definedtag>
+    if ( $partstring ne "" ) {
+      $output .= &Apache::run::evaluate($partstring,$safeeval,
+                  $parstack[$#parstack].$innerparstack[$#innerparstack]);
+    }
+    if ($tokenpat->[0] eq 'E') { pop @innerstack;pop @innerparstack; }
+  }
+  return $output;
+}
+
 sub callsub {
   my ($sub,$target,$token,$parstack,$parser,$safeeval,$style)=@_;
   my $currentstring='';