Bug 24776: Remove Borrowers.IsDebarred call
authorFridolin Somers <fridolin.somers@biblibre.com>
Mon, 2 Mar 2020 14:27:10 +0000 (17:27 +0300)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 16 Mar 2020 14:52:20 +0000 (14:52 +0000)
Borrowers Template Toolkit plugin contains method IsDebarred that can be directly called from Koha::Patron object.

Also in template circ/returns.tt, when patron is restricted and checkin was backdated, var "patron" is undefined, should be "issue.patron".

Test plan :
1) Enable system preference SpecifyReturnDate
2) For a patron without restriction create a checkout
3) Create a manual restriction to debarre patron
4) Go to return page and click on "Checkin settings"
5) Specify a date in the past, barcode and click "Check in"
6) You see message "Jon Doe is restricted. Please verify this patron should still be restricted."

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt

index 13dac5f..2a9717e 100644 (file)
@@ -3,7 +3,6 @@
 [% USE KohaDates %]
 [% USE Branches %]
 [% USE Koha %]
-[% USE Borrowers %]
 [% USE ItemTypes %]
 [% USE AuthorisedValues %]
 [% USE ColumnsSettings %]
                             [% END %]
 
                             <!-- Patron is restricted and checkin was backdated -->
-                            [% IF return_date_was_overriden && Borrowers.IsDebarred( borrower ) %]
+                            [% IF return_date_was_overriden && issue.patron.is_debarred %]
                                 <div id="restricted_backdated" class="dialog message">
                                     <h3>
-                                        <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% patron.borrowernumber | uri %]">
-                                            [% patron.firstname | html %] [% patron.surname | html %]
+                                        <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% issue.patron.borrowernumber | uri %]">
+                                            [% issue.patron.firstname | html %] [% issue.patron.surname | html %]
                                         </a>
                                         is restricted. Please verify this patron should still be restricted.
                                     </h3>