From ca8de07d2e68e98f813d852cf2ed260af66b3143 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Fri, 28 Jul 2017 11:38:54 +0200 Subject: [PATCH] Bug 18996: What about adding $circ->ok to the second clause ? Content-Type: text/plain; charset=utf-8 Proposal needs feedback from preferably Colin. --- C4/SIP/ILS.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/SIP/ILS.pm b/C4/SIP/ILS.pm index 35879af..1c90584 100644 --- a/C4/SIP/ILS.pm +++ b/C4/SIP/ILS.pm @@ -216,7 +216,7 @@ sub checkin { # It's ok to check it in if it exists, and if it was checked out # or it was not checked out but the checked_in_ok flag was set - $circ->ok( ( $checked_in_ok && $item ) || ( $item && $item->{patron} ) ); + $circ->ok( ( $checked_in_ok && $item ) || ( $item && $item->{patron} && $circ->ok ) ); syslog("LOG_DEBUG", "C4::SIP::ILS::checkin - using checked_in_ok") if $checked_in_ok; if ( !defined( $item->{patron} ) ) { -- 2.1.4