minor #20186 [HttpKernel] removed implicit dep on Console (fabpot)

fabpot · fabpot · commit 451cdac4778f · 2016-10-08T11:41:43.000-07:00
This PR was merged into the 3.2-dev branch. Discussion ---------- [HttpKernel] removed implicit dep on Console | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | n/a | License | MIT | Doc PR | n/a HttpKernel does not have console as a dependency, so we must ensure that this is not implicitly required (`DebugHandlersListener` already has this check). Commits ------- 414206d [HttpKernel] removed implicit dep on Console
diff --git a/src/Symfony/Component/HttpKernel/EventListener/DumpListener.php b/src/Symfony/Component/HttpKernel/EventListener/DumpListener.php
@@ -49,6 +49,10 @@ public function configure()
 
     public static function getSubscribedEvents()
     {
+        if (!class_exists(ConsoleEvents::class)) {
+            return array();
+        }
+
         // Register early to have a working dump() as early as possible
         return array(ConsoleEvents::COMMAND => array('configure', 1024));
     }

Original file line number	Diff line number	Diff line change
`@@ -49,6 +49,10 @@ public function configure()`
`49`	`49`
`50`	`50`	`public static function getSubscribedEvents()`
`51`	`51`	`{`
	`52`	`+ if (!class_exists(ConsoleEvents::class)) {`
	`53`	`+ return array();`
	`54`	`+ }`
	`55`	`+`
`52`	`56`	`// Register early to have a working dump() as early as possible`
`53`	`57`	`return array(ConsoleEvents::COMMAND => array('configure', 1024));`
`54`	`58`	`}`