[wp-trac] [WordPress Trac] #38763: Multisite sign-up notifications missing user login

[WordPress Trac]

#38763: Multisite sign-up notifications missing user login
------------------------------+------------------------
 Reporter:  johnjamesjacoby   |       Owner:  ocean90
     Type:  defect (bug)      |      Status:  closed
 Priority:  high              |   Milestone:  4.7
Component:  I18N              |     Version:  trunk
 Severity:  normal            |  Resolution:  fixed
 Keywords:  has-patch commit  |     Focuses:  multisite
------------------------------+------------------------
Changes (by ocean90):

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


Comment:

 In [changeset:"39203"]:
 {{{
 #!CommitTicketReference repository="" revision="39203"
 Multisite: Rename the `$user` parameter to `$user_login` in
 `wpmu_signup_blog_notification()` and `wpmu_signup_user_notification()`.

 [38961] added a `$user = get_user_by( 'login', $user );` call which was
 overriding the `$user` string variable for the login name. The change to
 `$user_login` makes this clear.

 Props johnjamesjacoby.
 Fixes #38763.
 }}}

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