[Bug 32216] New: Send MEMBERSHIP_EXPIRY notice as print
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Bug ID: 32216 Summary: Send MEMBERSHIP_EXPIRY notice as print Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Notices Assignee: koha-bugs@lists.koha-community.org Reporter: lucas@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org This is bug 23773 but for print versions of MEMBERSHIP_EXPIRY -- 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=32216 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com Assignee|koha-bugs@lists.koha-commun |jake.deery@ptfs-europe.com |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=32216 --- Comment #1 from Jake Deery <jake.deery@ptfs-europe.com> --- Created attachment 170799 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=170799&action=edit Bug 32216: Add print notices to membership_expiry.pl This patch adds print notice functionality to the membership_expiry.pl script, by adding both conditional and and forced modes. When passing only --confirm, the script will check the patron's record for an email address. If one is found, an email notice is generated. If one is not, a print notice is used instead. By adding a -p flag, print notices will be generated, even if an email address is present. To test: a) set the MembershipExpiryDaysNotice syspref to 7 days b) choose or set up half a dozen patrons with expiry dates 1) take heed: these expiry dates will need to be T+7 days c) add fake email addresses to two of the patrons 1) take heed: the other patrons should have no primary or secondary email d) set up the MEMBERSHIP_EXPIRY notice a) set content & subject in the email notice to contain the word email b) set content & subject in the print notice to contain the word print e) run ./misc/cronjobs/membership_expiry.pl -n -c 1) notice how the printed notices contain the word email f) run ./misc/cronjobs/membership_expiry.pl -c g) check the notices for the patrons from step b 1) notice how only email notices are generated h) APPLY PATCH i) repeat step e 1) notice how a mix of email and print notices are now generated j) run ./misc/cronjobs/membership_expiry.pl -n -p -c 1) notice how the output is the same as in step i - this is by design k) repeat steps f-g 1) notice how some patrons will only get an email notice, whilst others will only get a print notice 2) this should occur based on which patrons do, or do not have a valid primary or secondary email l) run ./misc/cronjobs/membership_expiry.pl -p -c m) repeat step g 1) notice that all patrons now get a print notice, whilst some get both a print notice, and an email notice. Again, this should occur based on whether the patron has a valid primary or secondary email n) SIGN OFF -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Jake Deery <jake.deery@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 --- Comment #2 from Jake Deery <jake.deery@ptfs-europe.com> --- Forgot to update the status on this one. Sorry, folks! Jake. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Jake Deery <jake.deery@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |lucas@bywatersolutions.com |y.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 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.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #170799|0 |1 is obsolete| | --- Comment #3 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Created attachment 174975 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=174975&action=edit Bug 32216: Add print notices to membership_expiry.pl This patch adds print notice functionality to the membership_expiry.pl script, by adding both conditional and and forced modes. When passing only --confirm, the script will check the patron's record for an email address. If one is found, an email notice is generated. If one is not, a print notice is used instead. By adding a -p flag, print notices will be generated, even if an email address is present. To test: a) set the MembershipExpiryDaysNotice syspref to 7 days b) choose or set up half a dozen patrons with expiry dates 1) take heed: these expiry dates will need to be T+7 days c) add fake email addresses to two of the patrons 1) take heed: the other patrons should have no primary or secondary email d) set up the MEMBERSHIP_EXPIRY notice a) set content & subject in the email notice to contain the word email b) set content & subject in the print notice to contain the word print e) run ./misc/cronjobs/membership_expiry.pl -n -c 1) notice how the printed notices contain the word email f) run ./misc/cronjobs/membership_expiry.pl -c g) check the notices for the patrons from step b 1) notice how only email notices are generated h) APPLY PATCH i) repeat step e 1) notice how a mix of email and print notices are now generated j) run ./misc/cronjobs/membership_expiry.pl -n -p -c 1) notice how the output is the same as in step i - this is by design k) repeat steps f-g 1) notice how some patrons will only get an email notice, whilst others will only get a print notice 2) this should occur based on which patrons do, or do not have a valid primary or secondary email l) run ./misc/cronjobs/membership_expiry.pl -p -c m) repeat step g 1) notice that all patrons now get a print notice, whilst some get both a print notice, and an email notice. Again, this should occur based on whether the patron has a valid primary or secondary email n) SIGN OFF Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Martin Renvoize (ashimema) <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andrew@bywatersolutions.com | |, | |lisette@bywatersolutions.co | |m, | |nick@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Martin Renvoize (ashimema) <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #174975|0 |1 is obsolete| | --- Comment #4 from Martin Renvoize (ashimema) <martin.renvoize@ptfs-europe.com> --- Created attachment 176745 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176745&action=edit Bug 32216: Add print notices to membership_expiry.pl This patch adds print notice functionality to the membership_expiry.pl script, by adding both conditional and and forced modes. When passing only --confirm, the script will check the patron's record for an email address. If one is found, an email notice is generated. If one is not, a print notice is used instead. By adding a -p flag, print notices will be generated, even if an email address is present. To test: a) set the MembershipExpiryDaysNotice syspref to 7 days b) choose or set up half a dozen patrons with expiry dates 1) take heed: these expiry dates will need to be T+7 days c) add fake email addresses to two of the patrons 1) take heed: the other patrons should have no primary or secondary email d) set up the MEMBERSHIP_EXPIRY notice a) set content & subject in the email notice to contain the word email b) set content & subject in the print notice to contain the word print e) run ./misc/cronjobs/membership_expiry.pl -n -c 1) notice how the printed notices contain the word email f) run ./misc/cronjobs/membership_expiry.pl -c g) check the notices for the patrons from step b 1) notice how only email notices are generated h) APPLY PATCH i) repeat step e 1) notice how a mix of email and print notices are now generated j) run ./misc/cronjobs/membership_expiry.pl -n -p -c 1) notice how the output is the same as in step i - this is by design k) repeat steps f-g 1) notice how some patrons will only get an email notice, whilst others will only get a print notice 2) this should occur based on which patrons do, or do not have a valid primary or secondary email l) run ./misc/cronjobs/membership_expiry.pl -p -c m) repeat step g 1) notice that all patrons now get a print notice, whilst some get both a print notice, and an email notice. Again, this should occur based on whether the patron has a valid primary or secondary email n) SIGN OFF Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Martin Renvoize (ashimema) <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Martin Renvoize (ashimema) <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|lucas@bywatersolutions.com |martin.renvoize@ptfs-europe | |.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Jake Deery <jake.deery@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #176745|0 |1 is obsolete| | --- Comment #5 from Jake Deery <jake.deery@ptfs-europe.com> --- Created attachment 177740 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177740&action=edit Bug 32216: Add print notices to membership_expiry.pl This patch adds print notice functionality to the membership_expiry.pl script, by adding both conditional and and forced modes. When passing only --confirm, the script will check the patron's record for an email address. If one is found, an email notice is generated. If one is not, a print notice is used instead. By adding a -p flag, print notices will be generated, even if an email address is present. To test: a) set the MembershipExpiryDaysNotice syspref to 7 days b) choose or set up half a dozen patrons with expiry dates 1) take heed: these expiry dates will need to be T+7 days c) add fake email addresses to two of the patrons 1) take heed: the other patrons should have no primary or secondary email d) set up the MEMBERSHIP_EXPIRY notice a) set content & subject in the email notice to contain the word email b) set content & subject in the print notice to contain the word print e) run ./misc/cronjobs/membership_expiry.pl -n -c 1) notice how the printed notices contain the word email f) run ./misc/cronjobs/membership_expiry.pl -c g) check the notices for the patrons from step b 1) notice how only email notices are generated h) APPLY PATCH i) repeat step e 1) notice how a mix of email and print notices are now generated j) run ./misc/cronjobs/membership_expiry.pl -n -p -c 1) notice how the output is the same as in step i - this is by design k) repeat steps f-g 1) notice how some patrons will only get an email notice, whilst others will only get a print notice 2) this should occur based on which patrons do, or do not have a valid primary or secondary email l) run ./misc/cronjobs/membership_expiry.pl -p -c m) repeat step g 1) notice that all patrons now get a print notice, whilst some get both a print notice, and an email notice. Again, this should occur based on whether the patron has a valid primary or secondary email n) SIGN OFF Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 --- Comment #6 from Jake Deery <jake.deery@ptfs-europe.com> --- Rebased on main -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |release-notes-needed --- Comment #7 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Jake, this does a bit more than the bug description advertises. Can you please write a little summary for release notes? (new option for the script etc.) At some point we need to clean up all the fallback behavior for print notices and the like. There should be a way for libraries to determine what should create a print notice if no email... but that's out of scope here. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to main Version(s)| |25.05.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 --- Comment #8 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Pushed for 25.05! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|release-notes-needed | Text to go in the| |This enhancement adds print release notes| |notice functionality to the | |membership_expiry.pl | |script, by adding both | |conditional and and forced | |modes. | |When passing only | |--confirm, the script will | |check the patron's record | |for an email address. | |If | |one is found, an email | |notice is generated. If one | |is not, a print notice is | |used instead. | |By adding a | |-p flag, print notices will | |be generated, even if an | |email address is present. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to main |Needs documenting CC| |fridolin.somers@biblibre.co | |m --- Comment #9 from Fridolin Somers <fridolin.somers@biblibre.com> --- Enhancement not pushed to 24.11.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32216 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs documenting |RESOLVED CC| |caroline.cyr-la-rose@inlibr | |o.com Resolution|--- |FIXED --- Comment #10 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- This was reverted by bug 30301. If any documentation is needed, it will be done on the other one. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org