]> git.koha-community.org Git - koha.git/commit
Bug 15757: Make GetLoanLength defaults to 0 instead of 21
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Thu, 7 Apr 2016 08:45:58 +0000 (09:45 +0100)
committerJulian Maurice <julian.maurice@biblibre.com>
Fri, 22 Apr 2016 11:42:00 +0000 (13:42 +0200)
commita0dd9770448322189dde29467e54d272f7d03071
tree1e61dd41189a8d7c4f137e77d5f1d849e4587a85
parent2471a0e3a9f222b756d01492b413232afa60c1c6
Bug 15757: Make GetLoanLength defaults to 0 instead of 21

GetLoanLength arbitrary defaulted to 21. The expected behavior seems to
be to default on 0 (loan will be dued today).

IMPORTANT NOTE: This patch will introduce a change in the behaviors for
configuration with a 0 in issuelength. Before this patch, the rule with
a issuelength==0 was skipped, now it's used!

Test plan:
1/ Do not define any rule: the due date will be today (before this patch
was +21 days)
2/ Define some rules which does not match the patron category, itemtype
or branchcode: the due date will be today (before this patch was +21
days).
3/ Modify a rule to match the checkout and set issuelength=0: the due
date will be today (before this patch, the rule was skipped)
4/ Modify this rule and set the issuelength to something > 0: the due
date will be adjusted (same behavior as before this patch)

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Works ok, checked 1-4
All test pass
No koha-qa errors

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit 769728015cea465bbb47d27f69077962a8f2429d)
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
C4/Circulation.pm
t/db_dependent/Circulation_Issuingrule.t
t/db_dependent/Circulation_issuingrules.t