magium / MagiumMagento

lib/Magento/AbstractMagentoTestCase.php

Doc Comments +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     }
 
     /**
-     * @param $method
+     * @param string $method
      * @return \Magium\Magento\Actions\Checkout\PaymentMethods\PaymentMethodInterface
      */
 

Spacing +7 added lines, -7 removed lines

@@ -27,7 +27,7 @@ 
 
         // If we are passed just the class name we will prepend it with Magium\Magento\Actions\Checkout\PaymentMethods
         if (strpos($method, '\\') === false) {
-            $method = 'Magium\Magento\Actions\Checkout\PaymentMethods\\' . $method;
+            $method = 'Magium\Magento\Actions\Checkout\PaymentMethods\\'.$method;
         }
 
         if (is_subclass_of($method, 'Magium\Magento\Actions\Checkout\PaymentMethods\PaymentMethodInterface')) {
@@ -69,7 +69,7 @@ 
 
         // If we are passed just the class name 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;
         }
 
         if (is_subclass_of($method, 'Magium\Magento\Actions\Checkout\ShippingMethods\ShippingMethodInterface')) {
@@ -86,12 +86,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 {

lib/Magento/Actions/Admin/Configuration/PaymentMethods/CashOnDelivery.php

Doc Comments -1 removed lines

@@ -23,7 +23,6 @@
     ];
 
     /**
-     * @param string $active
      */
     public function setEnabled($enabled)
     {

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
     const NAME = 'Cash On Delivery Payment';
     const ACTION = 'Admin\Configuration\PaymentMethods\CashOnDelivery';
 
-    protected $section = 'Payment Methods/' . self::NAME;
+    protected $section = 'Payment Methods/'.self::NAME;
 
     protected $settings = [
         'payment_cashondelivery_active'             => 0,

lib/Magento/Actions/Admin/Configuration/SettingModifier.php

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      * * Tab/Section::name=Text Of Setting (will ensure that the correct section is navigated to, uses setting name (exact), not ID)
      * * name=Text Of Setting (will not navigate, uses setting name (exact) not ID)
      *
-     * @param $identifier
+     * @param string $identifier
      * @param $value
      * @throws \Magium\InvalidInstructionException
      */

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
             $xpath = '';
             if (count($xpath) === 2) {
                 $nav = explode('/', $parts[0]);
-                $xpath = $this->themeConfiguration->getSystemConfigSectionDisplayCheckXpath($nav[1]) . '/descendant::';
+                $xpath = $this->themeConfiguration->getSystemConfigSectionDisplayCheckXpath($nav[1]).'/descendant::';
             }
             $xpath .= $this->themeConfiguration->getSystemConfigSettingLabelXpath($matches[1]);
             $labelElement = $this->webDriver->byXpath($xpath);

lib/Magento/Actions/Cart/AddItemToCart.php

Doc Comments -1 removed lines

@@ -96,7 +96,6 @@
     /**
      * Adds as configurable product to the cart from the product page.
      * 
-     * @param array $options
      */
     
     public function addConfigurableItemToCartFromProductPage($categoryNavigationPath = null)

lib/Magento/Extractors/Admin/Login/Messages.php

Doc Comments +3 added lines

@@ -43,6 +43,9 @@
         return $this->messages;
     }
 
+    /**
+     * @param string $message
+     */
     public function addMessage($message)
     {
         $this->messages[] = $message;

lib/Magento/Extractors/Admin/Xtree/CategoryTreeNode.php

Doc Comments +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 
     /**
      * @param $nodePath
-     * @return WebDriverElement
+     * @return null|\Facebook\WebDriver\Remote\RemoteWebElement
      */
 
     public function getNode($nodePath)

Spacing +4 added lines, -4 removed lines

@@ -58,8 +58,8 @@ 
         $previousExpandXpath = '';
         $element = null;
         foreach ($parts as $part) {
-            $xpath .= '/' . $this->theme->getXTreeChildXpath($part);
-            if(!$this->webDriver->elementExists($xpath, WebDriver::BY_XPATH)) {
+            $xpath .= '/'.$this->theme->getXTreeChildXpath($part);
+            if (!$this->webDriver->elementExists($xpath, WebDriver::BY_XPATH)) {
                 if ($previousExpandXpath) {
                     $this->testCase->assertElementExists($previousExpandXpath, WebDriver::BY_XPATH);
                     $this->webDriver->byXpath($previousExpandXpath)->click();
@@ -70,9 +70,9 @@ 
             $this->testCase->assertElementExists($xpath, WebDriver::BY_XPATH);
             $element = $this->webDriver->byXpath($xpath);
             if (!$element->isDisplayed()) {
-                throw new ElementNotVisibleException('Node element is not visible and cannot be made visible: ' . $part);
+                throw new ElementNotVisibleException('Node element is not visible and cannot be made visible: '.$part);
             }
-            $previousExpandXpath = $xpath . $this->theme->getXTreeChildNodeExpandPrefixXpath();
+            $previousExpandXpath = $xpath.$this->theme->getXTreeChildNodeExpandPrefixXpath();
             $xpath .= $this->theme->getXTreeChildNodePrefixXpath();
         }
         $this->node = $element;

lib/Magento/Extractors/Catalog/LayeredNavigation/LayeredNavigation.php

Doc Comments +3 added lines

@@ -80,6 +80,9 @@
     }
 
 
+    /**
+     * @param string $type
+     */
     public function addFilterType($type)
     {
         $type = $this->filterTypeName($type);

Spacing +7 added lines, -7 removed lines

@@ -55,11 +55,11 @@ 
     {
         $returnType = $type;
         if (strpos($type, '\\') === false) {
-            $returnType = $this->baseNamespace . '\\' . $type;
+            $returnType = $this->baseNamespace.'\\'.$type;
             if (!class_exists($returnType)) {
                 $returnType = $type;
                 if (!class_exists($returnType)) {
-                    throw new InvalidFilterException('Filter type must exist: ' . $returnType);
+                    throw new InvalidFilterException('Filter type must exist: '.$returnType);
                 }
             }
         }
@@ -69,12 +69,12 @@ 
 
     protected function validateFilterType($returnType)
     {
-        if ($returnType == $this->baseNamespace . '\AbstractFilterType') {
+        if ($returnType == $this->baseNamespace.'\AbstractFilterType') {
             return true;
         }
         $reflection = new \ReflectionClass($returnType);
-        if (!$reflection->isSubclassOf($this->baseNamespace . '\AbstractFilterType')) {
-            throw new InvalidFilterException('Filter type must extend AbstractFilterType: ' . $returnType);
+        if (!$reflection->isSubclassOf($this->baseNamespace.'\AbstractFilterType')) {
+            throw new InvalidFilterException('Filter type must extend AbstractFilterType: '.$returnType);
         }
         return true;
     }
@@ -131,7 +131,7 @@ 
             return $this->filterValues[$filterKey];
         }
 
-        throw new MissingFilterException('Could not find the filter: ' . $filter);
+        throw new MissingFilterException('Could not find the filter: '.$filter);
     }
 
     public function extract()
@@ -167,7 +167,7 @@ 
                         break;
                     }
                 } else {
-                    throw new InvalidFilterException('Filter type does not extend AbstractFilterType: ' . $type);
+                    throw new InvalidFilterException('Filter type does not extend AbstractFilterType: '.$type);
                 }
             }
         }

lib/Magento/Extractors/Catalog/Products/AbstractProductCollection.php

Doc Comments +4 added lines

@@ -25,6 +25,10 @@
         return $this->products;
     }
 
+    /**
+     * @param integer $count
+     * @param string $attribute
+     */
     protected function getByXpath($type, $count, $attribute = null, $getElement = false)
     {
         $xpath = $this->getElementXpath($type, $count);

lib/Magento/Identities/Customer.php

Doc Comments +1 added lines, -1 removed lines

@@ -192,7 +192,7 @@
     }
 
     /**
-     * @param string $billingFastName
+     * @param string $billingLastName
      */
     public function setBillingLastName($billingLastName)
     {

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 {
     const IDENTITY = 'Customer';
 
-    public $emailAddress          = 'test@example.com';
+    public $emailAddress = 'test@example.com';
 
     public $billingFirstName        = 'Kevin';
     public $billingLastName         = 'Schroeder';