bug #22144 [Console] Fixed fatal error when the command is not defined (lyrixx)

fabpot · fabpot · commit 869e68dd1fbd · 2017-03-25T08:31:41.000-07:00
This PR was merged into the 3.3-dev branch. Discussion ---------- [Console] Fixed fatal error when the command is not defined | Q | A | ------------- | --- | Branch? | master | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | https://github.com/symfony/symfony/pull/18140/files#r107871406 | License | MIT | Doc PR | Commits ------- d5b41b6 [Console] Fixed fatal error when the command is not defined
diff --git a/src/Symfony/Component/Console/Application.php b/src/Symfony/Component/Console/Application.php
@@ -127,7 +127,7 @@ public function run(InputInterface $input = null, OutputInterface $output = null
             $exception = new FatalThrowableError($e);
         }
 
-        if (null !== $e && null !== $this->dispatcher) {
+        if (null !== $this->runningCommand && null !== $e && null !== $this->dispatcher) {
             $event = new ConsoleErrorEvent($this->runningCommand, $input, $output, $e, $e->getCode());
             $this->dispatcher->dispatch(ConsoleEvents::ERROR, $event);
 

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ public function run(InputInterface $input = null, OutputInterface $output = null`
`127`	`127`	`$exception = new FatalThrowableError($e);`
`128`	`128`	`}`
`129`	`129`
`130`		`- if (null !== $e && null !== $this->dispatcher) {`
	`130`	`+ if (null !== $this->runningCommand && null !== $e && null !== $this->dispatcher) {`
`131`	`131`	`$event = new ConsoleErrorEvent($this->runningCommand, $input, $output, $e, $e->getCode());`
`132`	`132`	`$this->dispatcher->dispatch(ConsoleEvents::ERROR, $event);`
`133`	`133`