[Bug 7852] New: Hourly loan fines are not working
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Priority: P5 - low Change sponsored?: --- Bug ID: 7852 CC: gmcharlt@gmail.com Assignee: kyle.m.hall@gmail.com Summary: Hourly loan fines are not working QA Contact: koha.sekjal@gmail.com Severity: enhancement Classification: Unclassified OS: All Reporter: tdavis@uttyler.edu Hardware: All Status: NEW Version: master Component: Circulation Product: Koha When The fines cron runs there are no fines assessed for hourly items. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Elliott Davis <tdavis@uttyler.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 - low |P1 - high Severity|enhancement |major -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Elliott Davis <tdavis@uttyler.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Assignee|kyle.m.hall@gmail.com |tdavis@uttyler.edu --- Comment #1 from Elliott Davis <tdavis@uttyler.edu> --- Created attachment 8702 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8702&action=edit Add private function to calculate chargeable units and reconfigure Koha/Calendar.pm to use this function. To test: Check out an hourly item, manipulate the database to backdate the checkin or let the item go overdue whilst enjoying a sandwich. After the Item is overdue Check the item back in and wait for the fines cron job to run (or run it yourself). You should see a fine assessed for this item. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Elliott Davis <tdavis@uttyler.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|tdavis@uttyler.edu |kyle.m.hall@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Elliott Davis <tdavis@uttyler.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |7849 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Elliott Davis <tdavis@uttyler.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #8702|0 |1 is obsolete| | --- Comment #2 from Elliott Davis <tdavis@uttyler.edu> --- Created attachment 8779 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8779&action=edit Fix fines for large durations - from Colin -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Elliott Davis <tdavis@uttyler.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #8779|0 |1 is obsolete| | --- Comment #3 from Elliott Davis <tdavis@uttyler.edu> --- Created attachment 8780 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8780&action=edit Rebased fines to master -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Kyle M Hall <kyle.m.hall@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #8780|0 |1 is obsolete| | --- Comment #4 from Kyle M Hall <kyle.m.hall@gmail.com> --- Created attachment 8782 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8782&action=edit [Signed-Off] Use hour or day deltas to calculate overdue durations If durations are calculated by subtraction they will use units larger than those we care about and these are not convertable to the smaller units we are attempting to enumerate Use the appropriate delta methods to calculate theee fines Adds a separate hours_between method to calendar This should strictly be checking opening hours (of which closed days are a special case) of the relevant branch These need adding to branches http://bugs.koha-community.org/show_bug.cgi?id=7852 Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Kyle M Hall <kyle.m.hall@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 --- Comment #5 from Ian Walls <koha.sekjal@gmail.com> --- Created attachment 8816 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8816&action=edit [PASSED QA] Use hour or day deltas to calculate overdue durations If durations are calculated by subtraction they will use units larger than those we care about and these are not convertable to the smaller units we are attempting to enumerate Use the appropriate delta methods to calculate theee fines Adds a separate hours_between method to calendar This should strictly be checking opening hours (of which closed days are a special case) of the relevant branch These need adding to branches http://bugs.koha-community.org/show_bug.cgi?id=7852 Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Ian Walls <koha.sekjal@gmail.com> QA comment: renamed "days_minus_grace" variable to "units_minus_grace" Also added POD to _get_chargeable_units -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Ian Walls <koha.sekjal@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Ian Walls <koha.sekjal@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #8782|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7852 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |paul.poulain@biblibre.com Version|master |rel_3_8 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org