magium / Magium

example_configuration_not_used/Magium/Themes/ThemeConfiguration.php

Spacing +1 added lines, -1 removed lines

@@ -1,3 +1,3 @@
 <?php
 
-$this->baseUrl                      = 'http://magento19.loc//';
\ No newline at end of file
+$this->baseUrl = 'http://magento19.loc//';
\ No newline at end of file

lib/Extractors/Navigation/UnableToExtractMenuXpathException.php

Spacing +1 added lines, -1 removed lines

@@ -2,4 +2,4 @@
 
 namespace Magium\Extractors\Navigation;
 
-class UnableToExtractMenuXpathException extends \Exception{}
\ No newline at end of file
+class UnableToExtractMenuXpathException extends \Exception {}
\ No newline at end of file

lib/Extractors/Navigation/MissingNavigationSchemeException.php

Spacing +1 added lines, -1 removed lines

@@ -2,4 +2,4 @@
 
 namespace Magium\Extractors\Navigation;
 
-class MissingNavigationSchemeException extends \Exception{}
\ No newline at end of file
+class MissingNavigationSchemeException extends \Exception {}
\ No newline at end of file

lib/Extractors/AbstractAddressExtractor.php

Spacing +5 added lines, -5 removed lines

@@ -31,10 +31,10 @@ 
         $this->name = trim(array_pop($rows));
         while (count($rows) > 0) {
             $row = array_shift($rows);
-            if (strpos($row, 'T: ') ===0) {
+            if (strpos($row, 'T: ') === 0) {
                 $this->phone = trim(substr($row, 3));
                 continue;
-            } else if (strpos($row, 'F: ') ===0) {
+            } else if (strpos($row, 'F: ') === 0) {
                 $this->fax = trim(substr($row, 3));
                 continue;
             }
@@ -59,7 +59,7 @@ 
             $this->street1 = trim(array_shift($rows));
             $this->business = trim(array_shift($rows));
             //Easy
-        }else if (count($rows) == 1) {
+        } else if (count($rows) == 1) {
             $this->street1 = trim(array_shift($rows));
 
             // Not so easy
@@ -70,8 +70,8 @@ 
                 $this->street1 = $stOrBus;
                 $this->business = trim(array_shift($rows));
             } else {
-                $this->street1 =trim(array_shift($rows));
-                $this->street2 =  $stOrBus;
+                $this->street1 = trim(array_shift($rows));
+                $this->street2 = $stOrBus;
             }
 
         }

lib/Assertions/LoggingAssertionExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
         }
         try {
             $assertion->assert();
-            $this->logger->logAssertionSuccess( $assertion, $extra );
+            $this->logger->logAssertionSuccess($assertion, $extra);
         } catch (\Exception $e) {
             $this->logger->logAssertionFailure($assertion, $extra);
             throw $e;

lib/WebDriver/ExpectedCondition.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
     public static function elementExists($selector, $by = 'byId')
     {
         return new WebDriverExpectedCondition(
-            function ($driver) use ($selector, $by) {
+            function($driver) use ($selector, $by) {
                 try {
                     $element = $driver->$by($selector);
                     return $element instanceof WebDriverElement;
@@ -26,7 +26,7 @@ 
     public static function elementRemoved(WebDriverElement $element)
     {
         return new WebDriverExpectedCondition(
-            function () use ($element) {
+            function() use ($element) {
                 try {
                     $element->isDisplayed();
                     return false;

lib/WebDriver/WebDriverElementProxy.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     public function __call($name, $arguments)
     {
         if (strpos($name, '::') > 0) {
-            list(   , $name) = explode('::', $name);
+            list(, $name) = explode('::', $name);
         }
         if (!$this->element instanceof WebDriverElement) {
             $this->element = $this->webDriver->{$this->by}($this->selector);

lib/Cli/Command/ApiPing.php

Unused Use Statements -4 removed lines

@@ -5,13 +5,9 @@
 use Guzzle\Http\Client;
 use Magium\Cli\Command\Test\TestSkeleton;
 use Magium\InvalidConfigurationException;
-use Magium\NotFoundException;
 use Magium\Util\Api\ApiConfiguration;
 use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\ArrayInput;
-use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
 class ApiPing extends Command

Spacing +12 added lines, -12 removed lines

@@ -31,26 +31,26 @@ 
         $api = $test->get('Magium\Util\Api\ApiConfiguration');
         /* @var $api ApiConfiguration */
         if (!$api instanceof ApiConfiguration) {
-            throw new InvalidConfigurationException('Did not receive an instance of ApiConfiguration; received ' . get_class($api));
+            throw new InvalidConfigurationException('Did not receive an instance of ApiConfiguration; received '.get_class($api));
         }
         $api->setEnabled(true); // Gotta force this for this test
 
         $output->writeln('Sending un-authenticated payload...');
-        $client = new Client('http://' . $api->getApiHostname(). '/api/ping');
+        $client = new Client('http://'.$api->getApiHostname().'/api/ping');
         $request = $client->get();
         $response = $request->send();
         $output->writeln(
             'Checking for 200 status message... '
-            . ($response->getStatusCode() == '200'?'OK':'Failed')
+            . ($response->getStatusCode() == '200' ? 'OK' : 'Failed')
         );
         $output->writeln(
             'Checking for application/json content type... '
-            . (stripos($response->getContentType(), 'application/json') !== false?'OK':'Failed')
+            . (stripos($response->getContentType(), 'application/json') !== false ? 'OK' : 'Failed')
         );
         $content = json_decode($response->getBody(), true);
         $output->writeln(
             'Checking for successful response... '
-            . (is_array($content) && isset($content['status']) && $content['status'] === 'success'?'OK':'Failed')
+            . (is_array($content) && isset($content['status']) && $content['status'] === 'success' ? 'OK' : 'Failed')
         );
 
         $output->writeln('');
@@ -62,16 +62,16 @@ 
 
         $output->writeln(
             'Checking for 200 status message... '
-            . ($response->getStatusCode() == '200'?'OK':'Failed')
+            . ($response->getStatusCode() == '200' ? 'OK' : 'Failed')
         );
         $output->writeln(
             'Checking for application/json content type... '
-            . (stripos($response->getContentType(), 'application/json') !== false?'OK':'Failed')
+            . (stripos($response->getContentType(), 'application/json') !== false ? 'OK' : 'Failed')
         );
         $content = $request->getPayload($response);
         $output->writeln(
             'Checking for successful response... '
-            . (is_array($content) && isset($content['status']) && $content['status'] === 'success'?'OK':'Failed')
+            . (is_array($content) && isset($content['status']) && $content['status'] === 'success' ? 'OK' : 'Failed')
         );
 
 
@@ -82,20 +82,20 @@ 
         $response = $request->push('/api/echo-authed', ['message' => 'hello world']);
         $output->writeln(
             'Checking for 200 status message... '
-            . ($response->getStatusCode() == '200'?'OK':'Failed')
+            . ($response->getStatusCode() == '200' ? 'OK' : 'Failed')
         );
         $output->writeln(
             'Checking for application/json content type... '
-            . (stripos($response->getContentType(), 'application/json') !== false?'OK':'Failed')
+            . (stripos($response->getContentType(), 'application/json') !== false ? 'OK' : 'Failed')
         );
         $content = $request->getPayload($response);
         $output->writeln(
             'Checking for successful response... '
-            . (is_array($content) && isset($content['status']) && $content['status'] === 'success'?'OK':'Failed')
+            . (is_array($content) && isset($content['status']) && $content['status'] === 'success' ? 'OK' : 'Failed')
         );
         $output->writeln(
             'Checking for matching echo message... '
-            . (is_array($content) && isset($content['message']) && $content['message'] === 'hello world'?'OK':'Failed')
+            . (is_array($content) && isset($content['message']) && $content['message'] === 'hello world' ? 'OK' : 'Failed')
         );
     }
     

lib/Cli/CommandLoader.php

Spacing +4 added lines, -4 removed lines

@@ -19,7 +19,7 @@ 
     )
     {
         if (!is_dir($configDir)) {
-            throw new NotFoundException('Configuration dir not found: ' . $configDir);
+            throw new NotFoundException('Configuration dir not found: '.$configDir);
         }
         $this->config = $configDir;
 
@@ -28,14 +28,14 @@ 
 
     public function load()
     {
-        self::addCommandDir('Magium\Cli\Command', __DIR__ . '/Command');
+        self::addCommandDir('Magium\Cli\Command', __DIR__.'/Command');
 
         foreach (self::$dirs as $namespace => $dir) {
-            $files = glob($dir . '/*.php');
+            $files = glob($dir.'/*.php');
 
             foreach ($files as $file) {
                 $class = substr(basename($file), 0, -4);
-                $className = $namespace . '\\' . $class;
+                $className = $namespace.'\\'.$class;
                 if (class_exists($className)) {
                     $reflection = new \ReflectionClass($className);
                     if (!$reflection->isInstantiable()) {