8000 [Validator] Add `EnableAutoMapping` and `DisableAutoMapping` constraints by alexandre-daubois · Pull Request #18568 · symfony/symfony-docs · GitHub
[go: up one dir, main page]
Skip to content

[Validator] Add EnableAutoMapping and DisableAutoMapping constraints #18568

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 17, 2023
Merged

[Validator] Add EnableAutoMapping and DisableAutoMapping constraints #18568

merged 1 commit into from
Jul 17, 2023

Conversation

alexandre-daubois
Copy link
Member

Fix #12563

@carsonbot carsonbot added this to the 5.4 milestone Jul 14, 2023
@alexandre-daubois alexandre-daubois force-pushed the enable-disable-automapping branch from 9c76e68 to 16f65cc Compare July 14, 2023 20:03
xabbuh
xabbuh approved these changes Jul 15, 2023
View reviewed changes
@alexandre-daubois alexandre-daubois force-pushed the enable-disable-automapping branch from 16f65cc to 14b6375 Compare July 16, 2023 10:16
@alexandre-daubois alexandre-daubois force-pushed the enable-disable-automapping branch from 14b6375 to 2653b56 Compare July 16, 2023 10:17
@javiereguiluz javiereguiluz merged commit 19955a7 into symfony:5.4 Jul 17, 2023
@javiereguiluz
Copy link
Member

Thanks Alex. I didn't know that we were missing this. While merging in upper branches, I removed the example with annotations.

@alexandre-daubois alexandre-daubois mentioned this pull request Jul 17, 2023
Merged
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this pull request Jul 17, 2023
@javiereguiluz
minor symfony#18570 [Validator] Add missing types and return types (a…
49888ee 
…lexandre-daubois)

This PR was merged into the 6.2 branch.

Discussion
----------

[Validator] Add missing types and return types

Follow-up of symfony#18568

Also the first parameter of `ConstraintValidator::validate()` has been renamed to `$value` instead of `$object`, I updated the few places I could find it appears 🙂

Commits
-------

97c31c7 [Validator] Add missing types and return types
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xabbuh xabbuh xabbuh approved these changes

Assignees
No one assigned
Labels
Status: Reviewed Validator
Projects
None yet
Milestone
5.4
Development

Successfully merging this pull request may close these issues.
4 participants
@alexandre-daubois @javiereguiluz @xabbuh @carsonbot
0