spaark / composite-utils

src/Model/Common/Equatable.php

Doc Comments +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
     /**
      * Checks if the given class is equal to this one
      *
-     * @param Type $oject
+     * @param Type $object
      * @return boolean
      */
     public function equals($object) : bool;

src/Factory/Reflection/ReflectionMethodFactory.php

Indentation +2 added lines, -2 removed lines

@@ -97,7 +97,7 @@ 
         $this->initParams();
 
         $this->accessor->setRawValue('visibility',
-              ($this->reflector->isPublic() ? 'public'
+                ($this->reflector->isPublic() ? 'public'
             : ($this->reflector->isProtected() ? 'protected'
             : ($this->reflector->isPrivate() ? 'private'
             : (''))))
@@ -154,7 +154,7 @@ 
         {
             throw new \Exception
             (
-                  'Tried to set param annotation for non existant '
+                    'Tried to set param annotation for non existant '
                 . 'parameter: ' . $param
             );
         }

Spacing +7 added lines, -13 removed lines

@@ -63,8 +63,7 @@ 
      */
     public static function fromName(string $class, string $method)
     {
-        return new static(new PHPNativeReflectionMethod
-        (
+        return new static(new PHPNativeReflectionMethod(
             $class, $method
         ));
     }
@@ -84,12 +83,11 @@ 
      *     this method belongs to
      * @return ReflectionMethod
      */
-    public function build(?ReflectionComposite $parent = null)
+    public function build(? ReflectionComposite $parent = null)
     {
         $this->typeParser = new TypeParser($parent);
         $this->accessor->setRawValue('owner', $parent);
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'name',
             $this->reflector->getName()
         );
@@ -125,13 +123,11 @@ 
      */
     protected function initParams()
     {
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'parameters',
             new FixedList(count($this->reflector->getParameters()))
         );
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'nativeParameters',
             new FixedList(count($this->reflector->getParameters()))
         );
@@ -152,8 +148,7 @@ 
 
         if (!$this->parameters->containsKey($param))
         {
-            throw new \Exception
-            (
+            throw new \Exception(
                   'Tried to set param annotation for non existant '
                 . 'parameter: ' . $param
             );
@@ -178,8 +173,7 @@ 
      *
      * @param PHPNativeReflectionParameter $reflect
      */
-    protected function addParameter
-    (
+    protected function addParameter(
         PHPNativeReflectionParameter $reflect
     )
     : void

src/Model/Generic/GenericContext.php

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 use Spaark\CompositeUtils\Model\Reflection\Type\AbstractType;
 use Spaark\CompositeUtils\Model\Reflection\Type\ObjectType;
 use Spaark\CompositeUtils\Model\Reflection\ReflectionComposite;
-use Spaark\CompositeUtils\Traits\AllReadableTrait;
 use Spaark\CompositeUtils\Traits\AutoConstructTrait;
 
 /**

src/Exception/MissingContextException.php

Indentation +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     {
         parent::__construct
         (
-              'Cannot serialize object containing generics without '
+                'Cannot serialize object containing generics without '
             . 'context',
             0,
             $previous

Spacing +2 added lines, -4 removed lines

@@ -28,13 +28,11 @@
      *
      * @param Throwable $previous The exception which caused this
      */
-    public function __construct
-    (
+    public function __construct(
         Throwable $previous = null
     )
     {
-        parent::__construct
-        (
+        parent::__construct(
               'Cannot serialize object containing generics without '
             . 'context',
             0,

src/Factory/Reflection/GenericCompositeGenerator.php

Spacing +1 added lines, -2 removed lines

@@ -73,8 +73,7 @@
     public function generateClassCode(...$generics) : string
     {
         $object = $this->createObject(...$generics);
-        $this->nameProvider = new GenericNameProvider
-        (
+        $this->nameProvider = new GenericNameProvider(
             new GenericContext($object, $this->reflect)
         );
         $class = $this->nameProvider->inferName($object);

Indentation +2 added lines, -2 removed lines

@@ -82,7 +82,7 @@ 
         $i = 0;
 
         $code =
-              'namespace ' . $class->namespace . ';'
+                'namespace ' . $class->namespace . ';'
             . 'class ' . $class->classname . ' '
             .     'extends \\' . $originalClass
             . '{';
@@ -118,7 +118,7 @@ 
         }
 
         return
-              ($method->scope === 'static' ? 'static ' : '')
+                ($method->scope === 'static' ? 'static ' : '')
             . 'function ' . $method->name
             . '(' . implode(',', $params) . '){'
             . '__generic_' . $method->name

Unused Use Statements -7 removed lines

@@ -16,16 +16,9 @@
 
 use Spaark\CompositeUtils\Model\Reflection\ReflectionComposite;
 use Spaark\CompositeUtils\Model\Reflection\ReflectionMethod;
-use Spaark\CompositeUtils\Model\Reflection\Type\BooleanType;
-use Spaark\CompositeUtils\Model\Reflection\Type\CollectionType;
-use Spaark\CompositeUtils\Model\Reflection\Type\IntegerType;
-use Spaark\CompositeUtils\Model\Reflection\Type\MixedType;
 use Spaark\CompositeUtils\Model\Reflection\Type\ObjectType;
-use Spaark\CompositeUtils\Model\Reflection\Type\StringType;
-use Spaark\CompositeUtils\Model\Reflection\Type\GenericType;
 use Spaark\CompositeUtils\Model\Generic\GenericContext;
 use Spaark\CompositeUtils\Model\ClassName;
-use Spaark\CompositeUtils\Service\RawPropertyAccessor;
 use Spaark\CompositeUtils\Service\GenericNameProvider;
 use Spaark\CompositeUtils\Traits\AutoConstructPropertyAccessTrait;
 

src/Service/ReflectionCompositeProvider.php

Spacing +2 added lines, -4 removed lines

@@ -54,8 +54,7 @@ 
      *
      * @param ReflectionCompositeProviderInterface $default
      */
-    public static function setDefault
-    (
+    public static function setDefault(
         ReflectionCompositeProviderInterface $default
     )
     {
@@ -90,8 +89,7 @@ 
         {
             $this->cache[$classname] =
                 (
-                    ReflectionCompositeFactory::fromClassName
-                    (
+                    ReflectionCompositeFactory::fromClassName(
                         $classname
                     )
                 )

src/Model/Reflection/Type/ObjectType.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
     /**
      * Creates this ObjectType with the given classname
      *
-     * @param mixed $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($classname)

Spacing +1 added lines, -2 removed lines

@@ -59,8 +59,7 @@
      */
     public function equals($type) : bool
     {
-        if
-        (
+        if (
             $type instanceof ObjectType &&
             $type->classname->equals($this->classname) &&
             $type->generics->size() === $this->generics->size()

src/Model/ClassName.php

Braces +1 added lines, -2 removed lines

@@ -46,8 +46,7 @@
         if (!$matches)
         {
             $this->classname = $classname;
-        }
-        else
+        } else
         {
             $this->classname = $matches[2];
             $this->namespace = $matches[1];

Indentation +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
     public function __toString()
     {
         return
-              ($this->namespace ? $this->namespace . '\\' : '')
+                ($this->namespace ? $this->namespace . '\\' : '')
             . $this->classname;
     }
 

src/Service/PropertyAccessor.php

Unused Use Statements -5 removed lines

@@ -17,11 +17,6 @@
 use Spaark\CompositeUtils\Model\Reflection\ReflectionComposite;
 use Spaark\CompositeUtils\Model\Reflection\ReflectionProperty;
 use Spaark\CompositeUtils\Model\Reflection\Type\ObjectType;
-use Spaark\CompositeUtils\Model\Reflection\Type\MixedType;
-use Spaark\CompositeUtils\Model\Reflection\Type\StringType;
-use Spaark\CompositeUtils\Model\Reflection\Type\IntegerType;
-use Spaark\CompositeUtils\Model\Reflection\Type\BooleanType;
-use Spaark\CompositeUtils\Model\Reflection\Type\CollectionType;
 use Spaark\CompositeUtils\Model\Reflection\Type\ScalarType;
 use Spaark\CompositeUtils\Model\Reflection\Type\AbstractType;
 use Spaark\CompositeUtils\Exception\CannotWritePropertyException;

Spacing +8 added lines, -16 removed lines

@@ -69,8 +69,7 @@ 
         {
             if (!isset($args[$i]))
             {
-                throw new MissingRequiredParameterException
-                (
+                throw new MissingRequiredParameterException(
                     get_class($this->object),
                     $property->name
                 );
@@ -148,14 +147,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
         );
@@ -193,8 +190,7 @@ 
      * @param ScalarType $valueType The scalar type
      * @param mixed $value The value to set
      */
-    private function setScalarValue
-    (
+    private function setScalarValue(
         ReflectionProperty $property,
         ScalarType $valueType,
         $value
@@ -204,16 +200,14 @@ 
 
         if (is_scalar($value))
         {
-            $this->setRawValue
-            (
+            $this->setRawValue(
                 $property->name,
                 $property->type->cast($value)
             );
         }
         elseif (is_object($value) && method_exists([$value, $method]))
         {
-            $this->setScalarValue
-            (
+            $this->setScalarValue(
                 $property,
                 $valueType,
                 $value->$method()
@@ -231,14 +225,12 @@ 
      * @param ReflectionProperty $property The property being set
      * @param AbstractType $valueType The value being set
      */
-    private function throwError
-    (
+    private function throwError(
         ReflectionProperty $property,
         AbstractType $valueType
     )
     {
-        throw new IllegalPropertyTypeException
-        (
+        throw new IllegalPropertyTypeException(
             get_class($this->object),
             $property->name,
             $property->type,

Braces +7 added lines, -14 removed lines

@@ -87,15 +87,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);
@@ -119,8 +117,7 @@ 
         if (!$property->type instanceof ObjectType)
         {
             $this->setAnyValue($property, 0);
-        }
-        elseif ($property->builtInConstructor)
+        } elseif ($property->builtInConstructor)
         {
             $class = (string)$property->type->classname;
             $this->setRawValue($property->name, new $class());
@@ -175,12 +172,10 @@ 
         if ($comparator->compatible($property->type, $valueType))
         {
             $this->setRawValue($property->name, $value);
-        }
-        elseif ($property->type instanceof ScalarType)
+        } elseif ($property->type instanceof ScalarType)
         {
             $this->setScalarValue($property, $valueType, $value);
-        }
-        else
+        } else
         {
             $this->throwError($property, $valueType);
         }
@@ -209,8 +204,7 @@ 
                 $property->name,
                 $property->type->cast($value)
             );
-        }
-        elseif (is_object($value) && method_exists([$value, $method]))
+        } elseif (is_object($value) && method_exists([$value, $method]))
         {
             $this->setScalarValue
             (
@@ -218,8 +212,7 @@ 
                 $valueType,
                 $value->$method()
             );
-        }
-        else
+        } else
         {
             $this->throwError($property, $valueType);
         }