#2404 closed defect (bug) (fixed)
[patch] Clean up bp_core_signup_send_validation_email()
| Reported by: | cnorris23 | Owned by: | DJPaul |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.5 |
| Component: | Core | Version: | |
| Severity: | 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
I want to go over emails in a big way, so if that doesn't happen, we'll put something like this on.