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,
Created attachment 154579 [details] [review] 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
Created attachment 154681 [details] [review] 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>
Created attachment 154801 [details] [review] 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>
Pushed to master for 23.11. Nice work everyone, thanks!
Pushed to 23.05.x for 23.05.04
Nice work everyone! Pushed to oldstable for 22.11.x