From 4f5d525d66f1436fb1a74f4fc9da3a0d17168e75 Mon Sep 17 00:00:00 2001
From: Tomas Cohen Arazi <tomascohen@gmail.com>
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 <claire.gravely@bsz-bw.de>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
---
 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);