Bug 35587: Do not trigger transfer if lost item returned and BlockReturnOfLostItems...
authorAleisha Amohia <aleishaamohia@hotmail.com>
Wed, 20 Dec 2023 02:37:48 +0000 (02:37 +0000)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Fri, 5 Jan 2024 13:04:04 +0000 (14:04 +0100)
commit930ad0178d41483672ec52fc8bedc94fe0dacc5b
treed773b83b56aa5b4c08105f5ccbd153a6369a3be8
parent1f16ac11e228770872b0d2f40538e7816c03940c
Bug 35587: Do not trigger transfer if lost item returned and BlockReturnOfLostItems enabled

This fixes current buggy behaviour - when BlockReturnOfLostItems is enabled, no transfer should be triggered and the lost status should be retained.

To test:

1. Go to Koha Administration -> Global system preferences
2. Set the BlockReturnOfLostItems system preference to Block
3. Enable the AutomaticItemReturn system preference (this is simply to make testing a bit faster)
4. Take note of your logged in library
5. Search for an item where the home library is NOT the same as your logged in library
6. Edit this item and give it a lost status
7. Check in the item
8. Notice the item is returned and a transfer is automatically triggered
9. If you go to the item record page, the lost status has been remove

10. Apply the patch and restart services

11. Edit the item again and give it a lost status. This will also cancel the transfer
12. Check in the item
13. Confirm the transfer is NOT triggered and the lost status is retained as expected.
14. Go back to system preferences and disable the BlockReturnOfLostItems system preference (set to "Don't block")
15. Check in the item
16. Confirm the transfer is triggered and lost status is removed

17. Confirm tests pass

prove t/db_dependent/Circulation/Returns.t
prove t/db_dependent/Circulation/Branch.t

Sponsored-by: Pymble Ladies' College
Signed-off-by: Esther <esther@bywatersolutions.com>
Signed-off-by: Kelly <kelly@bywatersolutions.com>
Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
C4/Circulation.pm
t/db_dependent/Circulation/Returns.t