spaark / composite-utils

src/Factory/Reflection/ReflectionCompositeFactory.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 
 namespace Spaark\CompositeUtils\Factory\Reflection;
 
-use Spaark\CompositeUtils\Factory\BaseFactory;
 use Spaark\CompositeUtils\Model\Reflection\ReflectionComposite;
 use \ReflectionClass as PHPNativeReflectionClass;
 use \ReflectionProperty as PHPNativeReflectionProperty;

Spacing +4 added lines, -8 removed lines

@@ -61,13 +61,11 @@ 
             (new ReflectionFileFactory($this->reflector->getFileName()))
                 ->build();
         $this->accessor->setRawValue('file', $file);
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'classname',
             $this->reflector->name
         );
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'namespace',
             $file->namespaces[$this->reflector->getNamespaceName()]
         );
@@ -97,8 +95,7 @@ 
      *
      * @param PHPNativeReflectionProperty
      */
-    protected function buildProperty
-    (
+    protected function buildProperty(
         PHPNativeReflectionProperty $reflect
     )
     {
@@ -106,8 +103,7 @@ 
 
         $properties[$reflect->getName()] = 
             (new ReflectionPropertyFactory($reflect))
-                ->build
-                (
+                ->build(
                     $this->object,
                     $this->reflector
                         ->getDefaultProperties()[$reflect->getName()]

src/Model/Reflection/Type/ObjectType.php

Doc Comments +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     /**
      * Creates this ObjectType with the given classname
      *
-     * @param string $class The name of the class this must be an
+     * @param string $classname The name of the class this must be an
      *     instance of
      */
     public function __construct(string $classname)

src/Exception/PropertyAccessException.php

Indentation +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
     {
         parent::__construct
         (
-              'Cannot ' . static::ACCESS_TYPE . ' property: '
+                'Cannot ' . static::ACCESS_TYPE . ' property: '
             . $class . '::$' . $property . '. ' . static::ERROR_REASON,
             0,
             $previous

Spacing +2 added lines, -4 removed lines

@@ -47,15 +47,13 @@
      * @param string $property The property accessed
      * @param Throwable $previous The exception which caused this
      */
-    public function __construct
-    (
+    public function __construct(
         string $class,
         string $property,
         Throwable $previous = null
     )
     {
-        parent::__construct
-        (
+        parent::__construct(
               'Cannot ' . static::ACCESS_TYPE . ' property: '
             . $class . '::$' . $property . '. ' . static::ERROR_REASON,
             0,

src/Exception/MissingRequiredParameterException.php

Indentation +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     {
         parent::__construct
         (
-              'Missing required parameter in constructor. '
+                'Missing required parameter in constructor. '
             . $class . ' requires a value for ' . $parameter,
             0,
             $previous

Spacing +2 added lines, -4 removed lines

@@ -30,15 +30,13 @@
      * @param string $parameter The name of the missing parameter
      * @param Throwable The exception which caused this
      */
-    public function __construct
-    (
+    public function __construct(
         string $class,
         string $parameter,
         Throwable $previous = null
     )
     {
-        parent::__construct
-        (
+        parent::__construct(
               'Missing required parameter in constructor. '
             . $class . ' requires a value for ' . $parameter,
             0,

src/Factory/Reflection/ReflectionMethodFactory.php

Spacing +3 added lines, -5 removed lines

@@ -46,8 +46,7 @@ 
      */
     public static function fromName(string $class, string $method)
     {
-        return new static(new PHPNativeReflectionMethod
-        (
+        return new static(new PHPNativeReflectionMethod(
             $class, $method
         ));
     }
@@ -60,11 +59,10 @@ 
      *     this method belongs to
      * @return ReflectionMethod
      */
-    public function build(?ReflectionComposite $parent = null)
+    public function build(? ReflectionComposite $parent = null)
     {
         $this->accessor->setRawValue('owner', $parent);
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'name',
             $this->reflector->getName()
         );

src/Model/Collection/MapIterator.php

Unused Use Statements -2 removed lines

@@ -14,8 +14,6 @@
 
 namespace Spaark\CompositeUtils\Model\Collection;
 
-use Spaark\CompositeUtils\Traits\AutoConstructTrait;
-
 /**
  * Abstract Iterator for Map datatypes
  *

src/Service/PropertyAccessor.php

Spacing +11 added lines, -22 removed lines

@@ -65,8 +65,7 @@ 
         {
             if (!isset($args[$i]))
             {
-                throw new MissingRequiredParameterException
-                (
+                throw new MissingRequiredParameterException(
                     get_class($this->object),
                     $property->name
                 );
@@ -144,14 +143,12 @@ 
     {
         if (!$this->reflect->properties->containsKey($property))
         {
-            throw new CannotWritePropertyException
-            (
+            throw new CannotWritePropertyException(
                 get_class($this->object), $property
             );
         }
 
-        $this->setAnyValue
-        (
+        $this->setAnyValue(
             $this->reflect->properties[$property],
             $value
         );
@@ -198,8 +195,7 @@ 
      * @param ReflectionProperty $property The property to set
      * @param mixed $value The value to set
      */
-    private function setNonNullValue
-    (
+    private function setNonNullValue(
         ReflectionProperty $property,
         $value
     )
@@ -248,8 +244,7 @@ 
      * @param callable $cast Method to cast a value to the scalar data
      *     type
      */
-    private function setScalarValue
-    (
+    private function setScalarValue(
         ReflectionProperty $property,
         $value,
         string $name,
@@ -264,8 +259,7 @@ 
         }
         elseif (is_object($value) && method_exists([$value, $method]))
         {
-            $this->setScalarValue
-            (
+            $this->setScalarValue(
                 $property,
                 $value->$method(),
                 $method,
@@ -284,8 +278,7 @@ 
      * @param ReflectionProperty $property The property to set
      * @param mixed $value The value to set
      */
-    private function setObjectValue
-    (
+    private function setObjectValue(
         ReflectionProperty $property,
         $value
     )
@@ -296,8 +289,7 @@ 
         }
         else
         {
-            $this->throwError
-            (
+            $this->throwError(
                 $property,
                 $property->type->classname,
                 $value
@@ -311,8 +303,7 @@ 
      * @param ReflectionProperty $property The property to set
      * @param mixed The value to set
      */
-    private function setCollectionValue
-    (
+    private function setCollectionValue(
         ReflectionProperty $property,
         $value
     )
@@ -334,15 +325,13 @@ 
      * @param string $expected The expected datatype
      * @param string $value The value being set
      */
-    private function throwError
-    (
+    private function throwError(
         ReflectionProperty $property,
         string $expected,
         $value
     )
     {
-        throw new IllegalPropertyTypeException
-        (
+        throw new IllegalPropertyTypeException(
             get_class($this->object),
             $property->name,
             $expected,

Braces +9 added lines, -18 removed lines

@@ -83,15 +83,13 @@ 
             if ($building)
             {
                 $this->buildProperty($property);
-            }
-            else
+            } else
             {
                 if (isset($args[$i]))
                 {
                     $this->setAnyValue($property, $args[$i]);
                     $i++;
-                }
-                else
+                } else
                 {
                     $building = true;
                     $this->buildProperty($property);
@@ -116,8 +114,7 @@ 
         {
             $class = $property->type->classname;
             $this->setRawValue($property->name, new $class());
-        }
-        else
+        } else
         {
             $this->setAnyValue($property, 0);
         }
@@ -168,8 +165,7 @@ 
         if (is_null($value))
         {
             $this->setNullValue($property);
-        }
-        else
+        } else
         {
             $this->setNonNullValue($property, $value);
         }
@@ -185,8 +181,7 @@ 
         if ($property->type->nullable)
         {
             $this->setRawValue($property->name, null);
-        }
-        else
+        } else
         {
             $this->throwError($property, 'NonNull', null);
         }
@@ -261,8 +256,7 @@ 
         if (is_scalar($value))
         {
             $this->setRawValue($property->name, $cast($value));
-        }
-        elseif (is_object($value) && method_exists([$value, $method]))
+        } elseif (is_object($value) && method_exists([$value, $method]))
         {
             $this->setScalarValue
             (
@@ -271,8 +265,7 @@ 
                 $method,
                 $cast
             );
-        }
-        else
+        } else
         {
             $this->throwError($property, $name, $value);
         }
@@ -293,8 +286,7 @@ 
         if (is_a($value, $property->type->classname))
         {
             $this->setRawValue($property->name, $value);
-        }
-        else
+        } else
         {
             $this->throwError
             (
@@ -320,8 +312,7 @@ 
         if (is_a($value, \ArrayAccess::class) || is_array($value))
         {
             $this->setRawValue($property->name, $value);
-        }
-        else
+        } else
         {
             $this->throwError($property, 'Collection', $value);
         }

src/Model/Collection/ArrayList.php

Spacing +2 added lines, -3 removed lines

@@ -45,10 +45,9 @@
     /**
      * {@inheritDoc}
      */
-    public function splice
-    (
+    public function splice(
         int $offset,
-        ?int $length = null,
+        ? int $length = null,
         array $replacement = []
     )
     {

src/Model/Collection/ListInterface.php

Spacing +2 added lines, -3 removed lines

@@ -33,10 +33,9 @@
      * @param int $length The number of elements to remove
      * @param array $items Items to add
      */
-    public function splice
-    (
+    public function splice(
         int $offset,
-        ?int$length = null,
+        ? int$length = null,
         array $items = []
     );