Skip to content
This repository was archived by the owner on Sep 24, 2018. It is now read-only.

Throw an error when a user tries to update to an existing user's email #705

Merged
danielbachhuber merged 5 commits into from
Dec 10, 2014
Merged

Throw an error when a user tries to update to an existing user's email #705

danielbachhuber merged 5 commits into from
Dec 10, 2014

Conversation

@danielbachhuber
Copy link
Member

@danielbachhuber danielbachhuber commented Dec 9, 2014

Fixes #608

@danielbachhuber danielbachhuber added this to the 2.0 milestone Dec 9, 2014
@danielbachhuber
Copy link
Member Author

danielbachhuber commented Dec 9, 2014

Need to:

  • Check whether user_login or user_nicename fall into this same category.
  • Open a core ticket about returning WP_Error from wp_update_user() when user_email is the same as an existing user.

@danielbachhuber danielbachhuber mentioned this pull request Dec 9, 2014
Closed
@danielbachhuber
Copy link
Member Author

danielbachhuber commented Dec 10, 2014

Handling the two tasks in a new PR.

danielbachhuber added a commit that referenced this pull request Dec 10, 2014
@danielbachhuber
Merge pull request #705 from WP-API/fix-608
77e068d 
Throw an error when a user tries to update to an existing user's email
@danielbachhuber danielbachhuber merged commit 77e068d into develop Dec 10, 2014
@danielbachhuber danielbachhuber deleted the fix-608 branch December 10, 2014 00:30
@danielbachhuber danielbachhuber mentioned this pull request Dec 10, 2014
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

Bug

Projects

None yet

Milestone

2.0

Development

Successfully merging this pull request may close these issues.

While editing an user, there is no checking of previously existing email

1 participant

@danielbachhuber