Bugzilla – Attachment 85064 Details for
Bug 17229
ILS-DI HoldTitle and HoldItem should check if patron is expired
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 17229: Check if patron is expired in CanItemBeReserved
Bug-17229-Check-if-patron-is-expired-in-CanItemBeR.patch (text/plain), 1.43 KB, created by
Arthur Suzuki
on 2019-02-13 18:22:51 UTC
(
hide
)
Description:
Bug 17229: Check if patron is expired in CanItemBeReserved
Filename:
MIME Type:
Creator:
Arthur Suzuki
Created:
2019-02-13 18:22:51 UTC
Size:
1.43 KB
patch
obsolete
>From 858ab5ed5649c894247aa6a96c5aa9ba0593eb90 Mon Sep 17 00:00:00 2001 >From: Julian Maurice <julian.maurice@biblibre.com> >Date: Wed, 31 Aug 2016 15:27:44 +0200 >Subject: [PATCH] Bug 17229: Check if patron is expired in CanItemBeReserved > >This way, calls to ILS-DI HoldTitle and HoldItem do this check too > >Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com> >--- > C4/Reserves.pm | 5 +++++ > 1 file changed, 5 insertions(+) > >diff --git a/C4/Reserves.pm b/C4/Reserves.pm >index 974168f8a3..c413531eb1 100644 >--- a/C4/Reserves.pm >+++ b/C4/Reserves.pm >@@ -306,6 +306,7 @@ sub CanBookBeReserved{ > { status => libraryNotFound }, if given branchcode is not an existing library > { status => libraryNotPickupLocation }, if given branchcode is not configured to be a pickup location > { status => cannotBeTransferred }, if branch transfer limit applies on given item and branchcode >+ { status => patronExpired } if patron is expired and expired patrons are not allowed to place holds > > =cut > >@@ -325,6 +326,10 @@ sub CanItemBeReserved { > my $patron = Koha::Patrons->find( $borrowernumber ); > my $borrower = $patron->unblessed; > >+ if ($patron->category->effective_BlockExpiredPatronOpacActions && $patron->is_expired) { >+ return 'expired'; >+ } >+ > # If an item is damaged and we don't allow holds on damaged items, we can stop right here > return { status =>'damaged' } > if ( $item->{damaged} >-- >2.13.7
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 17229
:
55059
|
55060
|
62036
|
62037
|
67324
|
67325
|
70682
|
70683
|
72970
|
72971
|
85064
|
85065
|
85066
|
85070
|
85071
|
85072
|
85118
|
85883
|
89760
|
99701
|
99702
|
99703
|
99817
|
99818
|
99819
|
113741
|
113742
|
113743
|
115081
|
115082
|
115083
|
115411
|
115435