Bugzilla – Attachment 176383 Details for
Bug 38307
Make overdue_notices.pl quiet if there are no libraries with active overdue rules
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 38307: Make overdue_notices.pl die quietly if no branches have overdue rules
Bug-38307-Make-overduenoticespl-die-quietly-if-no-.patch (text/plain), 1.73 KB, created by
Imani
on 2025-01-10 17:53:22 UTC
(
hide
)
Description:
Bug 38307: Make overdue_notices.pl die quietly if no branches have overdue rules
Filename:
MIME Type:
Creator:
Imani
Created:
2025-01-10 17:53:22 UTC
Size:
1.73 KB
patch
obsolete
>From 6cb19378d6a3b2db4fb27298776b7497d4fddcbf Mon Sep 17 00:00:00 2001 >From: Aleisha Amohia <aleishaamohia@hotmail.com> >Date: Thu, 31 Oct 2024 03:18:33 +0000 >Subject: [PATCH] Bug 38307: Make overdue_notices.pl die quietly if no branches > have overdue rules > >If no patron categories require overdue notices, then there will be no overdue rules/status triggers. This isn't necessarily a bad thing and shouldn't be noisy. > >To test: > >1. Go to Administration -> Patron categories >2. Set all your category types to have overdue notice required: No >3. Go to Tools -> Overdue notices/status triggers >4. Notice you're unable to set any overdue rules >5. Run the cronjob misc/cronjobs/overdue_notices.pl and notice the error about no overdue rules is printed >6. Apply the patch and restart services >7. Run the cronjob misc/cronjobs/overdue_notices.pl and notice the error is not printed >8. Run the cronjob misc/cronjobs/overdue_notices.pl --verbose and notice the error is printed > >Sponsored-by: Catalyst IT >Signed-off-by: Imani Thomas <imani.thomas@bywatersolutions.com> >--- > misc/cronjobs/overdue_notices.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/misc/cronjobs/overdue_notices.pl b/misc/cronjobs/overdue_notices.pl >index 5e24361699..0b012582bd 100755 >--- a/misc/cronjobs/overdue_notices.pl >+++ b/misc/cronjobs/overdue_notices.pl >@@ -362,7 +362,7 @@ my $PrintNoticesMaxLines = C4::Context->preference('PrintNoticesMaxLines'); > if ($branchcount) { > $verbose and warn "Found $branchcount $overduebranch_word with first message enabled: " . join( ', ', map { "'$_'" } @overduebranches ), "\n"; > } else { >- die 'No branches with active overduerules'; >+ $verbose and die 'No branches with active overduerules'; > } > > if (@branchcodes) { >-- >2.39.5
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 38307
:
173742
|
176383
|
178476