Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #11210, comment 15


Ignore:
Timestamp:
06/11/2015 03:34:03 PM (9 years ago)
Author:
jfarthing84
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #11210, comment 15

    initial v1  
    11My patch creates two new functions: {{{wp_new_user_registration_notification()}}} and {{{wp_new_user_registration_admin_notification()}}}. The current {{{wp_new_user_notification()}}} simply becomes a wrapper function for these two functions. Additionally, I added filters to both functions: one to add the possibility to disable the notification, one to modify the e-mail subject and one to modify the e-mail message.
    22
    3 Ideally, I'd like to see the {{{wp_new_user_notification()}}} call within {{{register_new_user()}}} be replaced with an action, such as {{{new_user_registered}}}, and then the two new functions hooked to it as default actions, rendering {{{wp_new_user_notification()}}} deprecated. However, I know {{{wp_new_user_notification()}}} is used in other places, hence why I didn't propose this route in my patch.
     3Ideally, I'd like to see the {{{wp_new_user_notification()}}} call within {{{register_new_user()}}} be replaced with an action, such as {{{new_user_registered}}}, and then the two new functions hooked to it as default actions, rendering {{{wp_new_user_notification()}}} deprecated. However, I know {{{wp_new_user_notification()}}} is used in other places, hence why I didn't propose this route in my patch. This is actually why I added the filters to disable them instead.