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 +6 added lines, -12 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,20 +95,17 @@ 
      *
      * @param PHPNativeReflectionProperty
      */
-    protected function buildProperty
-    (
+    protected function buildProperty(
         PHPNativeReflectionProperty $reflect
     )
     {
-        $properties = $this->accessor->getRawValue
-        (
+        $properties = $this->accessor->getRawValue(
             'properties'
         );
 
         $properties[$reflect->getName()] = 
             (new ReflectionPropertyFactory($reflect))
-                ->build
-                (
+                ->build(
                     $this->object,
                     $this->reflector
                         ->getDefaultProperties()[$reflect->getName()]
@@ -125,8 +120,7 @@ 
      */
     protected function buildMethod(PHPNativeReflectionMethod $reflect)
     {
-        $this->accessor->rawAddToValue
-        (
+        $this->accessor->rawAddToValue(
             'methods',
             (new ReflectionMethodFactory($reflect))
                 ->build($this->object)

src/Factory/Reflection/ReflectionPropertyFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
      * Builds the ReflectionProperty from the provided parameters,
      * linking to a parent ReflectionComposite
      *
-     * @param ReflectionCompostite $parent The reflector for the class
+     * @param ReflectionComposite $parent The reflector for the class
      *     this property belongs to
      * @param mixed $default This property's default value
      * @return ReflectionProperty

Spacing +10 added lines, -20 removed lines

@@ -64,8 +64,7 @@ 
      */
     public static function fromName($class, $property)
     {
-        return new static(new PHPNativeReflectionProperty
-        (
+        return new static(new PHPNativeReflectionProperty(
             $class, $property
         ));
     }
@@ -83,8 +82,7 @@ 
     {
         $this->accessor->setRawValue('owner', $parent);
         $this->accessor->setRawValue('defaultValue', $default);
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'name',
             $this->reflector->getName()
         );
@@ -189,50 +187,42 @@ 
         switch ($value[0])
         {
             case 'required':
-                $this->accessor->setRawValue
-                (
+                $this->accessor->setRawValue(
                     'passedToConstructor',
                     true
                 );
-                $this->accessor->setRawValue
-                (
+                $this->accessor->setRawValue(
                     'requiredInConstructor',
                     true
                 );
-                $compositeAccessor->rawAddToValue
-                (
+                $compositeAccessor->rawAddToValue(
                     'requiredProperties',
                     $this->object
                 );
                 break;
             case 'new':
-                $this->accessor->setRawValue
-                (
+                $this->accessor->setRawValue(
                     'builtInConstructor',
                     true
                 );
-                $compositeAccessor->rawAddToValue
-                (
+                $compositeAccessor->rawAddToValue(
                     'builtProperties',
                     $this->object
                 );
                 break;
             case 'optional':
-                $this->accessor->setRawValue
-                (
+                $this->accessor->setRawValue(
                     'passedToConstructor',
                     true
                 );
-                $compositeAccessor->rawAddToValue
-                (
+                $compositeAccessor->rawAddToValue(
                     'optionalProperties',
                     $this->object
                 );
 
                 if (isset($value[1]) && $value[1] === 'new')
                 {
-                    $this->accessor->setRawValue
-                    (
+                    $this->accessor->setRawValue(
                         'builtInConstructor',
                         true
                     );

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/Exception/IllegalPropertyTypeException.php

Spacing +3 added lines, -5 removed lines

@@ -32,8 +32,7 @@ 
      * @param string $got The type that was provided
      * @param Exception $previous The exception which caused this
      */
-    public function __construct
-    (
+    public function __construct(
         string $class,
         string $property,
         string $expected,
@@ -41,10 +40,9 @@ 
         Exception $previous = null
     )
     {
-        parent::__construct
-        (
+        parent::__construct(
               'Tried to set an illegal property type for '
-            . $class .'::$' . $property . '. Excpected ' . $expected
+            . $class . '::$' . $property . '. Excpected ' . $expected
             . ', got ' . $got,
             0,
             $previous

src/Service/PropertyAccessor.php

Spacing +10 added lines, -20 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->contains($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
     )
@@ -306,8 +299,7 @@ 
      * @param ReflectionProperty $property The property to set
      * @param mixed The value to set
      */
-    private function setCollectionValue
-    (
+    private function setCollectionValue(
         ReflectionProperty $property,
         $value
     )
@@ -329,15 +321,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($property, $type->classname, $value);
         }
@@ -315,8 +307,7 @@ 
         if (is_a($value, \ArrayAccess::class) || is_array($value))
         {
             $this->setRawValue($property->name, $value);
-        }
-        else
+        } else
         {
             $this->throwError($property, 'Collection', $value);
         }

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/Factory/Reflection/ReflectorFactory.php

Spacing +3 added lines, -5 removed lines

@@ -71,8 +71,7 @@ 
      */
     protected function parseDocComment()
     {
-        preg_match_all
-        (
+        preg_match_all(
               '/^'
             .     '[ \t]*\*[ \t]*'
             .     '@([a-zA-Z]+)'
@@ -89,8 +88,7 @@ 
 
             if (isset($this->acceptedParams[$name]))
             {
-                call_user_func
-                (
+                call_user_func(
                     array($this, $this->acceptedParams[$name]),
                     $name, $value
                 );
@@ -106,7 +104,7 @@ 
      */
     protected function setBool($name, $value)
     {
-        switch(strtolower($value))
+        switch (strtolower($value))
         {
             case '':
             case 'true':