Skip to content

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

Closed
wants to merge 3 commits into from
Closed

Conversation

sloba88
Copy link

@sloba88 sloba88 commented Sep 15, 2015

No description provided.

@@ -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!');
Copy link
Member

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

Copy link
Author

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.

Copy link
Member

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.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so shall @sloba88 then remove the left-over @stof ?

@sloba88
Copy link
Author

sloba88 commented Sep 20, 2015

I'll just create another pull request for cleanup...

@OskarStark
Copy link
Contributor

thank you!

please don't forget to reference this PR here

@xabbuh
Copy link
Member

xabbuh commented Sep 23, 2015

I don't see why we need this PR at all. The issue should be fixed at all when merging #5711, isn't it?

xabbuh added a commit that referenced this pull request Sep 23, 2015
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
@xabbuh xabbuh closed this Sep 23, 2015
@xabbuh
Copy link
Member

xabbuh commented Sep 23, 2015

@sloba88 Closing here, feel free to reopen if you do not agree.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants