Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[Console] Fix autocompletion of argument with default value#44730

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

Merged
nicolas-grekas merged 1 commit intosymfony:5.4fromGromNaN:issue-44505
Dec 26, 2021

Conversation

@GromNaN
Copy link
Member

@GromNaNGromNaN commentedDec 20, 2021
edited
Loading

QA
Branch?5.4
Bug fix?yes
New feature?no
Deprecations?no
TicketsFix#44505
LicenseMIT
Doc PR-

Also fix completion for thehelp command when the command name have not been started.

TheInput::getArguments method merges input arguments with default values; which is not desired for completion. This fix uses the rawInput::$arguments property instead.

publicfunctiongetArguments()
{
returnarray_merge($this->definition->getArgumentDefaults(),$this->arguments);
}

@carsonbotcarsonbot added this to the5.4 milestoneDec 20, 2021
@carsonbotcarsonbot changed the title[Console] Fix autocompletion of argument with default value[Console]  Fix autocompletion of argument with default valueDec 20, 2021
@nicolas-grekasnicolas-grekas changed the title[Console]  Fix autocompletion of argument with default value[Console] Fix autocompletion of argument with default valueDec 26, 2021
@nicolas-grekas
Copy link
Member

Thank you@GromNaN.

@nicolas-grekasnicolas-grekas merged commit4ee271b intosymfony:5.4Dec 26, 2021
@GromNaNGromNaN deleted the issue-44505 branchDecember 26, 2021 17:11
This was referencedDec 29, 2021
dmolineus added a commit to phpcq/phpcq that referenced this pull requestJan 12, 2022
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@chalasrchalasrAwaiting requested review from chalasrchalasr is a code owner

@wouterjwouterjAwaiting requested review from wouterj

Assignees

No one assigned

Projects

None yet

Milestone

5.4

Development

Successfully merging this pull request may close these issues.

3 participants

@GromNaN@nicolas-grekas@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp