spaark / composite-utils

src/Service/PropertyAccessor.php

Spacing +11 added lines, -22 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->containsKey($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
     )
@@ -296,8 +289,7 @@ 
         }
         else
         {
-            $this->throwError
-            (
+            $this->throwError(
                 $property,
                 $property->type->classname,
                 $value
@@ -311,8 +303,7 @@ 
      * @param ReflectionProperty $property The property to set
      * @param mixed The value to set
      */
-    private function setCollectionValue
-    (
+    private function setCollectionValue(
         ReflectionProperty $property,
         $value
     )
@@ -334,15 +325,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);
@@ -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);
         }

src/Model/Collection/ArrayList.php

Spacing +2 added lines, -3 removed lines

@@ -45,10 +45,9 @@
     /**
      * {@inheritDoc}
      */
-    public function splice
-    (
+    public function splice(
         int $offset,
-        ?int $length = null,
+        ? int $length = null,
         array $replacement = []
     )
     {

src/Model/Collection/ListInterface.php

Spacing +2 added lines, -3 removed lines

@@ -33,10 +33,9 @@
      * @param int $length The number of elements to remove
      * @param array $items Items to add
      */
-    public function splice
-    (
+    public function splice(
         int $offset,
-        ?int$length = null,
+        ? int$length = null,
         array $items = []
     );
 

src/Model/Collection/HashMap.php

Spacing +3 added lines, -5 removed lines

@@ -41,8 +41,7 @@ 
         return static::$defaultHashProducer;
     }
 
-    public static function setDefaultHashProducer
-    (
+    public static function setDefaultHashProducer(
         HashProducerInterface $hashProducer
     )
     : HashProducerInterface
@@ -60,9 +59,8 @@ 
      */
     protected $hashProducer;
 
-    public function __construct
-    (
-        ?HashProducerInterface $hashProducer = null
+    public function __construct(
+        ? HashProducerInterface $hashProducer = null
     )
     {
         $this->hashProducer =

src/Model/Collection/FixedList.php

Spacing +2 added lines, -3 removed lines

@@ -45,10 +45,9 @@
     /**
      * {@inheritDoc}
      */
-    public function splice
-    (
+    public function splice(
         int $offset,
-        ?int $length = null,
+        ? int $length = null,
         array $replacement = []
     )
     {

src/Model/Collection/AbstractList.php

Braces +1 added lines, -2 removed lines

@@ -29,8 +29,7 @@
         if ($offset === null)
         {
             $this->add($value);
-        }
-        else
+        } else
         {
             $this->set($offset, $value);
         }

src/Factory/Reflection/ReflectionCompositeFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -130,7 +130,7 @@
      *
      * @param string $name
      * @param bool $checkFile
-     * @param string $singular
+     * @param string $signular
      */
     protected function addItems
     (

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 Spaark\CompositeUtils\Model\Reflection\ReflectionProperty;
 use Spaark\CompositeUtils\Model\Reflection\ReflectionMethod;

Indentation +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
             else
             {
                 $factory =
-                      '\Spaark\CompositeUtils\Factory\Reflection'
+                        '\Spaark\CompositeUtils\Factory\Reflection'
                     . '\Reflection' . $signular . 'Factory';
                 $item = $this->{'build' . $signular}
                 (

Spacing +10 added lines, -20 removed lines

@@ -78,8 +78,7 @@ 
      * @param PHPNativeReflectionClass $reflect
      * @param ReflectionCompositeProviderInterface $provider
      */
-    public function __construct
-    (
+    public function __construct(
         PHPNativeReflectionClass $reflect,
         ReflectionCompositeProviderInterface $provider
     )
@@ -112,8 +111,7 @@ 
             $this->addInheritance('interfaces', $interface);
         }
 
-        $this->parseDocComment
-        ([
+        $this->parseDocComment([
             'generic' => 'addGeneric'
         ]);
 
@@ -122,13 +120,11 @@ 
         $file = (new ReflectionFileFactory($fileName))->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()]
         );
@@ -151,8 +147,7 @@ 
             $size = count($this->reflector->{'get' . $name}());
             foreach ($prefixes as $prefix)
             {
-                $this->accessor->setRawValue
-                (
+                $this->accessor->setRawValue(
                     $prefix . $name,
                     new FixedList($size)
                 );
@@ -207,8 +202,7 @@ 
      * @param bool $checkFile
      * @param string $singular
      */
-    protected function addItems
-    (
+    protected function addItems(
         string $name,
         bool $checkFile,
         string $signular
@@ -239,8 +233,7 @@ 
                     new $factory($item),
                     $item
                 );
-                $this->accessor->rawAddToValue
-                (
+                $this->accessor->rawAddToValue(
                     'local' . ucfirst($name),
                     $item
                 );
@@ -257,8 +250,7 @@ 
      * @param PHPNativeReflectionClass $reflect
      * @param string $method
      */
-    protected function addInheritance
-    (
+    protected function addInheritance(
         string $group,
         PHPNativeReflectionClass $reflect,
         string $method = 'rawAddToValue'
@@ -283,15 +275,13 @@ 
      * @param ReflectionPropertyFactory $factory
      * @return ReflectionProperty
      */
-    protected function buildProperty
-    (
+    protected function buildProperty(
         ReflectionPropertyFactory $factory,
         PHPNativeReflectionProperty $reflect
     )
     : ReflectionProperty
     {
-        return $factory->build
-        (
+        return $factory->build(
             $this->object,
             $this->reflector
                 ->getDefaultProperties()[$reflect->getName()]

Braces +1 added lines, -2 removed lines

@@ -174,8 +174,7 @@
         {
             $key = $value;
             $type = '';
-        }
-        else
+        } else
         {
             $key = substr($value, 0, $space);
             $type = substr($value, $space);

src/Service/GenericNameProvider.php

Braces +1 added lines, -2 removed lines

@@ -68,8 +68,7 @@
         if ($reflect->generics->empty())
         {
             return $reflect->classname;
-        }
-        else
+        } else
         {
             $items = [];
             foreach ($reflect->generics as $generic)

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

Spacing +2 added lines, -2 removed lines

@@ -100,7 +100,7 @@ 
                     $item = $stack->pop();
                     if ($value{$i - 1} !== '>')
                     {
-                        $item->generics[] =$this->resolveName();
+                        $item->generics[] = $this->resolveName();
                     }
 
                     if ($i + 1 !== strlen($value) && $value{$i + 1} === '[')
@@ -148,7 +148,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');
             }

Braces +5 added lines, -10 removed lines

@@ -113,8 +113,7 @@ 
                     if ($stack->isEmpty())
                     {
                         return $item;
-                    }
-                    else
+                    } else
                     {
                         $stack->top()->generics[] = $item;
                     }
@@ -141,12 +140,10 @@ 
         if ($i + 1 === strlen($value))
         {
             throw new \Exception('Unexpected EOF');
-        }
-        elseif ($value{$i + 1} !== ']')
+        } elseif ($value{$i + 1} !== ']')
         {
             throw new \Exception('[ must be followed by ]');
-        }
-        elseif ($i + 2 !== strlen($value))
+        } elseif ($i + 2 !== strlen($value))
         {
             if (!in_array($value{$i + 2}, ['>',',']))
             {
@@ -245,12 +242,10 @@ 
         if ($useStatements->containsKey($this->currentValue))
         {
             return $useStatements[$this->currentValue]->classname;
-        }
-        elseif ($generics->containsKey($this->currentValue))
+        } elseif ($generics->containsKey($this->currentValue))
         {
             return new GenericType($this->currentValue);
-        }
-        else
+        } else
         {
             return $this->context->namespace->namespace
                 . '\\' . $this->currentValue;