https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43500 Bug ID: 43500 Summary: Extract mark_returned and lift_overdue_restrictions from MarkIssueReturned Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs@lists.koha-community.org Reporter: chloe.zermatten@openfifth.co.uk QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle@bywatersolutions.com Target Milestone: --- MarkIssueReturned currently completes two different tasks: - it completes the conversion of an issue into an old issue - it lifts overdue restrictions on a patron if AutoRemoveOverduesRestrictions is on In preparation for 39756, extract both tasks into two new separate methods: Koha::Checkout->mark_returned and Koha::Patron->lift_overdue_restrictions. This is because in the script 39756 will introduce, while the conversion to old_issue will happen once per issue, the intent is for the restriction lift logic to only run once per patron. I think it would be useful to decouple the two so that they can be called separately. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.