minor symfony#20974 [TwigBundle] do not remove the Twig ExceptionController service (xabbuh)

fabpot · fabpot · commit 2ef2264ba36c · 2016-12-17T22:57:23.000+01:00
This PR was merged into the 2.8 branch. Discussion ---------- [TwigBundle] do not remove the Twig ExceptionController service | Q | A | ------------- | --- | Branch? | 2.8 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | follow up of symfony#20799 Commits ------- c9e560f do not remove the Twig ExceptionController service
diff --git a/src/Symfony/Bundle/TwigBundle/DependencyInjection/Compiler/ExceptionListenerPass.php b/src/Symfony/Bundle/TwigBundle/DependencyInjection/Compiler/ExceptionListenerPass.php
@@ -27,8 +27,6 @@ public function process(ContainerBuilder $container)
             return;
         }
 
-        $container->removeDefinition('twig.controller.exception');
-
         // register the exception controller only if Twig is enabled and required dependencies do exist
         if (!class_exists('Symfony\Component\Debug\Exception\FlattenException') || !interface_exists('Symfony\Component\EventDispatcher\EventSubscriberInterface')) {
             $container->removeDefinition('twig.exception_listener');

Original file line number	Diff line number	Diff line change
`@@ -27,8 +27,6 @@ public function process(ContainerBuilder $container)`
`27`	`27`	`return;`
`28`	`28`	`}`
`29`	`29`
`30`		`- $container->removeDefinition('twig.controller.exception');`
`31`		`-`
`32`	`30`	`// register the exception controller only if Twig is enabled and required dependencies do exist`
`33`	`31`	`if (!class_exists('Symfony\Component\Debug\Exception\FlattenException') \|\| !interface_exists('Symfony\Component\EventDispatcher\EventSubscriberInterface')) {`
`34`	`32`	`$container->removeDefinition('twig.exception_listener');`