spaark / composite-utils

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
  *

src/Model/Collection/Map/OrderedMap.php

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;

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

src/Model/Collection/ListCollection/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/GenericCompositeGenerator.php

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

Unused Use Statements -7 removed lines

@@ -16,16 +16,9 @@
 
 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\Model\ClassName;
-use Spaark\CompositeUtils\Service\RawPropertyAccessor;
 use Spaark\CompositeUtils\Service\GenericNameProvider;
 use Spaark\CompositeUtils\Traits\AutoConstructPropertyAccessTrait;
 use Spaark\CompositeUtils\Traits\GenericTrait;

Indentation +2 added lines, -2 removed lines

@@ -92,7 +92,7 @@ 
         $i = 0;
 
         $code =
-              'namespace ' . $class->namespace . ';'
+                'namespace ' . $class->namespace . ';'
             . 'class ' . $class->classname . ' '
             .     'extends \\' . $originalClass . ' '
             .     'implements \\' . Generic::class
@@ -130,7 +130,7 @@ 
         }
 
         return
-              ($method->scope === 'static' ? 'static ' : '')
+                ($method->scope === 'static' ? 'static ' : '')
             . 'function ' . $method->name
             . '(' . implode(',', $params) . '){'
             . '__generic_' . $method->name

src/Factory/Reflection/ReflectionMethodFactory.php

Indentation +3 added lines, -3 removed lines

@@ -97,7 +97,7 @@ 
         $this->initParams();
 
         $this->accessor->setRawValue('visibility',
-              ($this->reflector->isPublic() ? 'public'
+                ($this->reflector->isPublic() ? 'public'
             : ($this->reflector->isProtected() ? 'protected'
             : ($this->reflector->isPrivate() ? 'private'
             : (''))))
@@ -154,7 +154,7 @@ 
         {
             throw new \Exception
             (
-                  'Tried to set param annotation for non existant '
+                    'Tried to set param annotation for non existant '
                 . 'parameter: ' . $param
             );
         }
@@ -168,7 +168,7 @@ 
         {
             throw new \Exception
             (
-                  'Types are incompatible for: '
+                    'Types are incompatible for: '
                 . $this->reflector->getName() . '::' . $items[1]
             );
         }

Spacing +8 added lines, -16 removed lines

@@ -64,8 +64,7 @@ 
      */
     public static function fromName(string $class, string $method)
     {
-        return new static(new PHPNativeReflectionMethod
-        (
+        return new static(new PHPNativeReflectionMethod(
             $class, $method
         ));
     }
@@ -89,8 +88,7 @@ 
     {
         $this->typeParser = new TypeParser($parent);
         $this->accessor->setRawValue('owner', $parent);
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'name',
             $this->reflector->getName()
         );
@@ -118,8 +116,7 @@ 
         $type = $this->reflector->getReturnType();
         if ($type)
         {
-            $this->accessor->setRawValue
-            (
+            $this->accessor->setRawValue(
                 'nativeReturnType',
                 !$type->isBuiltin() ? '\\' . $type : (string)$type
             );
@@ -136,13 +133,11 @@ 
      */
     protected function initParams()
     {
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'parameters',
             new FixedList(count($this->reflector->getParameters()))
         );
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'nativeParameters',
             new FixedList(count($this->reflector->getParameters()))
         );
@@ -163,8 +158,7 @@ 
 
         if (!$this->parameters->containsKey($param))
         {
-            throw new \Exception
-            (
+            throw new \Exception(
                   'Tried to set param annotation for non existant '
                 . 'parameter: ' . $param
             );
@@ -177,8 +171,7 @@ 
 
         if (!$comparator->compatible($nativeType, $type))
         {
-            throw new \Exception
-            (
+            throw new \Exception(
                   'Types are incompatible for: '
                 . $this->reflector->getName() . '::' . $items[1]
             );
@@ -193,8 +186,7 @@ 
      *
      * @param PHPNativeReflectionParameter $reflect
      */
-    protected function addParameter
-    (
+    protected function addParameter(
         PHPNativeReflectionParameter $reflect
     )
     : void

src/Traits/GenericTrait.php

Spacing +2 added lines, -4 removed lines

@@ -65,8 +65,7 @@ 
     {
         if (!$this->genericContext)
         {
-            $this->genericContext = new GenericContext
-            (
+            $this->genericContext = new GenericContext(
                 $this->getObjectType(),
                 static::getReflectionComposite()
             );
@@ -86,8 +85,7 @@ 
     {
         if ($this->genericContext)
         {
-            throw new ImmutablePropertyException
-            (
+            throw new ImmutablePropertyException(
                 (string)$this->getObjectType(),
                 'genericContext'
             );

src/Model/Collection/Map/HashMap.php

Spacing +2 added lines, -4 removed lines

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

src/Model/Collection/ListCollection/FlexibleList.php

Spacing +1 added lines, -2 removed lines

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

src/Model/Collection/ListCollection/FixedList.php

Spacing +1 added lines, -2 removed lines

@@ -56,8 +56,7 @@
     /**
      * {@inheritDoc}
      */
-    public function splice
-    (
+    public function splice(
         int $offset,
         ?int $length = null,
         array $replacement = []