From 2499226ffe1e6e4e202ac84fda544893674387ba Mon Sep 17 00:00:00 2001 From: Colin Campbell Date: Thu, 17 Apr 2014 15:55:03 +0100 Subject: [PATCH] Bug 11633 : Block Issue if fines require staff override If a patrons fines exceed noissuescharge and we allow staff to allow issue at their discretion via an override the SIP process allowed charges to go ahead. This patch closes the loophole which allowed self issue to subvert the usual library loan policy Signed-off-by: Chris Cormack --- C4/SIP/ILS/Transaction/Checkout.pm | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/C4/SIP/ILS/Transaction/Checkout.pm b/C4/SIP/ILS/Transaction/Checkout.pm index c61eca6..edc1d04 100644 --- a/C4/SIP/ILS/Transaction/Checkout.pm +++ b/C4/SIP/ILS/Transaction/Checkout.pm @@ -85,7 +85,9 @@ sub do_checkout { } elsif ($confirmation eq 'ISSUED_TO_ANOTHER') { $self->screen_msg("Item already checked out to another patron. Please return item for check-in."); $noerror = 0; - } elsif ($confirmation eq 'DEBT') { # don't do anything, it's the minor debt, and alarms fire elsewhere + } elsif ($confirmation eq 'DEBT') { + $self->screen_msg('Outstanding Fines block issue'); + $noerror = 0; } elsif ($confirmation eq 'HIGHHOLDS') { $overridden_duedate = $needsconfirmation->{$confirmation}->{returndate}; $self->screen_msg('Loan period reduced for high-demand item'); -- 1.9.1