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
Add Symfony 6 support #778
Add Symfony 6 support #778
Conversation
Requires: sonata-project/SonataDoctrineMongoDBAdminBundle#699 Edit: Done, it is merged. |
c7d2d43
to
50a1181
Compare
Some code need to be fixed. |
Maybe after this one we can do the 4.0.0 final release? @core23 |
@@ -74,14 +74,14 @@ public function listAction(Request $request): Response | |||
|
|||
public function treeAction(Request $request): Response | |||
{ | |||
$categoryManager = $this->get('sonata.classification.manager.category'); | |||
$categoryManager = $this->container->get('sonata.classification.manager.category'); |
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.
Do we have a test that covers this action?
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.
I dont know, but the get method was deprecated on 5.4, and removed on 6.0
Edit: I would say probably because test were failing without this change
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.
This is not related to this PR, but we had some problem with some admin actions.
I'll try to add some basic tests to cover this this before releasing the stable release.
Subject
I am targeting this branch, because this cannot be done on lower branch.
Part of: sonata-project/dev-kit#1675
Dev-kit PR: sonata-project/dev-kit#1786
Changelog