Skip to content
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

Escape % in NonExistingGeneratorException #1565

Merged

Conversation

umpirsky
Copy link
Contributor

No description provided.

@stloyd
Copy link
Contributor

stloyd commented May 29, 2014

👍, but to be honest, I would prefer to delete whole note about configuration from that exception, this is not place for such info.

@umpirsky
Copy link
Contributor Author

umpirsky commented Jun 2, 2014

Lol, someone requested it in previous PR. @pjedrzejewski iirc.

@stloyd
Copy link
Contributor

stloyd commented Jun 2, 2014

IMO there should be just something like: please make sure you have configured your generators correctly everything how to do that should be in docs.

@winzou
Copy link
Contributor

winzou commented Jun 2, 2014

What's wrong with having detailed exception messages? This is way better than nothing or "check the doc", and Symfony itself pushes in that way.

@QuingKhaos
Copy link
Contributor

👍 for the detailed exception. After all it appears only in dev environment and it useful here.

@stloyd
Copy link
Contributor

stloyd commented Jun 17, 2014

@pjedrzejewski ping, what do you think?

pjedrzejewski pushed a commit that referenced this pull request Jun 21, 2014
@pjedrzejewski pjedrzejewski merged commit 4cad306 into Sylius:master Jun 21, 2014
@pjedrzejewski
Copy link
Member

The more descriptive exception, the better, I do not think this one has too much. Thanks Sasha!

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.

None yet

5 participants