spaark / composite-utils

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)

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

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

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;

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/Model/Collection/OrderedMap.php

Spacing +1 added lines, -2 removed lines

@@ -44,8 +44,7 @@
     {
         if (!$pair instanceof OrderedPair)
         {
-            $pair = new OrderedPair
-            (
+            $pair = new OrderedPair(
                 $this->size(),
                 $pair->key,
                 $pair->value

Doc Comments +4 added lines, -3 removed lines

@@ -37,8 +37,6 @@ 
     /**
      * Adds an element to the Map
      *
-     * @param KeyType $key The key to add
-     * @param ValueType $value The value to add
      */
     public function insert(Pair $pair)
     {
@@ -59,7 +57,7 @@ 
     /**
      * Checks if a key exists
      *
-     * @param KeyType $key The key to search for
+     * @param string $key The key to search for
      * @return boolean
      */
     public function containsKey($key) : bool
@@ -103,6 +101,9 @@ 
         return $this->list->size();
     }
 
+    /**
+     * @param string $key
+     */
     public function indexOfKey($key)
     {
         return $this->getPair($key)->index;