[Validator] Fix fields without constraints in Collection

HypeMC · HypeMC · commit 9de3680da037 · 2024-02-04T21:12:24.000+01:00
diff --git a/src/Symfony/Component/Validator/Constraints/Collection.php b/src/Symfony/Component/Validator/Constraints/Collection.php
@@ -103,6 +103,10 @@ private static function isFieldsOption($options): bool
                 return true;
             }
 
+            if (null === $optionOrField) {
+                return true;
+            }
+
             if (!\is_array($optionOrField)) {
                 return false;
             }
diff --git a/src/Symfony/Component/Validator/Tests/Constraints/CollectionTest.php b/src/Symfony/Component/Validator/Tests/Constraints/CollectionTest.php
@@ -175,10 +175,15 @@ public function testEmptyFieldsInOptions()
         $this->assertSame('foo bar baz', $constraint->extraFieldsMessage);
     }
 
-    public function testEmptyConstraintListFor()
+    /**
+     * @testWith [[]]
+     *           [null]
+     */
+    public function testEmptyConstraintListForField(?array $fieldConstraint)
     {
-        $constraint = new Collection([
-                'foo' => [],
+        $constraint = new Collection(
+            [
+                'foo' => $fieldConstraint,
             ],
             null,
             null,
@@ -193,11 +198,15 @@ public function testEmptyConstraintListFor()
         $this->assertSame('foo bar baz', $constraint->extraFieldsMessage);
     }
 
-    public function testEmptyConstraintListForFieldInOptions()
+    /**
+     * @testWith [[]]
+     *           [null]
+     */
+    public function testEmptyConstraintListForFieldInOptions(?array $fieldConstraint)
     {
         $constraint = new Collection([
             'fields' => [
-                'foo' => [],
+                'foo' => $fieldConstraint,
             ],
             'allowExtraFields' => true,
             'extraFieldsMessage' => 'foo bar baz',

Original file line number	Diff line number	Diff line change
`@@ -103,6 +103,10 @@ private static function isFieldsOption($options): bool`
`103`	`103`	`return true;`
`104`	`104`	`}`
`105`	`105`
	`106`	`+ if (null === $optionOrField) {`
	`107`	`+ return true;`
	`108`	`+ }`
	`109`	`+`
`106`	`110`	`if (!\is_array($optionOrField)) {`
`107`	`111`	`return false;`
`108`	`112`	`}`