fix(validator): property path on deepObject style

Author: soyuka

src/Symfony/Validator/State/ParameterValidatorProvider.php

@@ -100,7 +100,12 @@ private function getProperty(Parameter $parameter, ConstraintViolationInterface
             return str_replace(':property', $violation->getPropertyPath(), $key);
         }
 
-        if ($p = $violation->getPropertyPath()) {
+        $openApi = $parameter->getOpenApi();
+        if (false === $openApi) {
+            $openApi = null;
+        }
+
+        if ('deepObject' === $openApi?->getStyle() && $p = $violation->getPropertyPath()) {
             return $key.$p;
         }
 

tests/Fixtures/TestBundle/ApiResource/WithParameter.php

@@ -92,7 +92,8 @@
 #[GetCollection(
     uriTemplate: 'validate_parameters{._format}',
     parameters: [
-        'enum' => new QueryParameter(schema: ['enum' => ['a', 'b'], 'uniqueItems' => true]),
+        'enum' => new QueryParameter(schema: ['enum' => ['a', 'b'], 'uniqueItems' => true], openApi: new OpenApiParameter(name: 'enum', in: 'query', style: 'deepObject')),
+        'enumNotDeepObject' => new QueryParameter(schema: ['enum' => ['a', 'b'], 'uniqueItems' => true]),
         'num' => new QueryParameter(
             schema: ['minimum' => 1, 'maximum' => 3],
             nativeType: new BuiltinType(TypeIdentifier::STRING),

tests/Functional/Parameters/ValidationTest.php

@@ -55,6 +55,20 @@ public function testValidation(string $queryString, array $expectedViolations):
     public static function provideQueryStrings(): array
     {
         return [
+            [
+                'enumNotDeepObject[]=c&enumNotDeepObject[]=c',
+                [
+                    [
+                        'propertyPath' => 'enumNotDeepObject', 'message' => 'The value you selected is not a valid choice.',
+                    ],
+                    [
+                        'propertyPath' => 'enumNotDeepObject', 'message' => 'The value you selected is not a valid choice.',
+                    ],
+                    [
+                        'message' => 'This collection should contain only unique elements.',
+                    ],
+                ],
+            ],
             [
                 'enum[]=c&enum[]=c',
                 [