[wp-trac] [WordPress Trac] #11210: Split wp_new_user_notification() into two functions

[WordPress Trac]

#11210: Split wp_new_user_notification() into two functions
------------------------------------+-----------------------------
 Reporter:  sirzooro                |       Owner:  westi
     Type:  enhancement             |      Status:  new
 Priority:  normal                  |   Milestone:  Future Release
Component:  Login and Registration  |     Version:  2.9
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |     Focuses:
------------------------------------+-----------------------------
Changes (by jfarthing84):

 * keywords:  has-patch => has-patch dev-feedback


Comment:

 My 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.

 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.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/11210#comment:15>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform