From 0f4f8d00092b6f9c1377c7353c3eb98e767061b9 Mon Sep 17 00:00:00 2001
From: James Fournie <jfournie@sitka.bclibraries.ca>
Date: Tue, 18 Dec 2012 12:45:51 -0800
Subject: [PATCH] Check for a selection_depth returned from
 title_hold.is_possible. See Launchpad 1064651 for details.  This is a quick
 fix which solves a fairly serious issue whereby copies outside of a hard
 boundary could fill holds when they are placed within the TPAC in the staff
 client.

Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
---
 Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm | 3 +++
 1 file changed, 3 insertions(+)

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 0b188bf2e2..d713fd32ac 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm
@@ -93,6 +93,9 @@ sub test_and_create_hold_batch {
         ($res) = $self->method_lookup(
             'open-ils.circ.title_hold.is_possible')->run($auth, $params, $override ? $oargs : {});
         if ($res->{'success'} == 1) {
+
+            $params->{'depth'} = $res->{'depth'} if $res->{'depth'};
+
             my $ahr = construct_hold_request_object($params);
             my ($res2) = $self->method_lookup(
                 $override
-- 
2.11.0