spaark / composite-utils

src/Traits/HasReflectorTrait.php

Spacing +2 added lines, -4 removed lines

@@ -41,8 +41,7 @@ 
         if (!static::$reflectionComposite)
         {
             static::$reflectionComposite =
-                ReflectionCompositeFactory::fromClassName
-                (
+                ReflectionCompositeFactory::fromClassName(
                     get_called_class()
                 )
                 ->build();
@@ -54,8 +53,7 @@ 
     /**
      * Set the default ReflectionComposite for this class
      */
-    protected static function setDefaultReflectionComposite
-    (
+    protected static function setDefaultReflectionComposite(
         ReflectionComposite $defaultReflectionComposite
     )
     {

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/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/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)

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 
 namespace Spaark\CompositeUtils\Model\Reflection\Type;
 
-use Spaark\CompositeUtils\Model\Collection\ListCollection\FlexibleList;
 use Spaark\CompositeUtils\Model\ClassName;
 
 /**

Spacing +1 added lines, -2 removed lines

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

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);
         }

src/Service/TypeComparator.php

Braces +2 added lines, -4 removed lines

@@ -46,12 +46,10 @@
         )
         {
             return true;
-        }
-        elseif ($parent instanceof ScalarType)
+        } elseif ($parent instanceof ScalarType)
         {
             return get_class($parent) === get_class($child);
-        }
-        elseif ($parent instanceof ObjectType)
+        } elseif ($parent instanceof ObjectType)
         {
             if
             (

Spacing +5 added lines, -10 removed lines

@@ -32,15 +32,13 @@ 
      * @param AbstractType $child
      * @return boolean
      */
-    public function compatible
-    (
+    public function compatible(
         AbstractType $parent,
         AbstractType $child
     )
     : bool
     {
-        if
-        (
+        if (
             ($child instanceof NullType && $parent->nullable) ||
             ($parent instanceof MixedType)
         )
@@ -53,11 +51,9 @@ 
         }
         elseif ($parent instanceof ObjectType)
         {
-            if
-            (
+            if (
                 $child instanceof ObjectType && 
-                is_a
-                (
+                is_a(
                     $child->classname->__toString(),
                     $parent->classname->__toString(),
                     true
@@ -70,8 +66,7 @@ 
             return false;
         }
 
-        throw new \DomainException
-        (
+        throw new \DomainException(
             'Unknown type: ' . get_class($parent)
         );
     }

src/Model/Collection/Map/MapIterator.php

Unused Use Statements -2 removed lines

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