bug #54468 [Translation] Fix LocaleSwitcher throws when intl not loaded (smnandre)

nicolas-grekas · nicolas-grekas · commit ff77282700c7 · 2024-04-03T10:56:26.000+02:00
This PR was merged into the 6.4 branch. Discussion ---------- [Translation] Fix LocaleSwitcher throws when intl not loaded | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | Fix issue [1505](symfony/demo#1505) (from Symfony demo) | License | MIT LocaleSwitcher::setDefault() throw an exception when intl is not installed > "The Symfony\Polyfill\Intl\Icu\Locale::setDefault() is not implemented. Please install the "intl" extension for full localization capabilities." Fixed by following `@stof` suggestion > Symfony should ignore this in the LocaleSwitcher, as already done in \Symfony\Component\HttpFoundation\Request::setLocale Commits ------- eff5e3c [Translation] Silence error when intl not loaded
diff --git a/src/Symfony/Component/Translation/LocaleSwitcher.php b/src/Symfony/Component/Translation/LocaleSwitcher.php
@@ -34,9 +34,14 @@ public function __construct(
 
     public function setLocale(string $locale): void
     {
-        if (class_exists(\Locale::class)) {
-            \Locale::setDefault($locale);
+        // Silently ignore if the intl extension is not loaded
+        try {
+            if (class_exists(\Locale::class, false)) {
+                \Locale::setDefault($locale);
+            }
+        } catch (\Exception) {
         }
+
         $this->locale = $locale;
         $this->requestContext?->setParameter('_locale', $locale);
 
