jclaveau / php-deferred-callchain

src/DeferredCallChain.php

Spacing +15 added lines, -15 removed lines

@@ -36,7 +36,7 @@ 
      * 
      * @param string $class_type_interface_or_instance The expected target class/type/interface/instance
      */
-    public function __construct($class_type_interface_or_instance=null)
+    public function __construct($class_type_interface_or_instance = null)
     {
         if ($class_type_interface_or_instance) {
             $this->expectedTarget = $class_type_interface_or_instance;
@@ -101,12 +101,12 @@ 
         }
         elseif (is_string($this->expectedTarget)) {
             if (class_exists($this->expectedTarget)) {
-                if (! $target instanceof $this->expectedTarget) {
+                if ( ! $target instanceof $this->expectedTarget) {
                     throw new BadTargetClassException($this, $this->expectedTarget, $target);
                 }
             }
             elseif (interface_exists($this->expectedTarget)) {
-                if (! $target instanceof $this->expectedTarget) {
+                if ( ! $target instanceof $this->expectedTarget) {
                     throw new BadTargetInterfaceException($this, $this->expectedTarget, $target);
                 }
             }
@@ -158,7 +158,7 @@ 
                     $class = $current_chained_subject;
                 }
                 
-                $callable = $class .'::'. $method_name;
+                $callable = $class.'::'.$method_name;
             }
             
             $current_chained_subject = call_user_func_array(
@@ -203,10 +203,10 @@ 
         
         return  
                 ($trace[0]['function'] == '__call' || $trace[0]['function'] == '__callStatic')
-            &&  $trace[0]['class']    == (is_string($current_chained_subject) 
+            &&  $trace[0]['class'] == (is_string($current_chained_subject) 
                                        ? $current_chained_subject 
                                        : get_class($current_chained_subject))
-            &&  $trace[0]['args'][0]  == $method_name
+            &&  $trace[0]['args'][0] == $method_name
             && (
                     $trace[$call_user_func_array_position]['file'] == __FILE__
                 &&  $trace[$call_user_func_array_position]['function'] == 'call_user_func_array'
@@ -220,7 +220,7 @@ 
      * @param  mixed $target The target to apply the callchain on
      * @return mixed The value returned once the call chain is called uppon $target
      */
-    public function __invoke($target=null)
+    public function __invoke($target = null)
     {
         $out = $this->checkTarget($target);
         
@@ -237,9 +237,9 @@ 
                         );
                     }
                     
-                    if (! $is_called && (
-                                (is_string($out) && is_callable($out .'::'.$call['method']))
-                            ||  (is_object($out) && is_callable(get_class($out) .'::'.$call['method']))
+                    if ( ! $is_called && (
+                                (is_string($out) && is_callable($out.'::'.$call['method']))
+                            ||  (is_object($out) && is_callable(get_class($out).'::'.$call['method']))
                         )
                     ) {
                         $is_called = $this->checkMethodIsReallyCallable(
@@ -250,22 +250,22 @@ 
                         );
                     }
                     
-                    if (! $is_called && is_callable($call['method'])) {
+                    if ( ! $is_called && is_callable($call['method'])) {
                         $arguments = $this->prepareArgs($call['arguments'], $out);
                         $out = call_user_func_array($call['method'], $arguments);
                         $is_called = true;
                     }
                     
-                    if (! $is_called) {
+                    if ( ! $is_called) {
                         throw new \BadMethodCallException(
-                            $call['method'] . "() is neither a method of "
+                            $call['method']."() is neither a method of "
                             . (is_string($out) ? $out : get_class($out))
                             . " nor a function"
                         );
                     }
                 }
                 else {
-                    $out = $out[ $call['entry'] ];
+                    $out = $out[$call['entry']];
                 }
             }
             catch (\Exception $e) {
@@ -280,7 +280,7 @@ 
                     'message',
                     $e->getMessage()
                     . "\nWhen applying $callchain_description defined at "
-                    . $call['file'] . ':' . $call['line']
+                    . $call['file'].':'.$call['line']
                 );
                 
                 // Throw $e with the good stack (usage exception)

src_5.6/DeferredCallChain.php

Spacing +8 added lines, -8 removed lines

@@ -81,11 +81,11 @@ 
             $out = $this->expectedTarget;
         } elseif (is_string($this->expectedTarget)) {
             if (class_exists($this->expectedTarget)) {
-                if (!$target instanceof $this->expectedTarget) {
+                if ( ! $target instanceof $this->expectedTarget) {
                     throw new BadTargetClassException($this, $this->expectedTarget, $target);
                 }
             } elseif (interface_exists($this->expectedTarget)) {
-                if (!$target instanceof $this->expectedTarget) {
+                if ( ! $target instanceof $this->expectedTarget) {
                     throw new BadTargetInterfaceException($this, $this->expectedTarget, $target);
                 }
             } elseif (type_exists($this->expectedTarget)) {
@@ -124,7 +124,7 @@ 
                 } elseif (is_string($current_chained_subject)) {
                     $class = $current_chained_subject;
                 }
-                $callable = $class . '::' . $method_name;
+                $callable = $class.'::'.$method_name;
             }
             $current_chained_subject = call_user_func_array($callable, $arguments);
         } catch (\BadMethodCallException $e) {
@@ -169,23 +169,23 @@ 
                     if (is_callable([$out, $call['method']])) {
                         $is_called = $this->checkMethodIsReallyCallable('->', $out, $call['method'], $call['arguments']);
                     }
-                    if (!$is_called && (is_string($out) && is_callable($out . '::' . $call['method']) || is_object($out) && is_callable(get_class($out) . '::' . $call['method']))) {
+                    if ( ! $is_called && (is_string($out) && is_callable($out.'::'.$call['method']) || is_object($out) && is_callable(get_class($out).'::'.$call['method']))) {
                         $is_called = $this->checkMethodIsReallyCallable('::', $out, $call['method'], $call['arguments']);
                     }
-                    if (!$is_called && is_callable($call['method'])) {
+                    if ( ! $is_called && is_callable($call['method'])) {
                         $arguments = $this->prepareArgs($call['arguments'], $out);
                         $out = call_user_func_array($call['method'], $arguments);
                         $is_called = true;
                     }
-                    if (!$is_called) {
-                        throw new \BadMethodCallException($call['method'] . "() is neither a method of " . (is_string($out) ? $out : get_class($out)) . " nor a function");
+                    if ( ! $is_called) {
+                        throw new \BadMethodCallException($call['method']."() is neither a method of ".(is_string($out) ? $out : get_class($out))." nor a function");
                     }
                 } else {
                     $out = $out[$call['entry']];
                 }
             } catch (\Exception $e) {
                 $callchain_description = $this->toString(['target' => $target, 'limit' => $i]);
-                VisibilityViolator::setHiddenProperty($e, 'message', $e->getMessage() . "\nWhen applying {$callchain_description} defined at " . $call['file'] . ':' . $call['line']);
+                VisibilityViolator::setHiddenProperty($e, 'message', $e->getMessage()."\nWhen applying {$callchain_description} defined at ".$call['file'].':'.$call['line']);
                 // Throw $e with the good stack (usage exception)
                 throw $e;
             }

Braces +22 added lines, -11 removed lines

@@ -79,24 +79,29 @@ 
                 throw new TargetAlreadyDefinedException($this, $this->expectedTarget, $target);
             }
             $out = $this->expectedTarget;
-        } elseif (is_string($this->expectedTarget)) {
+        }
+        elseif (is_string($this->expectedTarget)) {
             if (class_exists($this->expectedTarget)) {
                 if (!$target instanceof $this->expectedTarget) {
                     throw new BadTargetClassException($this, $this->expectedTarget, $target);
                 }
-            } elseif (interface_exists($this->expectedTarget)) {
+            }
+            elseif (interface_exists($this->expectedTarget)) {
                 if (!$target instanceof $this->expectedTarget) {
                     throw new BadTargetInterfaceException($this, $this->expectedTarget, $target);
                 }
-            } elseif (type_exists($this->expectedTarget)) {
+            }
+            elseif (type_exists($this->expectedTarget)) {
                 if (gettype($target) != $this->expectedTarget) {
                     throw new BadTargetTypeException($this, $this->expectedTarget, $target);
                 }
-            } else {
+            }
+            else {
                 throw new UndefinedTargetClassException($this, $this->expectedTarget);
             }
             $out = $target;
-        } else {
+        }
+        else {
             $out = $target;
         }
         return $out;
@@ -118,19 +123,23 @@ 
         try {
             if ($method_type == '->') {
                 $callable = [$current_chained_subject, $method_name];
-            } elseif ($method_type == '::') {
+            }
+            elseif ($method_type == '::') {
                 if (is_object($current_chained_subject)) {
                     $class = get_class($current_chained_subject);
-                } elseif (is_string($current_chained_subject)) {
+                }
+                elseif (is_string($current_chained_subject)) {
                     $class = $current_chained_subject;
                 }
                 $callable = $class . '::' . $method_name;
             }
             $current_chained_subject = call_user_func_array($callable, $arguments);
-        } catch (\BadMethodCallException $e) {
+        }
+        catch (\BadMethodCallException $e) {
             if ($this->exceptionTrownFromMagicCall($e->getTrace(), $current_chained_subject, $method_name)) {
                 $is_called = false;
-            } else {
+            }
+            else {
                 throw $e;
             }
         }
@@ -180,10 +189,12 @@ 
                     if (!$is_called) {
                         throw new \BadMethodCallException($call['method'] . "() is neither a method of " . (is_string($out) ? $out : get_class($out)) . " nor a function");
                     }
-                } else {
+                }
+                else {
                     $out = $out[$call['entry']];
                 }
-            } catch (\Exception $e) {
+            }
+            catch (\Exception $e) {
                 $callchain_description = $this->toString(['target' => $target, 'limit' => $i]);
                 VisibilityViolator::setHiddenProperty($e, 'message', $e->getMessage() . "\nWhen applying {$callchain_description} defined at " . $call['file'] . ':' . $call['line']);
                 // Throw $e with the good stack (usage exception)