spaark / composite-utils

src/Factory/Reflection/GenericCompositeGenerator.php

Unused Use Statements -7 removed lines

@@ -16,15 +16,8 @@
 
 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\Service\RawPropertyAccessor;
 use Spaark\CompositeUtils\Service\GenericNameProvider;
 use Spaark\CompositeUtils\Traits\AutoConstructTrait;
 

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 =
-              '<?php namespace ' . $class->namespace . ';'
+                '<?php 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

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/Service/GenericNameProvider.php

Unused Use Statements -1 removed lines

@@ -20,7 +20,6 @@
 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;

Doc Comments +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
      * Infers the serialized name of the given ObjectType
      *
      * @param ObjectType $reflect
-     * @return string
+     * @return ClassName
      */
     protected function inferObjectName(ObjectType $reflect)
     {

Indentation +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
 
             return new ClassName
             (
-                  self::BASE . $reflect->classname
+                    self::BASE . $reflect->classname
                 . '_g' . implode('_c', $items) . '_e'
             );
         }

Spacing +3 added lines, -6 removed lines

@@ -69,8 +69,7 @@ 
                 return $this->inferGenericName($reflect);
         }
 
-        throw new \DomainException
-        (
+        throw new \DomainException(
             'Unknown type: ' . get_class($reflect)
         );
     }
@@ -89,8 +88,7 @@ 
             throw new MissingContextException();
         }
 
-        return $this->inferName
-        (
+        return $this->inferName(
             $this->context->getGenericType($reflect->name)
         );
     }
@@ -115,8 +113,7 @@ 
                 $items[] = $this->inferName($generic);
             }
 
-            return new ClassName
-            (
+            return new ClassName(
                   self::BASE . $reflect->classname
                 . '_g' . implode('_c', $items) . '_e'
             );

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)

src/Model/ClassName.php

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

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];

src/Service/TypeComparator.php

Spacing +3 added lines, -6 removed lines

@@ -31,8 +31,7 @@ 
      * @param AbstractType $child
      * @return boolean
      */
-    public function compatible
-    (
+    public function compatible(
         AbstractType $parent,
         AbstractType $child
     )
@@ -50,16 +49,14 @@ 
         {
             return
                 $child instanceof ObjectType && 
-                is_a
-                (
+                is_a(
                     $child->classname->__toString(),
                     $parent->classname->__toString(),
                     true
                 );
         }
 
-        throw new \DomainException
-        (
+        throw new \DomainException(
             'Unknown type: ' . get_class($parent)
         );
     }

src/Service/PropertyAccessor.php

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);
@@ -115,8 +113,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());
@@ -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);
         }