[Koha-devel] [Koha-patches] [PATCH] include grace period when calculating overdue fine

Michael Hafen mdhafen at tech.washk12.org
Thu Sep 3 16:53:15 CEST 2009


Hello list,
  Here's the situation.  Currently the fines.pl calls CalcFine which
calculates the number of days overdue as ( date_due - today ) - grace
period.  My librarians want the calculation to be ( date_due - today ).
I have submitted a patch to this end.  The grace period is still
effective in deciding if a fine should be started.

  Here's the problem.  This is really a serious change in the way
overdues are processed.  We could add another system preference to
control which calculation is used.  Or we can come to some consensus on
the list as to which way it should be so we don't need another sys.
pref.

Here's the question.  Is there a consensus as to how this calculation
should be run?  If there is I presume the patch will be pushed to head,
and that's the end of the debate.  If there isn't a unanimous consensus
then we have to ask if it's worth making more work for a few in order to
not have another system preference here.

Thoughts please.

On Wed, 2009-09-02 at 18:40 -0400, Galen Charlton wrote:
> Hi,
> 
> On Wed, Sep 2, 2009 at 4:14 PM, Michael Hafen<mdhafen at tech.washk12.org> wrote:
> > Before the grace period was not included in the number of days overdue when
> > calculating the fine.  My librarians seem to think the number of days overdue
> > should include the grace period.
> 
> Interesting.  Unfortunately for keeping the list of configuration
> options low, I could see this as being a case where the desired policy
> could vary from library to library.  Could you ask on the Koha list
> whether there's a consensus one way or the other, or whether we'll
> need another syspref?
> 
> Regards,
> 
> Galen
-- 
Michael Hafen
Systems Analyst and Programmer
Washington County School District
Utah, USA

for Koha checkout
http://development.washk12.org/gitweb/
or
git://development.washk12.org/koha





More information about the Koha-devel mailing list