Bug 32450: Noissuescharge debit type exclusions
authorMatt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Fri, 6 Jan 2023 13:51:00 +0000 (13:51 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Wed, 17 May 2023 13:25:06 +0000 (10:25 -0300)
commit355d7de9a58dd315f22accea60dfef19016597c3
tree0c932941885510d1c8e2a6f3a4e954e2b20044e8
parent0866ecc4f9f8086de71225d3ceb945462e998560
Bug 32450: Noissuescharge debit type exclusions

Currently the debit types to be excluded from the noissuescharge syspref are
hardcoded in non_issues_charges which gives no flexibility for selecting which
debit types should be included. This patch amends the subroutine to use a
database flag to identify which debit types should be included. It also adds a
column to the table in the Debit Types area of System Preferences which shows
which debit types are included. The ability to edit all debit types has been
added rather than just the non-system ones and the flag to include/exclude the
debit type from noissuescharge can be changed by clicking that edit button.

Test plan:
1) Choose a patron and add some fines to this patron that have different debit_types
2) Navigate to system preferences and observe that currently you can only amend
   the noissuescharge included debit types using three preferences:
   ManInvInNoissuesCharge, RentalsInNoissuesCharge, HoldsInNoissuesCharge
3) Apply both commits attached to this bug
4) Navigate as above and observe that these three system preferences are now gone
5) Navigate to Debit Types in System Preferences, the table should have a column
   called No issues charge that shows whether a debit_type is Included or Not
   included
6) Click the edit button and there should be a checkbox for Included in
   noissuescharge
7) Change some of the debit_types using this option and observe that the patron
   you added fines to will either be blocked from checkouts or able to checkout
   depending on which debit_types you include and the value of these fines.

Mentored-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Koha/Account.pm
admin/debit_types.pl
koha-tmpl/intranet-tmpl/prog/en/modules/admin/debit_types.tt