From f2a40ab35ce77f41eda6a7101026dadd5e8e4380 Mon Sep 17 00:00:00 2001 From: Thomas Berezansky Date: Mon, 30 Jan 2012 10:09:17 -0500 Subject: [PATCH] Fix "Clear Shelf Expired Holds" checkin modifier By not looking up empty arrays there either. Signed-off-by: Thomas Berezansky Signed-off-by: Bill Erickson --- Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm index fe1b93075c..17ae306f26 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm @@ -3445,7 +3445,8 @@ sub clear_shelf_process { # refetch the holds to pick up the caclulated cancel_time, # which may be needed by Action/Trigger $e->xact_begin; - my $updated_holds = $e->search_action_hold_request({id => \@canceled_holds}, {substream => 1}); + my $updated_holds = []; + $updated_holds = $e->search_action_hold_request({id => \@canceled_holds}, {substream => 1}) if (@canceled_holds > 0); $e->rollback; $U->create_events_for_hook( -- 2.11.0