netz98 / n98-magerun

src/N98/Magento/Command/System/Check/Settings/BaseUrlCheckAbstract.php

Spacing +6 added lines, -6 removed lines

@@ -20,7 +20,7 @@ 
 
     public function initConfigPaths()
     {
-        $this->registerStoreConfigPath('baseUrl', 'web/' . $this->class . '/base_url');
+        $this->registerStoreConfigPath('baseUrl', 'web/'.$this->class.'/base_url');
     }
 
     /**
@@ -33,17 +33,17 @@ 
         $errorMessage = 'Wrong hostname configured. <info>Hostname must contain a dot</info>';
 
         $host    = parse_url($baseUrl, PHP_URL_HOST);
-        $isValid = (bool) strstr($host, '.');
+        $isValid = (bool)strstr($host, '.');
         $result->setStatus($isValid);
         if ($isValid) {
             $result->setMessage(
-                '<info>' . ucfirst($this->class) . ' BaseURL: <comment>' . $baseUrl . '</comment> of Store: <comment>' .
-                $store->getCode() . '</comment> - OK'
+                '<info>'.ucfirst($this->class).' BaseURL: <comment>'.$baseUrl.'</comment> of Store: <comment>'.
+                $store->getCode().'</comment> - OK'
             );
         } else {
             $result->setMessage(
-                '<error>Invalid ' . ucfirst($this->class) . ' BaseURL: <comment>' . $baseUrl .
-                '</comment> of Store: <comment>' . $store->getCode() . '</comment> ' . $errorMessage . '</error>'
+                '<error>Invalid '.ucfirst($this->class).' BaseURL: <comment>'.$baseUrl.
+                '</comment> of Store: <comment>'.$store->getCode().'</comment> '.$errorMessage.'</error>'
             );
         }
     }

src/N98/Magento/Command/System/Check/PHP/BytecodeCacheExtensionsCheck.php

Spacing +2 added lines, -2 removed lines

@@ -36,8 +36,8 @@
             $result->setMessage("<info>Bytecode Cache <comment>$bytecodeCacheExtension</comment> found.</info>");
         } else {
             $result->setMessage(
-                "<error>No Bytecode-Cache found!</error> <comment>It's recommended to install anyone of " .
-                implode(', ', $bytecopdeCacheExtensions) . ".</comment>"
+                "<error>No Bytecode-Cache found!</error> <comment>It's recommended to install anyone of ".
+                implode(', ', $bytecopdeCacheExtensions).".</comment>"
             );
         }
     }

src/N98/Magento/Command/System/Check/Security/LocalConfigAccessableCheck.php

Spacing +3 added lines, -3 removed lines

@@ -20,13 +20,13 @@
     {
         $result = $results->createResult();
         $filePath = 'app/etc/local.xml';
-        $defaultUnsecureBaseURL = (string) \Mage::getConfig()->getNode(
-            'default/' . \Mage_Core_Model_Store::XML_PATH_UNSECURE_BASE_URL
+        $defaultUnsecureBaseURL = (string)\Mage::getConfig()->getNode(
+            'default/'.\Mage_Core_Model_Store::XML_PATH_UNSECURE_BASE_URL
         );
 
         $http = new \Varien_Http_Adapter_Curl();
         $http->setConfig(array('timeout' => $this->_verificationTimeOut));
-        $http->write(\Zend_Http_Client::POST, $defaultUnsecureBaseURL . $filePath);
+        $http->write(\Zend_Http_Client::POST, $defaultUnsecureBaseURL.$filePath);
         $responseBody = $http->read();
         $responseCode = \Zend_Http_Response::extractCode($responseBody);
         $http->close();

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

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
                 'format',
                 null,
                 InputOption::VALUE_OPTIONAL,
-                'Output Format. One of [' . implode(',', RendererFactory::getFormats()) . ']'
+                'Output Format. One of ['.implode(',', RendererFactory::getFormats()).']'
             )
         ;
     }
@@ -53,7 +53,7 @@ 
         $timezone = $input->getOption('timezone')
             ? $input->getOption('timezone') : \Mage::app()->getStore()->getConfig('general/locale/timezone');
 
-        $output->writeln('<info>Times shown in <comment>' . $timezone . '</comment></info>');
+        $output->writeln('<info>Times shown in <comment>'.$timezone.'</comment></info>');
 
         $date       = \Mage::getSingleton('core/date');
         $offset     = $date->calculateOffset($timezone);

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

Spacing +4 added lines, -4 removed lines

@@ -61,7 +61,7 @@ 
             $runConfig = $jobConfig->run;
 
             if ($runConfig->model) {
-                if (!preg_match(self::REGEX_RUN_MODEL, (string) $runConfig->model, $run)) {
+                if (!preg_match(self::REGEX_RUN_MODEL, (string)$runConfig->model, $run)) {
                     throw new RuntimeException('Invalid model/method definition, expecting "model/class::method".');
                 }
                 if (!($model = \Mage::getModel($run[1])) || !method_exists($model, $run[2])) {
@@ -69,7 +69,7 @@ 
                 }
                 $callback = array($model, $run[2]);
 
-                $output->write('<info>Run </info><comment>' . get_class($model) . '::' . $run[2] . '</comment> ');
+                $output->write('<info>Run </info><comment>'.get_class($model).'::'.$run[2].'</comment> ');
 
                 \Mage::getConfig()->init()->loadEventObservers('crontab');
                 \Mage::app()->addEventArea('crontab');
@@ -118,9 +118,9 @@ 
         $question = array();
 
         foreach ($jobs as $key => $job) {
-            $question[] = '<comment>[' . ($index++) . ']</comment> ' . $job['Job'] . PHP_EOL;
+            $question[] = '<comment>['.($index++).']</comment> '.$job['Job'].PHP_EOL;
         }
-        $question[] = '<question>Please select job: </question>' . PHP_EOL;
+        $question[] = '<question>Please select job: </question>'.PHP_EOL;
 
         /** @var $dialogHelper DialogHelper */
         $dialogHelper = $this->getHelperSet()->get('dialog');

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

Spacing +13 added lines, -13 removed lines

@@ -32,7 +32,7 @@ 
                 'format',
                 null,
                 InputOption::VALUE_OPTIONAL,
-                'Output Format. One of [' . implode(',', RendererFactory::getFormats()) . ']'
+                'Output Format. One of ['.implode(',', RendererFactory::getFormats()).']'
             )
         ;
     }
@@ -77,7 +77,7 @@ 
                 $this->categoryCount();
                 $this->productCount();
             } catch (Exception $e) {
-                $output->writeln('<error>' . $e->getMessage() . '</error>');
+                $output->writeln('<error>'.$e->getMessage().'</error>');
             }
         }
 
@@ -90,9 +90,9 @@ 
             $settingArgument = strtolower($settingArgument);
             $this->infos = array_change_key_case($this->infos, CASE_LOWER);
             if (!isset($this->infos[$settingArgument])) {
-                throw new InvalidArgumentException('Unknown key: ' . $settingArgument);
+                throw new InvalidArgumentException('Unknown key: '.$settingArgument);
             }
-            $output->writeln((string) $this->infos[$settingArgument]);
+            $output->writeln((string)$this->infos[$settingArgument]);
         } else {
             $this->getHelper('table')
                 ->setHeaders(array('name', 'value'))
@@ -120,10 +120,10 @@ 
     protected function findCoreOverwrites()
     {
         $folders = array(
-            $this->_magentoRootFolder . '/app/code/local/Mage',
-            $this->_magentoRootFolder . '/app/code/local/Enterprise',
-            $this->_magentoRootFolder . '/app/code/community/Mage',
-            $this->_magentoRootFolder . '/app/code/community/Enterprise',
+            $this->_magentoRootFolder.'/app/code/local/Mage',
+            $this->_magentoRootFolder.'/app/code/local/Enterprise',
+            $this->_magentoRootFolder.'/app/code/community/Mage',
+            $this->_magentoRootFolder.'/app/code/community/Enterprise',
         );
         foreach ($folders as $key => $folder) {
             if (!is_dir($folder)) {
@@ -147,12 +147,12 @@ 
     protected function findVendors()
     {
         $codePools = array(
-            'core'      => $this->_magentoRootFolder . '/app/code/core/',
-            'community' => $this->_magentoRootFolder . '/app/code/community/',
+            'core'      => $this->_magentoRootFolder.'/app/code/core/',
+            'community' => $this->_magentoRootFolder.'/app/code/community/',
         );
 
-        if (is_dir($this->_magentoRootFolder . '/app/code/local/')) {
-            $codePools['local'] = $this->_magentoRootFolder . '/app/code/local/';
+        if (is_dir($this->_magentoRootFolder.'/app/code/local/')) {
+            $codePools['local'] = $this->_magentoRootFolder.'/app/code/local/';
         }
 
         foreach ($codePools as $codePool => $codePoolDir) {
@@ -172,7 +172,7 @@ 
                 $vendors
             );
 
-            $this->infos['Vendors (' . $codePool . ')'] = implode(', ', $vendors);
+            $this->infos['Vendors ('.$codePool.')'] = implode(', ', $vendors);
         }
     }
 

src/N98/Magento/Command/System/Setup/RunCommand.php

Spacing +3 added lines, -3 removed lines

@@ -98,8 +98,8 @@ 
         foreach ($trace as $row) {
             $rows[] = array(
                 $i++,
-                $row['file'] . ':' . $row['line'],
-                $row['class'] . '::' . $row['function']
+                $row['file'].':'.$row['line'],
+                $row['class'].'::'.$row['function']
             );
         }
         $table->setHeaders(array('#', 'File/Line', 'Method'));
@@ -114,7 +114,7 @@ 
      */
     protected function printException(OutputInterface $output, $e)
     {
-        $output->writeln('<error>' . $e->getMessage() . '</error>');
+        $output->writeln('<error>'.$e->getMessage().'</error>');
     }
 
     /**

src/N98/Magento/Command/System/Setup/CompareVersionsCommand.php

Spacing +6 added lines, -6 removed lines

@@ -27,7 +27,7 @@ 
                 'format',
                 null,
                 InputOption::VALUE_OPTIONAL,
-                'Output Format. One of [' . implode(',', RendererFactory::getFormats()) . ']'
+                'Output Format. One of ['.implode(',', RendererFactory::getFormats()).']'
             )
             ->setDescription('Compare module version with core_resource table.');
         $help = <<<HELP
@@ -62,11 +62,11 @@ 
             $errorCounter = 0;
             $table = array();
             foreach ($setups as $setupName => $setup) {
-                $moduleName     = (string) $setup->setup->module;
-                $moduleVersion  = (string) $modules->{$moduleName}->version;
-                $dbVersion      = (string) $resourceModel->getDbVersion($setupName);
+                $moduleName     = (string)$setup->setup->module;
+                $moduleVersion  = (string)$modules->{$moduleName}->version;
+                $dbVersion      = (string)$resourceModel->getDbVersion($setupName);
                 if (!$ignoreDataUpdate) {
-                    $dataVersion = (string) $resourceModel->getDataVersion($setupName);
+                    $dataVersion = (string)$resourceModel->getDataVersion($setupName);
                 }
                 $ok = $dbVersion == $moduleVersion;
                 if ($ok && !$ignoreDataUpdate) {
@@ -171,7 +171,7 @@ 
     {
         $document = new JUnitXmlDocument();
         $suite = $document->addTestSuite();
-        $suite->setName('n98-magerun: ' . $this->getName());
+        $suite->setName('n98-magerun: '.$this->getName());
         $suite->setTimestamp(new \DateTime());
         $suite->setTime($duration);
 

src/N98/Magento/Command/System/Setup/IncrementalCommand.php

Spacing +18 added lines, -18 removed lines

@@ -56,7 +56,7 @@ 
             ->setDescription('List new setup scripts to run, then runs one script')
             ->addOption('stop-on-error', null, InputOption::VALUE_NONE, 'Stops execution of script on error')
             ->setHelp(
-                'Examines an un-cached configuration tree and determines which ' .
+                'Examines an un-cached configuration tree and determines which '.
                 'structure and data setup resource scripts need to run, and then runs them.'
             );
     }
@@ -310,12 +310,12 @@ 
             $output->writeln(
                 array(
                     '+--------------------------------------------------+',
-                    'Resource Name:             ' . $name,
-                    'For Module:                ' . $moduleConfig->getName(),
-                    'Class:                     ' . get_class($setupResource),
-                    'Current Structure Version: ' . $dbVersion,
-                    'Current Data Version:      ' . $dbDataVersion,
-                    'Configured Version:        ' . $configVersion
+                    'Resource Name:             '.$name,
+                    'For Module:                '.$moduleConfig->getName(),
+                    'Class:                     '.get_class($setupResource),
+                    'Current Structure Version: '.$dbVersion,
+                    'Current Data Version:      '.$dbDataVersion,
+                    'Configured Version:        '.$configVersion
                 )
             );
 
@@ -356,7 +356,7 @@ 
             return;
         }
         foreach ($files as $file) {
-            $output->writeln(str_replace(\Mage::getBaseDir() . '/', '', $file['fileName']));
+            $output->writeln(str_replace(\Mage::getBaseDir().'/', '', $file['fileName']));
         }
     }
 
@@ -386,11 +386,11 @@ 
     {
         $output = $this->_output;
         if (!in_array($type, array(self::TYPE_MIGRATION_STRUCTURE, self::TYPE_MIGRATION_DATA))) {
-            throw new RuntimeException('Invalid Type [' . $type . ']: structure, data is valid');
+            throw new RuntimeException('Invalid Type ['.$type.']: structure, data is valid');
         }
 
         if (!array_key_Exists($name, $needsUpdate)) {
-            $output->writeln('<error>No updates to run for ' . $name . ', skipping </error>');
+            $output->writeln('<error>No updates to run for '.$name.', skipping </error>');
 
             return;
         }
@@ -407,17 +407,17 @@ 
         }
         //recreate our specific node in <global><resources></resource></global>
         //allows for theoretical multiple runs
-        $setupResourceConfig = $this->_secondConfig->getNode('global/resources/' . $name);
+        $setupResourceConfig = $this->_secondConfig->getNode('global/resources/'.$name);
         $moduleName = $setupResourceConfig->setup->module;
         $className = $setupResourceConfig->setup->class;
 
-        $specificResource = $realConfig->getNode('global/resources/' . $name);
+        $specificResource = $realConfig->getNode('global/resources/'.$name);
         $setup = $specificResource->addChild('setup');
         if ($moduleName) {
             $setup->addChild('module', $moduleName);
         } else {
             $output->writeln(
-                '<error>No module node configured for ' . $name . ', possible configuration error </error>'
+                '<error>No module node configured for '.$name.', possible configuration error </error>'
             );
         }
 
@@ -520,7 +520,7 @@ 
     protected function _runStructureOrDataScripts($toUpdate, array $needsUpdate, $type)
     {
         $output = $this->_output;
-        $output->writeln('The next ' . $type . ' update to run is <info>' . $toUpdate . '</info>');
+        $output->writeln('The next '.$type.' update to run is <info>'.$toUpdate.'</info>');
         $this->getHelper('dialog')->askAndValidate(
             $output,
             '<question>Press Enter to Run this update: </question>'
@@ -530,8 +530,8 @@ 
         $this->_runNamedSetupResource($toUpdate, $needsUpdate, $type);
         $time_ran = microtime(true) - $start;
         $output->writeln('');
-        $output->writeln(ucwords($type) . ' update <info>' . $toUpdate . '</info> complete.');
-        $output->writeln('Ran in ' . floor($time_ran * 1000) . 'ms');
+        $output->writeln(ucwords($type).' update <info>'.$toUpdate.'</info> complete.');
+        $output->writeln('Ran in '.floor($time_ran * 1000).'ms');
     }
 
     /**
@@ -590,10 +590,10 @@ 
         $needsUpdate = $this->_getAllSetupResourceObjectThatNeedUpdates($setupResources);
 
         $output->writeln(
-            'Found <info>' . count($setupResources) . '</info> configured setup resource(s)</info>'
+            'Found <info>'.count($setupResources).'</info> configured setup resource(s)</info>'
         );
         $output->writeln(
-            'Found <info>' . count($needsUpdate) . '</info> setup resource(s) which need an update</info>'
+            'Found <info>'.count($needsUpdate).'</info> setup resource(s) which need an update</info>'
         );
 
         return $needsUpdate;