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

Skip to content

Fix exception controller config#9344

Merged
pamil merged 1 commit into
Sylius:1.0from
teohhanhui:fix/exception-controller
Apr 25, 2018
Merged

Fix exception controller config#9344
pamil merged 1 commit into
Sylius:1.0from
teohhanhui:fix/exception-controller

Conversation

@teohhanhui
Copy link
Copy Markdown
Contributor

@teohhanhui teohhanhui commented Apr 16, 2018

Q A
Branch? 1.0
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets N/A
License MIT

The exception controller was configured incorrectly. You can easily tell by going to /app_dev.php/_error/404.html

@teohhanhui
Copy link
Copy Markdown
Contributor Author

ping @pamil

Comment thread app/config/config.yml

fos_rest:
exception: ~
exception: true
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didnt need this change in order to make exceptions work. Is it used on the Api?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is for the API.

prefix: /_profiler

_errors:
resource: '@TwigBundle/Resources/config/routing/errors.xml'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

using “ instead of ‘ would be better, to maintain consistency on config files

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is copied straight from Symfony Standard. I suggest not to change it for now.

@pamil pamil merged commit f2cb788 into Sylius:1.0 Apr 25, 2018
@pamil
Copy link
Copy Markdown
Contributor

pamil commented Apr 25, 2018

Thank you, Teoh, and sorry for the delay! :)

@teohhanhui teohhanhui deleted the fix/exception-controller branch April 25, 2018 08:23
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