Skip to content

[Debug] preserve modified error level #10978

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

Conversation

nicolas-grekas
Copy link
Member

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets none
License MIT
Doc PR none

Minor edge case, but still a bug fix.

@Koc
Copy link
Contributor

Koc commented May 23, 2014

Does it fixes #10612 and replaces #10614?

@nicolas-grekas
Copy link
Member Author

No, it's not related. It's for a rare edge case where someone changes the error reporting level while errors are stacked, which currently could happen only while loading a class (which should not execute code but only load the declaration)

fabpot added a commit that referenced this pull request May 25, 2014
This PR was merged into the 2.5 branch.

Discussion
----------

[Debug] preserve modified error level

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | none
| License       | MIT
| Doc PR        | none

Minor edge case, but still a bug fix.
Replaces #10978

Commits
-------

e40b717 [Debug] preserve modified error level
fabpot added a commit to symfony/debug that referenced this pull request May 25, 2014
This PR was merged into the 2.5 branch.

Discussion
----------

[Debug] preserve modified error level

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | none
| License       | MIT
| Doc PR        | none

Minor edge case, but still a bug fix.
Replaces symfony/symfony#10978

Commits
-------

e40b717 [Debug] preserve modified error level
@nicolas-grekas nicolas-grekas deleted the user-error-level branch May 25, 2014 16:40
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.

2 participants