magium / MagiumMagento

lib/Magento/Extractors/Admin/Widget/Attribute.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     }
 
     /**
-     * @param $label
+     * @param string $label
      * @return WebDriverElement
      */
 

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