netz98 / n98-magerun2

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

Indentation +4 added lines, -4 removed lines

@@ -28,10 +28,10 @@
             @chmod($varCacheFolder, 0777);
 
             $mediaFolder = $installationFolder
-                         . DIRECTORY_SEPARATOR
-                         . 'pub'
-                         . DIRECTORY_SEPARATOR
-                         . 'media';
+                            . DIRECTORY_SEPARATOR
+                            . 'pub'
+                            . DIRECTORY_SEPARATOR
+                            . 'media';
             if (!is_dir($mediaFolder)) {
                 @mkdir($mediaFolder);
             }

Doc Comments +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
      *
      * @param Bootstrap $bootstrap
      * @param \Exception $exception
-     * @return bool
+     * @return boolean|null
      */
     public function catchException(Bootstrap $bootstrap, \Exception $exception)
     {

src/N98/Magento/Command/SelfUpdateCommand.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
     protected function execute(InputInterface $input, OutputInterface $output)
     {
         $localFilename = realpath($_SERVER['argv'][0]) ?: $_SERVER['argv'][0];
-        $tempFilename = dirname($localFilename) . '/' . basename($localFilename, '.phar').'-temp.phar';
+        $tempFilename = dirname($localFilename) . '/' . basename($localFilename, '.phar') . '-temp.phar';
 
         // check for permissions in local filesystem before start connection process
         if (!is_writable($tempDirectory = dirname($tempFilename))) {
@@ -128,7 +128,7 @@ 
                 if (!$e instanceof \UnexpectedValueException && !$e instanceof \PharException) {
                     throw $e;
                 }
-                $output->writeln('<error>The download is corrupted ('.$e->getMessage().').</error>');
+                $output->writeln('<error>The download is corrupted (' . $e->getMessage() . ').</error>');
                 $output->writeln('<error>Please re-run the self-update command to try again.</error>');
             }
         } else {

src/N98/Magento/Command/SubCommand/SubCommandFactory.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
         }
 
         $subCommand = new $className();
-        if (! $subCommand instanceof SubCommandInterface) {
+        if (!$subCommand instanceof SubCommandInterface) {
             throw new \InvalidArgumentException('Subcommand must implement SubCommandInterface.');
         }
 

src/N98/Magento/Command/System/Cron/HistoryCommand.php

Spacing +1 added lines, -2 removed lines

@@ -47,8 +47,7 @@
         }
 
         $timezone = $input->getOption('timezone')
-            ? $input->getOption('timezone') :
-            $this->_scopeConfig->getValue('general/locale/timezone');
+            ? $input->getOption('timezone') : $this->_scopeConfig->getValue('general/locale/timezone');
 
         if (!$input->getOption('format')) {
             $output->writeln('<info>Times shown in <comment>' . $timezone . '</comment></info>');

src/N98/Magento/Command/System/InfoCommand.php

Indentation +11 added lines, -11 removed lines

@@ -151,32 +151,32 @@
     protected function addProductCount()
     {
         $this->infos['Product Count'] = $this->productFactory
-                                             ->create()
-                                             ->getCollection()
-                                             ->getSize();
+                                                ->create()
+                                                ->getCollection()
+                                                ->getSize();
     }
 
     protected function addCustomerCount()
     {
         $this->infos['Customer Count'] = $this->customerFactory->create()
-                                              ->getCollection()
-                                              ->getSize();
+                                                ->getCollection()
+                                                ->getSize();
     }
 
     protected function addCategoryCount()
     {
         $this->infos['Category Count'] = $this->categoryFactory
-                                              ->create()
-                                              ->getCollection()
-                                              ->getSize();
+                                                ->create()
+                                                ->getCollection()
+                                                ->getSize();
     }
 
     protected function addAttributeCount()
     {
         $this->infos['Attribute Count'] = $this->attributeFactory
-                                               ->create()
-                                               ->getCollection()
-                                               ->getSize();
+                                                ->create()
+                                                ->getCollection()
+                                                ->getSize();
     }
 
     protected function addCacheInfos()

Spacing +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
 
     public function hasInfo()
     {
-        return ! empty($this->infos);
+        return !empty($this->infos);
     }
 
     public function getInfo($key = null)

src/N98/Util/Console/Helper/InjectionHelper.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
             $argumentsToInject[] = $objectManager->get($parameter[1]);
         }
 
-        call_user_func_array([ $object, $methodName ], $argumentsToInject);
+        call_user_func_array([$object, $methodName], $argumentsToInject);
     }
 
     /**

tests/N98/Magento/Command/Cache/EnableCommandTest.php

Indentation +4 added lines, -4 removed lines

@@ -1,9 +1,9 @@
 <?php
 /**
- * @todo    writing unit tests for toggling caches is complicated because environments differ
- *          touching caches changes state and will result in different test result second time
- *          solutions: disabling/enabling, faking&mocking or having one defined test environment
- */
+     * @todo    writing unit tests for toggling caches is complicated because environments differ
+     *          touching caches changes state and will result in different test result second time
+     *          solutions: disabling/enabling, faking&mocking or having one defined test environment
+     */
 namespace N98\Magento\Command\Cache;
 
 use Symfony\Component\Console\Tester\CommandTester;

tests/N98/Magento/Command/Cache/ListCommandTest.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
         $disabledCacheTypes = 0;
 
         foreach ($cacheTypes as $cacheType) {
-            if (! $cacheType->getStatus()) {
+            if (!$cacheType->getStatus()) {
                 $disabledCacheTypes++;
             }
         }

tests/N98/Magento/Command/Config/DeleteCommandTest.php

Indentation +8 added lines, -8 removed lines

@@ -47,11 +47,11 @@ 
             $commandTester = new CommandTester($setCommand);
             $commandTester->execute(
                 array(
-                     'command'    => $setCommand->getName(),
-                     'path'       => 'n98_magerun/foo/bar',
-                     '--scope'    => 'stores',
-                     '--scope-id'  => $store->getId(),
-                     'value'      => 'store-' . $store->getId(),
+                        'command'    => $setCommand->getName(),
+                        'path'       => 'n98_magerun/foo/bar',
+                        '--scope'    => 'stores',
+                        '--scope-id'  => $store->getId(),
+                        'value'      => 'store-' . $store->getId(),
                 )
             );
         }
@@ -59,9 +59,9 @@ 
         $commandTester = new CommandTester($deleteCommand);
         $commandTester->execute(
             array(
-                 'command' => $deleteCommand->getName(),
-                 'path'    => 'n98_magerun/foo/bar',
-                 '--all'   => true,
+                    'command' => $deleteCommand->getName(),
+                    'path'    => 'n98_magerun/foo/bar',
+                    '--all'   => true,
             )
         );