-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
service called on wrong variable #5696
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
Conversation
@@ -226,7 +226,7 @@ from the authorization checker is called. | |||
|
|||
$authChecker = $this->get('security.authorization_checker'); | |||
|
|||
$this->denyAccessUnlessGranted('view', $post, 'Unauthorized access!'); | |||
$authChecker->denyAccessUnlessGranted('view', $post, 'Unauthorized access!'); |
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 wrong. the AuthorizationChecker does not have such method
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.
My bad. But I still don't understand why is service instantiated in this example and then not used.
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.
it is a left-over of the time when the controller shortcut was not used in the doc.
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'll just create another pull request for cleanup... |
thank you! please don't forget to reference this PR here |
I don't see why we need this PR at all. The issue should be fixed at all when merging #5711, isn't it? |
This PR was submitted for the 2.8 branch but it was merged into the 2.7 branch instead (closes #5711). Discussion ---------- removed service call from controller | Q | A | ------------- | --- | Doc fix? | [yes] | New docs? | [no] | Applies to | [2.6, 2.7, 2.8] | Fixed tickets | [] As said in #5696 removed left-over unnecessary service call from controller Commits ------- 42fa416 removed service call from controller
@sloba88 Closing here, feel free to reopen if you do not agree. |
No description provided.