From 8c72a71371a534e2c56be128d073d8ec0e228e44 Mon Sep 17 00:00:00 2001 From: Bill Erickson Date: Thu, 19 Jan 2017 12:21:11 -0500 Subject: [PATCH] LP#1596595 Hold copy permit test thinko repair Pickup and requesting org unit IDs were passed in the wrong order to the copy permit test. This resulted in some items, particularly age protected copies, appearing on the holds pull list even though they were not (yet) permitted for hold capture at a remoted library. Signed-off-by: Bill Erickson Signed-off-by: Chris Sharp Signed-off-by: Kathy Lussier --- Open-ILS/src/perlmods/lib/OpenILS/Utils/HoldTargeter.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Utils/HoldTargeter.pm b/Open-ILS/src/perlmods/lib/OpenILS/Utils/HoldTargeter.pm index 5555c86b81..125b395ac7 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Utils/HoldTargeter.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Utils/HoldTargeter.pm @@ -1043,8 +1043,8 @@ sub copy_is_permitted { my $resp = $self->editor->json_query({ from => [ 'action.hold_retarget_permit_test', - $self->hold->request_lib, $self->hold->pickup_lib, + $self->hold->request_lib, $copy->{id}, $self->hold->usr, $self->hold->requestor -- 2.11.0