Bug 12862: OPAC bootstrap - clean up language="javascript"
authorMark Tompsett <mtompset@hotmail.com>
Tue, 2 Sep 2014 20:43:32 +0000 (16:43 -0400)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Sat, 18 Oct 2014 14:13:07 +0000 (11:13 -0300)
There was a <script language="javascript"> tag.
It was cleaned up to be <script type="text/javascript">.
Also, the div was being chopped because the script was
in the middle of the div. Moving it to head solved the issues.

This is difficult to test, since the printslip is triggered
when you click Finish.

TEST PLAN
---------
 1) Home -> Koha administration -> Global system parameters -> OPAC
 2) Set the OPAC system preference 'opacthemes' to 'bootstrap'.
 3) Start the self-checkout client (/cgi-bin/koha/sco/sco-main.pl)
 4) Log in.
 5) Check something out.
 6) Change the address url text box to
    .../cgi-bin/koha/sco/printslip.pl?borrowernumber=##&amp;print=qslip
    Where ## is your borrower number which just checked something out.
 7) Click Cancel on the print dialogue
 8) View Page Source
    -- The HTML validation plugin will point out multiple errors.
 9) Apply patch
10) Refresh page
11) View Page Source
    -- Much happier validation results.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/printslip.tt

index 8f02235..3cb63db 100644 (file)
@@ -6,6 +6,12 @@
 [% IF stylesheet %]
 <link rel="stylesheet" type="text/css" href="[% stylesheet %]" />
 [% END %]
+<script type="text/javascript">
+    function printThenClose() {
+        window.print();
+        setTimeout('window.close()', 1);
+    }
+</script>
 </head>
 
 <body id="circ_printslip" class="circ" onload="printThenClose();">
 
     </div>
 
-<script language="javascript">
-    function printThenClose() {
-        window.print();
-        setTimeout('window.close()', 1);
-    }
-</script>
 </body>
 </html>