ctasca / magefix

magefix/src/Magefix/Yaml/Parser.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      */
     public function parse()
     {
-        return Yaml::parse($this->getFixturesLocation() . $this->_filename);
+        return Yaml::parse($this->getFixturesLocation().$this->_filename);
     }
 
     /**

magefix/src/Magefix/Fixture/Instanciator.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     public static function instance($fixtureType, array $instanceData, $hook)
     {
         $mageModel = Mage::getModel($instanceData['fixture']['model']);
-        $class = new \ReflectionClass(static::BUILDER_NAMESPACE_PREFIX . $fixtureType);
+        $class = new \ReflectionClass(static::BUILDER_NAMESPACE_PREFIX.$fixtureType);
         $fixture = $class->newInstanceArgs([$instanceData, $mageModel, $hook]);
 
         return $fixture;

magefix/src/Magefix/Fixture/Builder/Helper/ShippingMethod.php

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
      */
     public static function isEnabled($carrier)
     {
-        return Mage::getStoreConfig('carriers/' . $carrier . '/active');
+        return Mage::getStoreConfig('carriers/'.$carrier.'/active');
     }
 
     /**
@@ -90,7 +90,7 @@ 
         $carriers = Mage::getSingleton('shipping/config')->getActiveCarriers();
 
         if (array_key_exists($carrier, $carriers)) {
-            Mage::app()->getStore()->setConfig('carriers/' . $carrier . '/active', $enable);
+            Mage::app()->getStore()->setConfig('carriers/'.$carrier.'/active', $enable);
         }
     }
 }

magefix/src/Magefix/Fixture/Builder/Helper/MediaGalleryImage.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     {
         if (!@file_exists($gallery['image'])) {
             throw new ProductMediaGalleryImageNotFound(
-                'Specified product fixture gallery image does not exists -> ' . $gallery['image']
+                'Specified product fixture gallery image does not exists -> '.$gallery['image']
             );
         }
 

magefix/src/Magefix/Fixture/DataIterator.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     protected function _throwMethodNotFoundExceptionForProvider(Provider $provider, $method)
     {
         throw new ProviderMethodNotFound(
-            "Given provider does not have the method " . $method . ' -> ' . get_class($provider)
+            "Given provider does not have the method ".$method.' -> '.get_class($provider)
         );
     }
 

magefix/src/Magefix/Plugin/Spinner.php

Spacing +6 added lines, -6 removed lines

@@ -43,7 +43,7 @@ 
      */
     public function spinUntilVisible($element, $wait = 60)
     {
-        $this->spin(function ($context) use ($element) {
+        $this->spin(function($context) use ($element) {
             return $context->getElement($element)->isVisible();
         }, $wait);
     }
@@ -56,7 +56,7 @@ 
      */
     public function spinUntilInvisible($element, $wait = 60)
     {
-        $this->spin(function ($context) use ($element) {
+        $this->spin(function($context) use ($element) {
             return ($context->getElement($element)->isVisible() == false);
         }, $wait);
     }
@@ -69,7 +69,7 @@ 
      */
     public function spinAndClick($element, $wait = 60)
     {
-        $this->spin(function ($context) use ($element) {
+        $this->spin(function($context) use ($element) {
             $context->getElement($element)->click();
             return true;
         }, $wait);
@@ -83,7 +83,7 @@ 
      */
     public function spinAndPress($element, $wait = 60)
     {
-        $this->spin(function ($context) use ($element) {
+        $this->spin(function($context) use ($element) {
             $context->getElement($element)->press();
             return true;
         }, $wait);
@@ -97,8 +97,8 @@ 
         $backtrace = debug_backtrace();
 
         throw new \Exception(
-            "Timeout thrown by " . $backtrace[1]['class'] . "::" . $backtrace[1]['function'] . "()\n" .
-            $backtrace[1]['file'] . ", line " . $backtrace[1]['line']
+            "Timeout thrown by ".$backtrace[1]['class']."::".$backtrace[1]['function']."()\n".
+            $backtrace[1]['file'].", line ".$backtrace[1]['line']
         );
     }
 }

Doc Comments +4 added lines, -4 removed lines

@@ -13,9 +13,9 @@ 
 trait Spinner
 {
     /**
-     * @param $lambda
+     * @param \Closure $lambda
      * @param int $wait
-     * @return bool
+     * @return boolean|null
      * @throws \Exception
      */
     public function spin($lambda, $wait = 60)
@@ -103,8 +103,8 @@ 
     /**
      * @param $context
      * @param $element
-     * @param $action
-     * @param null $condition
+     * @param string $action
+     * @param boolean $condition
      * @return bool
      */
     private function _spinnerAction($context, $element, $action, $condition = null)

magefix/src/Magefix/Fixtures/Registry.php

Spacing +1 added lines, -1 removed lines

@@ -120,7 +120,7 @@
     protected static function _registerFixture($type, $fixtureId, $hook)
     {
         self::_throwNullFixtureIdException($fixtureId);
-        self::register($type . "_{$hook}_" . $fixtureId, $fixtureId);
+        self::register($type."_{$hook}_".$fixtureId, $fixtureId);
     }
 
     /**