The Wayback Machine - https://web.archive.org/web/20230128074557/https://github.com/symfony/symfony/pull/48951
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

Jump to bottom

[Validator] Handle object properties in Unique validator #48951

Open
wants to merge 8 commits into
base: 6.3
Choose a base branch
from
Open

[Validator] Handle object properties in Unique validator #48951

wants to merge 8 commits into from
+284 −56

Conversation

plfort
Copy link
Contributor

@plfort plfort commented Jan 11, 2023

Q A
Branch? 6.3
Bug fix? no
New feature? yes
Deprecations? no
Tickets Fix #48072
License MIT
Doc PR

Allow Unique constraint to access object properties.

Simple public object properties are handled directly, otherwise we rely on PropertyAccess component to read the value.

This PR is an improvement of #47714

@carsonbot carsonbot added this to the 6.3 milestone Jan 11, 2023
@plfort plfort force-pushed the unique_validator_object branch from 3953218 to c14e90e Compare Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Seb33300 Seb33300

@alexandre-daubois alexandre-daubois

@artyuum artyuum

Assignees
No one assigned
Labels
Feature Status: Needs Work Validator
Projects
None yet
Milestone
6.3
Development

Successfully merging this pull request may close these issues.

[Validator] Unique constraint can't be applied to collection of object
5 participants
@plfort @Seb33300 @alexandre-daubois @artyuum @carsonbot