--- loncom/xml/scripttag.pm	2010/05/23 12:29:25	1.153
+++ loncom/xml/scripttag.pm	2011/12/09 22:04:55	1.158
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # <script> definiton
 #
-# $Id: scripttag.pm,v 1.153 2010/05/23 12:29:25 www Exp $
+# $Id: scripttag.pm,v 1.158 2011/12/09 22:04:55 www Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -116,17 +116,16 @@ sub start_script {
 	    my $function_name = 
 		join('_','LONCAPA_scriptvars',$uname,$udom,
 		     $env{'form.counter'},$Apache::lonxml::curdepth);
-	    my $script_var ="<script type=\"text/javascript\">
-// <![CDATA[
-    function $function_name() {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;
-	    $script_var.=$listing;
-	    $script_var.= "<\\/pre>$end_page');newWindow.document.close();newWindow.focus()}
-// ]]>
-</script><a href=\"javascript:$function_name();void(0);\">".&mt('Script Vars')."</a><br />";
-	    &Apache::lonxml::add_script_result($script_var);
+            my $scriptblocklog=$start_page.
+                               &Apache::loncommon::start_scrollbox('500px','480px','480px').
+                               '<pre>'.&Apache::run::dump($target,$safeeval).'</pre>'.
+                               &Apache::loncommon::end_scrollbox().
+                               $end_page;
+            $scriptblocklog=~s/\'/\\'/gs;
+            &Apache::lonxml::add_script_result(
+	             &Apache::loncommon::modal_adhoc_script($function_name,500,500,$scriptblocklog).
+                     "<a href=\"javascript:$function_name();void(0);\">".&mt('Script Vars')."</a><br />"
+                                              );
 	}
     } elsif ($target eq "edit" ) {
       #&Apache::run::run($bodytext,$safeeval);
@@ -385,9 +384,17 @@ sub start_import {
       return "";
     }
     my $importmode=&Apache::lonxml::get_param('importmode',$parstack,$safeeval);
-    if ($importmode eq 'problem') {
-       $file=~s/^\s*<problem>/<library>/s;
-       $file=~s/<\/problem>\s*$/<\/library>/s;
+    if (($importmode eq 'problem') || ($importmode eq 'part')) {
+# We are using import to import published problems
+       if (($importmode eq 'problem') || ($file=~/<part[^<]*>/s)) {
+# We explicitly don't want this to be a separate part or the problem already has parts
+          $file=~s/^\s*<problem>/<library>/s;
+	  $file=~s/<\/problem>\s*$/<\/library>/s;
+       } else {
+# We want this to be a separate part, but it currently is not
+          $file=~s/^\s*<problem>/<library><part>/s;
+	  $file=~s/<\/problem>\s*$/<\/part><\/library>/s;
+       }
     }
     my $dir=$location;
     $dir=~s:/[^/]*$::;
@@ -406,14 +413,17 @@ sub start_import {
     $location=~s/\s*$//s;
     $result.=&Apache::edit::editline($location,$bodytext,'',40);
     $result.=&Apache::edit::browse(undef,'textnode');
-        $result.= '<label>'.&mt('Import as:').
+    $result.= '&nbsp;<label>'.&mt('Import as:').
               '<select name="importmode_'.$Apache::lonxml::curdepth.'">';
-    foreach my $option ('','problem') {
+    my %options=&Apache::lonlocal::texthash(''        => 'as standard library',
+                                            'problem' => 'as problem',
+                                            'part'    => 'as problem part(s)');
+    foreach my $option (sort(keys(%options))) {
        $result.='<option value="'.$option.'"';
        if ($option eq &Apache::lonxml::get_param('importmode',$parstack,$safeeval)) {
           $result.=' selected="selected"';
        }
-       $result.='>'.($option?&mt('as problem'):&mt('as standard library')).'</option>';
+       $result.='>'.$options{$option}.'</option>';
     }
     $result.='</select></label>';
     #FIXME this need to convert $bodytext to be a contruction space reference
@@ -429,7 +439,7 @@ sub start_import {
     if ($id) {
       $result.='" id="'.$id;
     }
-    $result.='">';
+    $result.='" importmode="'.$token->[2]{'importmode'}.'">';
     $result.=$bodytext;
     $result.='</import>';
   }
@@ -616,7 +626,7 @@ sub start_algebra {
 	$inside = &Apache::run::evaluate($inside,$safeeval,$$parstack[-1]);
 	if ($target eq 'web' || $target eq 'tex' || $target eq 'analyze') {
 	    my $style=&Apache::lonxml::get_param('style',$parstack,$safeeval);
-	    $result=&Apache::lontexconvert::algebra($inside,undef,$target,$style,$parstack,$safeeval);
+	    $result=&Apache::lontexconvert::algebra($inside,$target,$style,$parstack,$safeeval);
 	}
 	$Apache::lonxml::post_evaluate=0;
     }