AC-3208: Fixed return types for deploy command

Author: anzin

n98-magerun.yaml

@@ -6,20 +6,24 @@
 
 namespace MagentoHackathon\Composer\Magento\Command;
 
+use Composer\Command\BaseCommand;
 use MagentoHackathon\Composer\Magento\Deploy\Manager\Entry;
 use MagentoHackathon\Composer\Magento\DeployManager;
+use MagentoHackathon\Composer\Magento\Installer;
 use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
-use Composer\Downloader\VcsDownloader;
-use MagentoHackathon\Composer\Magento\Installer;
 
 /**
  * @author Tiago Ribeiro <[email protected]>
  * @author Rui Marinho <[email protected]>
  */
-class DeployCommand extends \Composer\Command\BaseCommand
+class DeployCommand extends BaseCommand
 {
+    private const SUCCESS_EXIT_CODE = 0;
+
+    /**
+     * @inheritdoc
+     */
     protected function configure()
     {
         $this
@@ -28,70 +32,56 @@ protected function configure()
             ->setDefinition([
             // we dont need to define verbose, because composer already defined it internal
             //new InputOption('verbose', 'v', InputOption::VALUE_NONE, 'Show modified files for each directory that contains changes.'),
-        ])
+            ])
             ->setHelp(<<<EOT
 This command deploys all magento Modules
 
 EOT
-        )
-        ;
+        );
     }
 
+    /**
+     * @inheritdoc
+     */
     protected function execute(InputInterface $input, OutputInterface $output)
     {
         // init repos
         $composer = $this->getComposer();
         $installedRepo = $composer->getRepositoryManager()->getLocalRepository();
-
-        $dm = $composer->getDownloadManager();
         $im = $composer->getInstallationManager();
 
-        /**
-         * @var $moduleInstaller \MagentoHackathon\Composer\Magento\Installer
-         */
+        /** @var $moduleInstaller Installer */
         $moduleInstaller = $im->getInstaller("magento-module");
-
-
-        $deployManager = new DeployManager( $this->getIO() );
-
-        $extra          = $composer->getPackage()->getExtra();
-        $sortPriority   = $extra['magento-deploy-sort-priority'] ?? [];
+        $deployManager = new DeployManager($this->getIO());
+        $extra = $composer->getPackage()->getExtra();
+        $sortPriority = $extra['magento-deploy-sort-priority'] ?? [];
         $deployManager->setSortPriority( $sortPriority );
-
-
-
         $moduleInstaller->setDeployManager( $deployManager );
-        
 
         foreach ($installedRepo->getPackages() as $package) {
-
             if ($input->getOption('verbose')) {
-                $output->writeln( $package->getName() );
-                $output->writeln( $package->getType() );
+                $output->writeln($package->getName());
+                $output->writeln($package->getType());
             }
-
-            if( $package->getType() != "magento-module" ){
+            if ($package->getType() != "magento-module"){
                 continue;
             }
             if ($input->getOption('verbose')) {
                 $output->writeln("package {$package->getName()} recognized");
             }
-
             $strategy = $moduleInstaller->getDeployStrategy($package);
+
             if ($input->getOption('verbose')) {
                 $output->writeln("used " . get_class($strategy) . " as deploy strategy");
             }
             $strategy->setMappings($moduleInstaller->getParser($package)->getMappings());
-
             $deployManagerEntry = new Entry();
             $deployManagerEntry->setPackageName($package->getName());
             $deployManagerEntry->setDeployStrategy($strategy);
             $deployManager->addPackage($deployManagerEntry);
-            
         }
-
         $deployManager->doDeploy();
 
-        return;
+        return self::SUCCESS_EXIT_CODE;
     }
 }

src/MagentoHackathon/Composer/Magento/Command/DeployCommand.php

@@ -215,8 +215,6 @@ protected function deployLibraries()
             }
             return;
         }
-
-
         $vendorDir = rtrim($this->composer->getConfig()->get('vendor-dir'), '/');
 
         $filesystem = $this->filesystem;
@@ -255,7 +253,8 @@ protected function deployLibraries()
             if ($this->io->isDebug()) {
                 $this->io->write('Magento deployLibraries executes autoload generator');
             }
-            $process = new Process($executable . " -o {$libraryPath}/autoload.php  " . implode(' ', $autoloadDirectories));
+            $command = $executable . " -o {$libraryPath}/autoload.php  " . implode(' ', $autoloadDirectories);
+            $process = method_exists(Process::class, 'fromShellCommandline') ? Process::fromShellCommandline($command) : new Process($command);
             $process->run();
         } else {
             if ($this->io->isDebug()) {
@@ -264,8 +263,6 @@ protected function deployLibraries()
 
             }
         }
-
-
     }