From d019a68079ccaf2948561e73fee619d1a9892251 Mon Sep 17 00:00:00 2001 From: Fridolin Somers 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 Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi (cherry picked from commit f4f4feb50b3d2b58b9212b692daf2c4e3e000b7d) Signed-off-by: Fridolin Somers --- 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 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.39.5