Skip to content

Commit c28fc39

Browse files
bug #44359 Avoid duplicated session listener registration in tests (alexander-schranz)
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 ------- 366cb1ad3a Avoid duplicated session listener registration in tests
2 parents bc62124 + 200d84a commit c28fc39

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

DependencyInjection/FrameworkExtension.php

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -350,6 +350,11 @@ public function load(array $configs, ContainerBuilder $container)
350350

351351
$this->sessionConfigEnabled = true;
352352
$this->registerSessionConfiguration($config['session'], $container, $loader);
353+
if (!empty($config['test'])) {
354+
// test listener will replace the existing session listener
355+
// as we are aliasing to avoid duplicated registered events
356+
$container->setAlias('session_listener', 'test.session.listener');
357+
}
353358
} elseif (!empty($config['test'])) {
354359
$container->removeDefinition('test.session.listener');
355360
}

0 commit comments

Comments
 (0)