From 5e5168f106883fef3d7919ba723e9037d7da3897 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 18 Sep 2020 13:53:29 +0000 Subject: [PATCH] Bug 12224: (follow-up) Move button to patron column in checkins table This patch moves the "Print checkin slip" out of its own column and into the colum containing patron information. I think this makes more sense because the slip is specific to a patron, not the individual item which has been checked in. Also changed: printcheckinslip function should be triggered via a class, not a non-unique ID. The function should handle the click using "preventDefault" rather than "return false." To test, apply the patch and check in some items checked out to various patrons. - In the table of checked-in items, confirm that the "Print checkin slip" appears correctly in the column of patron information. - Confirm that the button works correctly to print the correct slip for the associated patron. --- koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt index e313bf3f96..e15baee5c9 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt @@ -794,7 +794,6 @@ Collection Patron Note -   @@ -862,6 +861,7 @@ [% END %] + Print checkin slip [% ELSE %] Not checked out [% END %] @@ -877,9 +877,6 @@

[% riloo.itemnotes_nonpublic | html %]

[% END %] - - Print checkin slip - [% END # /FOREACH riloo %] @@ -1076,10 +1073,10 @@ }); $('[data-toggle="tooltip"]').tooltip(); - $("#printcheckinslip").click(function(){ + $(".printcheckinslip").on("click", function(e){ + e.preventDefault(); var borrowernumber = $(this).data('borrowernumber'); window.open("/cgi-bin/koha/members/printslip.pl?borrowernumber=" + borrowernumber + "&print=checkinslip", "printwindow"); - return false; }); }); -- 2.11.0