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

Make --format=list work with commands such as core-status #136

Closed
greg-1-anderson opened this issue Sep 19, 2013 · 3 comments
Closed

Make --format=list work with commands such as core-status #136

greg-1-anderson opened this issue Sep 19, 2013 · 3 comments
Assignees

Comments

@greg-1-anderson
Copy link
Member

This is related to #130.

Works:

$ drush status 'Drupal root' --format=list

Does not work:

$ drush status --fields=root --format=list

These should be the same. The first works, because it first filters the result rows down to just one consisting of a simple string. The second fails, because it attempts to validate to output from the status command first, and finds values that are arrays. The key-value format flattens these arrays into space-delimited strings, whereas the 'list' format does not support this.

@weitzman
Copy link
Member

This is fixed now?

@greg-1-anderson
Copy link
Member Author

Nope. --list is still failing if the command output contains any values that are not simple strings. key-value handles this by imploding the array. This issue is to unify the implementation of the list and key-value output formats, so they behave the same. I think the other issue can be committed, though, as this is kind of an edge case.

@ghost ghost assigned greg-1-anderson Sep 20, 2013
@greg-1-anderson greg-1-anderson added this to the Drush7 milestone Feb 6, 2015
@greg-1-anderson greg-1-anderson removed this from the Drush7 milestone Mar 17, 2015
@greg-1-anderson
Copy link
Member Author

Drush 9 and the output formatters project does this correctly. Closing.

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

No branches or pull requests

2 participants