Fix #4541: Add again uid and mail options for user commands

[obriat]

Fix #4541: Add again uid and mail options for user command.

[q0rban]

This would be nice!

[q0rban]

As a workaround, you can do things like this:

vendor/bin/drush user:block "$(vendor/bin/drush user:information --uid=1 --fields=name --format=string)"

[deviantintegral]

We discussed the above, and one issue we have is that it's not very discoverable for junior developers or those unfamiliar with the shell. Making it an argument exposes it in the help docs, where the above requires understanding of shell string substitution.

@weitzman I see you closed the related issue. Is there anything you would like to see in order to get a feature like this merged in?

[weitzman]

In general, this looks great. I am looking at our more strict validation in user:login and wondering if the warnings proposed here are sufficient. See

drush/src/Commands/core/LoginCommands.php

Lines 55 to 70 in 2cb8999

	$account = null;
	if (!is_null($options['name']) && !$account = user_load_by_name($options['name'])) {
	throw new \Exception(dt('Unable to load user by name: !name', ['!name' => $options['name']]));
	}
	if (!is_null($options['uid']) && !$account = User::load($options['uid'])) {
	throw new \Exception(dt('Unable to load user by uid: !uid', ['!uid' => $options['uid']]));
	}
	if (!is_null($options['mail']) && !$account = user_load_by_mail($options['mail'])) {
	throw new \Exception(dt('Unable to load user by mail: !mail', ['!mail' => $options['mail']]));
	}
	if (empty($account)) {
	$account = User::load(1);
	}

[weitzman]

Sorry for the notification noise while I coax CircleCI run its tests on this PR.

[miromichalicka]

@weitzman sorry to bother you, but could this be backported to 10.x as well since it's still a stable release? Seems to be mergeable without any changes.