[wp-trac] [WordPress Trac] #33358: Passwords: Second param of wp_new_user_notification() has been removed

[WordPress Trac]

#33358: Passwords: Second param of wp_new_user_notification() has been removed
-------------------------------------+--------------------------
 Reporter:  ocean90                  |       Owner:  markjaquith
     Type:  defect (bug)             |      Status:  reviewing
 Priority:  normal                   |   Milestone:  4.3
Component:  Users                    |     Version:  trunk
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:
-------------------------------------+--------------------------

Comment (by valendesigns):

 > We could add 'admin' as an alias for '' (empty, which will skip the user
 e-mail). And then for back compat, any other non-empty value means 'both'
 (as the patch currently does). Doing that now.

 Sounds good to me.

 > I'm also not worried about this being pluggable. Things overriding
 pluggable functions are on their own.

 I agree, you're on your own if you want to override pluggable functions.

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