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

Add support for validation from metadata (e.g., annotations) #10

Merged
merged 1 commit into from
Dec 11, 2017

Conversation

ddeboer
Copy link
Member

@ddeboer ddeboer commented Dec 9, 2017

Add support for validation constraints from annotations (or other validation metadata sources).

Fix portphp/portphp#57.

@Baachi Baachi merged commit 05965b9 into master Dec 11, 2017
@ddeboer ddeboer deleted the validate-from-metadata branch December 11, 2017 07:25
ddeboer added a commit to portphp/portphp that referenced this pull request Dec 11, 2017
@alexdruhet
Copy link

alexdruhet commented Sep 25, 2020

Hi, this is quite old PR but that lead to wrong behaviour since constraints are this way applied on the root object. Don't you think validator should be applied after object transformation/mapping?

@Baachi
Copy link
Contributor

Baachi commented Sep 28, 2020

Hey @Pixadelic,

this depends strongly of your use case, this is why I introduced the step system. It allows you to configure your own workflow which fits your usecase. If you need to to validation after the mapping, just add the step after that.

@alexdruhet
Copy link

Hi @Baachi, thanks for your reply. It helps me to understand how I was wrong dealing with priority arguments in my workflow setup... It's clearer now.

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.

Using validation annotations in doctrine entity
3 participants