Bugzilla – Attachment 178427 Details for
Bug 19336
Add option to fines.pl to apply maximal fine amount after delay
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 19336: Add --maxfinesday option to fines.pl
Bug-19336-Add---maxfinesday-option-to-finespl.patch (text/plain), 2.14 KB, created by
William Lavoie
on 2025-02-20 16:05:38 UTC
(
hide
)
Description:
Bug 19336: Add --maxfinesday option to fines.pl
Filename:
MIME Type:
Creator:
William Lavoie
Created:
2025-02-20 16:05:38 UTC
Size:
2.14 KB
patch
obsolete
>From 02651d9e9c32395506bcf699f64d718a23b9b8e5 Mon Sep 17 00:00:00 2001 >From: David Bourgault <david.bourgault@inlibro.com> >Date: Mon, 18 Sep 2017 13:24:13 -0400 >Subject: [PATCH] Bug 19336: Add --maxfinesday option to fines.pl > >This is a functionality we have for some of our clients we want to push >to the community. > >Adds the --maxfinesday argument to fines.pl which creates or updates a >fine with the maximum amount set in the circulation rules after the >delay has passed. > >e.g.: >./fines.pl --maxfinesdays 30 >All item over 30 days late will have the maximal fine applied. > >Here is a test plan I followed based on above. >Test plan: >1. Create circulation rule with Overdue fines cap set > >2. Set finesMode systempreference to 'Calculate and charge' > >3. Check out an item with the due date set 30 days ago > >4. In Koha shell run ./fines.pl --maxfinesdays 30 > >5. There should now be a new fine in the accountlines table of the >amount set in overdue fines cap > >Followed test plan, patch worked as intended > >Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz> >Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> >--- > misc/cronjobs/fines.pl | 3 +++ > 1 file changed, 3 insertions(+) > >diff --git a/misc/cronjobs/fines.pl b/misc/cronjobs/fines.pl >index 684b4a1841..3dd05259b3 100755 >--- a/misc/cronjobs/fines.pl >+++ b/misc/cronjobs/fines.pl >@@ -49,6 +49,7 @@ my $output_dir; > my $log; > my $maxdays; > my $verify_issue; >+my $maxfinesdays; > > my $command_line_options = join( " ", @ARGV ); > cronlogaction( { info => $command_line_options } ); >@@ -60,6 +61,7 @@ GetOptions( > 'o|out:s' => \$output_dir, > 'm|maxdays:i' => \$maxdays, > 'i|verifyissue' => \$verify_issue, >+ 'maxfinesdays=i' => \$maxfinesdays > ); > my $usage = << 'ENDUSAGE'; > >@@ -76,6 +78,7 @@ This script has the following parameters : > -m --maxdays: how many days back of overdues to process > -i --verifyissue: verify the issue before updating the fine in case the > item is returned while the fines job is running >+ --maxfinesdays: number of days after which normal fine amount should jump to maximal amount > > ENDUSAGE > >-- >2.43.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 19336
:
67191
|
68318
|
68335
|
68336
|
75379
|
75380
|
75381
|
85914
|
85915
|
85916
|
85917
|
85918
|
85919
|
85920
|
85921
|
85922
|
85923
|
124197
|
124198
|
124199
|
124200
|
124201
|
153504
|
153505
|
153506
|
153507
|
153508
|
168517
|
168518
|
168519
|
168520
|
168521
|
172982
|
172983
|
172984
|
172985
|
172986
|
172987
|
176915
|
176916
|
176917
|
176918
|
176919
|
176920
| 178427 |
178428
|
178429
|
178430
|
178431
|
178432