From bb621b4b743b9c6615a6d77756bfe73e1e6fc429 Mon Sep 17 00:00:00 2001 From: Jason Boyer Date: Mon, 9 Dec 2019 09:21:24 -0500 Subject: [PATCH] LP1853363 Followup: Permit Lost Cards There is a short circuit check for the active field on the user's card, forcing the Patron Status Response field to return charge, renewal, recall, and hold privileges denied and returns a minimal set of patron information. This followup commit will allow the patron_status_always_permit_loans to also override this. I should note that I don't personally think that this is a good idea, (lost card is lost) but in the interst of discussion and allowing libraries to choose what they feel works best for their users, here it is. Signed-off-by: Jason Boyer --- Open-ILS/src/perlmods/lib/OpenILS/SIP/Patron.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/SIP/Patron.pm b/Open-ILS/src/perlmods/lib/OpenILS/SIP/Patron.pm index 16cff51aee..db61a38e49 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/SIP/Patron.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/SIP/Patron.pm @@ -87,7 +87,7 @@ sub new { my $card = $e->search_actor_card([{barcode => $patron_id}, $usr_flesh])->[0]; - if(!$card or !$U->is_true($card->active)) { + if(!$card or (!$U->is_true($card->active) && !$self->patron_status_always_permit_loans_set())) { syslog("LOG_WARNING", "No such patron barcode: $patron_id"); return undef; } -- 2.11.0