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;

src/Exceptions/BadTargetTypeException.php

Spacing +1 added lines, -1 removed lines

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

src/Exceptions/TargetAlreadyDefinedException.php

Spacing +1 added lines, -1 removed lines

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

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

Spacing +1 added lines, -1 removed lines

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

src/Exceptions/BadTargetClassException.php

Spacing +1 added lines, -1 removed lines

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

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/ExportTrait.php

Spacing +5 added lines, -5 removed lines

@@ -40,7 +40,7 @@ 
      *
      * @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;
@@ -56,7 +56,7 @@ 
         foreach ($this->stack as $i => $call) {
             if (isset($call['method'])) {
                 $string .= '->';
-                $string .= $call['method'].'(';
+                $string .= $call['method'] . '(';
                 $string .= implode(', ', array_map(function($argument) use ($max_param_length, $short_objects) {
                     return static::varExport($argument, [
                         'short_objects' => $short_objects, 
@@ -72,7 +72,7 @@ 
                 ]) . ']';
             }
             
-            if (! empty($options['limit']) && $options['limit'] == $i) {
+            if (!empty($options['limit']) && $options['limit'] == $i) {
                 break;
             }
         }
@@ -87,10 +87,10 @@ 
      * @param  array  $options max_length | alias_instances
      * @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);
         

Braces +2 added lines, -4 removed lines

@@ -64,8 +64,7 @@ 
                     ]);
                 }, $call['arguments']));
                 $string .= ')';
-            }
-            else {
+            } else {
                 $string .= '[' . static::varExport($call['entry'], [
                     'short_objects' => $short_objects, 
                     'max_length' => $max_param_length,
@@ -105,8 +104,7 @@ 
             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)

src/DeferredCallChain.php

Spacing +13 added lines, -13 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);
                 }
             }
@@ -157,7 +157,7 @@ 
                     $class = $current_chained_subject;
                 }
                 
-                $callable = $class .'::'. $method_name;
+                $callable = $class . '::' . $method_name;
             }
             
             $current_chained_subject = call_user_func_array(
@@ -202,8 +202,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'
@@ -217,7 +217,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);
         
@@ -234,9 +234,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(
@@ -247,13 +247,13 @@ 
                         );
                     }
                     
-                    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)
                             . " nor a function"
@@ -261,7 +261,7 @@ 
                     }
                 }
                 else {
-                    $out = $out[ $call['entry'] ];
+                    $out = $out[$call['entry']];
                 }
             }
             catch (\Exception $e) {

Braces +11 added lines, -22 removed lines

@@ -98,30 +98,25 @@ 
             }
             
             $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;
         }
         
@@ -148,12 +143,10 @@ 
         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;
                 }
                 
@@ -164,16 +157,14 @@ 
                 $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;
             }
         }
@@ -259,12 +250,10 @@ 
                             . " nor a function"
                         );
                     }
-                }
-                else {
+                } else {
                     $out = $out[ $call['entry'] ];
                 }
-            }
-            catch (\Exception $e) {
+            } catch (\Exception $e) {
                 
                 $callchain_description = $this->toString([
                     'target' => $target,