magium / MagiumMagento

lib/Magento/Actions/Checkout/ShippingMethods/ByName.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
                 )
             );
         } catch (\Exception $e) {
-            throw new NoSuchShippingMethodException('Unable to find the shipping method: ' . $this->name);
+            throw new NoSuchShippingMethodException('Unable to find the shipping method: '.$this->name);
         }
 
         // Some products, such as virtual products, do not get shipped

lib/Magento/Actions/Customer/Register.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 
         $this->form->execute($registerForNewsletter);
 
-        $submitElement     = $this->webdriver->byXpath($this->theme->getRegisterSubmitXpath());
+        $submitElement = $this->webdriver->byXpath($this->theme->getRegisterSubmitXpath());
 
         $submitElement->click();
 

lib/Magento/Actions/Checkout/Steps/BillingAddress.php

Doc Comments +5 added lines, -1 removed lines

@@ -38,7 +38,7 @@ 
      *
      * @see Magium\Magento\Actions\Checkout\Steps\CustomerBillingAddress
      *
-     * @param $element The name of the element
+     * @param string $element The name of the element
      */
 
     public function bypassElement($element)
@@ -88,6 +88,10 @@ 
         return array_search($xpath, $this->bypass) === false;
     }
 
+    /**
+     * @param string $xpath
+     * @param string $data
+     */
     protected function sendData($xpath, $data)
     {
         if ($this->shouldProcess($xpath)) {

lib/Magento/Actions/Checkout/Steps/ShippingAddress.php

Doc Comments +4 added lines

@@ -62,6 +62,10 @@
         return false;
     }
 
+    /**
+     * @param string $xpath
+     * @param string $data
+     */
     protected function sendData($xpath, $data)
     {
         if ($xpath) {

lib/Magento/Actions/Checkout/PaymentMethods/SagePay.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
 
         $this->webDriver->byId('sagepaydirectpro_cc_owner')->clear();
         $this->webDriver->byId('sagepaydirectpro_cc_owner')->sendKeys(
-            $this->customer->getBillingFirstName() . ' ' . $this->customer->getBillingLastName()
+            $this->customer->getBillingFirstName().' '.$this->customer->getBillingLastName()
         );
 
     }

lib/Magento/Actions/Customer/FillRegistrationForm.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
         $emailElement      = $this->webdriver->byXpath($this->theme->getRegisterEmailXpath());
         $passwordElement   = $this->webdriver->byXpath($this->theme->getRegisterPasswordXpath());
         $confirmElement    = $this->webdriver->byXpath($this->theme->getRegisterConfirmPasswordXpath());
-        $registerElement     = $this->webdriver->byXpath($this->theme->getRegisterNewsletterXpath());
+        $registerElement = $this->webdriver->byXpath($this->theme->getRegisterNewsletterXpath());
 
         $firstnameElement->sendKeys($this->customerIdentity->getFirstName());
         $lastnameElement->sendKeys($this->customerIdentity->getLastName());

lib/Magento/Navigators/BaseMenu.php

Spacing +2 added lines, -2 removed lines

@@ -36,9 +36,9 @@
     {
 
         usleep(500000); // Give the UI some time to update
-        $xpath .= '/descendant::' . $this->themeConfiguration->getNavigationChildXPathSelector($path);
+        $xpath .= '/descendant::'.$this->themeConfiguration->getNavigationChildXPathSelector($path);
 
-        $element = $this->webdriver->byXpath($xpath . '/a');
+        $element = $this->webdriver->byXpath($xpath.'/a');
 
         $this->execute($element);
 

lib/Magento/AbstractMagentoTestCase.php

Spacing +7 added lines, -7 removed lines

@@ -29,7 +29,7 @@ 
 
         // If we are passed just the class name we will prepend it with Magium\Magento\Actions\Checkout\PaymentMethods
         if (strpos($method, '\\') === false) {
-            $method = 'Checkout\PaymentMethods\\' . $method;
+            $method = 'Checkout\PaymentMethods\\'.$method;
             $method = self::resolveClass($method, 'Actions');
         }
         $reflection = new \ReflectionClass($method);
@@ -81,7 +81,7 @@ 
 
         // When just the class name is passed we will prepend it with Magium\Magento\Actions\Checkout\PaymentMethods
         if (strpos($method, '\\') === false) {
-            $method = 'Magium\Magento\Actions\Checkout\ShippingMethods\\' . $method;
+            $method = 'Magium\Magento\Actions\Checkout\ShippingMethods\\'.$method;
         }
         $reflection = new \ReflectionClass($method);
         if ($reflection->implementsInterface('Magium\Magento\Actions\Checkout\ShippingMethods\ShippingMethodInterface')) {
@@ -106,12 +106,12 @@ 
         if ($reflection->isSubclassOf('Magium\Magento\Themes\NavigableThemeInterface')) {
             // Not entirely sure of hardcoding the various interface types.  May make this configurable
             parent::switchThemeConfiguration($fullyQualifiedClassName);
-            $this->setTypePreference('Magium\Magento\Themes\AbstractThemeConfiguration',$fullyQualifiedClassName);
-            $this->setTypePreference('Magium\Magento\Themes\NavigableThemeInterface',$fullyQualifiedClassName);
-            $this->setTypePreference('Magium\Themes\BaseThemeInterface',$fullyQualifiedClassName);
+            $this->setTypePreference('Magium\Magento\Themes\AbstractThemeConfiguration', $fullyQualifiedClassName);
+            $this->setTypePreference('Magium\Magento\Themes\NavigableThemeInterface', $fullyQualifiedClassName);
+            $this->setTypePreference('Magium\Themes\BaseThemeInterface', $fullyQualifiedClassName);
 
-            $this->setTypePreference('Magium\Magento\Themes\Customer\AbstractThemeConfiguration',$this->getTheme()->getCustomerThemeClass());
-            $this->setTypePreference('Magium\Magento\Themes\OnePageCheckout\AbstractThemeConfiguration',$this->getTheme()->getCheckoutThemeClass());
+            $this->setTypePreference('Magium\Magento\Themes\Customer\AbstractThemeConfiguration', $this->getTheme()->getCustomerThemeClass());
+            $this->setTypePreference('Magium\Magento\Themes\OnePageCheckout\AbstractThemeConfiguration', $this->getTheme()->getCheckoutThemeClass());
         } else {
             throw new InvalidConfigurationException('The theme configuration extend Magium\Magento\Themes\NavigableThemeInterface');
         }

lib/Magento/Themes/MagentoEE112/ThemeConfiguration.php

Spacing +23 added lines, -23 removed lines

@@ -15,25 +15,25 @@ 
     /**
      * @var string The Xpath string that finds the base of the navigation menu
      */
-    public $navigationBaseXPathSelector          = '//ul[@id="nav"]';
+    public $navigationBaseXPathSelector = '//ul[@id="nav"]';
 
     /**
      * @var string The Xpath string that can be used iteratively to find child navigation nodes
      */
 
-    public $navigationChildXPathSelector         = 'a[concat(" ",normalize-space(.)," ") = " %s "]/..';
+    public $navigationChildXPathSelector = 'a[concat(" ",normalize-space(.)," ") = " %s "]/..';
 
     /**
      * @var string A simple, default path to use for categories.
      */
 
-    public $navigationPathToSimpleProductCategory      = '{{Electronics}}/{{Cell Phones}}';
+    public $navigationPathToSimpleProductCategory = '{{Electronics}}/{{Cell Phones}}';
 
     public $defaultSimpleProductName = '{{Nokia 2610 Phone}}';
 
     public $productPagePriceXpath = '(//form[@id="product_addtocart_form"]/descendant::span[contains(concat(" ",normalize-space(@class)," ")," regular-price ")]/span[contains(concat(" ",normalize-space(@class)," ")," price ")])[1]';
 
-    public $navigationPathToConfigurableProductCategory      = '{{Apparel}}/{{Shirts}}';
+    public $navigationPathToConfigurableProductCategory = '{{Apparel}}/{{Shirts}}';
 
     public $defaultConfigurableProductName = '{{Zolof The Rock And Roll Destroyer: LOL Cat T-shirt}}';
 
@@ -41,7 +41,7 @@ 
      * @var string Xpath to add a Simple product to the cart from the product's page
      */
 
-    public $addToCartXpath          = '//button[@title="{{Add to Cart}}" and contains(concat(" ",normalize-space(@class)," ")," btn-cart ")]';
+    public $addToCartXpath = '//button[@title="{{Add to Cart}}" and contains(concat(" ",normalize-space(@class)," ")," btn-cart ")]';
 
     /**
      * @var string Xpath to add a Simple product to the cart from the category page
@@ -53,15 +53,15 @@ 
      * @var string Xpath to find a product's link on a category page.  Used to navigate to the product from the category
      */
 
-    public $categoryProductPageXpath             = '//h2[@class="product-name"]/descendant::a';
+    public $categoryProductPageXpath = '//h2[@class="product-name"]/descendant::a';
 
-    public $categorySpecificProductPageXpath             = '//h2[@class="product-name"]/descendant::a[.="%s"]';
+    public $categorySpecificProductPageXpath = '//h2[@class="product-name"]/descendant::a[.="%s"]';
 
     /**
      * @var string Xpath used after a product has been added to the cart to verify that the product has been added to the cart
      */
 
-    public $addToCartSuccessXpath        = '//li[@class="success-msg" and contains(., "{{was added to your shopping cart}}")]';
+    public $addToCartSuccessXpath = '//li[@class="success-msg" and contains(., "{{was added to your shopping cart}}")]';
 
     /**
      * @var string The base URL of the installation
@@ -87,12 +87,12 @@ 
      * @var array Instructions in an Xpath array syntax to get to the customer registration page
      */
 
-    public $logoutNavigationInstructions         = [
+    public $logoutNavigationInstructions = [
         [\Magium\WebDriver\WebDriver::INSTRUCTION_MOUSE_CLICK, '//ul[@class="links"]/descendant::a[.="{{Log Out}}"]'],
 //        [\Magium\WebDriver\WebDriver::INSTRUCTION_MOUSE_CLICK, '//div[@id="header-account"]/descendant::a[@title="{{Log Out}}"]']
     ];
 
-    public $cartNavigationInstructions            = [
+    public $cartNavigationInstructions = [
         [\Magium\WebDriver\WebDriver::INSTRUCTION_MOUSE_CLICK, '//string[@id="cartHeader"]'],
         [\Magium\WebDriver\WebDriver::INSTRUCTION_MOUSE_CLICK, '//div[@id="topCartContent"]/descendant::a/span[.="{{Go to Shopping Cart}}"]']
     ];
@@ -101,7 +101,7 @@ 
      * @var array Instructions in an Xpath array syntax to get to the login page.
      */
     
-    public $navigateToCustomerPageInstructions            = [
+    public $navigateToCustomerPageInstructions = [
         [\Magium\WebDriver\WebDriver::INSTRUCTION_MOUSE_CLICK, '//ul[@class="links"]/descendant::a[.="{{My Account}}"]'],
         //[\Magium\WebDriver\WebDriver::INSTRUCTION_MOUSE_CLICK, '//div[@id="header-account"]/descendant::a[@title="My Account"]']
     ];
@@ -110,7 +110,7 @@ 
      * @var array Instructions in an Xpath array syntax to get to the start of the checkout page
      */
 
-    public $checkoutNavigationInstructions         = [
+    public $checkoutNavigationInstructions = [
         [\Magium\WebDriver\WebDriver::INSTRUCTION_MOUSE_CLICK, '//ul[@class="links"]/descendant::a[@title="{{Checkout}}"]'],
 //        [\Magium\WebDriver\WebDriver::INSTRUCTION_MOUSE_CLICK, '//div[@class="minicart-actions"]/descendant::a[@title="Checkout"]']
     ];
@@ -120,7 +120,7 @@ 
      * @var array Instructions in an Xpath array syntax to get to the customer registration page
      */
 
-    public $registrationNavigationInstructions         = [
+    public $registrationNavigationInstructions = [
         [\Magium\WebDriver\WebDriver::INSTRUCTION_MOUSE_CLICK, '//ul[@class="links"]/descendant::a[.="{{My Account}}"]'],
         [\Magium\WebDriver\WebDriver::INSTRUCTION_MOUSE_CLICK, '//button/span/span[.="{{Register}}"]/../..']
     ];
@@ -141,10 +141,10 @@ 
     public $cartSummaryCheckoutProductLoopQtyXpath = '(//table[@id="checkout-review-table"]/tbody/descendant::td[@data-rwd-label="Qty"])[%d]';
     public $cartSummaryCheckoutProductLoopSubtotalXpath = '(//table[@id="checkout-review-table"]/tbody/descendant::td[@data-rwd-label="Subtotal"])[%d]';
 
-    public $cartSummaryCheckoutSubTotal              = '//table[@id="checkout-review-table"]/tfoot/tr/td[concat(" ",normalize-space(.)," ") = " Subtotal "]/../td[2]';
-    public $cartSummaryCheckoutTax              = '//table[@id="checkout-review-table"]/tfoot/tr/td[concat(" ",normalize-space(.)," ") = " Tax "]/../td[2]';
-    public $cartSummaryCheckoutGrandTotal              = '//table[@id="checkout-review-table"]/tfoot/tr/td[concat(" ",normalize-space(.)," ") = " Grand Total "]/../td[2]';
-    public $cartSummaryCheckoutShippingTotal              = '//table[@id="checkout-review-table"]/tfoot/tr/td[contains(concat(" ",normalize-space(.)," "), " Shipping & Handling (")]/../td[2]';
+    public $cartSummaryCheckoutSubTotal = '//table[@id="checkout-review-table"]/tfoot/tr/td[concat(" ",normalize-space(.)," ") = " Subtotal "]/../td[2]';
+    public $cartSummaryCheckoutTax = '//table[@id="checkout-review-table"]/tfoot/tr/td[concat(" ",normalize-space(.)," ") = " Tax "]/../td[2]';
+    public $cartSummaryCheckoutGrandTotal = '//table[@id="checkout-review-table"]/tfoot/tr/td[concat(" ",normalize-space(.)," ") = " Grand Total "]/../td[2]';
+    public $cartSummaryCheckoutShippingTotal = '//table[@id="checkout-review-table"]/tfoot/tr/td[contains(concat(" ",normalize-space(.)," "), " Shipping & Handling (")]/../td[2]';
 
     public $breadCrumbXpath                  = '//div[@class="breadcrumbs"]';
 
@@ -157,7 +157,7 @@ 
     public $productListOriginalPriceXpath    = '/descendant::div[@class="price-box"]/descendant::p[@class="old-price"]/descendant::*[@class="price"]';
     public $productListPriceXpath            = '/descendant::div[@class="price-box"]/descendant::*[@class="regular-price" or @class="special-price"]/descendant::span[@class="price"]';
     public $productListWishlistLinkXpath     = '/descendant::ul[@class="add-to-links"]/descendant::a[@class="link-wishlist"]';
-    public $productListAddToCartLinkXpath     = '/descendant::div[@class="product-shop"]/descendant::button[contains(concat(" ",normalize-space(@class)," ")," btn-cart ")]';
+    public $productListAddToCartLinkXpath = '/descendant::div[@class="product-shop"]/descendant::button[contains(concat(" ",normalize-space(@class)," ")," btn-cart ")]';
 
     public $productGridBaseXpath             = '//ul[contains(concat(" ",normalize-space(@class)," ")," products-grid ")]/li[%d]';
     public $productGridDescriptionXpath      = '/*[.="no description in the grid view"]';
@@ -168,15 +168,15 @@ 
     public $productGridOriginalPriceXpath    = '/descendant::div[@class="price-box"]/descendant::p[@class="old-price"]/descendant::*[@class="price"]';
     public $productGridPriceXpath            = '/descendant::div[@class="price-box"]/descendant::*[@class="regular-price" or @class="special-price"]/descendant::span[@class="price"]';
     public $productGridWishlistLinkXpath     = '/descendant::ul[@class="add-to-links"]/descendant::a[@class="link-wishlist"]';
-    public $productGridAddToCartLinkXpath     = '/descendant::div[@class="actions"]/descendant::button[contains(concat(" ",normalize-space(@class)," ")," btn-cart ")]';
+    public $productGridAddToCartLinkXpath = '/descendant::div[@class="actions"]/descendant::button[contains(concat(" ",normalize-space(@class)," ")," btn-cart ")]';
 
     public $productCollectionViewModeXpath   = '//p[@class="view-mode"]/strong';
     public $productCollectionSortByXpath     = '//div[@class="sort-by"]/descendant::option[@selected]'; // We select using the div, not the title because the title may be translated
     public $productCollectionShowCountXpath  = '//div[@class="limiter"]/descendant::option[@selected]'; // dittos
-    public $productCollectionShowCountOptionsXpath  = '//div[@class="limiter"]/descendant::option';
+    public $productCollectionShowCountOptionsXpath = '//div[@class="limiter"]/descendant::option';
     public $productCollectionProductCountXpath = '//div[contains(concat(" ",normalize-space(@class)," ")," pager ")]/descendant::p[contains(concat(" ",normalize-space(@class)," ")," amount ")]';
 
-    public $layeredNavigationBaseXpath        = '//div[contains(concat(" ",normalize-space(@class)," ")," block-layered-nav ")]';
+    public $layeredNavigationBaseXpath = '//div[contains(concat(" ",normalize-space(@class)," ")," block-layered-nav ")]';
 
     public $searchInputXpath                 = '//input[@id="search"]';
     public $searchSubmitXpath                = '//form[@id="search_mini_form"]/descendant::button[@type="submit"]';
@@ -195,11 +195,11 @@ 
     public $breadCrumbMemberXpath = '/descendant::a[concat(" ",normalize-space(.)," ")=" {{%s}} "]';
     public $breadCrumbSelectorXpath = '/descendant::a[%d]';
 
-    public $layeredNavigationFilterNameXpath =  '//dl[@id="narrow-by-list"]/dt';
+    public $layeredNavigationFilterNameXpath = '//dl[@id="narrow-by-list"]/dt';
 
     public $layeredNavigationFilterTypesXpath = '//dt[.="%s"]/following-sibling::dd[1]/descendant::li';
     public $layeredNavigationFilterLinkXpath = '//dt[.="%s"]/following-sibling::dd[1]/descendant::li/descendant::a';
-    public $layeredNavigationFilterNameElementXpath =  '//dl[@id="narrow-by-list"]/dt[normalize-space(.) = "%s"]';
+    public $layeredNavigationFilterNameElementXpath = '//dl[@id="narrow-by-list"]/dt[normalize-space(.) = "%s"]';
 
     public $storeSwitcherInstructionsXpath   = [
         [\Magium\WebDriver\WebDriver::INSTRUCTION_MOUSE_CLICK, '//select[@id="select-language"]/descendant::option[contains(@href,"___store=%s"]'],