#2404 closed defect (bug) (fixed)
[patch] Clean up bp_core_signup_send_validation_email()
Reported by: | cnorris23 | Owned by: | DJPaul |
---|---|---|---|
Milestone: | 1.5 | Priority: | normal |
Severity: | Version: | ||
Component: | Core | Keywords: | has-patch |
Cc: |
Description
The function, bp_core_signup_send_validation_email(), contains a lot of code from the function, bp_core_activation_signup_user_notification(), for WPMU. Some of this code is redundant. The $message_headers variable, for one, isn't needed, because wp_mail() gives the exact same results by default. The $from_name and $admin_email variables are already handled by the bp_core_email_from_name_filter() and bp_core_email_from_address_filter, respectively.
The filters for $to, $subject, and $message are named after the WPMU specific function.
It would also be nice to have some extra parameters for the $message filter so that the email can be more easily customized and personalized.
I want to go over emails in a big way, so if that doesn't happen, we'll put something like this on.