Skip to content

[FrameworkBundle] Update deprecated service call#12787

Merged
fabpot merged 1 commit intosymfony:2.6from
lrlopez:security-shortcut-fix
Dec 1, 2014
Merged

[FrameworkBundle] Update deprecated service call#12787
fabpot merged 1 commit intosymfony:2.6from
lrlopez:security-shortcut-fix

Conversation

@lrlopez
Copy link
Copy Markdown
Contributor

@lrlopez lrlopez commented Nov 30, 2014

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets none, but related to #11593 and #11690
License MIT
Doc PR none

Service security.context is now deprecated due to PR #11690. This commit updates PR #11593 replacing security.context with security.authorization_checker in the isGranted() controller shortcut introduced in Symfony 2.6.

The service `security.context` is now deprecated due to PR symfony#11690.

This commit updates PR symfony#11593 replacing `security.context` with
`security.authorization_checker` in the `isGranted()` controller
shortcut.
@timglabisch
Copy link
Copy Markdown

👍

@fabpot
Copy link
Copy Markdown
Member

fabpot commented Dec 1, 2014

Thank you @lrlopez.

@fabpot fabpot merged commit 2625193 into symfony:2.6 Dec 1, 2014
fabpot added a commit that referenced this pull request Dec 1, 2014
This PR was merged into the 2.6 branch.

Discussion
----------

[FrameworkBundle] Update deprecated service call

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | none, but related to #11593 and #11690
| License       | MIT
| Doc PR        | none

Service `security.context` is now deprecated due to PR #11690. This commit updates PR #11593 replacing `security.context` with `security.authorization_checker` in the `isGranted()` controller shortcut introduced in Symfony 2.6.

Commits
-------

2625193 [FrameworkBundle] Update deprecated service call
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.

3 participants