From 9ad28f7e615f255809d04dfb7c45f9d97bb5be1b 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 --- 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