Bug 11055 - Add 'minutes' as a load period unit
Summary: Add 'minutes' as a load period unit
Status: BLOCKED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Maxime Beaulieu
QA Contact: Testopia
URL:
Keywords:
Depends on: 8133
Blocks:
  Show dependency treegraph
 
Reported: 2013-10-15 19:26 UTC by Blou
Modified: 2024-04-26 19:28 UTC (History)
11 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 11055 - Add 'Minutes' as a load period unit (13.54 KB, patch)
2014-06-27 20:54 UTC, Maxime Beaulieu
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Blou 2013-10-15 19:26:16 UTC
Our user likes to be able to fine by the (5) minutes for hourly loans (2h, 4h, etc...).

One solution would be to change (hack) the fine amount and grace and... to allow for fraction.  An easier solution is simply to add "minutes" as a unit.

Code change was very simple, but the Calendar and Holiday mechanisms have changed since, so I'll see how to do it all cute again.

ALSO: One area of discussion with the user was about the meaning of a 24h loan.  Unit=Hours, Period=24.  If the library is opened only 10h per day, this is not a 3 day rental.  This should be a "same time tomorrow, fines by the hour".

Comments welcomed.
Comment 1 Maxime Beaulieu 2014-06-27 20:54:44 UTC
Created attachment 29337 [details] [review]
Bug 11055 - Add 'Minutes' as a load period unit

WARNING: This patch is dependent on http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8133.
My patch on bug 8133 ensures proper calculation of overdues.

Minutes are added as a unit in circulation rules.

A test script is provided: 't/Circulation/minuteloan.t'.
Comment 2 Jonathan Druart 2015-02-26 14:26:03 UTC
Blocked by bug 8133 (which is blocked too...)