[Koha-bugs] [Bug 8640] New: GetHardDueDate functions incorrectly

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Aug 14 19:16:11 CEST 2012


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8640

          Priority: P5 - low
 Change sponsored?: ---
            Bug ID: 8640
                CC: gmcharlt at gmail.com, kyle.m.hall at gmail.com
          Assignee: koha-bugs at lists.koha-community.org
           Summary: GetHardDueDate functions incorrectly
          Severity: enhancement
    Classification: Unclassified
                OS: All
          Reporter: kyle at bywatersolutions.com
          Hardware: All
            Status: NEW
           Version: master
         Component: Circulation
           Product: Koha

GetHardDueDate works just like GetIssuingRule, but it not only tests for having
found an row of results before returning, but checks for the existence of a
hard due date as well. That means that even if it finds a matching rule, unless
that rule has a hardduedate set it will toss it and keep looking!

So, for example, even though there is a matching rule for say "STAFF / VIDEO /
*", but it has no hardduedate. So it gets tossed and the subroutine keeps
looking. If the next match, "STAFF / * / *" *has* a hardduedate, it is the one
returned.

This means that if a more specific rule has no hard due date, it is overridden
by a less specific rule that does have a hard due date.

TEST PLAN:
1) Create 2 issuing rules, one more specific with no hard due date, one less
specific *with* a hard due date. See my description above for an example.
2) Issue an item to a borrower where the more specific rule should apply, and
the checkout should not receive a hard due it.
3) Observe that it *does* receive a hard due date.
4) Apply the patch, and repeat step 2.
5) Observe that the checkout *does not* receive a hard due date.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.


More information about the Koha-bugs mailing list