[LON-CAPA-cvs] cvs: loncom /interface lonpickcourse.pm

[raeburn]

raeburn		Mon Mar 25 04:05:30 2013 EDT

  Modified files:              
    /loncom/interface	lonpickcourse.pm 
  Log:
  - No need to pass output from js_changer() to lonhtmlcommon::scripttag().
  - Remove unnecessary concatenator.
  
  
Index: loncom/interface/lonpickcourse.pm
diff -u loncom/interface/lonpickcourse.pm:1.106 loncom/interface/lonpickcourse.pm:1.107
--- loncom/interface/lonpickcourse.pm:1.106	Fri Mar  1 04:55:35 2013
+++ loncom/interface/lonpickcourse.pm	Mon Mar 25 04:05:30 2013
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Pick a course
 #
-# $Id: lonpickcourse.pm,v 1.106 2013/03/01 04:55:35 raeburn Exp $
+# $Id: lonpickcourse.pm,v 1.107 2013/03/25 04:05:30 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -109,8 +109,7 @@
         $loaditem{'onload'} .= 'setDefaultCredits();setRoles();setSections();';
     }
     my $js = &js_changer();
-    $r->print(&Apache::loncommon::start_page($title,
-                                             &Apache::lonhtmlcommon::scripttag($js),
+    $r->print(&Apache::loncommon::start_page($title,$js,
 					     {'add_entries' => \%loaditem,
 					      'no_nav_bar'  => 1, }));
 
@@ -217,7 +216,7 @@
     }
     document.filterpicker.submit();
 }
-// ]]
+// ]]>
 </script>
 
 ENDJS
@@ -1029,7 +1028,7 @@
                                  list  => 'courselist',
                               },
                     );
-    my $output .= qq|
+    my $output = qq|
 function gochoose(cname,cdom,cdesc) {
     var openerForm = "$env{'form.form'}";
     courseCount = 0;