--- loncom/xml/scripttag.pm	2002/03/22 20:05:19	1.67
+++ loncom/xml/scripttag.pm	2002/04/25 17:25:36	1.71
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # <script> definiton
 #
-# $Id: scripttag.pm,v 1.67 2002/03/22 20:05:19 albertel Exp $
+# $Id: scripttag.pm,v 1.71 2002/04/25 17:25:36 matthew Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -62,13 +62,7 @@ sub start_script {
       if (($target eq 'answer') && ($Apache::lonhomework::viewgrades == 'F')) {
 	$Apache::lonxml::evaluate--;
 	$result.="<a href=\"javascript:newWindow=open(\'\',\'new_W\',\'width=500,height=200,scrollbars=1\');newWindow.document.open(\'text/html\',\'replace\');newWindow.document.writeln(\'<html><head><title>newwindow</title></head><body bgcolor=&quot;#FFFFFF&quot;><pre>";
-	my $listing= &Apache::run::dump($target,$safeeval);
-	$listing =~ s/\n/\\n/g;
-	$listing =~ s/\"/&quot;/g;
-	$listing =~ s/\'/&rsquo;/g;
-	$listing =~ s/</&lt;/g;
-	$listing =~ s/>/&gt;/g;
-	$listing =~ s/&/&amp;/g;
+	my $listing= &HTML::Entities::encode(&Apache::run::dump($target,$safeeval));
 	$result.=$listing;
 	$result.= "</pre></body></html>\');newWindow.document.close();void(0);\">Script Vars</a><br />";
       }
@@ -161,7 +155,7 @@ sub start_scriptlib {
   if ($target eq "edit" ) {
     $result=
       &Apache::edit::tag_start($target,$token,'New Script Functions').
-	&Apache::edit::editline($token->[1],$bodytext,'',40,1).
+	&Apache::edit::editline($token->[1],$bodytext,'',40).
 	  $error.'</td></tr>'.
 	    &Apache::edit::end_table();
   }
@@ -206,7 +200,7 @@ sub start_parserlib {
   if ($target eq "edit" ) {
     $result=
       &Apache::edit::tag_start($target,$token,'New Tag Definitions').
-	&Apache::edit::editline($token->[1],$bodytext,'',40,1).
+	&Apache::edit::editline($token->[1],$bodytext,'',40).
 	  $error.'</td></tr>'.
 	    &Apache::edit::end_table();
   }
@@ -262,6 +256,7 @@ sub start_import {
     # FIXME this probably needs to be smart about construction vs.
     # non construction space.
     my $location=&Apache::lonnet::filelocation($Apache::lonxml::pwd['-1'],$bodytext);
+    &Apache::lonnet::logthis("$location,$Apache::lonxml::pwd['-1'],$bodytext");
     my $file=&Apache::lonnet::getfile($location);
     if ($file == -1) {
       &Apache::lonxml::error("<b> Unable to find <i>$bodytext as $location</i> for import</b>");
@@ -277,7 +272,7 @@ sub start_import {
     &Apache::lonxml::newparser($parser,\$file,$dir);
   } elsif ($target eq "edit" ) {
     $result.=&Apache::edit::tag_start($target,$token);
-    $result.=&Apache::edit::editline($token->[1],$bodytext,'',40,1);
+    $result.=&Apache::edit::editline($token->[1],$bodytext,'',40);
     #FIXME this need to convert $bodytext to be a contruction space reference
     #my $location=&Apache::lonnet::filelocation($Apache::lonxml::pwd['-1'],$bodytext);
     #$result.="Click<a href=\"$location\">here</a> to edit<br />"
@@ -299,8 +294,11 @@ sub start_import {
 }
 
 sub end_import {
+  my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
   pop(@Apache::inputtags::import);
-  return '';
+  my $result;
+  if ($target eq 'edit' ) { $result=&Apache::edit::end_table(); }
+  return $result;
 }
 
 sub start_storetc {
