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

updated validation.rst #5608

Closed
wants to merge 1 commit into from
Closed

Conversation

issei-m
Copy link
Contributor

@issei-m issei-m commented Aug 9, 2015

I'm not sure this change is correct but FieldError was renamed to FormError, right?

@javiereguiluz
Copy link
Member

The UPGRADE guide doesn't say anything about this change: https://github.com/symfony/symfony/blob/2.8/UPGRADE-2.8.md 😕

@issei-m
Copy link
Contributor Author

issei-m commented Aug 12, 2015

@javiereguiluz I'm not sure where this PR should be merged to, but FieldError class is not found in Symfony project anywhere.
But I'm guessing this class had been introduced in a early stage of development, thereafter renamed.

@issei-m
Copy link
Contributor Author

issei-m commented Aug 12, 2015

ping @webmozart

@wouterj
Copy link
Member

wouterj commented Aug 12, 2015

This change is 100% correct and a left-over from way before Symfony 2.0.0: symfony/form@889a160

So big 👍 and thanks @issei-m for finding & fixing this very old issue.

@wouterj
Copy link
Member

wouterj commented Aug 12, 2015

Thanks again @issei-m, I've now merged it into the docs in version 2.3 (the oldest, still maintained, version) and will merge it up the other versions from there.

wouterj added a commit that referenced this pull request Aug 12, 2015
This PR was submitted for the 2.8 branch but it was merged into the 2.3 branch instead (closes #5608).

Discussion
----------

updated validation.rst

I'm not sure this change is correct but FieldError was renamed to FormError, right?

Commits
-------

74da3cf updated validation.rst
@wouterj wouterj closed this Aug 12, 2015
@issei-m issei-m deleted the fix-validation-doc branch August 16, 2015 13:41
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.

3 participants