spaark / composite-utils

src/Factory/Reflection/TypeParser.php

Doc Comments +3 added lines

@@ -225,6 +225,9 @@
             : new ObjectType($context);
     }
 
+    /**
+     * @param string $var
+     */
     public function scalarToType($var) : ?AbstractType
     {
         switch (strtolower($var))

Spacing +4 added lines, -5 removed lines

@@ -88,8 +88,7 @@ 
      */
     public function parseObjectName($var) : ObjectType
     {
-        return $this->parse(str_replace
-        (
+        return $this->parse(str_replace(
             ['_g', '_e', '_c'],
             ['<', '>', ','],
             get_class($var)
@@ -134,7 +133,7 @@ 
                     $item = $stack->pop();
                     if ($value{$i - 1} !== '>')
                     {
-                        $item->generics[] =$this->resolveName();
+                        $item->generics[] = $this->resolveName();
                     }
 
                     if ($i + 1 !== strlen($value) && $value{$i + 1} === '[')
@@ -182,7 +181,7 @@ 
         }
         elseif ($i + 2 !== strlen($value))
         {
-            if (!in_array($value{$i + 2}, ['>',',']))
+            if (!in_array($value{$i + 2}, ['>', ',']))
             {
                 throw new \Exception('Unexpected char after collection');
             }
@@ -225,7 +224,7 @@ 
             : new ObjectType($context);
     }
 
-    public function scalarToType($var) : ?AbstractType
+    public function scalarToType($var) : ? AbstractType
     {
         switch (strtolower($var))
         {

src/Service/GenericNameProvider.php

Unused Use Statements -2 removed lines

@@ -20,10 +20,8 @@
 use Spaark\CompositeUtils\Model\Reflection\Type\FloatType;
 use Spaark\CompositeUtils\Model\Reflection\Type\MixedType;
 use Spaark\CompositeUtils\Model\Reflection\Type\IntegerType;
-use Spaark\CompositeUtils\Model\Reflection\Type\CollectionType;
 use Spaark\CompositeUtils\Model\Reflection\Type\StringType;
 use Spaark\CompositeUtils\Model\Reflection\Type\GenericType;
-use Spaark\CompositeUtils\Model\Generic\GenericContext;
 use Spaark\CompositeUtils\Exception\MissingContextException;
 use Spaark\CompositeUtils\Traits\HasGenericContextTrait;
 use Spaark\CompositeUtils\Model\ClassName;

Spacing +2 added lines, -4 removed lines

@@ -63,8 +63,7 @@ 
                 return $this->inferGenericName($reflect);
         }
 
-        throw new \DomainException
-        (
+        throw new \DomainException(
             'Unknown type: ' . get_class($reflect)
         );
     }
@@ -101,8 +100,7 @@ 
                 $items[] = $this->inferName($generic);
             }
 
-            return new ClassName
-            (
+            return new ClassName(
                   self::BASE . $reflect->classname
                 . '_g' . implode('_c', $items) . '_e'
             );

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/Model/Reflection/Type/ObjectType.php

Spacing +1 added lines, -2 removed lines

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

src/Service/TypeComparator.php

Spacing +6 added lines, -12 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
@@ -66,8 +62,7 @@ 
             {
                 foreach ($child->generics as $i => $generic)
                 {
-                    $compare = $this->compatible
-                    (
+                    $compare = $this->compatible(
                         $parent->generics[$i],
                         $generic
                     );
@@ -84,8 +79,7 @@ 
             return false;
         }
 
-        throw new \DomainException
-        (
+        throw new \DomainException(
             'Unknown type: ' . get_class($parent)
         );
     }

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
             (