[Bug 29555] New: Auto-renewals digest reports on 'auto_too_soon'
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29555 Bug ID: 29555 Summary: Auto-renewals digest reports on 'auto_too_soon' Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Circulation Assignee: koha-bugs@lists.koha-community.org Reporter: nick@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle.m.hall@gmail.com When a patron has requested auto-renewal digests the cron sends emails when all items are 'auto_too_soon' -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29555 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- 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=29555 --- Comment #1 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 127929 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127929&action=edit Bug 29555: Don't push auto_too_soon issues into report This patch adds a conditional to avoid reporting on issues that are too soon to be renewed To test: 1 - Set AutoRenewalNotices preference to 'according to patron messaging preferences' 2 - Set a patront o request auto renewal emails digested 3 - Checkout an item to the patron, back dated to be opverdue, and marked for auto renewal 4 - Add an email to the patron for testing 5 - perl misc/cronjobs/automatic_renewals.pl -v -c 6 - See the issue is renewed (ensure it is now not overdue) 7 - Confirm the patron was sent a digest with the renewal (on patron 'Notices' tab) 8 - perl misc/cronjobs/automatic_renewals.pl -v -c 9 - See the issue was not renewed 10 - See a notice was sent, with no indication of what happened to the issue 11 - Apply patch 12 - perl misc/cronjobs/automatic_renewals.pl -v -c 13 - Confrim no renewal or notice 14 - Checkout a second item to patron, backdated and set for auto renewal 15 - perl misc/cronjobs/automatic_renewals.pl -v -c 16 - Confirm the correct item is renewed and notice sent 17 - Confirm notice does not include the too soon issue 18 - Checkout a third item 19 - perl misc/cronjobs/automatic_renewals.pl -v -c -b 20 - Confirm correct item is renewed and notice sent, nbot including other two 21 - perl misc/cronjobs/automatic_renewals.pl -v -c -b 22 - Confirm no notice sent -- 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=29555 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- 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=29555 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #127929|0 |1 is obsolete| | --- Comment #2 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Created attachment 127934 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127934&action=edit Bug 29555: Don't push auto_too_soon issues into report This patch adds a conditional to avoid reporting on issues that are too soon to be renewed To test: 1 - Set AutoRenewalNotices preference to 'according to patron messaging preferences' 2 - Set a patront o request auto renewal emails digested 3 - Checkout an item to the patron, back dated to be opverdue, and marked for auto renewal 4 - Add an email to the patron for testing 5 - perl misc/cronjobs/automatic_renewals.pl -v -c 6 - See the issue is renewed (ensure it is now not overdue) 7 - Confirm the patron was sent a digest with the renewal (on patron 'Notices' tab) 8 - perl misc/cronjobs/automatic_renewals.pl -v -c 9 - See the issue was not renewed 10 - See a notice was sent, with no indication of what happened to the issue 11 - Apply patch 12 - perl misc/cronjobs/automatic_renewals.pl -v -c 13 - Confrim no renewal or notice 14 - Checkout a second item to patron, backdated and set for auto renewal 15 - perl misc/cronjobs/automatic_renewals.pl -v -c 16 - Confirm the correct item is renewed and notice sent 17 - Confirm notice does not include the too soon issue 18 - Checkout a third item 19 - perl misc/cronjobs/automatic_renewals.pl -v -c -b 20 - Confirm correct item is renewed and notice sent, nbot including other two 21 - perl misc/cronjobs/automatic_renewals.pl -v -c -b 22 - Confirm no notice sent Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29555 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au Resolution|--- |DUPLICATE Status|Signed Off |RESOLVED --- Comment #3 from David Cook <dcook@prosentient.com.au> --- Marking this as a duplicate of bug 29381 since it's older and signed off (also Andrew can't sign off Nick's patches). Although after comparing the patches on both issues... I'm not sure that either is 100% correct... *** This bug has been marked as a duplicate of bug 29381 *** -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org