Skip to content

[FrameworkBundle] don't register the MailerTestCommand symfony/console is not installed #48416

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

Merged
merged 1 commit into from
Dec 1, 2022

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Dec 1, 2022

Q A
Branch? 6.2
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #48415
License MIT
Doc PR

@fabpot
Copy link
Member

fabpot commented Dec 1, 2022

Thank you @xabbuh.

@fabpot fabpot merged commit 40731aa into symfony:6.2 Dec 1, 2022
@xabbuh xabbuh deleted the issue-48415 branch December 1, 2022 08:52
@featuriz
Copy link

featuriz commented Dec 2, 2022

fabpot merged commit 40731aa into

This not fixed my issue. #48415

I double checked the file src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php

@fabpot fabpot mentioned this pull request Dec 6, 2022
kbond added a commit that referenced this pull request Dec 7, 2022
…ilable (kbond)

This PR was merged into the 6.2 branch.

Discussion
----------

[FrameworkBundle] fix removing commands if console not available

| Q             | A
| ------------- | ---
| Branch?       | 6.2
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | #48415
| License       | MIT
| Doc PR        | n/a

#48416 didn't quite solve and we had the same issue with the new `messenger:stats` command.

Commits
-------

df69810 [FrameworkBundle] fix removing commands if console not available
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants