[Intl] Fix locale validator when canonicalize is true

rdavaillaud · nicolas-grekas · commit fd5b24b95aa4 · 2025-06-26T09:25:45.000+02:00
When canonicalize is set to true, and the value length exceeds INTL_MAX_LOCALE_LEN the validator throws an exception.

The Intl Locale::canonicalize() method returns null when the value is too long and Locales::exists() only accept non null string.

This commit allows to handle the null value as it should.

[Intl] windows / php 8.1 INTL_MAX_LOCALE_LEN
diff --git a/src/Symfony/Component/Validator/Constraints/LocaleValidator.php b/src/Symfony/Component/Validator/Constraints/LocaleValidator.php
@@ -47,7 +47,7 @@ public function validate(mixed $value, Constraint $constraint)
             $value = \Locale::canonicalize($value);
         }
 
-        if (!Locales::exists($value)) {
+        if (null === $value || !Locales::exists($value)) {
             $this->context->buildViolation($constraint->message)
                 ->setParameter('{{ value }}', $this->formatValue($inputValue))
                 ->setCode(Locale::NO_SUCH_LOCALE_ERROR)
diff --git a/src/Symfony/Component/Validator/Tests/Constraints/LocaleValidatorTest.php b/src/Symfony/Component/Validator/Tests/Constraints/LocaleValidatorTest.php
@@ -91,6 +91,21 @@ public static function getInvalidLocales()
         ];
     }
 
+    public function testTooLongLocale()
+    {
+        $constraint = new Locale([
+            'message' => 'myMessage',
+        ]);
+
+        $locale = str_repeat('a', (\defined('INTL_MAX_LOCALE_LEN') ? \INTL_MAX_LOCALE_LEN : 85) + 1);
+        $this->validator->validate($locale, $constraint);
+
+        $this->buildViolation('myMessage')
+            ->setParameter('{{ value }}', '"' . $locale . '"')
+            ->setCode(Locale::NO_SUCH_LOCALE_ERROR)
+            ->assertRaised();
+    }
+
     /**
      * @dataProvider getUncanonicalizedLocales
      */

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ public function validate(mixed $value, Constraint $constraint)`
`47`	`47`	`$value = \Locale::canonicalize($value);`
`48`	`48`	`}`
`49`	`49`
`50`		`- if (!Locales::exists($value)) {`
	`50`	`+ if (null === $value \|\| !Locales::exists($value)) {`
`51`	`51`	`$this->context->buildViolation($constraint->message)`
`52`	`52`	`->setParameter('{{ value }}', $this->formatValue($inputValue))`
`53`	`53`	`->setCode(Locale::NO_SUCH_LOCALE_ERROR)`