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
[AdminBundle][AdminListBundle] Revert code changes for container access deprecation to make symfony 4 upgrade easier #2167
[AdminBundle][AdminListBundle] Revert code changes for container access deprecation to make symfony 4 upgrade easier #2167
Conversation
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 @acrobat, your PR needs some changes
- your PR title should look like
[SomeBundle] Fixed some code
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 @acrobat, your PR needs some changes
- your PR title should look like
[SomeBundle] Fixed some code
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 @acrobat, your PR needs some changes
- your PR title should look like
[SomeBundle] Fixed some code
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 @acrobat, your PR needs some changes
- your PR title should look like
[SomeBundle] Fixed some code
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 @acrobat, your PR needs some changes
- your PR title should look like
[SomeBundle] Fixed some code
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 @acrobat, your PR passed all our requirements.
Thank you for contributing!
* 'master' of github.com:Kunstmaan/KunstmaanBundlesCMS: [NodeSearchBundle][Tests] fix namespace tests (Kunstmaan#2136) [PagePartBundle][Tests] fix namespace tests and remove old mocks (Kunstmaan#2137) [RedirectBundle][Tests] fix namespace tests (Kunstmaan#2138) [SearchBundle][Tests] fix namespace tests (Kunstmaan#2139) [SeoBundle][Tests] fix namespace tests (Kunstmaan#2140) [SitemapBundle][Tests] fix namespace tests (Kunstmaan#2141) [TaggingBundle] [Tests] fix namespace tests (Kunstmaan#2142) [TranslatorBundle][Tests] fix namespace tests (Kunstmaan#2144) [UsermanagementBundle] [Tests] fix namespace tests (Kunstmaan#2145) [UtilitiesBundle][Tests] fix namespace tests and old mocks (Kunstmaan#2146) [VotingBundle] [Test] fix namespace tests (Kunstmaan#2147) Revert code changes for container access deprecation to make symfony 4 upgrade easier (Kunstmaan#2167) [MediaBundle] Folder types as constants (part 2) (Kunstmaan#2150) [MediaBundle] Folder types as constants (Kunstmaan#2125) [DashboardBundle] Fixed duplicate service id and missing command in command services config (Kunstmaan#2121)
The method we override here is an internal method so it has changes from 3.4 -> 4.0. This makes it currently hard to support both versions without BC breaks. So I suggest to revert this deprecation warning for now (until we can come up with a better fix). We currently have a notice about removing container access in these controller in our upgrade guide so this will do for now.