jclaveau / php-deferred-callchain

src/FunctionCallTrait.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
      * @param mixed $value
      * @return array
      */
-    protected function prepareArgs (array $args, $value)
+    protected function prepareArgs(array $args, $value)
     {
         return $this->hasPlaceholder($args)
             ? $this->replacePlaceholderWithValue($args, $value)
@@ -37,7 +37,7 @@ 
      * @param array $args
      * @return bool
      */
-    protected function hasPlaceholder (array $args)
+    protected function hasPlaceholder(array $args)
     {
         return in_array($this->placeholder, $args, true);
     }
@@ -48,7 +48,7 @@ 
      * @param mixed $value
      * @return array
      */
-    protected function addValueAsFirstArg (array $args, $value)
+    protected function addValueAsFirstArg(array $args, $value)
     {
         array_unshift($args, $value);
 
@@ -61,9 +61,9 @@ 
      * @param mixed $value
      * @return array
      */
-    protected function replacePlaceholderWithValue (array $args, $value)
+    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;

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 +6 added lines, -6 removed lines

@@ -3,7 +3,7 @@ 
  */ 
 function scandir_r($directory) {
     // $files = array_slice(scandir($directory, SCANDIR_SORT_NONE), 2);
-    if (! $entries = scandir($directory)) {
+    if ( ! $entries = scandir($directory)) {
         throw new \InvalidArgumentException("Error during scandir");
     }
     
@@ -11,12 +11,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;
         }
     }
     
@@ -56,11 +56,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);
     if (! $entries = scandir($directory)) {
         throw new \InvalidArgumentException("Error during scandir");

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/Exceptions/UndefinedTargetClassException.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      */
     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;
     }
     
     /**/

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,8 +203,8 @@ 
         
         return  
                 ($trace[0]['function'] == '__call' || $trace[0]['function'] == '__callStatic')
-            &&  $trace[0]['class']    == get_class($current_chained_subject)
-            &&  $trace[0]['args'][0]  == $method_name
+            &&  $trace[0]['class'] == get_class($current_chained_subject)
+            &&  $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'
@@ -218,7 +218,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);
         
@@ -235,9 +235,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(
@@ -248,21 +248,21 @@ 
                         );
                     }
                     
-                    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 " . get_class($out)
+                            $call['method']."() is neither a method of ".get_class($out)
                             . " nor a function"
                         );
                     }
                 }
                 else {
-                    $out = $out[ $call['entry'] ];
+                    $out = $out[$call['entry']];
                 }
             }
             catch (\Exception $e) {
@@ -277,7 +277,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/ExportTrait.php

Spacing +11 added lines, -11 removed lines

@@ -41,17 +41,17 @@ 
      * @param  array  $options target: mixed | max_parameter_length: int | short_objects: bool
      * @return string The PHP code corresponding to this call chain
      */
-    public function toString(array $options=[])
+    public function toString(array $options = [])
     {
         $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, [
+        $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) {
@@ -67,13 +67,13 @@ 
                 $string .= ')';
             }
             else {
-                $string .= '[' . static::varExport($call['entry'], [
+                $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;
             }
         }
@@ -88,16 +88,16 @@ 
      * @param  array  $options max_length: int | short_objects: bool
      * @return string The PHP code of the variable
      */
-    protected static function varExport($variable, array $options=[])
+    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).' ';
             }
         }
         
@@ -105,7 +105,7 @@ 
             
             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);

src/functions.php

Spacing +3 added lines, -3 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.
      * 
@@ -27,14 +27,14 @@ 
     }
 }
 
-if (! function_exists('later')) {
+if ( ! function_exists('later')) {
     /**
      * Create a deferred call chain in a functionnal way.
      * 
      * @param  string $class_type_interface_or_instance The expected target class/type/interface/instance
      * @return \JClaveau\Async\DeferredCallChain
      */
-    function later($class_type_interface_or_instance=null)
+    function later($class_type_interface_or_instance = null)
     {
         return new \JClaveau\Async\DeferredCallChain(...func_get_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