Bug 27676: Correct finesMode ne off condition
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Thu, 11 Feb 2021 10:21:43 +0000 (11:21 +0100)
committerFridolin Somers <fridolin.somers@biblibre.com>
Fri, 12 Feb 2021 14:53:40 +0000 (15:53 +0100)
commitd22780d917fa48ffe39aa08a736a3124df9023d2
tree9a11cce4b801b51e9c25d0d2040936eca2f72485
parenta0fb677b554acd0ec6972c7da5bd3752ae7b049b
Bug 27676: Correct finesMode ne off condition

finesMode is 'off' by default (sysprefs.sql), but if you modify its value from
the UI and set it to 'production' then back to 'off', the DB value becomes an
empty string '', because of $YAML::Syck::ImplicitTyping

This has been found when working on on removing YAML::Syck (bug 22824),
so it's not perfect but the situation will be cleared in the follow-up
bug report.

Test plan:
0. Don't apply the patch
1. reset_all
=> finesMode eq 'off' in DB
2. Set the pref's value to production
3. Switch it back to 'off'
4. Value is '' in DB
5. Check an item in that should generate overdue charges
=> Charges are not generated
6. Apply the patch
7. Check an item in that should generate overdue charges
=> Charges are generated

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit f831130525c62f799c831dd7eb1b36065b60c172)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
circ/returns.pl