Attachment 99926 Details for Bug 24777 – Bug 24777: Use patron.is_debarred instead of patron.debarred in return.tt

[patch] Bug 24777: Use patron.is_debarred instead of patron.debarred in return.tt

Bug-24777-Use-patronisdebarred-instead-of-patronde.patch (text/plain), 2.80 KB, created by Fridolin Somers on 2020-03-02 15:09:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Fridolin Somers

Created: 2020-03-02 15:09:59 UTC

Size: 2.80 KB

patch

obsolete

>From 9038e8cc11589d0278bd2bac583a88bd4e1a56f3 Mon Sep 17 00:00:00 2001
>From: Fridolin Somers <fridolin.somers@biblibre.com>
>Date: Mon, 2 Mar 2020 18:03:54 +0300
>Subject: [PATCH] Bug 24777: Use patron.is_debarred instead of patron.debarred
> in return.tt
>
>In return page return.tt, when there is an hold message, patron is said as restricted even if restriction end date is in the past.
>
>Test plan :
>1) For a patron Jon Doe create a manual restriction with end date in the past
>2) Create an hold for this patron on an item
>3) Check in this item => You don't see message "Patron is RESTRICTED"
>4) For a patron Kevin Doe create a manual restriction with end date in the future
>5) Create an hold for this patron on an item
>6) Check in this item => You see message "Patron is RESTRICTED"
>7) Repeate 1-6 with system preference HoldsAutoFill enabled
>8) Repeate 1-6 with a waiting hold
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt | 6 +++---
> 1 file changed, 3 insertions(+), 3 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 29818882c9..13dac5f2b2 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt
>@@ -89,7 +89,7 @@
>                                             [% INCLUDE display_bormessagepref %]
>                                         [% END %]
> 
>-                                        [% IF ( patron.debarred ) %]
>+                                        [% IF ( patron.is_debarred ) %]
>                                             <li class="error">Patron is RESTRICTED</li>
>                                         [% END %]
> 
>@@ -431,7 +431,7 @@
>                                                             <li><a id="boremail" href="mailto:[% patron.email | html %]">[% patron.email | html %]</a></li>
>                                                         [% END %]
> 
>-                                                        [% IF ( patron.debarred ) %]
>+                                                        [% IF ( patron.is_debarred ) %]
>                                                             <li class="error">Patron is RESTRICTED</li>
>                                                         [% END %]
> 
>@@ -601,7 +601,7 @@
>                                                             [% INCLUDE display_bormessagepref %]
>                                                         [% END %]
> 
>-                                                        [% IF ( patron.debarred ) %]
>+                                                        [% IF ( patron.is_debarred ) %]
>                                                             <li class="error">Patron is RESTRICTED</li>
>                                                         [% END %]
> 
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 24777: 99926 | 100276 | 100457