Bug 27993

Summary: Koha::Item::Transfer->in_transit should not count cancelled transfers
Product: Koha Reporter: Martin Renvoize (ashimema) <martin.renvoize>
Component: CirculationAssignee: Martin Renvoize (ashimema) <martin.renvoize>
Status: CLOSED FIXED QA Contact: Joonas Kylmälä <joonas.kylmala>
Severity: normal    
Priority: P5 - low CC: fridolin.somers, gmcharlt, jonathan.druart, joonas.kylmala, kyle.m.hall, stalkernoid, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
21.05.00
Circulation function:
Bug Depends on: 26481    
Bug Blocks: 27281    
Attachments: Bug 27993: Add unit tests
Bug 27993: Add handling for datecancelled in in_transit
Bug 27993: Add handling for datecancelled in in_transit
Bug 27993: Add unit tests
Bug 27993: Add handling for datecancelled in in_transit
Bug 27993: Add unit tests
Bug 27993: Add handling for datecancelled in in_transit

Description Martin Renvoize (ashimema) 2021-03-19 08:25:20 UTC
Petro Vashchuk has identified a bug in the in_transit method. We should also filter out cancelled transfers.
Comment 1 Martin Renvoize (ashimema) 2021-03-19 13:33:04 UTC
Created attachment 118539 [details] [review]
Bug 27993: Add unit tests

This patch adds a unit test to check that 'in_transit' takes
datecancelled into account.
Comment 2 Martin Renvoize (ashimema) 2021-03-19 13:33:07 UTC
Created attachment 118540 [details] [review]
Bug 27993: Add handling for datecancelled in in_transit

This patch adds handling for the datecancelled field in the in_transit
method.

Test plan
1/ Run the new unit test in t/db_dependent/Koha/Item/Transfer.t
Comment 3 Peter Vashchuk 2021-03-22 10:52:40 UTC Comment hidden (obsolete)
Comment 4 Peter Vashchuk 2021-03-22 11:28:10 UTC
Created attachment 118586 [details] [review]
Bug 27993: Add unit tests

This patch adds a unit test to check that 'in_transit' takes
datecancelled into account.

Signed-off-by: Petro Vashchuk <stalkernoid@gmail.com>
Comment 5 Peter Vashchuk 2021-03-22 11:28:14 UTC
Created attachment 118587 [details] [review]
Bug 27993: Add handling for datecancelled in in_transit

This patch adds handling for the datecancelled field in the in_transit
method.

Test plan
1/ Run the new unit test in t/db_dependent/Koha/Item/Transfer.t

Signed-off-by: Petro Vashchuk <stalkernoid@gmail.com>
Comment 6 Joonas Kylmälä 2021-03-22 11:32:53 UTC
Created attachment 118588 [details] [review]
Bug 27993: Add unit tests

This patch adds a unit test to check that 'in_transit' takes
datecancelled into account.

Signed-off-by: Petro Vashchuk <stalkernoid@gmail.com>
Signed-off-by: Joonas Kylmälä <joonas.kylmala@helsinki.fi>
Comment 7 Joonas Kylmälä 2021-03-22 11:32:57 UTC
Created attachment 118589 [details] [review]
Bug 27993: Add handling for datecancelled in in_transit

This patch adds handling for the datecancelled field in the in_transit
method.

Test plan
1/ Run the new unit test in t/db_dependent/Koha/Item/Transfer.t

Signed-off-by: Petro Vashchuk <stalkernoid@gmail.com>
Signed-off-by: Joonas Kylmälä <joonas.kylmala@helsinki.fi>
Comment 8 Jonathan Druart 2021-04-01 16:55:04 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 9 Fridolin Somers 2021-04-13 14:51:52 UTC
Depends on Bug 26481 not in 20.11.x