From 09ff5cd260b1e5f4240ef806c9219f46925f50a8 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Wed, 26 May 2021 11:16:13 -0400 Subject: [PATCH] Bug 28464: Remove useless check that gives the incorrect error message Signed-off-by: David Nind --- C4/SIP/ILS.pm | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/C4/SIP/ILS.pm b/C4/SIP/ILS.pm index 3e5b6a4824..b2d5008ba9 100644 --- a/C4/SIP/ILS.pm +++ b/C4/SIP/ILS.pm @@ -365,13 +365,7 @@ sub cancel_hold { return $trans; } # Remove the hold from the patron's record first - $trans->ok($patron->drop_hold($item_id)); # different than the transaction drop! - - unless ($trans->ok) { - # We didn't find it on the patron record - $trans->screen_msg("No such hold on patron record."); - return $trans; - } + $patron->drop_hold($item_id); # different than the transaction drop! # Now, remove it from the item record. If it was on the patron # record but not on the item record, we'll treat that as success. -- 2.20.1