Skip to content

[WIP]REFS #10487 - [Console] Fixed Options arguments behaviour #10603

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

Closed
wants to merge 1 commit into from
Closed

[WIP]REFS #10487 - [Console] Fixed Options arguments behaviour #10603

wants to merge 1 commit into from

Conversation

mickaelandrieu
Copy link
Contributor

Q A
Bug fix? yes
New feature? no
BC breaks? yes
Deprecations? no
Tests pass? no
Fixed tickets #10487, #9576
License MIT
Doc PR no

Hi,

I'm actualy working on it.
The main problem is that we can actualy do:
php app/console --foo bar and foo value is bar.

This behavior is not correct with the actual documentation: http://symfony.com/doc/2.3/components/console/introduction.html#using-command-arguments

The options values should be passed with the = operator and don't accept "space" notation described before.

That's why a call like php app/console -v foo bar crashes becauses the actual behavior try to use the next arguments as a value for the current option.

So... do we have to fix the behavior on verbose or remove the "space" notation ?

Regards,

@fabpot
Copy link
Member

fabpot commented Apr 1, 2014

The behavior is correct. You can use --foo bar or --foo=bar or even --foo="bar" and --foo "bar".

@fabpot fabpot closed this Apr 1, 2014
@fabpot
Copy link
Member

fabpot commented Apr 1, 2014

To be more precise, the verbose flag has a "special" and "weird" behavior that we cannot fix unfortunately.

@mickaelandrieu
Copy link
Contributor Author

True :/

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

Successfully merging this pull request may close these issues.

2 participants