[Console] fixed PHP7 Errors when not using Dispatcher

keradus · fabpot · commit 899fa7936bcc · 2016-12-05T14:00:57.000+01:00
diff --git a/src/Symfony/Component/Console/Application.php b/src/Symfony/Component/Console/Application.php
@@ -843,7 +843,13 @@ protected function doRunCommand(Command $command, InputInterface $input, OutputI
         }
 
         if (null === $this->dispatcher) {
-            return $command->run($input, $output);
+            try {
+                return $command->run($input, $output);
+            } catch (\Exception $e) {
+                throw $e;
+            } catch (\Throwable $e) {
+                throw new FatalThrowableError($e);
+            }
         }
 
         $event = new ConsoleCommandEvent($command, $input, $output);
diff --git a/src/Symfony/Component/Console/Tests/ApplicationTest.php b/src/Symfony/Component/Console/Tests/ApplicationTest.php
@@ -951,6 +951,24 @@ public function testRunDispatchesAllEventsWithException()
         $this->assertContains('before.foo.caught.after.', $tester->getDisplay());
     }
 
+    public function testRunWithError()
+    {
+        $this->setExpectedException('Exception', 'dymerr');
+
+        $application = new Application();
+        $application->setAutoExit(false);
+        $application->setCatchExceptions(false);
+
+        $application->register('dym')->setCode(function (InputInterface $input, OutputInterface $output) {
+            $output->write('dym.');
+
+            throw new \Error('dymerr');
+        });
+
+        $tester = new ApplicationTester($application);
+        $tester->run(array('command' => 'dym'));
+    }
+
     /**
      * @expectedException        \LogicException
      * @expectedExceptionMessage caught

Original file line number	Diff line number	Diff line change
`@@ -843,7 +843,13 @@ protected function doRunCommand(Command $command, InputInterface $input, OutputI`
`843`	`843`	`}`
`844`	`844`
`845`	`845`	`if (null === $this->dispatcher) {`
`846`		`- return $command->run($input, $output);`
	`846`	`+ try {`
	`847`	`+ return $command->run($input, $output);`
	`848`	`+ } catch (\Exception $e) {`
	`849`	`+ throw $e;`
	`850`	`+ } catch (\Throwable $e) {`
	`851`	`+ throw new FatalThrowableError($e);`
	`852`	`+ }`
`847`	`853`	`}`
`848`	`854`
`849`	`855`	`$event = new ConsoleCommandEvent($command, $input, $output);`