Fix for Bug 6531 - clicking print button in opac causes 404 error
authorOwen Leonard <oleonard@myacpl.org>
Thu, 7 Jul 2011 14:01:34 +0000 (10:01 -0400)
committerChris Cormack <chrisc@catalyst.net.nz>
Sun, 10 Jul 2011 09:18:29 +0000 (21:18 +1200)
opac-detail and opac-ISBDdetail both use window.print(), so I've
added that to opac-MARCdetail as well. I've also eliminated the
"javascript:" pseudo-protocol because it's pointless. I'd
prefer to have a real link for non-js users but that will have
to wait for another bug.

After applying this patch users with JS enabled should get a
print prompt after clicking the "Print" link on opac-MARCdetail.pl

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
koha-tmpl/opac-tmpl/prog/en/modules/opac-ISBDdetail.tt
koha-tmpl/opac-tmpl/prog/en/modules/opac-MARCdetail.tt
koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt

index 35fffc8..f9984a7 100644 (file)
@@ -62,7 +62,7 @@
                [% END %]
        [% END %]
 [% END %]
-        <li><a class="print" href="javascript:window.print();">Print</a></li>
+        <li><a class="print" href="#" onclick="window.print();">Print</a></li>
 [% IF ( virtualshelves ) %]
    [% IF ( opacuserlogin ) %] [% IF ( loggedinusername ) %]
         <li><a class="addtoshelf" href="/cgi-bin/koha/opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]" onclick="Dopop('opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]'); return false;">
index 68b0148..284154e 100644 (file)
@@ -217,7 +217,7 @@ $(document).ready(function(){
             [% END %]
     [% END %]
 [% END %]
-        <li><a class="print" href="/cgi-bin/koha/opac-detailprint.pl?biblionumber=[% biblionumber %]" onclick="Dopop('opac-detailprint.pl?biblionumber=[% biblionumber %]'); return false;">
+        <li><a class="print" href="#" onclick="window.print();">
             Print
         </a></li>
 [% IF ( virtualshelves ) %]
index 60972cd..3430cbb 100644 (file)
@@ -814,7 +814,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
                [% END %]
        [% END %]
 [% END %]
-        <li><a class="print" href="javascript:window.print();">Print</a></li>
+        <li><a class="print" href="#" onclick="window.print();">Print</a></li>
 [% IF ( virtualshelves ) %]
     [% IF ( opacuserlogin ) %][% IF ( loggedinusername ) %]
         <li><a class="addtoshelf" href="/cgi-bin/koha/opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]" onclick="Dopop('opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]'); return false;">