From 3c7704d2bf7bddc54aaf58d996ab8fe990c7956f 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 --- 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 2927a7ad0a..2833f39f19 100644 --- a/Koha/REST/V1/Hold.pm +++ b/Koha/REST/V1/Hold.pm @@ -144,6 +144,7 @@ sub edit { branchcode => $branchcode, rank => $priority, suspend_until => $suspend_until, + itemnumber => $hold->itemnumber }; C4::Reserves::ModReserve($params); -- 2.16.2