https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33900 --- Comment #1 from Pedro Amorim <pedro.amorim@ptfs-europe.com> --- Created attachment 152003 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=152003&action=edit Bug 33900: Fix handling of due_digest Test plan, on k-t-d 1) Go to 'my account' 2) On 'Patron messaging preferences', click 'Edit' 3) On the 'Item due' row, check the 'Email' and 'Digests only' checkboxes and save 4) On the top search bar, press 'Check out' and enter '42' (koha user cardnumber) 5) On the checkout input bar, enter a barcode e.g. 39999000001372 and press checkout 6) Run the following query to force the due_date to be equal to 'todays' date: NOTE: change the YYYY-MM-DD below to whatever day it is you're running this test plan UPDATE issues SET date_due = '2023-06-05 23:59:00' where issue_id = 1; 7) Run the cronjob: ./koha/misc/cronjobs/advance_notices.pl -c --digest-per-branch 8) Notice it hangs, it's stuck on an infinite loop on the 2nd while cycle after if () 9) Cancel the cronjob script, apply patch 10) Run the cronjob script again (step 7), notice it finishes 11) Access patron's notices: /cgi-bin/koha/members/notices.pl?borrowernumber=51 Check that the 'Item due reminder' notice is there. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33900 Sam Lau <samalau@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #152003|0 |1 is obsolete| | --- Comment #2 from Sam Lau <samalau@gmail.com> --- Created attachment 152004 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=152004&action=edit Bug 33900: Fix handling of due_digest Test plan, on k-t-d 1) Go to 'my account' 2) On 'Patron messaging preferences', click 'Edit' 3) On the 'Item due' row, check the 'Email' and 'Digests only' checkboxes and save 4) On the top search bar, press 'Check out' and enter '42' (koha user cardnumber) 5) On the checkout input bar, enter a barcode e.g. 39999000001372 and press checkout 6) Run the following query to force the due_date to be equal to 'todays' date: NOTE: change the YYYY-MM-DD below to whatever day it is you're running this test plan UPDATE issues SET date_due = '2023-06-05 23:59:00' where issue_id = 1; 7) Run the cronjob: ./koha/misc/cronjobs/advance_notices.pl -c --digest-per-branch 8) Notice it hangs, it's stuck on an infinite loop on the 2nd while cycle after if () 9) Cancel the cronjob script, apply patch 10) Run the cronjob script again (step 7), notice it finishes 11) Access patron's notices: /cgi-bin/koha/members/notices.pl?borrowernumber=51 Check that the 'Item due reminder' notice is there. Signed-off-by: Sam Lau <samalau@gmail.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33900 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |normal -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33900 --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Sam, seen you do a lot of sign-offs - THANK YOU :) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33900 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #152004|0 |1 is obsolete| | --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 152159 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=152159&action=edit Bug 33900: Fix handling of due_digest Test plan, on k-t-d 1) Go to 'my account' 2) On 'Patron messaging preferences', click 'Edit' 3) On the 'Item due' row, check the 'Email' and 'Digests only' checkboxes and save 4) On the top search bar, press 'Check out' and enter '42' (koha user cardnumber) 5) On the checkout input bar, enter a barcode e.g. 39999000001372 and press checkout 6) Run the following query to force the due_date to be equal to 'todays' date: NOTE: change the YYYY-MM-DD below to whatever day it is you're running this test plan UPDATE issues SET date_due = '2023-06-05 23:59:00' where issue_id = 1; 7) Run the cronjob: ./koha/misc/cronjobs/advance_notices.pl -c --digest-per-branch 8) Notice it hangs, it's stuck on an infinite loop on the 2nd while cycle after if () 9) Cancel the cronjob script, apply patch 10) Run the cronjob script again (step 7), notice it finishes 11) Access patron's notices: /cgi-bin/koha/members/notices.pl?borrowernumber=51 Check that the 'Item due reminder' notice is there. Signed-off-by: Sam Lau <samalau@gmail.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33900 --- Comment #5 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 23.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33900 Pedro Amorim <pedro.amorim@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |additional_work_needed --- Comment #6 from Pedro Amorim <pedro.amorim@ptfs-europe.com> --- There's a bug here I think, I'm working on it. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33900 Pedro Amorim <pedro.amorim@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|additional_work_needed | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33900 --- Comment #7 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Thanks for all the hard work! Pushed to 23.05.x for the next release -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33900 --- Comment #8 from Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> --- Nice work everyone! Pushed to oldstable for 22.11.x -- You are receiving this mail because: You are watching all bug changes.