--- loncom/xml/scripttag.pm	2006/03/29 22:41:39	1.137
+++ loncom/xml/scripttag.pm	2007/08/30 22:56:43	1.143
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # <script> definiton
 #
-# $Id: scripttag.pm,v 1.137 2006/03/29 22:41:39 albertel Exp $
+# $Id: scripttag.pm,v 1.143 2007/08/30 22:56:43 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -80,13 +80,21 @@ sub end_LONCAPA_INTERNAL_TURN_STYLE_OFF
     return ('','no');
 }
 
+=pod 
+
+=item start_LONCAPA_INTERNAL_LONHTTPD_PORT
+
+emits a pice of javascript that says a global js variable to the
+current lonhttp port, currently used by the remote control to find out
+where to load images from
+
+=cut
+
 sub start_LONCAPA_INTERNAL_LONHTTPD_PORT {
     my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
     if ($target eq 'web' || $target eq 'webgrade') {
-	my $lonhttpdPort=$Apache::lonnet::perlvar{'lonhttpdPort'};
-	if (!defined($lonhttpdPort)) { $lonhttpdPort='8080'; }
 	return '<script type="text/javascript">var lonhttpdport=\''.
-	    $lonhttpdPort.'\';</script>';
+	    &Apache::loncommon::lonhttpd_port().'\';</script>';
     }
     return ('','no');
 }
@@ -116,7 +124,7 @@ sub start_script {
 	    ($env{'form.answer_output_mode'} ne 'tex') &&
 	    ($Apache::lonhomework::viewgrades == 'F')) {
 	    $Apache::lonxml::evaluate--;
-	    my (undef,undef,$udom,$uname)=&Apache::lonxml::whichuser();
+	    my (undef,undef,$udom,$uname)=&Apache::lonnet::whichuser();
 	    my $windowopen=&Apache::lonhtmlcommon::javascript_docopen();
 	    my $start_page =
 		&Apache::loncommon::start_page('Script Vars', undef,
@@ -126,16 +134,19 @@ sub start_script {
 	    my $end_page =
 		&Apache::loncommon::end_page({'js_ready' => 1,});
 
-	    $result.="<script type=\"text/javascript\">
+	    $uname =~s/\W//g;
+	    $udom  =~s/\W//g;
+	    my $script_var ="<script type=\"text/javascript\">
 // <![CDATA[
     function LONCAPA_scriptvars_".$uname."_".$udom."_"."$Apache::lonxml::curdepth() {newWindow=open('','new_W','width=500,height=500,scrollbars=1,resizable=yes');newWindow.$windowopen;newWindow.document.writeln('$start_page<pre>";
 	    my $listing=&Apache::run::dump($target,$safeeval);
 	    $listing=~s/\\/\\\\/g;
 	    $listing=~s/\'/\\\'/g;
-	    $result.=$listing;
-	    $result.= "</pre>$end_page');newWindow.document.close();newWindow.focus()}
+	    $script_var.=$listing;
+	    $script_var.= "<\\/pre>$end_page');newWindow.document.close();newWindow.focus()}
 // ]]>
 </script><a href=\"javascript:LONCAPA_scriptvars_".$uname."_".$udom."_$Apache::lonxml::curdepth();void(0);\">Script Vars</a><br />";
+	    &Apache::lonxml::add_script_result($script_var);
 	}
     } elsif ($target eq "edit" ) {
       #&Apache::run::run($bodytext,$safeeval);
@@ -388,7 +399,7 @@ sub start_import {
     my $dir=$location;
     $dir=~s:/[^/]*$::;
     #  &Apache::lonxml::debug("directory $dir $location file $file \n<b>END</b>\n");
-    my $id= &Apache::lonxml::get_param('id',$parstack,$safeeval);
+    my $id= &Apache::lonxml::get_id($parstack,$safeeval);
     if (!$id) { $id=$Apache::lonxml::curdepth; }
     push(@Apache::inputtags::import,$id);
     push(@Apache::inputtags::importlist,$id);
@@ -405,7 +416,7 @@ sub start_import {
   } elsif ($target eq 'modified') {
       $result=$token->[4].&Apache::edit::modifiedfield("/import",$parser);
   } elsif ($target eq 'meta') {
-    my $id= &Apache::lonxml::get_param('id',$parstack,$safeeval);
+    my $id= &Apache::lonxml::get_id($parstack,$safeeval);
     $result.='<import part="'.$Apache::inputtags::part;
     if ($id) {
       $result.='" id="'.$id;