]> git.koha-community.org Git - koha.git/commit
Bug 24553: Use 'barcode' not 'item_id' when removing hold from SIP patron
authorNick Clemens <nick@bywatersolutions.com>
Tue, 25 Feb 2020 14:42:29 +0000 (14:42 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Fri, 20 Mar 2020 15:39:24 +0000 (15:39 +0000)
commit64035301fcde844c899253c6e21174397c34acbf
tree46583eb889a144de39e76540e76fbcae85f731a3
parent5dcb02ec88d589512e18022ad36d7753381b88f6
Bug 24553: Use 'barcode' not 'item_id' when removing hold from SIP patron

The drop_hold routine was using the wrong parameter for item info in the patron's
array of holds. We store it as 'barcode' not 'item_id'. This is true for both waiting and unavailable
holds

To test:
1 - Apply unit tests patch
2 - prove -v t/db_dependent/SIP/ILS.t
3 - It fails
4 - Apply second patch
5 - prove -v t/db_dependent/SIP/ILS.t
6 - It passes!

You can also see bug 24175 - you cna hack sip_cli_emulator to allow cancelling a hold
and check the messages, test coverage is improved by this patch and should be sufficient
for testing

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
C4/SIP/ILS/Patron.pm