NotificationsYou must be signed in to change notification settings
Fork9.7k
Star30.8k

[Console] Fixed BC bug when mixing short & long options#25574

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Closed

stloyd wants to merge1 commit intosymfony:2.7fromstloyd:patch-6

Closed

[Console] Fixed BC bug when mixing short & long options#25574

stloyd wants to merge1 commit intosymfony:2.7fromstloyd:patch-6

Conversation

Copy link

Contributor

stloyd commentedDec 21, 2017

Q	A
Branch?	2.7+
Bug fix?	yes
New feature?	no
BC breaks?	no
Deprecations?	no
Tests pass?	yes
License	MIT

PR#25487 introduced BC break when call to command was mixing both, short & long options.

v2.7.38 (notice-e=test at end):

$ app/console cache:warmup --no-ansi --no-interaction --no-debug -e=test // Warming up the cachefor thetest environment with debugfalse [OK] Cachefor the"test" environment (debug=false) was successfully warmed.

v2.7.39:

$ app/console cache:warmup --no-ansi --no-interaction --no-debug -e=test      _____                  __     / ____|                / _|| (___  _   _ _ __ ___||_ ___  _ __  _   _\___\||||'_ ` _ \|  _/ _ \|'_\||||     ____)||_||||||||| (_)|||||_|||_____/\__,|_||_||_|_|\___/|_||_|\__,|             __/|                          __/||___/|___/Welcome to the Symfony shell (2.8.32 - app/test).At the prompt,typehelpfor some help,or list to get a list of available commands.Toexit the shell,type ^D.Symfony>