[Koha-bugs] [Bug 20125] Add a cron script to claim late orders

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125 --- Comment #28 from Arthur Suzuki <arthur.suzuki@biblibre.com> --- (In reply to Jonathan Druart from comment #26)

1. We should avoid such raw SQL queries.

Cannot you do something like - loop on orders - use ->claims->filter_by_late (or use ->claimed_date)

And use Koha::Objects everywhere?

2. We need to tests (and so move the code to methods)

3. We need try-catch and better error handling.

ups uploaded a new patch a bit too soon, I only fixed the first part so far. I will push a new patch soon. -- You are receiving this mail because: You are watching all bug changes.