fix(validator): custom message was not translated

fixes #7336

Author: soyuka

src/Validator/Util/ParameterValidationConstraints.php

@@ -133,7 +133,7 @@ public static function getParameterValidationConstraints(Parameter $parameter, ?
         }
 
         if ($required && false !== $allowEmptyValue) {
-            $assertions[] = new NotNull(message: \sprintf('The parameter "%s" is required.', $parameter->getKey()));
+            $assertions[] = new NotNull();
         }
 
         if (isset($schema['minItems']) || isset($schema['maxItems'])) {

tests/Fixtures/TestBundle/ApiResource/TranslateValidationError.php

@@ -0,0 +1,43 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <dunglas@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace ApiPlatform\Tests\Fixtures\TestBundle\ApiResource;
+
+use ApiPlatform\Metadata\GetCollection;
+use ApiPlatform\Metadata\Operation;
+use ApiPlatform\Metadata\QueryParameter;
+use Symfony\Component\Validator\Constraints as Assert;
+
+#[GetCollection(
+    uriTemplate: '/translate_validation_error',
+    parameters: [
+        'name' => new QueryParameter(
+            description: 'Nome della persona',
+            constraints: [
+                new Assert\NotBlank(),
+            ]
+        ),
+        'surname' => new QueryParameter(
+            description: 'Cognome della persona',
+            required: true,
+        ),
+    ],
+    provider: [self::class, 'provide']
+)]
+class TranslateValidationError
+{
+    public static function provide(Operation $operation, array $uriVariables = []): array
+    {
+        return [];
+    }
+}

tests/Fixtures/app/AppKernel.php

@@ -139,6 +139,7 @@ protected function configureContainer(ContainerBuilder $c, LoaderInterface $load
                 'annotations' => false,
                 'handle_all_throwables' => true,
                 'uid' => ['default_uuid_version' => 7, 'time_based_uuid_version' => 7],
+                'set_locale_from_accept_language' => true,
             ];
         } else {
             $config = [
@@ -155,6 +156,7 @@ protected function configureContainer(ContainerBuilder $c, LoaderInterface $load
                 'router' => ['utf8' => true],
                 'http_method_override' => false,
                 'annotations' => false,
+                'set_locale_from_accept_language' => true,
             ];
         }
 

tests/Functional/Parameters/ParameterTest.php

@@ -14,6 +14,7 @@
 namespace ApiPlatform\Tests\Functional\Parameters;
 
 use ApiPlatform\Symfony\Bundle\Test\ApiTestCase;
+use ApiPlatform\Tests\Fixtures\TestBundle\ApiResource\TranslateValidationError;
 use ApiPlatform\Tests\Fixtures\TestBundle\ApiResource\WithParameter;
 use ApiPlatform\Tests\SetupClassResourcesTrait;
 use PHPUnit\Framework\Attributes\DataProvider;
@@ -29,7 +30,7 @@ final class ParameterTest extends ApiTestCase
      */
     public static function getResources(): array
     {
-        return [WithParameter::class];
+        return [WithParameter::class, TranslateValidationError::class];
     }
 
     public function testWithGroupFilter(): void

tests/Functional/Parameters/ValidationTest.php

@@ -14,7 +14,7 @@
 namespace ApiPlatform\Tests\Functional\Parameters;
 
 use ApiPlatform\Symfony\Bundle\Test\ApiTestCase;
-use ApiPlatform\Tests\Fixtures\TestBundle\ApiResource\ValidateParameterBeforeProvider;
+use ApiPlatform\Tests\Fixtures\TestBundle\ApiResource\TranslateValidationError;
 use ApiPlatform\Tests\Fixtures\TestBundle\ApiResource\WithParameter;
 use ApiPlatform\Tests\SetupClassResourcesTrait;
 
@@ -29,7 +29,7 @@ final class ValidationTest extends ApiTestCase
      */
     public static function getResources(): array
     {
-        return [WithParameter::class, ValidateParameterBeforeProvider::class];
+        return [WithParameter::class, TranslateValidationError::class];
     }
 
     public function testWithGroupFilter(): void
@@ -188,4 +188,14 @@ public function testValidatePattern(): void
         self::createClient()->request('GET', 'validate_parameters?pattern=2');
         $this->assertResponseIsSuccessful();
     }
+
+    public function testTranslationValidation(): void
+    {
+        $res = self::createClient()->request('GET', 'translate_validation_error', ['headers' => ['accept-language' => 'es']]);
+        $this->assertSame([
+            ['propertyPath' => 'name', 'message' => 'Este valor no debería estar vacío.', 'code' => 'c1051bb4-d103-4f74-8988-acbcafc7fdc3'],
+            ['propertyPath' => 'surname', 'message' => 'Este valor no debería ser nulo.', 'code' => 'ad32d13f-c3d4-423b-909a-857b961eb720']
+
+        ], $res->toArray(false)['violations']);
+    }
 }