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

Drush user:login cumbersome to use #4020

Closed
BramDriesen opened this issue Mar 28, 2019 · 5 comments
Closed

Drush user:login cumbersome to use #4020

BramDriesen opened this issue Mar 28, 2019 · 5 comments

Comments

@BramDriesen
Copy link
Contributor

BramDriesen commented Mar 28, 2019

Describe the bug
While I understand you can use a drush command inside a drush command likse so: drush user:login --browser=firefox --name=$(drush user:information --mail="drush@example.org" --fields=name --format=string)

This is very cumbersome to use and requires a lot of typing. Therefore I'm asking why the --uid and --mail options are removed in drush 9.

Expected behavior
drush uli --uid=123
drush uli --mail=foo@bar.com

Workaround
Using commands in commands.

System Configuration

Q A
Drush version? 9.3.0
Drupal version? 8.6.5
PHP version 7.1
OS? Mac/Linux
@BramDriesen BramDriesen changed the title Drush userl:l Drush user:login cumbersome to use Mar 28, 2019
@BramDriesen
Copy link
Contributor Author

It's quite easy to implement this back into the command from what I can see looking at the code.

@weitzman
Copy link
Member

Yeah, seems like those could two be added

@BramDriesen
Copy link
Contributor Author

Okay, I'll port them to Drush 9 :)

weitzman pushed a commit that referenced this issue Mar 29, 2019
…#4021)

* #4020 Implemented the command changes

* #4020 Added extra tests for the new command options

* #4020 Fixed phpcs remark.

* #4020 Forgot to add the options to the options array.

* #4020 And fixed phpcs again.
@BramDriesen
Copy link
Contributor Author

Awesome 🎉

@obriat
Copy link
Contributor

obriat commented Aug 27, 2020

This also should be done for all user commands. At least for user:block & user:unblock, if you want to script disabling uid 1, specially if you already change is name. => #4541

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants