From fa20c7a4f562bd3ff3952276f62c6b5e4e7f85c4 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Mon, 5 Mar 2018 13:36:50 -0300 Subject: [PATCH] Bug 20167: Make changing a hold pass the itemnumber to ModReserve If you have an item-level hold, changing an attribute (like priority, pickup location or suspended until date) makes the hold a biblio-level hold, because ModReserve is not passed the current itemnumber. This patch uses the hold's itemnumber and passes it to the ModReserve call. To test: - Run the regression tests from the previous patch: $ kshell k$ prove t/db_dependent/api/v1/holds.t => FAIL: Tests fail - Apply this patch - Run the tests: $ kshell k$ prove t/db_dependent/api/v1/holds.t => SUCCESS: Tests pass! - Sign off :-D Signed-off-by: Claire Gravely Signed-off-by: Josef Moravec Signed-off-by: Nick Clemens Signed-off-by: Nick Clemens (cherry picked from commit e854b570021784d8e6ce8bb9d90e0b02d6f8c9a7) Signed-off-by: Fridolin Somers --- Koha/REST/V1/Hold.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/Koha/REST/V1/Hold.pm b/Koha/REST/V1/Hold.pm index d72e5df829..ce7320e026 100644 --- a/Koha/REST/V1/Hold.pm +++ b/Koha/REST/V1/Hold.pm @@ -136,6 +136,7 @@ sub edit { branchcode => $branchcode, rank => $priority, suspend_until => $suspend_until, + itemnumber => $reserve->{itemnumber} }; C4::Reserves::ModReserve($params); -- 2.39.5