jclaveau / php-deferred-callchain

src_5.6/ExportTrait.php

Spacing +10 added lines, -10 removed lines

@@ -45,21 +45,21 @@ 
         $target = isset($options['target']) ? $options['target'] : $this->expectedTarget;
         $max_param_length = isset($options['max_parameter_length']) ? $options['max_parameter_length'] : 56;
         $short_objects = isset($options['short_objects']) ? $options['short_objects'] : true;
-        $string = '(new ' . get_called_class();
-        $target && ($string .= '(' . static::varExport($target, ['short_objects' => $short_objects, 'max_length' => $max_param_length]) . ')');
+        $string = '(new '.get_called_class();
+        $target && ($string .= '('.static::varExport($target, ['short_objects' => $short_objects, 'max_length' => $max_param_length]).')');
         $string .= ')';
         foreach ($this->stack as $i => $call) {
             if (isset($call['method'])) {
                 $string .= '->';
-                $string .= $call['method'] . '(';
-                $string .= implode(', ', array_map(function ($argument) use($max_param_length, $short_objects) {
+                $string .= $call['method'].'(';
+                $string .= implode(', ', array_map(function($argument) use($max_param_length, $short_objects) {
                     return static::varExport($argument, ['short_objects' => $short_objects, 'max_length' => $max_param_length]);
                 }, $call['arguments']));
                 $string .= ')';
             } else {
-                $string .= '[' . static::varExport($call['entry'], ['short_objects' => $short_objects, 'max_length' => $max_param_length]) . ']';
+                $string .= '['.static::varExport($call['entry'], ['short_objects' => $short_objects, 'max_length' => $max_param_length]).']';
             }
-            if (!empty($options['limit']) && $options['limit'] == $i) {
+            if ( ! empty($options['limit']) && $options['limit'] == $i) {
                 break;
             }
         }
@@ -75,20 +75,20 @@ 
     protected static function varExport($variable, array $options = [])
     {
         $options['max_length'] = isset($options['max_length']) ? $options['max_length'] : 56;
-        $options['short_objects'] = !empty($options['short_objects']) || in_array('short_objects', $options);
+        $options['short_objects'] = ! empty($options['short_objects']) || in_array('short_objects', $options);
         $export = var_export($variable, true);
         if ($options['short_objects']) {
             if (is_object($variable)) {
-                $export = ' ' . get_class($variable) . ' #' . spl_object_id($variable) . ' ';
+                $export = ' '.get_class($variable).' #'.spl_object_id($variable).' ';
             }
         }
         if (strlen($export) > $options['max_length']) {
             if (is_object($variable)) {
                 // shortening short objects would only slow the workflow
-                $export = get_class($variable) . ' #' . spl_object_id($variable);
+                $export = get_class($variable).' #'.spl_object_id($variable);
             } elseif (is_string($variable)) {
                 $keep_length = floor(($options['max_length'] - 5) / 2);
-                $export = substr($variable, 0, (int) $keep_length) . ' ... ' . substr($variable, -$keep_length);
+                $export = substr($variable, 0, (int) $keep_length).' ... '.substr($variable, -$keep_length);
             }
         }
         return $export;

Braces +4 added lines, -2 removed lines

@@ -56,7 +56,8 @@ 
                     return static::varExport($argument, ['short_objects' => $short_objects, 'max_length' => $max_param_length]);
                 }, $call['arguments']));
                 $string .= ')';
-            } else {
+            }
+            else {
                 $string .= '[' . static::varExport($call['entry'], ['short_objects' => $short_objects, 'max_length' => $max_param_length]) . ']';
             }
             if (!empty($options['limit']) && $options['limit'] == $i) {
@@ -86,7 +87,8 @@ 
             if (is_object($variable)) {
                 // shortening short objects would only slow the workflow
                 $export = get_class($variable) . ' #' . spl_object_id($variable);
-            } elseif (is_string($variable)) {
+            }
+            elseif (is_string($variable)) {
                 $keep_length = floor(($options['max_length'] - 5) / 2);
                 $export = substr($variable, 0, (int) $keep_length) . ' ... ' . substr($variable, -$keep_length);
             }

src_5.6/functions.php

Spacing +2 added lines, -2 removed lines

@@ -3,7 +3,7 @@ 
 /**
  * File gathering functions
  */
-if (!function_exists('type_exists')) {
+if ( ! function_exists('type_exists')) {
     /**
      * Checks if a type is a valid PHP one.
      * 
@@ -15,7 +15,7 @@ 
         return in_array($type_name, ["boolean", "integer", "double", "string", "array", "object", "resource", "resource (closed)", "NULL", "unknown type"]);
     }
 }
-if (!function_exists('later')) {
+if ( ! function_exists('later')) {
     /**
      * Create a deferred call chain in a functionnal way.
      * 

autoloader.php

Spacing +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@
 <?php
 
-$version = explode('.',phpversion());
-$version = $version[0] . '.' . $version[1];
+$version = explode('.', phpversion());
+$version = $version[0].'.'.$version[1];
 
 if (PHP_VERSION_ID >= 70000) {
     $root = 'src';

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)