Summary: | tool statisticfines.pl(bz 6858) and hourly loan(bz5549) | ||
---|---|---|---|
Product: | Koha | Reporter: | delaye <stephane.delaye> |
Component: | Circulation | Assignee: | Fridolin Somers <fridolin.somers> |
Status: | CLOSED FIXED | QA Contact: | Mason James <mtj> |
Severity: | major | ||
Priority: | P3 | CC: | chris, fridolin.somers, gmcharlt, kyle.m.hall, mtj, paul.poulain |
Version: | 3.8 | ||
Hardware: | All | ||
OS: | All | ||
See Also: |
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6858 http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5549 |
||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Attachments: |
Proposed patch
Bug 8420: tool statisticfines.pl and hourly loan Proposed patch (for 3.8.X) |
Description
delaye
2012-07-11 15:04:27 UTC
Created attachment 11838 [details] [review] Proposed patch Proposed patch. The main change is the use of Koha::DateUtils::dt_from_string to use DateTime objects in fine computing. Patch tested with a sandbox, by Delaye Stephane <stephane.delaye@biblibre.com> Created attachment 11844 [details] [review] Bug 8420: tool statisticfines.pl and hourly loan Signed-off-by: Delaye Stephane <stephane.delaye@biblibre.com> > Signed-off-by: Delaye Stephane <stephane.delaye@biblibre.com> looks good, passing QA... i've changed version from 3.8 to master $ koha-qa.pl testing 1 commit(s) (applied to commit 60790d0) * e0debe5 Bug 8420: tool statisticfines.pl and hourly loan misc/cronjobs/staticfines.pl * misc/cronjobs/staticfines.pl OK Patch pushed to master Very local fix to a cron job. Even if the patch is written & signed-off by BibLibre, no side-effect risk at all, i've pushed it. Doesn't apply cleanly to 3.8.x, please supply a patch for it. Created attachment 12271 [details] [review] Proposed patch (for 3.8.X) Rebased patch for 3.8.X version. Only one existing line is different form master version. Pushed to 3.8.x, will be in 3.8.5 |