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
[DashboardBundle] Deprecated service container usage in dashboard com… #2022
[DashboardBundle] Deprecated service container usage in dashboard com… #2022
Conversation
deZinc
commented
Jun 15, 2018
•
edited by Devolicious
edited by Devolicious
Q | A |
---|---|
Bug fix? | no |
New feature? | no |
BC breaks? | no |
Deprecations? | yes |
Fixed tickets |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @deZinc, your PR needs some changes
- It seems that our checklist is missing or incomplete
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @deZinc, your PR needs some changes
- This PR seems to need a milestone of a minor release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @deZinc, your PR passed all our requirements.
Thank you for contributing!
UPGRADE-5.1.md
Outdated
DashboardBundle | ||
--------------- | ||
|
||
* Injecting the container in the `DashboardCommand` is deprecated and will be removed in 6.0. Inject the required parameters instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The upgrade log should be more like this, as the current message is not correct (the container is not injected in the commands, first argument is actually an optional command name)
`XXXXCommand::__construct()` now takes an instance of `XXXX` as the first argument. Not passing it is deprecated and will throw a `TypeError` in 6.0.
Same for the other upgrade notes