Add an entry on json export format

Pascal Montoya · Pascal Montoya · commit 637598d27f21 · 2018-05-03T17:39:31.000+02:00
diff --git a/src/Symfony/Bridge/Twig/Command/DebugCommand.php b/src/Symfony/Bridge/Twig/Command/DebugCommand.php
@@ -88,8 +88,22 @@ protected function execute(InputInterface $input, OutputInterface $output)
             }
             $data['tests'] = array_keys($data['tests']);
             $data['loader_paths'] = $this->getLoaderPaths();
+            $wrongBundles = $this->findWrongBundleOverrides();
+            if ($wrongBundles) {
+                $data['warnings'] = array();
+
+                $messages = $this->buildWarningMessages($this->findWrongBundleOverrides());
+                $data['warnings'] = array_reduce(
+                    $messages,
+                    function ($carry, $message) {
+                        $carry[] = $message;
+                        return $carry;
+                    },
+                    array()
+                );
+            }
+
             $io->writeln(json_encode($data));
-            $this->displayAlternatives($this->findWrongBundleOverrides(), $io);
 
             return 0;
         }
@@ -130,7 +144,13 @@ protected function execute(InputInterface $input, OutputInterface $output)
         array_pop($rows);
         $io->section('Loader Paths');
         $io->table(array('Namespace', 'Paths'), $rows);
-        $this->displayAlternatives($this->findWrongBundleOverrides(), $io);
+        $messages = $this->buildWarningMessages($this->findWrongBundleOverrides());
+        array_walk(
+            $messages,
+            function ($message) use ($io) {
+                 $io->warning(trim($message));
+             }
+         );
 
         return 0;
     }
@@ -312,8 +332,9 @@ function ($carry, $absolutePath) use ($relativePath) {
         return $alternatives;
     }
 
-    private function displayAlternatives(array $wrongBundles, SymfonyStyle $io): void
+    private function buildWarningMessages(array $wrongBundles): array
     {
+        $messages = array();
         foreach ($wrongBundles as $path => $alternatives) {
             $message = sprintf('Path "%s" not matching any bundle found', $path);
             if ($alternatives) {
@@ -326,7 +347,9 @@ private function displayAlternatives(array $wrongBundles, SymfonyStyle $io): voi
                     }
                 }
             }
-            $io->warning(trim($message));
+            $messages[] = $message;
         }
+
+        return $messages;
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -88,8 +88,22 @@ protected function execute(InputInterface $input, OutputInterface $output)`
`88`	`88`	`}`
`89`	`89`	`$data['tests'] = array_keys($data['tests']);`
`90`	`90`	`$data['loader_paths'] = $this->getLoaderPaths();`
	`91`	`+ $wrongBundles = $this->findWrongBundleOverrides();`
	`92`	`+ if ($wrongBundles) {`
	`93`	`+ $data['warnings'] = array();`
	`94`	`+`
	`95`	`+ $messages = $this->buildWarningMessages($this->findWrongBundleOverrides());`
	`96`	`+ $data['warnings'] = array_reduce(`
	`97`	`+ $messages,`
	`98`	`+ function ($carry, $message) {`
	`99`	`+ $carry[] = $message;`
	`100`	`+ return $carry;`
	`101`	`+ },`
	`102`	`+ array()`
	`103`	`+ );`
	`104`	`+ }`
	`105`	`+`
`91`	`106`	`$io->writeln(json_encode($data));`
`92`		`- $this->displayAlternatives($this->findWrongBundleOverrides(), $io);`
`93`	`107`
`94`	`108`	`return 0;`
`95`	`109`	`}`
`@@ -130,7 +144,13 @@ protected function execute(InputInterface $input, OutputInterface $output)`
`130`	`144`	`array_pop($rows);`
`131`	`145`	`$io->section('Loader Paths');`
`132`	`146`	`$io->table(array('Namespace', 'Paths'), $rows);`
`133`		`- $this->displayAlternatives($this->findWrongBundleOverrides(), $io);`
	`147`	`+ $messages = $this->buildWarningMessages($this->findWrongBundleOverrides());`
	`148`	`+ array_walk(`
	`149`	`+ $messages,`
	`150`	`+ function ($message) use ($io) {`
	`151`	`+ $io->warning(trim($message));`
	`152`	`+ }`
	`153`	`+ );`
`134`	`154`
`135`	`155`	`return 0;`
`136`	`156`	`}`
`@@ -312,8 +332,9 @@ function ($carry, $absolutePath) use ($relativePath) {`
`312`	`332`	`return $alternatives;`
`313`	`333`	`}`
`314`	`334`
`315`		`- private function displayAlternatives(array $wrongBundles, SymfonyStyle $io): void`
	`335`	`+ private function buildWarningMessages(array $wrongBundles): array`
`316`	`336`	`{`
	`337`	`+ $messages = array();`
`317`	`338`	`foreach ($wrongBundles as $path => $alternatives) {`
`318`	`339`	`$message = sprintf('Path "%s" not matching any bundle found', $path);`
`319`	`340`	`if ($alternatives) {`
`@@ -326,7 +347,9 @@ private function displayAlternatives(array $wrongBundles, SymfonyStyle $io): voi`
`326`	`347`	`}`
`327`	`348`	`}`
`328`	`349`	`}`
`329`		`- $io->warning(trim($message));`
	`350`	`+ $messages[] = $message;`
`330`	`351`	`}`
	`352`	`+`
	`353`	`+ return $messages;`
`331`	`354`	`}`
`332`	`355`	`}`