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.

Sign up for GitHub

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

Jump to bottom

Drush 9; Argument 'en' is always replaced with 'pm:enable' #3295

Closed
TessBakker opened this issue Jan 9, 2018 · 6 comments
Closed

Drush 9; Argument 'en' is always replaced with 'pm:enable' #3295

TessBakker opened this issue Jan 9, 2018 · 6 comments
Labels
type-bug

Comments

@TessBakker
Copy link
Contributor

TessBakker commented Jan 9, 2018 •
edited
Loading

If a Drush 9 commands allows arguments, the argument 'en' will be replaced with its full command name pm:enable. In this way it isn't possible to create an argument that accepts the language code 'en'.

Drush 8 doesn't have this issue.

Some test cases:

$ drush @site_alias en 'en'
[error]  Unable to install modules pm:enable due to missing modules pm:enable.

$ drush @site_alias pm:enable 'en'
[error]  Unable to install modules pm:enable due to missing modules pm:enable.
@greg-1-anderson
Copy link
Member

😢 Thought we fixed this. Thanks for reporting.

@weitzman
Copy link
Member

weitzman commented Jan 9, 2018 via email

@TessBakker
Copy link
Contributor Author

@weitzman c905926 a.k.a. 9.0.0-rc2

@weitzman
Copy link
Member

Our attempted fix was #3066. That should help identify where we went wrong.

@Sutharsan Sutharsan mentioned this issue Jan 26, 2018
Merged
@Sutharsan
Copy link
Contributor

It's caused by typo's in #3066

@weitzman
Copy link
Member

Lets talk in the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type-bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@weitzman @Sutharsan @greg-1-anderson @TessBakker