netz98 / n98-magerun2

src/bootstrap.php

Doc Comments +4 added lines

@@ -3,6 +3,10 @@
 if (!class_exists('N98_Magerun_Bootstrap')) {
     class N98_Magerun_Bootstrap
     {
+
+        /**
+         * @param string $file
+         */
         public static function includeIfExists($file)
         {
             if (file_exists($file)) {

Spacing +5 added lines, -5 removed lines

@@ -16,11 +16,11 @@
          */
         public static function getLoader()
         {
-            if ((!$loader = \N98_Magerun_Bootstrap::includeIfExists(__DIR__.'/../vendor/autoload.php'))
-                && (!$loader = \N98_Magerun_Bootstrap::includeIfExists(__DIR__.'/../../../autoload.php'))) {
-                throw new \ErrorException('You must set up the project dependencies, run the following commands:'.PHP_EOL.
-                    'curl -s http://getcomposer.org/installer | php'.PHP_EOL.
-                    'php composer.phar install'.PHP_EOL);
+            if ((!$loader = \N98_Magerun_Bootstrap::includeIfExists(__DIR__ . '/../vendor/autoload.php'))
+                && (!$loader = \N98_Magerun_Bootstrap::includeIfExists(__DIR__ . '/../../../autoload.php'))) {
+                throw new \ErrorException('You must set up the project dependencies, run the following commands:' . PHP_EOL .
+                    'curl -s http://getcomposer.org/installer | php' . PHP_EOL .
+                    'php composer.phar install' . PHP_EOL);
             }
 
             return $loader;

src/N98/Magento/Application.php

Doc Comments +2 added lines, -2 removed lines

@@ -147,7 +147,7 @@ 
     }
 
     /**
-     * @return \Symfony\Component\Console\Input\InputDefinition|void
+     * @return \Symfony\Component\Console\Input\InputDefinition
      */
     protected function getDefaultInputDefinition()
     {
@@ -387,7 +387,7 @@ 
      * @TODO Move logic into "EventSubscriber"
      *
      * @param OutputInterface $output
-     * @return bool
+     * @return null|false
      */
     public function checkVarDir(OutputInterface $output)
     {

Indentation +1 added lines, -1 removed lines

@@ -407,7 +407,7 @@
                     $this->initMagento();
                 } catch (\Exception $e) {
                     $message = 'Cannot initialize Magento. Please check your configuration. '
-                             . 'Some n98-magerun command will not work. Got message: ';
+                                . 'Some n98-magerun command will not work. Got message: ';
                     if (OutputInterface::VERBOSITY_VERY_VERBOSE <= $output->getVerbosity()) {
                         $message .= $e->getTraceAsString();
                     } else {

Spacing +2 added lines, -2 removed lines

@@ -392,7 +392,7 @@ 
     public function checkVarDir(OutputInterface $output)
     {
         if (OutputInterface::VERBOSITY_NORMAL <= $output->getVerbosity()) {
-            $tempVarDir = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'magento' . DIRECTORY_SEPARATOR .  'var';
+            $tempVarDir = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'magento' . DIRECTORY_SEPARATOR . 'var';
 
             if (is_dir($tempVarDir)) {
                 $this->detectMagento(null, $output);
@@ -659,7 +659,7 @@ 
             $this->partialConfig = $configLoader->getPartialConfig($loadExternalConfig);
             $this->detectMagento($input, $output);
             $configLoader->loadStageTwo($this->_magentoRootFolder, $loadExternalConfig, $this->_magerunStopFileFolder);
-            $this->config = $configLoader->toArray();;
+            $this->config = $configLoader->toArray(); ;
             $this->dispatcher = new EventDispatcher();
             $this->setDispatcher($this->dispatcher);
             if ($this->autoloader) {

src/N98/Magento/Command/AbstractMagentoCommand.php

Doc Comments +3 added lines, -3 removed lines

@@ -83,7 +83,7 @@ 
     /**
      * @param array $codeArgument
      * @param bool  $status
-     * @return bool
+     * @return boolean|null
      */
     protected function saveCacheStatus($codeArgument, $status)
     {
@@ -288,8 +288,8 @@ 
     /**
      * brings locally cached repository up to date if it is missing the requested tag
      *
-     * @param $package
-     * @param $targetFolder
+     * @param PackageInterface $package
+     * @param string $targetFolder
      */
     protected function checkRepository($package, $targetFolder)
     {

Spacing +3 added lines, -3 removed lines

@@ -202,7 +202,7 @@ 
 
         if (!$silent) {
             $editionString = ($this->_magentoEnterprise ? ' (Enterprise Edition) ' : '');
-            $output->writeln('<info>Found Magento '. $editionString . 'in folder "' . $this->_magentoRootFolder . '"</info>');
+            $output->writeln('<info>Found Magento ' . $editionString . 'in folder "' . $this->_magentoRootFolder . '"</info>');
         }
 
         if (!empty($this->_magentoRootFolder)) {
@@ -266,7 +266,7 @@ 
         $preferSource = true
     ) {
         $dm = $this->getComposerDownloadManager($input, $output);
-        if (! $config instanceof PackageInterface) {
+        if (!$config instanceof PackageInterface) {
             $package = $this->createComposerPackageByConfig($config);
         } else {
             $package = $config;
@@ -310,7 +310,7 @@ 
                 escapeshellarg($targetFolder),
                 escapeshellarg($package->getSourceReference())
             );
-            $existingTag =  shell_exec($command);
+            $existingTag = shell_exec($command);
             if ($existingTag === $package->getSourceReference()) {
                 $command = sprintf('cd %s && hg pull', escapeshellarg($targetFolder));
                 shell_exec($command);

src/N98/Magento/Command/Developer/Module/CreateCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
     }
 
     /**
-     * @param $configBag
+     * @param ConfigBag $configBag
      */
     private function initConfigBagDefaultValues($configBag)
     {

src/N98/Magento/Command/Installer/SubCommand/InstallMagento.php

Doc Comments +1 added lines, -1 removed lines

@@ -252,7 +252,7 @@
     }
 
     /**
-     * @param array $installArgs
+     * @param string $installArgs
      *
      * @throws \Exception
      */

Spacing +2 added lines, -2 removed lines

@@ -120,7 +120,7 @@ 
             if (!preg_match('|^http(s)?://[a-z0-9-]+(.[a-z0-9-]+)*(:[0-9]+)?(/.*)?$|i', $url)) {
                 throw new \InvalidArgumentException('Please enter a valid URL');
             }
-            if (parse_url($url, \PHP_URL_HOST) ==  'localhost') {
+            if (parse_url($url, \PHP_URL_HOST) == 'localhost') {
                 throw new \InvalidArgumentException('localhost cause problems! Please use 127.0.0.1 or another hostname');
             }
 
@@ -211,7 +211,7 @@ 
      */
     protected function getInstallScriptPath()
     {
-        $installerScript  = $this->config->getString('installationFolder') . DIRECTORY_SEPARATOR . 'bin/magento';
+        $installerScript = $this->config->getString('installationFolder') . DIRECTORY_SEPARATOR . 'bin/magento';
         if (!file_exists($installerScript)) {
             throw new \RuntimeException('Installation script was not found.', 1);
         }

src/N98/Magento/Command/Installer/SubCommand/InstallSampleData.php

Doc Comments +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 class InstallSampleData extends AbstractSubCommand
 {
     /**
-     * @return bool
+     * @return false|null
      */
     public function execute()
     {

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
         $process = $processBuilder->getProcess();
         $process->setTimeout(86400);
         $process->start();
-        $process->wait(function ($type, $buffer) {
+        $process->wait(function($type, $buffer) {
             $this->output->write($buffer, false);
         });
 
@@ -70,7 +70,7 @@ 
         $process = $processBuilder->getProcess();
         $process->setTimeout(86400);
         $process->start();
-        $process->wait(function ($type, $buffer) {
+        $process->wait(function($type, $buffer) {
             $this->output->write($buffer, false);
         });
     }

src/N98/Magento/Command/Installer/SubCommand/RewriteHtaccessFile.php

Doc Comments +4 added lines, -1 removed lines

@@ -7,7 +7,7 @@ 
 class RewriteHtaccessFile extends AbstractSubCommand
 {
     /**
-     * @return bool
+     * @return boolean|null
      */
     public function execute()
     {
@@ -53,6 +53,9 @@ 
         $this->_replaceContent($htaccessFile, $baseUrl);
     }
 
+    /**
+     * @param string $htaccesFile
+     */
     protected function _backupOriginalFile($htaccesFile)
     {
         copy(

Indentation +4 added lines, -4 removed lines

@@ -44,10 +44,10 @@
     protected function replaceHtaccessFile($baseUrl)
     {
         $htaccessFile = $this->config->getString('installationFolder')
-                      . DIRECTORY_SEPARATOR
-                      . 'pub'
-                      . DIRECTORY_SEPARATOR
-                      . '.htaccess';
+                        . DIRECTORY_SEPARATOR
+                        . 'pub'
+                        . DIRECTORY_SEPARATOR
+                        . '.htaccess';
 
         $this->_backupOriginalFile($htaccessFile);
         $this->_replaceContent($htaccessFile, $baseUrl);

src/N98/Magento/Command/Script/Repository/ScriptLoader.php

Doc Comments +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
     /**
      * Reads the first line. If it's a comment return it.
      *
-     * @param $file
+     * @param string $file
      *
      * @return string
      */

src/N98/Magento/Command/ScriptCommand.php

Doc Comments +3 added lines, -3 removed lines

@@ -254,7 +254,7 @@ 
     /**
      * @param InputInterface $input
      * @param OutputInterface $output
-     * @param $commandString
+     * @param string $commandString
      * @throws \RuntimeException
      */
     protected function runMagerunCommand(InputInterface $input, OutputInterface $output, $commandString)
@@ -270,7 +270,7 @@ 
 
     /**
      * @param string $commandString
-     * @return mixed|string
+     * @return string
      */
     protected function _prepareShellCommand($commandString)
     {
@@ -320,7 +320,7 @@ 
 
     /**
      * @param $commandString
-     * @return mixed
+     * @return string
      */
     protected function _replaceScriptVars($commandString)
     {

Spacing +1 added lines, -1 removed lines

@@ -171,7 +171,7 @@
                 if (isset($parts[1])) {
                     $value = $parts[1];
                 }
-                $this->scriptVars['${' . $variable. '}'] = $value;
+                $this->scriptVars['${' . $variable . '}'] = $value;
             }
         }
     }