Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reset user password send error when no user is found #271

Merged
merged 10 commits into from Nov 12, 2019

Conversation

@raphael-concil
Copy link
Contributor

raphael-concil commented Oct 18, 2019

This fixes issue #258
The success message was changed also to show the number of successful password reset. This was done to resolve the case when there are both valid and invalid users.

`
$ ./vendor/bin/wp user reset-password 0 invalid-user
Warning: Invalid user ID, email or login: '0'
Warning: Invalid user ID, email or login: 'invalid-user'
Error: No user found to reset password.

$ ./vendor/bin/wp user reset-password 0 invalid-user user1
Warning: Invalid user ID, email or login: '0'
Warning: Invalid user ID, email or login: 'invalid-user'
Reset password for user1.
Success: Password reset for 1 user.
`

zzap and others added 2 commits Sep 13, 2019
This fixes issue #258
The success message was changed also to show the number of successful password reset. This was done to resolve the case when there are both valid and invalid users.
@raphael-concil raphael-concil requested a review from wp-cli/committers as a code owner Oct 18, 2019
thrijith and others added 8 commits Nov 4, 2019
Docs: Disable curl buffer for fetching content example
Fix minor PHPCS violation
This fixes issue #258
The success message was changed also to show the number of successful password reset. This was done to resolve the case when there are both valid and invalid users.
Fix Travis space and quotes errors
@schlessera

This comment has been minimized.

Copy link
Member

schlessera commented Nov 12, 2019

Thanks for the PR, @raphael-concil !

@schlessera schlessera merged commit 3c31e7e into wp-cli:master Nov 12, 2019
1 check failed
1 check failed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
This was referenced Dec 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants
You can’t perform that action at this time.