[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:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  4.3
Component:  Users         |     Version:  trunk
 Severity:  normal        |  Resolution:
 Keywords:  dev-feedback  |     Focuses:
--------------------------+--------------------
Changes (by ocean90):

 * keywords:   => dev-feedback


Old description:

> Originally reported by @Justin_K in [comment:ticket:33209:9].
>
> [https://core.trac.wordpress.org/browser/tags/4.2.4/src/wp-
> includes/pluggable.php?marks=1694-1695#L1670 In 4.2]
> `wp_new_user_notification()` has a second param `$plaintext_pass`. If
> it's not set the user won't get a notification, only the site admin. Now
> in trunk this param has been removed and the user get's always an email,
> see [comment:ticket:33209:9.
>
> * @jeremyfelt: Maybe repurpose the second param as a flag for sending the
> notification instead of plain text? If empty don't send.
> * This should have a changelog entry.
> * @kraft: `wp_new_user_notification()` is still called with the second
> param:
> https://github.com/WordPress/WordPress/search?utf8=%E2%9C%93&q=wp_new_user_notification
> * (This is a pluggable function.)

New description:

 Originally reported by @Justin_K in [comment:ticket:33209:9].

 [https://core.trac.wordpress.org/browser/tags/4.2.4/src/wp-
 includes/pluggable.php?marks=1694-1695#L1670 In 4.2]
 `wp_new_user_notification()` has a second param `$plaintext_pass`. If it's
 not set the user won't get a notification, only the site admin. Now in
 trunk this param has been removed ([33023]) and the user get's always an
 email, see [comment:ticket:33209:9].

 * @jeremyfelt: Maybe repurpose the second param as a flag for sending the
 notification instead of plain text? If empty don't send.
 * This should have a changelog entry.
 * @kraft: `wp_new_user_notification()` is still called with the second
 param:
 https://github.com/WordPress/WordPress/search?utf8=%E2%9C%93&q=wp_new_user_notification
 * (This is a pluggable function.)

--

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