]> git.koha-community.org Git - koha.git/commit
Bug 30254: [22.05.x] Don't charge overdue fines unless some fine exists
authorNick Clemens <nick@bywatersolutions.com>
Mon, 31 Oct 2022 18:38:04 +0000 (18:38 +0000)
committerLucas Gass <lucas@bywatersolutions.com>
Fri, 24 Feb 2023 13:27:54 +0000 (13:27 +0000)
commit93ee31403510318636b37ccc97305724db14b46b
treea2aa55dd30c4657c76187d088c71071a2439e3a1
parent262ab33d2486fb92e8ce69941b4be8f2e9ea9fe3
Bug 30254: [22.05.x] Don't charge overdue fines unless some fine exists

We need to determine if a book was lost by a patron, the clues we have
are previous charges. If we don't find any, we shouldn't charge a new fine

To test:
 1 - set Lost item fee refund on return policy to "Refund lost item charge and charge new overdue fine", turn on FinesMode, make sure your circ rules charge fines
 2 - have an itemtype / patron combo that charges an overdue fine
 3 - check item out (with a due date in the future) and then right back in again
 4 - confirm patron doesn't have a fine because the item was not late
 5 - set the item to Lost
 6 - in the database, edit the date_due of your checkout to a date in the past
 7 - check the item in, it is marked found
 8 - confirm your patron now has a fine
 9 - Apply patch
10 - Repeat with a new item and patron
11 - Confirm no charges

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Bug 30254: Unit tests

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Bug 30254: (QA follow-up) Remove warn from tests

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
https://bugs.koha-community.org/show_bug.cgi?id=32054

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Koha/Item.pm
t/db_dependent/Koha/Item.t