Attachment 166121 Details for Bug 35169 – Bug 35169: Make long overdue borrower category options configurable in interface

[patch] Bug 35169: Make long overdue borrower category options configurable in interface

Bug-35169-Make-long-overdue-borrower-category-opti.patch (text/plain), 3.23 KB, created by Kyle M Hall (khall) on 2024-05-03 14:11:11 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2024-05-03 14:11:11 UTC

Size: 3.23 KB

patch

obsolete

>From 8f4930a89aa40680e01130f31057aa84d7acb488 Mon Sep 17 00:00:00 2001
>From: Matthias Le Gac <matthias.le-gac@inlibro.com>
>Date: Thu, 22 Feb 2024 14:46:22 -0500
>Subject: [PATCH] Bug 35169: Make long overdue borrower category options
> configurable in interface
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> .../modules/admin/preferences/circulation.pref | 17 +++++++++++++++++
> misc/cronjobs/longoverdue.pl | 11 +++++++++++
> 2 files changed, 28 insertions(+)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>index 61c300be8a8..1040c700485 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>@@ -451,6 +451,23 @@ Circulation:
> - "."
> - Leave this field empty if you don't want to skip any <a href="/cgi-bin/koha/admin/authorised_values.pl?searchfield=LOST">LOST</a> statuses.
> - Set to a list of comma separated values, e.g. 5,6,7.
>+ -
>+ - pref: DefaultLongOverdueBorrowerCategories
>+ choices: patron-categories
>+ class: multiple
>+ - the borrower categories that will be listed. Exclude all others.
>+ - WARNING â This preference will be active only if DefaultLongOverdueSkipBorrowerCategories is empty.
>+ - " NOTE: This system preference requires the <code>misc/cronjobs/longoverdue.pl</code> cronjob. Ask your system administrator to schedule it."
>+
>+
>+ -
>+ - pref: DefaultLongOverdueSkipBorrowerCategories
>+ choices: patron-categories
>+ class: multiple
>+ - the borrower categories that will not be listed. Include all others.
>+ - WARNING - This preference overrides the DefaultLongOverdueBorrowerCategories preference, so the DefaultLongOverdueBorrowerCategories filter will not take effect.
>+ - " NOTE: This system preference requires the <code>misc/cronjobs/longoverdue.pl</code> cronjob. Ask your system administrator to schedule it."
>+
> -
> - "When issuing an item that has been marked as lost, "
> - pref: IssueLostItem
>diff --git a/misc/cronjobs/longoverdue.pl b/misc/cronjobs/longoverdue.pl
>index 2d313b77b24..06cd6b28305 100755
>--- a/misc/cronjobs/longoverdue.pl
>+++ b/misc/cronjobs/longoverdue.pl
>@@ -275,6 +275,17 @@ if ( ! defined($charge) ) {
> $charge = $charge_value;
> }
> }
>+
>+if ( C4::Context->preference('DefaultLongOverdueBorrowerCategories') ) {
>+ my $categories = C4::Context->preference('DefaultLongOverdueBorrowerCategories');
>+ $borrower_category = [ split( ',', $categories ) ];
>+}
>+
>+if ( C4::Context->preference('DefaultLongOverdueSkipBorrowerCategories') ) {
>+ my $categories = C4::Context->preference('DefaultLongOverdueSkipBorrowerCategories');
>+ $skip_borrower_category = [ split( ',', $categories ) ];
>+}
>+
> unless ($confirm) {
> $verbose = 1; # If you're not running it for real, then the whole point is the print output.
> print "### TEST MODE -- NO ACTIONS TAKEN ###\n";
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 35169: 162340 | 162345 | 162346 | 162611 | 162667 | 162688 | 163008 | 166121 | 166122 | 166123 | 166124 | 166125 | 166126 | 166127 | 166256