ctasca / magefix

magefix/src/Magefix/Plugin/DriverSwitcher.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
         if ($iFrame instanceof Element) {
             $reflectedElement = new \ReflectionClass($iFrame);
             if ($reflectedElement->hasMethod('getCssId') === false) {
-                throw new \ReflectionException("getCssId method not in Element " . get_class($iFrame));
+                throw new \ReflectionException("getCssId method not in Element ".get_class($iFrame));
             }
             $iFrameId = $iFrame->getCssId();
             $this->getDriver()->switchToIFrame($iFrameId);

magefix/src/Magefix/Plugin/Spinner.php

Doc Comments +3 added lines, -3 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)
@@ -119,7 +119,7 @@ 
      * @param $context
      * @param $element
      * @param $action
-     * @param null $condition
+     * @param boolean|null $condition
      * @return bool
      */
     private function _spinnerAction($context, $element, $action, $condition = null)

Spacing +7 added lines, -7 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 $this->_spinnerAction($context, $element, 'isVisible', true);
         }, $wait);
     }
@@ -56,7 +56,7 @@ 
      */
     public function spinUntilInvisible($element, $wait = 60)
     {
-        $this->spin(function ($context) use ($element) {
+        $this->spin(function($context) use ($element) {
             return $this->_spinnerAction($context, $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) {
             return $this->_spinnerAction($context, $element, 'click');
         }, $wait);
     }
@@ -82,7 +82,7 @@ 
      */
     public function spinAndPress($element, $wait = 60)
     {
-        $this->spin(function ($context) use ($element) {
+        $this->spin(function($context) use ($element) {
             return $this->_spinnerAction($context, $element, 'press');
         }, $wait);
     }
@@ -97,7 +97,7 @@ 
      */
     public function spinAndDo($element, $action, $condition = null, $wait = 60)
     {
-        $this->spin(function ($context) use ($element, $action, $condition) {
+        $this->spin(function($context) use ($element, $action, $condition) {
             return $this->_spinnerAction($context, $element, $action, $condition);
         }, $wait);
     }
@@ -110,8 +110,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']
         );
     }