bug #33805 [FrameworkBundle] Fix wrong returned status code in ConfigDebugCommand (jschaedl)

xabbuh · xabbuh · commit fb5f8fb96adb · 2019-10-02T14:13:41.000+02:00
This PR was merged into the 3.4 branch. Discussion ---------- [FrameworkBundle] Fix wrong returned status code in ConfigDebugCommand | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | yes | New feature? | no  | Deprecations? | no  | Tickets | Fix #33747 | License | MIT | Doc PR | - This is a follow-up PR caused by #33775 (comment) Commits ------- 9b5ced2 [FrameworkBundle] Fix wrong returned status code in ConfigDebugCommand
diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/ConfigDebugCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/ConfigDebugCommand.php
@@ -104,7 +104,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         } catch (LogicException $e) {
             $errorIo->error($e->getMessage());
 
-            return;
+            return 1;
         }
 
         $io->title(sprintf('Current configuration for "%s.%s"', $extensionAlias, $path));

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ protected function execute(InputInterface $input, OutputInterface $output)`
`104`	`104`	`} catch (LogicException $e) {`
`105`	`105`	`$errorIo->error($e->getMessage());`
`106`	`106`
`107`		`- return;`
	`107`	`+ return 1;`
`108`	`108`	`}`
`109`	`109`
`110`	`110`	`$io->title(sprintf('Current configuration for "%s.%s"', $extensionAlias, $path));`