[wp-trac] [WordPress Trac] #33654: wp_new_user_notification breaking change.

WordPress Trac noreply at wordpress.org
Mon Sep 14 12:42:36 UTC 2015


#33654: wp_new_user_notification breaking change.
-------------------------------+----------------------
 Reporter:  alexander.rohmann  |       Owner:  ocean90
     Type:  defect (bug)       |      Status:  closed
 Priority:  normal             |   Milestone:  4.3.1
Component:  Users              |     Version:  4.3
 Severity:  normal             |  Resolution:  fixed
 Keywords:  has-patch commit   |     Focuses:
-------------------------------+----------------------
Changes (by ocean90):

 * status:  accepted => closed
 * resolution:   => fixed


Comment:

 In [changeset:"34116"]:
 {{{
 #!CommitTicketReference repository="" revision="34116"
 Passwords: Deprecate second parameter of `wp_new_user_notification()`.

 The second parameter `$plaintext_pass` was removed in [33023] and restored
 as `$notify` in [33620] with a different behavior. If you have a plugin
 overriding `wp_new_user_notification()` which hasn't been updated you
 would get a notification with your username and the password "both".
 To prevent this the second parameter is now deprecated and reintroduced as
 the third parameter.

 Adds unit tests.

 Props kraftbj, adamsilverstein, welcher, ocean90.
 Fixes #33654.

 (Don't ask for new pluggables kthxbye)
 }}}

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


More information about the wp-trac mailing list