Skip to content
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

[2.7][SecurityBundle] Remove SecurityContext from Compile #14837

Merged
merged 1 commit into from Jun 2, 2015
Merged

[2.7][SecurityBundle] Remove SecurityContext from Compile #14837

merged 1 commit into from Jun 2, 2015

Conversation

zanbaldwin
Copy link
Member

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? no change
Fixed tickets #14831
License MIT
Doc PR n/a

Remove the deprecated SecurityContext from the list of classes to compile into classes.map and classes.php. Prevents E_USER_DEPRECATED notices from being thrown on every request.

Remove the deprecated SecurityContext from the list of classes to compile into `classes.map` and `classes.php`.
@stof
Copy link
Member

stof commented Jun 2, 2015

👍

@xabbuh xabbuh added the Ready label Jun 2, 2015
@xabbuh
Copy link
Member

xabbuh commented Jun 2, 2015

👍

@fabpot
Copy link
Member

fabpot commented Jun 2, 2015

Thank you @zanderbaldwin.

@fabpot fabpot merged commit 1778e02 into symfony:2.7 Jun 2, 2015
fabpot added a commit that referenced this pull request Jun 2, 2015
…(Zander Baldwin)

This PR was merged into the 2.7 branch.

Discussion
----------

[2.7][SecurityBundle] Remove SecurityContext from Compile

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | no change
| Fixed tickets | #14831
| License       | MIT
| Doc PR        | n/a

Remove the deprecated [SecurityContext](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Security/Core/SecurityContext.php) from the list of classes to compile into `classes.map` and `classes.php`. Prevents `E_USER_DEPRECATED` notices from being thrown on every request.

Commits
-------

1778e02 [2.7][SecurityBundle] Remove SecurityContext from Compile
@rparsi
Copy link

rparsi commented Jun 17, 2015

I've installed version 2.7 but am still seeing this in my php error log. How do I get this fix?

@zanbaldwin
Copy link
Member Author

Remove line 103 from src/Symfony/Bundle/SecurityBundle/DependencyInjection/SecurityExtension.php.
Delete your cache folder after, probably the easiest way.

@stof
Copy link
Member

stof commented Jun 18, 2015

@rparsi update to 2.7.1

@zanbaldwin zanbaldwin deleted the 14831-remove-deprecated-securitycontext branch July 1, 2015 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants