Bug 28210

Summary: C4::Circulation::LostItem should pass through skip_record_index to MarkIssueReturned
Product: Koha Reporter: Martin Renvoize <martin.renvoize>
Component: Architecture, internals, and plumbingAssignee: Martin Renvoize <martin.renvoize>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, jonathan.druart, nick
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
Bug Depends on: 24446, 27281    
Bug Blocks:    
Attachments: Bug 28210: Unit test
Bug 28210: Pass skip_record_index through LostItem
Bug 28210: Unit test
Bug 28210: Add issue, replacementprice, and set onloan so check-in tests work
Bug 28210: Pass skip_record_index through LostItem

Description Martin Renvoize 2021-04-23 15:46:38 UTC
With Bug 27281 we removed a superflous call to update the homebranch of items in transfer.

However, this highlighted a test that needed to be removed and lead to the removal of the 'skip_record_index' parameter from the method signature.

This lead Nick to spot that we never passed the parameter on to MarkIssueAsReturned when we should..

This bug will add back the parameter in LostItem, add a test for the MarkIssueReturned case and start passing the parameters through properly.
Comment 1 Martin Renvoize 2021-04-23 15:54:22 UTC
Created attachment 120118 [details] [review]
Bug 28210: Unit test

Add a unit test to check that skip_record_index is adhered to when
passed to LostItem with the force_mark_returned option enabled.
Comment 2 Martin Renvoize 2021-04-23 15:54:26 UTC
Created attachment 120119 [details] [review]
Bug 28210: Pass skip_record_index through LostItem

This patch restores the skip_record_index optional parameter for
LostItem and passes it through to MarkIssueReturned as required.
Comment 3 Jonathan Druart 2021-04-26 14:48:50 UTC
*** Bug 28209 has been marked as a duplicate of this bug. ***
Comment 4 Nick Clemens 2021-04-26 15:58:34 UTC
Created attachment 120179 [details] [review]
Bug 28210: Unit test

Add a unit test to check that skip_record_index is adhered to when
passed to LostItem with the force_mark_returned option enabled.

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 5 Nick Clemens 2021-04-26 15:58:37 UTC
Created attachment 120180 [details] [review]
Bug 28210: Add issue, replacementprice, and set onloan so check-in tests work

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 6 Nick Clemens 2021-04-26 15:58:40 UTC
Created attachment 120181 [details] [review]
Bug 28210: Pass skip_record_index through LostItem

This patch restores the skip_record_index optional parameter for
LostItem and passes it through to MarkIssueReturned as required.

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 7 Nick Clemens 2021-04-26 16:00:19 UTC
I added some data needed to make the tests pass, can you verify Martin?

For testing this also requires applying the follow-ups on 24446 and 27281

While we remove/readd some code in the series I think it makes sense to remove it to fix the tests, then add it as it should have been here
Comment 8 Jonathan Druart 2021-04-27 09:42:09 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 9 Fridolin Somers 2021-04-30 08:29:08 UTC
Depends on Bug 24446 not in 20.11.x