fix handling of special "value" constraint option

symfony · Jun 14, 2024 · b7ae744 · b7ae744
1 parent 2c36eca
commit b7ae744
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/Tests/Validator/Constraints/UniqueEntityTest.php b/Tests/Validator/Constraints/UniqueEntityTest.php
@@ -60,6 +60,13 @@ public function testAttributeWithGroupsAndPaylod()
         self::assertSame('some attached data', $constraint->payload);
         self::assertSame(['some_group'], $constraint->groups);
     }
+
+    public function testValueOptionConfiguresFields()
+    {
+        $constraint = new UniqueEntity(['value' => 'email']);
+
+        $this->assertSame('email', $constraint->fields);
+    }
 }
 
 #[UniqueEntity(['email'], message: 'myMessage')]

diff --git a/Validator/Constraints/UniqueEntity.php b/Validator/Constraints/UniqueEntity.php
@@ -60,7 +60,7 @@ public function __construct(
         $payload = null,
         array $options = [],
     ) {
-        if (\is_array($fields) && \is_string(key($fields)) && [] === array_diff(array_keys($fields), array_keys(get_class_vars(static::class)))) {
+        if (\is_array($fields) && \is_string(key($fields)) && [] === array_diff(array_keys($fields), array_merge(array_keys(get_class_vars(static::class)), ['value']))) {
             $options = array_merge($fields, $options);
         } else {
             $options['fields'] = $fields;