bug #44359 Avoid duplicated session listener registration in tests (alexander-schranz)

nicolas-grekas · nicolas-grekas · commit 2ace406ad73f · 2021-12-01T15:57:19.000+01:00
This PR was squashed before being merged into the 5.4 branch. Discussion ---------- Avoid duplicated session listener registration in tests | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #44345 | License | MIT | Doc PR | - Avoid duplicated session listener registration in tests. Commits ------- 366cb1a Avoid duplicated session listener registration in tests
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
@@ -350,6 +350,11 @@ public function load(array $configs, ContainerBuilder $container)
 
             $this->sessionConfigEnabled = true;
             $this->registerSessionConfiguration($config['session'], $container, $loader);
+            if (!empty($config['test'])) {
+                // test listener will replace the existing session listener
+                // as we are aliasing to avoid duplicated registered events
+                $container->setAlias('session_listener', 'test.session.listener');
+            }
         } elseif (!empty($config['test'])) {
             $container->removeDefinition('test.session.listener');
         }
