Skip to content

Commit a9144aa

Browse files
committed
[FrameworkBundle][Config] fix: do not add resource checkers for debug=false
1 parent 4acaf21 commit a9144aa

File tree

2 files changed

+16
-0
lines changed

2 files changed

+16
-0
lines changed

DependencyInjection/FrameworkExtension.php

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -361,6 +361,11 @@ public function load(array $configs, ContainerBuilder $container)
361361
$container->registerForAutoconfiguration(ObjectInitializerInterface::class)
362362
->addTag('validator.initializer');
363363

364+
if (!$container->getParameter('kernel.debug')) {
365+
// remove tagged iterator argument for resource checkers
366+
$container->getDefinition('config_cache_factory')->setArguments(array());
367+
}
368+
364369
if (\PHP_VERSION_ID < 70000) {
365370
$this->addClassesToCompile(array(
366371
'Symfony\\Component\\Config\\ConfigCache',

Tests/DependencyInjection/FrameworkExtensionTest.php

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -988,6 +988,17 @@ public function testCachePoolServices()
988988
$this->assertCachePoolServiceDefinitionIsCreated($container, 'cache.def', 'cache.app', 11);
989989
}
990990

991+
public function testRemovesResourceCheckerConfigCacheFactoryArgumentOnlyIfNoDebug()
992+
{
993+
$container = $this->createContainer(array('kernel.debug' => true));
994+
(new FrameworkExtension())->load(array(), $container);
995+
$this->assertCount(1, $container->getDefinition('config_cache_factory')->getArguments());
996+
997+
$container = $this->createContainer(array('kernel.debug' => false));
998+
(new FrameworkExtension())->load(array(), $container);
999+
$this->assertEmpty($container->getDefinition('config_cache_factory')->getArguments());
1000+
}
1001+
9911002
protected function createContainer(array $data = array())
9921003
{
9931004
return new ContainerBuilder(new ParameterBag(array_merge(array(

0 commit comments

Comments
 (0)