Bug 21780 - LostItem should be called from ModItem
Summary: LostItem should be called from ModItem
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-07 10:53 UTC by Martin Renvoize
Modified: 2018-11-07 10:53 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Renvoize 2018-11-07 10:53:24 UTC
Whilst QAing bug 21754 and bug 20884 I realised that the functionality related to losing an item is not very well encapsulated.  Currently LostItem is called in various places following a ModItem call.  We should move the call to LostItem inside of ModItem to lower the risk of LostItem not being applied after a ModItem call has been made.

This would be a good first step towards a more thorough refactoring of this area of the codebase and the move to the Koha:: namespace.