Bugzilla – Attachment 154801 Details for
Bug 34569
misc/cronjobs/holds/holds_reminder.pl problem with trigger arg
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 34569: Fix misc/cronjobs/holds/holds_reminder.pl trigger arg
Bug-34569-Fix-misccronjobsholdsholdsreminderpl-tri.patch (text/plain), 1.67 KB, created by
Katrin Fischer
on 2023-08-25 10:48:49 UTC
(
hide
)
Description:
Bug 34569: Fix misc/cronjobs/holds/holds_reminder.pl trigger arg
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2023-08-25 10:48:49 UTC
Size:
1.67 KB
patch
obsolete
>From b0af7d8c19e71beb0d3abfefc5a3a51d1f399ca9 Mon Sep 17 00:00:00 2001 >From: Fridolin Somers <fridolin.somers@biblibre.com> >Date: Thu, 17 Aug 2023 10:42:31 -1000 >Subject: [PATCH] Bug 34569: Fix misc/cronjobs/holds/holds_reminder.pl trigger > arg > >misc/cronjobs/holds/holds_reminder.pl help says one can use arge -t for --triggered >But not implemented in code. >Should be like in misc/cronjobs/overdue_notices.pl : > 't|triggered' => \$triggered, > >Test plan : >Play with misc/cronjobs/holds/holds_reminder.pl with arg -t and >--trigger > >Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >--- > misc/cronjobs/holds/holds_reminder.pl | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/misc/cronjobs/holds/holds_reminder.pl b/misc/cronjobs/holds/holds_reminder.pl >index 813eb0eb2e..a5cbbb6f41 100755 >--- a/misc/cronjobs/holds/holds_reminder.pl >+++ b/misc/cronjobs/holds/holds_reminder.pl >@@ -87,7 +87,7 @@ Optional parameter, number of days an items has been 'waiting' on hold > to send a message for. If not included a notice will be sent to all > patrons with waiting holds. > >-=item B<-triggered> >+=item B<-t> | B<--triggered> > > Optional parameter, only send notices for holds exactly <days> waiting. > If not included a notice will be sent to all patrons with waiting holds >@@ -184,7 +184,7 @@ GetOptions( > 'v' => \$verbose, > 'c|confirm' => \$confirm, > 'days=s' => \$days, >- 'triggered' => \$triggered, >+ 't|triggered' => \$triggered, > 'lettercode=s' => \$lettercode, > 'library=s' => \@branchcodes, > 'date=s' => \$date_input, >-- >2.30.2
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 34569
:
154579
|
154681
| 154801