Bug 29704: Add a triggered option to holds_reminder.pl
authorNick Clemens <nick@bywatersolutions.com>
Fri, 17 Dec 2021 15:49:17 +0000 (15:49 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Wed, 19 Jan 2022 07:15:04 +0000 (21:15 -1000)
commitd1a701573e9005fdfb770a40b3a04c4fcc0029b6
treea551d936e45c8e120d7ec31cf431bb7b924765e9
parent40da37caa66840280f421d994e56ecdf6eb446cf
Bug 29704: Add a triggered option to holds_reminder.pl

This patch adds an option to only trigger notices matching the number of
days waiting specified

You will need to define HOLD_REMINDER notices for the specific branch of the
patron and ensure the patron has hold reminder notices in their messaging preferences

TO test:
1 - Place a hold for a patron and check in to confirm
2 - Set the waiting date back a few days:
    update reserves set waitingdate = DATE_SUB(CURDATE(), INTERVAL 5 DAY);
3 - Run the cron and see that patron would be notified if running for 4 days weaiting
    perl misc/cronjobs/holds/holds_reminder.pl -v --days 4
4 - Apply patch
5 - perl misc/cronjobs/holds/holds_reminder.pl -v --days 4 --triggered
6 - Note patron would not be notified
7 - perl misc/cronjobs/holds/holds_reminder.pl -v --days 5 --triggered
8 - Note patron is notified when days waiting matches exactly

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
misc/cronjobs/holds/holds_reminder.pl