From fa09748ded7d7301ce77052931f8d0c035328b95 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Fri, 18 Sep 2020 13:53:29 +0000
Subject: [PATCH] Bug 12224: (follow-up) Move button to patron column in
checkins table
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
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.
Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Alexandra Speer <alexandra.speer@bsz-bw.de>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Christian Stelzenmüller <christian.stelzenmueller@bsz-bw.de>
---
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 180ac7d9e5..df59220144 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt
@@ -838,7 +838,6 @@
<th class="ci-collection">Collection</th>
<th class="ci-patron">Patron</th>
<th class="ci-note">Note</th>
- <th class="ci-printslip"> </th>
</tr>
</thead>
@@ -909,6 +908,7 @@
</span>
</span>
[% END %]
+ <a class="btn btn-default btn-xs printcheckinslip" href="#" data-borrowernumber="[% riloo.patron.borrowernumber | html %]"><i class="fa fa-print"></i> Print checkin slip</a>
[% ELSE %]
Not checked out
[% END %]
@@ -924,9 +924,6 @@
<p><span class="circ-hlt item-note-nonpublic">[% riloo.itemnotes_nonpublic | html %]</span></p>
[% END %]
</td>
- <td class="ci-printslip actions">
- <a id="printcheckinslip" href="#" class="btn btn-default" data-borrowernumber="[% riloo.patron.borrowernumber | html %]"><i class="fa fa-print"></i> Print checkin slip</a>
- </td>
</tr>
[% END # /FOREACH riloo %]
</table> <!-- /#checkedintable -->
@@ -1125,10 +1122,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