jclaveau / php-deferred-callchain

classmap.php

Indentation +10 added lines, -10 removed lines

@@ -37,16 +37,16 @@
  * the code needing it.
  */
 $sorted_map = [
-  'ArrayAccessTrait.php',
-  'FunctionCallTrait.php',
-  'ExportTrait.php',
-  'DeferredCallChain.php',
-  'Exceptions/BadTargetClassException.php',
-  'Exceptions/BadTargetInterfaceException.php',
-  'Exceptions/BadTargetTypeException.php',
-  'Exceptions/TargetAlreadyDefinedException.php',
-  'Exceptions/UndefinedTargetClassException.php',
-  'functions.php',
+    'ArrayAccessTrait.php',
+    'FunctionCallTrait.php',
+    'ExportTrait.php',
+    'DeferredCallChain.php',
+    'Exceptions/BadTargetClassException.php',
+    'Exceptions/BadTargetInterfaceException.php',
+    'Exceptions/BadTargetTypeException.php',
+    'Exceptions/TargetAlreadyDefinedException.php',
+    'Exceptions/UndefinedTargetClassException.php',
+    'functions.php',
 ];
 
 if ($missing = array_diff($sorted_map, $map)) {

Spacing +5 added lines, -5 removed lines

@@ -7,12 +7,12 @@ 
 
     $map = [];
     foreach ($files as $file) {
-        if (is_dir( $directory . '/' . $file)) {
-            $submap = scandir_r($directory . '/' . $file);
+        if (is_dir($directory.'/'.$file)) {
+            $submap = scandir_r($directory.'/'.$file);
             $map = array_merge($map, $submap);
         }
         else {
-            $map[] = $directory . '/' . $file;
+            $map[] = $directory.'/'.$file;
         }
     }
     
@@ -52,11 +52,11 @@ 
 if ($missing = array_diff($sorted_map, $map)) {
     throw new \Exception(
         "Missing file in your classmap. Please add it manually to "
-        . __FILE__ . "\n"
+        . __FILE__."\n"
         . var_export($missing, true)
     );
 }
 
 foreach ($sorted_map as $filepath) {
-    require_once($root . '/' . $filepath);
+    require_once($root.'/'.$filepath);
 }

Braces +2 added lines, -1 removed lines

@@ -1,7 +1,8 @@
 <?php
 /**
  */ 
-function scandir_r($directory) {
+function scandir_r($directory)
+{
     // $files = array_slice(scandir($directory, SCANDIR_SORT_NONE), 2);
     $files = array_slice(scandir($directory), 2);
 

src_5.6/FunctionCallTrait.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      */
     protected function replacePlaceholderWithValue(array $args, $value)
     {
-        return array_map(function ($arg) use($value) {
+        return array_map(function($arg) use($value) {
             return $arg === $this->placeholder ? $value : $arg;
         }, $args);
     }

src_5.6/Exceptions/TargetAlreadyDefinedException.php

Spacing +1 added lines, -1 removed lines

@@ -25,6 +25,6 @@
      */
     public function __construct(DeferredCallChain $callchain, $expected_target, $target)
     {
-        $this->message = "You are trying to define the target " . spl_object_id($target) . " for the {$callchain} which already has one: " . spl_object_id($expected_target);
+        $this->message = "You are trying to define the target ".spl_object_id($target)." for the {$callchain} which already has one: ".spl_object_id($expected_target);
     }
 }
\ No newline at end of file

src_5.6/Exceptions/BadTargetTypeException.php

Spacing +1 added lines, -1 removed lines

@@ -25,6 +25,6 @@
      */
     public function __construct(DeferredCallChain $callchain, $expected_target, $target)
     {
-        $this->message = "You are trying to define a target of type " . gettype($target) . " for the {$callchain} allowing only: " . $expected_target;
+        $this->message = "You are trying to define a target of type ".gettype($target)." for the {$callchain} allowing only: ".$expected_target;
     }
 }
\ No newline at end of file

src_5.6/Exceptions/BadTargetClassException.php

Spacing +1 added lines, -1 removed lines

@@ -25,6 +25,6 @@
      */
     public function __construct(DeferredCallChain $callchain, $expected_target, $target)
     {
-        $this->message = "You are trying to define a target of class " . get_class($target) . " for the {$callchain} allowing only targets of class " . $expected_target;
+        $this->message = "You are trying to define a target of class ".get_class($target)." for the {$callchain} allowing only targets of class ".$expected_target;
     }
 }
\ No newline at end of file

src_5.6/Exceptions/UndefinedTargetClassException.php

Spacing +1 added lines, -1 removed lines

@@ -24,6 +24,6 @@
      */
     public function __construct(DeferredCallChain $callchain, $expected_target)
     {
-        $this->message = "The expected target of {$callchain} is neither a existing class or interface nor a native type: " . $expected_target;
+        $this->message = "The expected target of {$callchain} is neither a existing class or interface nor a native type: ".$expected_target;
     }
 }
\ No newline at end of file

src_5.6/Exceptions/BadTargetInterfaceException.php

Spacing +1 added lines, -1 removed lines

@@ -25,6 +25,6 @@
      */
     public function __construct(DeferredCallChain $callchain, $expected_target, $target)
     {
-        $this->message = "You are trying to define a target of class " . get_class($target) . " for the {$callchain} allowing only targets implementing " . $expected_target;
+        $this->message = "You are trying to define a target of class ".get_class($target)." for the {$callchain} allowing only targets implementing ".$expected_target;
     }
 }
\ No newline at end of file

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 " . get_class($out) . " nor a function");
+                    if ( ! $is_called) {
+                        throw new \BadMethodCallException($call['method']."() is neither a method of ".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 " . 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)

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);
             }