Bug 17758 - SIP checkin does not handle holds correctly
authorBenjamin Rokseth <benjamin.rokseth@kul.oslo.kommune.no>
Tue, 13 Dec 2016 12:38:19 +0000 (13:38 +0100)
committerMason James <mtj@kohaaloha.com>
Sun, 23 Apr 2017 23:07:38 +0000 (11:07 +1200)
commit7a8c0af22c46370c2a5187449da01d591c11ba75
tree9bd8ea8c49b701b6c382ade6505f1e72442bd3b6
parent753eda6bae0a58282e134beadc96690ac2d26226
Bug 17758 - SIP checkin does not handle holds correctly

This patch updates SIP checkin to use reserve_id as param, to fix regression
introduced by Bug 14695.

To test:

Given 1 patron and 1 biblio with two items attached:

1) create two record level holds on one biblio on same user
Using SIP client or telnet:
2) checkin item x on pickup branch, observe item is waiting and hold 1 is filled
3) checkin item y on pickup branch, observe item y now is waiting and x is available
4) apply this patch and repeat 1-3, now expected behaviour should be
   that both holds are filled and both items waiting

Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit 72de37e24bd02ffccad3cd1a76e0d74202cf7794)
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
C4/SIP/ILS/Transaction/Checkin.pm