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

Reset user password send error when no user is found

Author: schlessera

README.md

@@ -5216,7 +5216,7 @@ wp user reset-password <user>... [--skip-email]
     $ wp user reset-password admin editor
     Reset password for admin.
     Reset password for editor.
-    Success: Passwords reset.
+    Success: Passwords reset for 2 users.
 
 
 

features/user-reset-password.feature

@@ -11,7 +11,7 @@ Feature: Reset passwords for one or more WordPress users.
     Then STDOUT should contain:
       """
       Reset password for admin.
-      Success: Password reset.
+      Success: Password reset for 1 user.
       """
     And an email should be sent
 
@@ -32,7 +32,7 @@ Feature: Reset passwords for one or more WordPress users.
     Then STDOUT should contain:
       """
       Reset password for admin.
-      Success: Password reset.
+      Success: Password reset for 1 user.
       """
     And an email should not be sent
 

src/User_Command.php

@@ -1079,7 +1079,7 @@ public function import_csv( $args, $assoc_args ) {
 	 *     $ wp user reset-password admin editor
 	 *     Reset password for admin.
 	 *     Reset password for editor.
-	 *     Success: Passwords reset.
+	 *     Success: Passwords reset for 2 users.
 	 *
 	 * @subcommand reset-password
 	 */
@@ -1102,7 +1102,15 @@ public function reset_password( $args, $assoc_args ) {
 		if ( $skip_email ) {
 			remove_filter( 'send_password_change_email', '__return_false' );
 		}
-		WP_CLI::success( count( $users ) > 1 ? 'Passwords reset.' : 'Password reset.' );
+
+		$reset_user_count = count( $users );
+		if ( 1 === $reset_user_count ) {
+			WP_CLI::success( "Password reset for {$reset_user_count} user." );
+		} elseif ( $reset_user_count > 1 ) {
+			WP_CLI::success( "Passwords reset for {$reset_user_count} users." );
+		} else {
+			WP_CLI::error( 'No user found to reset password.' );
+		}
 	}
 
 	/**