8000 [TwigBundle] show correct fallback exception template in debug mode by Tobion · Pull Request #12196 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[TwigBundle] show correct fallback exception template in debug mode #12196

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
Oct 10, 2014

Conversation

Tobion
Copy link
Contributor
@Tobion Tobion commented Oct 9, 2014
Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

In debug mode, the exception controller did not show the correct template in case the format is unknown.
It falls back to html but the logic for exception_full was not applied there. So it was just returning the partial html.

@stof
Copy link
Member
stof commented Oct 10, 2014

👍

@fabpot
Copy link
Member
fabpot commented Oct 10, 2014

Thank you @Tobion.

@fabpot fabpot merged commit b97acd9 into symfony:2.3 Oct 10, 2014
fabpot added a commit that referenced this pull request Oct 10, 2014
…ebug mode (Tobion)

This PR was merged into the 2.3 branch.

Discussion
----------

[TwigBundle] show correct fallback exception template in debug mode

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

In debug mode, the exception controller did not show the correct template in case the format is unknown.
It falls back to html but the logic for `exception_full` was not applied there. So it was just returning the partial html.

Commits
-------

b97acd9 [TwigBundle] show correct fallback exception template in debug mode
@Tobion Tobion deleted the exceptioncontroller-tpl branch October 10, 2014 17:07
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
0