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 undefined index for inconsistent command name definition#38024

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
fabpot merged 1 commit intosymfony:3.4fromchalasr:cmd-multi-name
Sep 2, 2020

Conversation

@chalasr
Copy link
Member

QA
Branch?3.4
Bug fix?yes
New feature?no
Deprecations?no
TicketsFixes#38015
LicenseMIT
Doc PR-

The issue happens when the command name is set via construct/setName() and is routed via a command loader under a different name, which causesApplication::get(): Command to return null (return type violation) with a notice. This makes it throws a proper CommandNotFoundException as expected.

@fabpot
Copy link
Member

Thank you@chalasr.

@fabpotfabpot merged commit33cacad intosymfony:3.4Sep 2, 2020
This was referencedSep 2, 2020
@chalasrchalasr deleted the cmd-multi-name branchSeptember 2, 2020 11:05
@fabpotfabpot mentioned this pull requestSep 27, 2020
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@fabpotfabpotfabpot approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@chalasr@fabpot@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp