[wp-trac] [WordPress Trac] #57967: Regression: Username check introduced in WP 6.2 should allow updates to the same user

WordPress Trac noreply at wordpress.org
Thu Mar 23 13:52:40 UTC 2023


#57967: Regression: Username check introduced in WP 6.2 should allow updates to the
same user
-------------------------------------------------+-------------------------
 Reporter:  polevaultweb                         |       Owner:  audrasjb
     Type:  defect (bug)                         |      Status:  accepted
 Priority:  normal                               |   Milestone:  6.2
Component:  Users                                |     Version:  trunk
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests needs-      |     Focuses:
  testing changes-requested close                |
-------------------------------------------------+-------------------------

Comment (by audrasjb):

 In [changeset:"55584" 55584]:
 {{{
 #!CommitTicketReference repository="" revision="55584"
 Login and Registration: Revert [55358] and [55360].

 This reverts the changes implemented in [55358] and [55360].

 Changeset [55358] was committed to prevent login name collision when one
 user registers with the email address `user at example.com` and a second user
 tries to register with the username `user at example.com`. However, it also
 introduced a potential backward compatibility issues for plugins that use
 `wp_update_user()`. When updating an existing user, it throws an
 `existing_user_email_as_login` error if the email address is also used for
 the user login, due to the code introduced in [55358].

 This changeset removes the new scenario added in [55358] and [55360],
 restoring the `wp_insert_user()` function back to its previous state.

 Props polevaultweb, audrasjb, costdev, peterwilsoncc, hellofromTonya,
 SergeyBiryukov, azaozz.
 See #57967, #57394.
 }}}

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


More information about the wp-trac mailing list