Skip to content

[Routing] Fixed correct class name in thrown exception (fixes #17388) #17389

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
Jan 15, 2016
Merged

[Routing] Fixed correct class name in thrown exception (fixes #17388) #17389

merged 1 commit into from
Jan 15, 2016

Conversation

robinvdvleuten
Copy link
Contributor

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

This fixes the thrown exception. Instead of the complete file contents, the class name is used in the exception message.

@xabbuh
Copy link
Member

xabbuh commented Jan 15, 2016

👍

Status: Reviewed

@paradajozsef
Copy link
Contributor

👍

@fabpot
Copy link
Member

fabpot commented Jan 15, 2016

Thank you @robinvdvleuten.

@fabpot fabpot merged commit c847feb into symfony:2.3 Jan 15, 2016
fabpot added a commit that referenced this pull request Jan 15, 2016
 #17388) (robinvdvleuten)

This PR was merged into the 2.3 branch.

Discussion
----------

[Routing] Fixed correct class name in thrown exception (fixes #17388)

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

This fixes the thrown exception. Instead of the complete file contents, the class name is used in the exception message.

Commits
-------

c847feb Fixed correct class name in thrown exception
@fabpot fabpot mentioned this pull request Feb 3, 2016
This was referenced Feb 28, 2016
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.

5 participants