Summary: | Send MEMBERSHIP_EXPIRY notice as print | ||
---|---|---|---|
Product: | Koha | Reporter: | Lucas Gass (lukeg) <lucas> |
Component: | Notices | Assignee: | Jake Deery <jake.deery> |
Status: | Signed Off --- | QA Contact: | Lucas Gass (lukeg) <lucas> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | andrew, lisette, martin.renvoize, nick |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 32216: Add print notices to membership_expiry.pl
Bug 32216: Add print notices to membership_expiry.pl |
Description
Lucas Gass (lukeg)
2022-11-15 21:54:29 UTC
Created attachment 170799 [details] [review] 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 Forgot to update the status on this one. Sorry, folks! Jake. Created attachment 174975 [details] [review] 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> |