Move magentoVersion dependency and code to ShowStatus process instead of Status command

Author: peterjaap

src/Command/Process/ShowStatus.php

@@ -8,6 +8,7 @@
 namespace Magento\CloudPatches\Command\Process;
 
 use Magento\CloudPatches\Command\Process\Action\ReviewAppliedAction;
+use Magento\CloudPatches\Composer\MagentoVersion;
 use Magento\CloudPatches\Console\QuestionFactory;
 use Magento\CloudPatches\Patch\Data\AggregatedPatch;
 use Magento\CloudPatches\Patch\Data\AggregatedPatchInterface;
@@ -45,6 +46,11 @@ class ShowStatus implements ProcessInterface
      */
     private $localPool;
 
+    /**
+     * @var MagentoVersion
+     */
+    private $magentoVersion;
+
     /**
      * @var StatusPool
      */
@@ -88,7 +94,8 @@ public function __construct(
         ReviewAppliedAction $reviewAppliedAction,
         Renderer $renderer,
         QuestionHelper $questionHelper,
-        QuestionFactory $questionFactory
+        QuestionFactory $questionFactory,
+        MagentoVersion $magentoVersion
     ) {
         $this->aggregator = $aggregator;
         $this->optionalPool = $optionalPool;
@@ -98,6 +105,7 @@ public function __construct(
         $this->renderer = $renderer;
         $this->questionHelper = $questionHelper;
         $this->questionFactory = $questionFactory;
+        $this->magentoVersion = $magentoVersion;
     }
 
     /**
@@ -134,6 +142,10 @@ public function run(InputInterface $input, OutputInterface $output)
         } else {
             $this->renderer->printTable($output, array_values($patches));
         }
+
+        if (!$isJsonFormat) {
+            $output->writeln('<info>' . $this->magentoVersion->get() . '</info>');
+        }
     }
 
     /**

src/Command/Status.php

@@ -9,7 +9,6 @@
 
 use Magento\CloudPatches\App\RuntimeException;
 use Magento\CloudPatches\Command\Process\ShowStatus;
-use Magento\CloudPatches\Composer\MagentoVersion;
 use Psr\Log\LoggerInterface;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
@@ -33,24 +32,16 @@ class Status extends AbstractCommand
      */
     private $logger;
 
-    /**
-     * @var MagentoVersion
-     */
-    private $magentoVersion;
-
     /**
      * @param ShowStatus $showStatus
      * @param LoggerInterface $logger
-     * @param MagentoVersion $magentoVersion
      */
     public function __construct(
         ShowStatus $showStatus,
-        LoggerInterface $logger,
-        MagentoVersion $magentoVersion
+        LoggerInterface $logger
     ) {
         $this->showStatus = $showStatus;
         $this->logger = $logger;
-        $this->magentoVersion = $magentoVersion;
 
         parent::__construct(self::NAME);
     }
@@ -74,9 +65,6 @@ public function execute(InputInterface $input, OutputInterface $output)
     {
         try {
             $this->showStatus->run($input, $output);
-            if ($input->getOption('format') !== 'json') {
-                $output->writeln('<info>' . $this->magentoVersion->get() . '</info>');
-            }
         } catch (RuntimeException $e) {
             $output->writeln('<error>' . $e->getMessage() . '</error>');
             $this->logger->error($e->getMessage());