[core] Fix WPStyle and update getConfigGlobalAsArray function (#39)

Author: hjuarez20

src/Core/Command/Settings/SetCommand.php

@@ -13,6 +13,7 @@
 use Symfony\Component\Yaml\Dumper;
 use Symfony\Component\Yaml\Parser;
 use WP\Console\Core\Command\Command;
+use WP\Console\Core\Style\WPStyle;
 use WP\Console\Core\Utils\ConfigurationManager;
 use WP\Console\Core\Utils\NestedArray;
 
@@ -76,6 +77,8 @@ protected function configure()
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {
+        $io = new WPStyle($input, $output);
+
         $parser = new Parser();
         $dumper = new Dumper();
 
@@ -93,7 +96,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         );
 
         if (!file_exists($userConfigFile)) {
-            $this->getIo()->error(
+            $io->error(
                 sprintf(
                     $this->trans('commands.settings.set.messages.missing-file'),
                     $userConfigFile
@@ -107,7 +110,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
                 file_get_contents($userConfigFile)
             );
         } catch (\Exception $e) {
-            $this->getIo()->error(
+            $io->error(
                 $this->trans(
                     'commands.settings.set.messages.error-parsing'
                 ) . ': ' . $e->getMessage()
@@ -127,7 +130,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         try {
             $userConfigFileDump = $dumper->dump($userConfigFileParsed, 10);
         } catch (\Exception $e) {
-            $this->getIo()->error(
+            $io->error(
                 [
                     $this->trans('commands.settings.set.messages.error-generating'),
                     $e->getMessage()
@@ -144,7 +147,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
             $translatorLanguage = $this->getApplication()->getTranslator()->getLanguage();
             if ($translatorLanguage != $settingValue) {
-                $this->getIo()->error(
+                $io->error(
                     sprintf(
                         $this->trans('commands.settings.set.messages.missing-language'),
                         $settingValue
@@ -158,7 +161,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         try {
             file_put_contents($userConfigFile, $userConfigFileDump);
         } catch (\Exception $e) {
-            $this->getIo()->error(
+            $io->error(
                 [
                     $this->trans('commands.settings.set.messages.error-writing'),
                     $e->getMessage()
@@ -167,5 +170,15 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
             return 1;
         }
+
+        $io->success(
+            sprintf(
+                $this->trans('commands.settings.set.messages.success'),
+                $settingName,
+                $settingValue
+            )
+        );
+
+        return 0;
     }
 }

src/Core/EventSubscriber/CalculateStatisticsListener.php

@@ -59,10 +59,10 @@ public function calculateStatistics(ConsoleTerminateEvent $event)
             return;
         }
 
-        $globalConfig = $this->configurationManager->getConfigAsArray();
+        $configGlobalAsArray = $this->configurationManager->getConfigGlobalAsArray();
 
         //Validate if the config is enable.
-        if (is_null($globalConfig) || !$globalConfig['application']['share']['statistics']) {
+        if (is_null($configGlobalAsArray) || !$configGlobalAsArray['application']['share']['statistics']) {
             return;
         }
 

src/Core/EventSubscriber/SaveStatisticsListener.php

@@ -47,9 +47,9 @@ class SaveStatisticsListener implements EventSubscriberInterface
     /**
      * SaveStatisticsListener constructor.
      *
-     * @param CountCodeLines             $countCodeLines
-     * @param ConfigurationManager       $configurationManager
-     * @param TranslatorManager $translator
+     * @param CountCodeLines       $countCodeLines
+     * @param ConfigurationManager $configurationManager
+     * @param TranslatorManager    $translator
      */
     public function __construct(
         CountCodeLines $countCodeLines,
@@ -72,10 +72,10 @@ public function saveStatistics(ConsoleTerminateEvent $event)
             return;
         }
 
-        $globalConfig = $this->configurationManager->getConfigAsArray();
+        $configGlobalAsArray = $this->configurationManager->getConfigGlobalAsArray();
 
         //Validate if the config is enable.
-        if (is_null($globalConfig) || !$globalConfig['application']['share']['statistics']) {
+        if (is_null($configGlobalAsArray) || !$configGlobalAsArray['application']['share']['statistics']) {
             return;
         }
 

src/Core/Generator/InitGenerator.php

@@ -6,6 +6,7 @@
  */
 namespace WP\Console\Core\Generator;
 
+use WP\Console\Core\Style\WPStyle;
 use WP\Console\Core\Utils\NestedArray;
 use Symfony\Component\Yaml\Dumper;
 use Symfony\Component\Yaml\Parser;
@@ -116,12 +117,6 @@ private function resetStatisticsConfig($homeDirectory, $statisticsValue)
         $userConfigFile = $homeDirectory . 'config.yml';
 
         if (!file_exists($userConfigFile)) {
-            $this->getIo()->error(
-                sprintf(
-                    $this->trans('commands.settings.set.messages.missing-file'),
-                    $userConfigFile
-                )
-            );
             return 1;
         }
 

src/Core/Utils/ConfigurationManager.php

@@ -256,11 +256,11 @@ public function loadExtendConfiguration()
     }
 
     /**
-     * Get the config as array.
+     * Get config global as array.
      *
      * @return array
      */
-    public function getConfigAsArray()
+    public function getConfigGlobalAsArray()
     {
         $filePath = sprintf(
             '%s/.wp-console/config.yml',
@@ -269,13 +269,13 @@ public function getConfigAsArray()
 
         $fs = new Filesystem();
 
-        if ($fs->exists($filePath)) {
-            $yaml = new Parser();
-            $configGlobal = $yaml->parse(file_get_contents($filePath), true);
-
-            return $configGlobal;
+        if (!$fs->exists($filePath)) {
+            return null;
         }
 
-        return null;
+        $yaml = new Parser();
+        $configGlobal = $yaml->parse(file_get_contents($filePath), true);
+
+        return $configGlobal;
     }
 }