[Bug 41260] New: Running holds reminder cronjob with both triggered and holidays parameter may leave hold reminders unformed
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41260 Bug ID: 41260 Summary: Running holds reminder cronjob with both triggered and holidays parameter may leave hold reminders unformed Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Command-line Utilities Assignee: koha-bugs@lists.koha-community.org Reporter: emmi.takkinen@koha-suomi.fi QA Contact: testopia@bugs.koha-community.org CC: robin@catalyst.net.nz If one runs cronjob holds_reminder.pl with both triggered and holidays parameter, reminders are not formed for holds which waiting date is same as library's closing date. This happens because triggered parameter changes crons functionality to handle only holds which waiting date is exactly the same one as variable $waiting_since. If date in holds waiting date is bypassed by above conditions reminder is never formed. To reproduce: 1. Find hold which waiting date is 2 days from day you're testing this or update one holds waitingdate value directly from database. 2. Set day which is same as the holds waiting date as closing day from that library's calendar where hold is waiting in. 3. Run perl holds_reminder.pl -v -days 2 --triggered -holidays -c => Note that no reminder is formed for the hold. 4. Wait for one day and run the cron again. => Note that no reminder is formed for the hold. 5. Now run cron without triggered parameter. => Reminder id formed. Triggered parameter is supposed to prevent spamming patrons with hold reminders, but it shouldn't prevent libraries from sending out reminders in cases where hold just happens to have waiting date on a closing date. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41260 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |normal -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41260 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|koha-bugs@lists.koha-commun |emmi.takkinen@koha-suomi.fi |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41260 --- Comment #1 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 190105 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190105&action=edit Bug 41260: Ensure hold reminders are send for holds with waiting date on holiday If one runs cronjob holds_reminder.pl with both triggered and holidays parameter, reminders are not formed for holds which have same date as waiting date as library's closing date. This happens because triggered parameter changes crons functionality to handle only holds which waiting date is exactly the same one as variable $waiting_since. If date in holds waiting date is bypassed by above conditions, reminder is never formed. This patch adds new condition which checks that if triggered parameter is used and the date used in variable $waiting_date_static is a holiday. If both are true we use $waiting_date_static as a condition to search for holds waiting for pick up instead of skipping holidays with addDays method. To test: 1. Find hold which waiting date is 2 days from the day when you're testing this or update one holds waitingdate value directly from database. 2. Set day which is same as the holds waiting date as closing day from that library's calendar where hold is waiting in. 3. Run perl holds_reminder.pl -v -days 2 --triggered -holidays -c => Note that no reminder is formed for the hold. 4. Now run cron without triggered parameter. => Note that reminder is formed. 5. Apply this patch. 6. Run cron again with triggerred parameter. => Note that reminder is now formed. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41260 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41260 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |ASSIGNED --- Comment #2 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- There is still a major problem with this. If the day when script is driven is closing day notices with certain waiting date are not formed. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org