From 98355071ba4bd0801a0c809a1ec3e9eaaf9aecbf Mon Sep 17 00:00:00 2001 From: Thomas Berezansky Date: Tue, 31 Jul 2012 15:35:21 -0400 Subject: [PATCH] Check for filled hold on transit checkin "Fill related hold" can grab a hold out from under a transit, so check for filled holds in addition to canceled ones. Signed-off-by: Thomas Berezansky Signed-off-by: Bill Erickson --- Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm index 262e0ba188..83a46a84d5 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm @@ -2588,9 +2588,9 @@ sub do_checkin { $self->hold($hold); - if( $hold and $hold->cancel_time ) { # this transited hold was cancelled mid-transit + if( $hold and ( $hold->cancel_time or $hold->fulfillment_time ) ) { # this transited hold was cancelled or filled mid-transit - $logger->info("circulator: we received a transit on a cancelled hold " . $hold->id); + $logger->info("circulator: we received a transit on a cancelled or filled hold " . $hold->id); $self->reshelve_copy(1); $self->cancelled_hold_transit(1); $self->notify_hold(0); # don't notify for cancelled holds -- 2.11.0