[Translation] Improve LocaleSwitcher a bit

Author: nicolas-grekas

DependencyInjection/Compiler/ConfigureLocaleSwitcherPass.php

@@ -205,6 +205,7 @@
 use Symfony\Component\Translation\Bridge\Loco\LocoProviderFactory;
 use Symfony\Component\Translation\Bridge\Lokalise\LokaliseProviderFactory;
 use Symfony\Component\Translation\Command\XliffLintCommand as BaseXliffLintCommand;
+use Symfony\Component\Translation\LocaleSwitcher;
 use Symfony\Component\Translation\PseudoLocalizationTranslator;
 use Symfony\Component\Translation\Translator;
 use Symfony\Component\Uid\Factory\UuidFactory;
@@ -1082,7 +1083,6 @@ private function registerRouterConfiguration(array $config, ContainerBuilder $co
             $container->removeDefinition('console.command.router_debug');
             $container->removeDefinition('console.command.router_match');
             $container->removeDefinition('messenger.middleware.router_context');
-            $container->removeDefinition('translation.locale_aware_request_context');
 
             return;
         }
@@ -1295,6 +1295,11 @@ private function registerTranslatorConfiguration(array $config, ContainerBuilder
         }
 
         $loader->load('translation.php');
+
+        if (!ContainerBuilder::willBeAvailable('symfony/translation', LocaleSwitcher::class, ['symfony/framework-bundle'])) {
+            $container->removeDefinition('translation.locale_switcher');
+        }
+
         $loader->load('translation_providers.php');
 
         // Use the "real" translator instead of the identity default

DependencyInjection/FrameworkExtension.php

@@ -15,7 +15,6 @@
 use Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\AddDebugLogProcessorPass;
 use Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\AddExpressionLanguageProvidersPass;
 use Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\AssetsContextPass;
-use Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\ConfigureLocaleSwitcherPass;
 use Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\ContainerBuilderDebugDumpPass;
 use Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\DataCollectorTranslatorPass;
 use Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\LoggingTranslatorPass;
@@ -164,7 +163,6 @@ public function build(ContainerBuilder $container)
         $container->addCompilerPass(new RegisterReverseContainerPass(true));
         $container->addCompilerPass(new RegisterReverseContainerPass(false), PassConfig::TYPE_AFTER_REMOVING);
         $container->addCompilerPass(new RemoveUnusedSessionMarshallingHandlerPass());
-        $container->addCompilerPass(new ConfigureLocaleSwitcherPass());
 
         if ($container->getParameter('kernel.debug')) {
             $container->addCompilerPass(new AddDebugLogProcessorPass(), PassConfig::TYPE_BEFORE_OPTIMIZATION, 2);

FrameworkBundle.php

@@ -13,7 +13,6 @@
 
 use Psr\Container\ContainerInterface;
 use Symfony\Bundle\FrameworkBundle\CacheWarmer\TranslationsCacheWarmer;
-use Symfony\Bundle\FrameworkBundle\Translation\LocaleAwareRequestContext;
 use Symfony\Bundle\FrameworkBundle\Translation\Translator;
 use Symfony\Component\Translation\Dumper\CsvFileDumper;
 use Symfony\Component\Translation\Dumper\IcuResFileDumper;
@@ -46,6 +45,7 @@
 use Symfony\Component\Translation\Reader\TranslationReaderInterface;
 use Symfony\Component\Translation\Writer\TranslationWriter;
 use Symfony\Component\Translation\Writer\TranslationWriterInterface;
+use Symfony\Contracts\Translation\LocaleAwareInterface;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
 return static function (ContainerConfigurator $container) {
@@ -164,22 +164,15 @@
             ->args([service(ContainerInterface::class)])
             ->tag('container.service_subscriber', ['id' => 'translator'])
             ->tag('kernel.cache_warmer')
-    ;
-
-    if (class_exists(LocaleSwitcher::class)) {
-        $container->services()
-            ->set('translation.locale_switcher', LocaleSwitcher::class)
-            ->args([
-                param('kernel.default_locale'),
-                abstract_arg('LocaleAware services'),
-            ])
-            ->alias(LocaleSwitcher::class, 'translation.locale_switcher')
 
-            ->set('translation.locale_aware_request_context', LocaleAwareRequestContext::class)
+        ->set('translation.locale_switcher', LocaleSwitcher::class)
             ->args([
-                service('router.request_context'),
                 param('kernel.default_locale'),
+                tagged_iterator('kernel.locale_aware'),
+                service('router.request_context')->ignoreOnInvalid(),
             ])
-        ;
-    }
+            ->tag('kernel.reset', ['method' => 'reset'])
+        ->alias(LocaleAwareInterface::class, 'translation.locale_switcher')
+        ->alias(LocaleSwitcher::class, 'translation.locale_switcher')
+    ;
 };

Resources/config/translation.php

@@ -31,7 +31,6 @@
 use Symfony\Component\Cache\Adapter\TagAwareAdapter;
 use Symfony\Component\Cache\DependencyInjection\CachePoolPass;
 use Symfony\Component\Config\Definition\Exception\InvalidConfigurationException;
-use Symfony\Component\DependencyInjection\Argument\AbstractArgument;
 use Symfony\Component\DependencyInjection\Argument\ServiceClosureArgument;
 use Symfony\Component\DependencyInjection\Argument\ServiceLocatorArgument;
 use Symfony\Component\DependencyInjection\Argument\TaggedIteratorArgument;
@@ -2019,15 +2018,13 @@ public function testLocaleSwitcherServiceRegistered()
         $container = $this->createContainerFromFile('full');
 
         $this->assertTrue($container->has('translation.locale_switcher'));
-        $this->assertTrue($container->has('translation.locale_aware_request_context'));
 
         $switcherDef = $container->getDefinition('translation.locale_switcher');
-        $localeAwareRequestContextDef = $container->getDefinition('translation.locale_aware_request_context');
 
         $this->assertSame('%kernel.default_locale%', $switcherDef->getArgument(0));
-        $this->assertInstanceOf(AbstractArgument::class, $switcherDef->getArgument(1));
-        $this->assertEquals(new Reference('router.request_context'), $localeAwareRequestContextDef->getArgument(0));
-        $this->assertSame('%kernel.default_locale%', $localeAwareRequestContextDef->getArgument(1));
+        $this->assertInstanceOf(TaggedIteratorArgument::class, $switcherDef->getArgument(1));
+        $this->assertSame('kernel.locale_aware', $switcherDef->getArgument(1)->getTag());
+        $this->assertEquals(new Reference('router.request_context', ContainerBuilder::IGNORE_ON_INVALID_REFERENCE), $switcherDef->getArgument(2));
     }
 
     protected function createContainer(array $data = [])