From: Thomas Berezansky Date: Tue, 31 Jul 2012 19:35:21 +0000 (-0400) Subject: Check for filled hold on transit checkin X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=f489194ee39bb59cad75d644f9fbe9d23c7a20e3;p=contrib%2FConifer.git 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 --- 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