From 7c793f9cbf9423deb92842f41a4a2ab268090e17 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 --- .../lib/OpenILS/Application/Circ/Circulate.pm | 4 ++-- 1 files 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 262e0ba..83a46a8 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 -- 1.7.2.5