-
-
Notifications
You must be signed in to change notification settings - Fork 156
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
Check the attribute type the BackendAccessVoter::supports() method #1013
Check the attribute type the BackendAccessVoter::supports() method #1013
Conversation
@Toflar The tests all seem to fail here because of the DebugClassLoader used in |
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 agree with fixing this, it does not harm and might upstream-fix Symfony for now
@AndreasA Thank you for your pull request. Can you please rebase it onto the latest master branch? This should fix the CI issues. |
Rebased to latest master and now pipelines succeeded. |
Thank you @AndreasA. |
Ensure that the e.g. ExpressionLanguage objects are ignored when checking for support in the BackendAccessVoter.
Fixes: contao/core-bundle#1710
See also previous PR (and the discussions) for core-bundle: contao/core-bundle#1711
Even if it is a bug in Symfony in the end, I think we should still add the check (at least until we have to update the Access Voter anyway to include the missing type hinting.