The AutoEmailPrimaryAddress preference no longer only affects AutoEmailNewUser (formerly known as AutoEmailOpacUser). As such I feel it should be clarified and renamed to EmailFieldPrimary to show it is actually being used for the majority of notices now. I chose EmailField* to sit it next to the new preference I attempt to introduce in bug 29046.
Martin, Can you specify what it affects exactly? I'll make a note to add this in the manual... Caroline
The preferece is used in only one routine, Koha::Patron->notice_email_address. But it appears at some point, this method pretty much supplanted Koha::Patron->first_valid_email_address and is used in a lot more of the codebase. notice_email_address is used in: C4::Auth_With_shibboleth C4::Letters (as the default) C4::Auth - For 2FA C4::Auth_with_ldap C4::Reserves shareshelf.pl memberentry.pl opac-registration-varify.pl two_factor_auth.pl notice.pl memberentry.pl pendingreserves.pl Koha::Ticket Patron::Import.pm suggestion.pl overdue_notices.pl TalkingTech_iteive_outbound.pl notice_unprocessed_suggestions.pm first_valid_email is now only used in: clubs-tab.tt waiting_holds.inc transferstoreceive.tt opac-sendbasket.pl Koha::SharedContent And there's likely a few other cases where we refer more directly to the email fields rather than using the methods.
Created attachment 148118 [details] [review] Bug 33192: Update AutoEmailPrimaryAddress to EmailFieldPrimary This patch updates the database to reflect the new name for the AutoEmailPrimaryAddress system preferences
Created attachment 148119 [details] [review] Bug 33192: Update all occurrences of AutoEmailPrimaryAddress This patch updates all references to AutoEmailPrimaryAddress to refer to EmailFieldPrimary instead.
1. Add primary, secondary and alternate email addresses in a patron's account - Primary email = primary@... - Secondary email = secondary@... - Email, under Alternate address = alternate@... 2. Set EmailFieldPrimary to home (primary email) 3. In the patron's account click More > Send welcome email 4. Check the message_queue, the to_address should be primary@... select to_address, message_transport_type, letter_code, time_queued from message_queue where date(time_queued) = curdate(); 5. Set EmailFieldPrimary to work (secondary email) 6. Repeat steps 3 and 4, the to_address should be secondary@... 7. Set EmailFieldPrimary to alternate 8. Repeat steps 3 and 4, the to_address should be alternate@... 9. Set EmailFieldPrimary to cardnumber as 10. Repeat steps 3 and 4, the to_address should be the patron's cardnumber 11. Set EmailFieldPrimary to first valid 12. Repeat steps 3 and 4, the to_address should be primary@... 13. Remove the primary email from the patron's account 14. Repeat steps 3 and 4, the to_address should be secondary@... 15. Remove the secondary email from the patron's account 16. Repeat steps 3 and 4, the to_address should be alternate@... If there are no email addresses at all, the notice is not generated.
Created attachment 148139 [details] [review] Bug 33192: Update AutoEmailPrimaryAddress to EmailFieldPrimary This patch updates the database to reflect the new name for the AutoEmailPrimaryAddress system preferences Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Created attachment 148140 [details] [review] Bug 33192: Update all occurrences of AutoEmailPrimaryAddress This patch updates all references to AutoEmailPrimaryAddress to refer to EmailFieldPrimary instead. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Shouldn't it be EmailAddress instead of Email? It's not a verb, and we have AutoEmailOpacUser where it is used as a verb.
Created attachment 149381 [details] [review] Bug 33192: Update AutoEmailPrimaryAddress to EmailFieldPrimary This patch updates the database to reflect the new name for the AutoEmailPrimaryAddress system preferences Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 149382 [details] [review] Bug 33192: Update all occurrences of AutoEmailPrimaryAddress This patch updates all references to AutoEmailPrimaryAddress to refer to EmailFieldPrimary instead. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 149383 [details] [review] Bug 33192: Fix it-IT sample data and unit tests To test: * prove t/db_dependent/Message.t * Run the it-IT web installer and make sure 'email' is set. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 149384 [details] [review] Bug 33192: (QA follow-up) Fix it-IT sample data and unit tests To test: * prove t/db_dependent/Message.t * Run the it-IT web installer and make sure 'email' is set. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Should not we add the original name in the description, so that the search will return the new one when then old one is searched for?
(In reply to Jonathan Druart from comment #13) > Should not we add the original name in the description, so that the search > will return the new one when then old one is searched for? Formerly known as... ? :) I am not sure to be honest. I hesitate as that will mean that we will probably never get rid of it and we might end up with a lot of those entries over time.
AFAIK, there is no precedent for this... gist -> TaxRates, gist is not mentioned in the description NotesBlacklist -> NotesToHide, no mention RequestOnOpac -> OPACHoldRequests, no mention etc.
Pushed to master for 23.05. Nice work everyone, thanks!
Enhancement - not backporting to 22.11.x Nice work everyone!