magium / Magium

lib/Util/Translator/Translator.php

Doc Comments +3 added lines

@@ -91,6 +91,9 @@
     }
 
 
+    /**
+     * @param string $textDomain
+     */
     protected function translatePart($translate, $result, $textDomain, $locale)
     {
         $newResult = parent::translate($result, $textDomain, $locale);

Spacing +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
     protected function translatePart($translate, $result, $textDomain, $locale)
     {
         $newResult = parent::translate($result, $textDomain, $locale);
-        $translate = str_replace('{{' . $result . '}}', $newResult, $translate);
+        $translate = str_replace('{{'.$result.'}}', $newResult, $translate);
         return $translate;
     }
 }
\ No newline at end of file

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/DateTime.php

Braces +3 added lines, -1 removed lines

@@ -53,7 +53,9 @@
         $max = false;
         $min = PHP_INT_MAX;
         foreach ($matchedParts as $part) {
-            if ($part === null) continue;
+            if ($part === null) {
+                continue;
+            }
             if ($part >= $max) {
                 $max = $part;
             }

Spacing +8 added lines, -8 removed lines

@@ -43,7 +43,7 @@ 
                  * We re-run the test method each time because Dec 01 01:01:01 could match only the first 01.  So we
                  * have to be a bit greedy here.
                  */
-                $fn = 'test' . ucfirst($term);
+                $fn = 'test'.ucfirst($term);
                 if ($this->$fn($part) && $matchedParts[$term] === null) {
                     $matchedParts[$term] = $key;
                 }
@@ -67,7 +67,7 @@ 
         }
         $foundDate = '';
         for ($i = (int)$min; $i <= (int)$max; $i++) {
-            $foundDate .= ' ' . $parts[$i];
+            $foundDate .= ' '.$parts[$i];
         }
         $parse = date_parse(trim($foundDate));
         if (isset($parse['error_count']) && $parse['error_count'] == 0) {
@@ -113,7 +113,7 @@ 
             return true;
         }
         $parse = date_parse($part);
-        if (!empty($parse) && $parse['day'] !== false && $parse['error_count'] == 0 ) {
+        if (!empty($parse) && $parse['day'] !== false && $parse['error_count'] == 0) {
             return true;
         } else if (preg_match('/^\d{1,2}[\/\\-]\d{1,2}[\/\\-]\d{1,2}$/', $part)) {
             return true;
@@ -127,7 +127,7 @@ 
             return true;
         }
         $parse = date_parse($part);
-        if (!empty($parse) && $parse['month'] !== false && $parse['error_count'] == 0 ) {
+        if (!empty($parse) && $parse['month'] !== false && $parse['error_count'] == 0) {
             return true;
         } else if (preg_match('/^\d{1,2}[\/\\-]\d{1,2}[\/\\-]\d{1,2}$/', $part)) {
             return true;
@@ -141,7 +141,7 @@ 
         if (!empty($parse) && $parse['year'] !== false && $parse['error_count'] == 0) {
             return true;
             // DateTime parses 2015 as 20:15:00, so this is to account for the unique condition.
-        } else if (strlen($part) ==4 ) {
+        } else if (strlen($part) == 4) {
             if ($parse['second'] === 0 && $parse['hour'] == substr($part, 0, 2) && $parse['minute'] == substr($part, 2, 2)) {
                 return true;
             }
@@ -162,7 +162,7 @@ 
             return false;
         }
         $parse = date_parse($part);
-        if (!empty($parse) && $parse['hour'] !== false && $parse['error_count'] == 0 ) {
+        if (!empty($parse) && $parse['hour'] !== false && $parse['error_count'] == 0) {
             return true;
         }
         return false;
@@ -179,7 +179,7 @@ 
             return false;
         }
         $parse = date_parse($part);
-        if (!empty($parse) && $parse['minute'] !== false && $parse['error_count'] == 0 ) {
+        if (!empty($parse) && $parse['minute'] !== false && $parse['error_count'] == 0) {
             return true;
 
         }
@@ -197,7 +197,7 @@ 
             return false;
         }
         $parse = date_parse($part);
-        if (!empty($parse) && $parse['second'] !== false && $parse['error_count'] == 0 ) {
+        if (!empty($parse) && $parse['second'] !== false && $parse['error_count'] == 0) {
             return true;
         }
         return false;

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;
             }
 
         }

Braces +1 added lines, -1 removed lines

@@ -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

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/AbstractTestCase.php

Indentation -1 removed lines

@@ -279,7 +279,6 @@
     }
 
     /**
-
      * @param string $theme
      * @return \Magium\Themes\ThemeConfigurationInterface
      */

Spacing +7 added lines, -7 removed lines

@@ -196,7 +196,7 @@ 
         }
     }
 
-    public static function resolveClass( $class, $prefix = null)
+    public static function resolveClass($class, $prefix = null)
     {
         $origClass = $class;
         if ($prefix !== null) {
@@ -210,7 +210,7 @@ 
         }
 
         foreach (self::$baseNamespaces as $namespace) {
-            $fqClass = $namespace . '\\' . $class;
+            $fqClass = $namespace.'\\'.$class;
             try {
                 if (class_exists($fqClass)) {
                     return $fqClass;
@@ -497,7 +497,7 @@ 
         try {
             $this->byXpath(sprintf('//%s[contains(., "%s")]', $node, addslashes($text)));
         } catch (\Exception $e) {
-            $this->fail('The body did not contain the text: ' . $text);
+            $this->fail('The body did not contain the text: '.$text);
         }
     }
 
@@ -507,7 +507,7 @@ 
             $this->webdriver->byXpath(sprintf('//body[contains(., "%s")]', $text));
             // If the element is not found then an exception will be thrown
         } catch (\Exception $e) {
-            $this->fail('The body did not contain the text: ' . $text);
+            $this->fail('The body did not contain the text: '.$text);
         }
 
     }
@@ -516,7 +516,7 @@ 
     {
         try {
             $this->webdriver->byXpath(sprintf('//body[contains(., "%s")]', $text));
-            $this->fail('The page contains the words: ' . $text);
+            $this->fail('The page contains the words: '.$text);
         } catch (NoSuchElementException $e) {
             // Exception thrown is a success
         }
@@ -556,7 +556,7 @@ 
     {
 
         if ($parentElementSelector !== null) {
-            $xpathTemplate = $parentElementSelector . $xpathTemplate;
+            $xpathTemplate = $parentElementSelector.$xpathTemplate;
         }
         if ($specificNodeType !== null) {
             return $this->byXpath(sprintf($xpathTemplate, $specificNodeType, $this->getTranslator()->translatePlaceholders($text)));
@@ -569,7 +569,7 @@ 
             }
         }
         // This is here for consistency with the other by* methods
-        WebDriverException::throwException(7, 'Could not find element with text: ' . $this->getTranslator()->translatePlaceholders($text), []);
+        WebDriverException::throwException(7, 'Could not find element with text: '.$this->getTranslator()->translatePlaceholders($text), []);
     }
 
     /**

Doc Comments +19 added lines, -1 removed lines

@@ -175,6 +175,9 @@ 
         }
     }
 
+    /**
+     * @param null|string $selector
+     */
     public function assertElementClickable($selector, $by = WebDriver::BY_ID)
     {
         $this->elementAssertion($selector, $by, Clickable::ASSERTION);
@@ -187,6 +190,9 @@ 
 
     }
 
+    /**
+     * @param string $namespace
+     */
     public static function addBaseNamespace($namespace)
     {
         if (!in_array($namespace, self::$baseNamespaces)) {
@@ -194,6 +200,9 @@ 
         }
     }
 
+    /**
+     * @param string $prefix
+     */
     public static function resolveClass( $class, $prefix = null)
     {
         $origClass = $class;
@@ -235,6 +244,9 @@ 
 
     }
 
+    /**
+     * @return string
+     */
     protected function normalizeClassRequest($class)
     {
         return str_replace('/', '\\', $class);
@@ -521,7 +533,7 @@ 
     }
 
     /**
-     * @param $xpath
+     * @param string $xpath
      * @return \Facebook\WebDriver\Remote\RemoteWebElement
      */
 
@@ -550,6 +562,12 @@ 
         return $this->webdriver->byCssSelector($selector);
     }
 
+    /**
+     * @param string $xpathTemplate
+     * @param string $text
+     * @param string $specificNodeType
+     * @param string $parentElementSelector
+     */
     protected function getElementByTextXpath($xpathTemplate, $text, $specificNodeType = null, $parentElementSelector = null)
     {
 

lib/Assertions/Element/Clickable.php

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 namespace Magium\Assertions\Element;
 
 use Facebook\WebDriver\WebDriverBy;
-use Magium\AbstractTestCase;
-use Magium\InvalidTestTypeException;
 use Magium\WebDriver\ExpectedCondition;
 
 class Clickable extends AbstractSelectorAssertion