https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14591 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |josef.moravec@gmail.com |y.org | --- Comment #23 from Josef Moravec <josef.moravec@gmail.com> --- (In reply to Martin Renvoize from comment #21)
I really like this solution! It tidies up the code significantly and corrects the logic considerably!
Signing off as it all works in testing as described...
Some QA comments for the next in line however.
* Can we/Should we remove the 'lastincrement' field here as I believe this was the only use case for it? * The logic here puts 'dropboxmode/dropboxdate' in precidence to any specified return date.. I think we perhaps need a bit more in this regard, but perhaps that can be done within bug 13044 as there's a fair bit of contextual conversation over there around this issue.
I agree to continue on these two points on other reports, as this patchset is ok and fixes the bug. -- You are receiving this mail because: You are watching all bug changes.