Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[FrameworkBundle][HttpKernel] Remove remaining $triggerDeprecation flag #16992

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

Merged
merged 1 commit into from
Dec 13, 2015
Merged

[FrameworkBundle][HttpKernel] Remove remaining $triggerDeprecation flag #16992

merged 1 commit into from
Dec 13, 2015

Conversation

ogizanagi
Copy link
Contributor

Q A
Fixed tickets n/a
License MIT

Introduced in #14665, this flag controlled the deprecation notice triggering when using the ContainerAwareHttpKernel, in order to avoid triggering it when using this class in the Symfony framework. This class no longer exists now, neither the flag.

@Tobion
Copy link
Contributor

Tobion commented Dec 13, 2015

Good catch, thanks @ogizanagi.

@Tobion Tobion merged commit ac15454 into symfony:3.0 Dec 13, 2015
Tobion added a commit that referenced this pull request Dec 13, 2015
…eprecation flag (ogizanagi)

This PR was merged into the 3.0 branch.

Discussion
----------

[FrameworkBundle][HttpKernel] Remove remaining $triggerDeprecation flag

| Q             | A
| ------------- | ---
| Fixed tickets | n/a
| License       | MIT

Introduced in #14665, this flag controlled the deprecation notice triggering when using the `ContainerAwareHttpKernel`, in order to avoid triggering it when using this class in the Symfony framework. This class no longer exists now, neither the flag.

Commits
-------

ac15454 [FrameworkBundle][HttpKernel] Remove remaining $triggerDeprecation flag
@ogizanagi ogizanagi deleted the http_kernel_remove_deprecation_triggering_flag branch December 13, 2015 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants