spaark / composite-utils

src/Service/TypeComparator.php

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
             (

Spacing +6 added lines, -12 removed lines

@@ -35,8 +35,7 @@ 
      * @param AbstractType $child
      * @return boolean
      */
-    public function compatible
-    (
+    public function compatible(
         AbstractType $parent,
         AbstractType $child
     )
@@ -47,8 +46,7 @@ 
             $parent = $this->getGenericType($parent);
         }
 
-        if
-        (
+        if (
             ($child instanceof NullType && $parent->nullable) ||
             ($parent instanceof MixedType)
         )
@@ -61,11 +59,9 @@ 
         }
         elseif ($parent instanceof ObjectType)
         {
-            if
-            (
+            if (
                 $child instanceof ObjectType && 
-                is_a
-                (
+                is_a(
                     $child->classname->__toString(),
                     $parent->classname->__toString(),
                     true
@@ -74,8 +70,7 @@ 
             {
                 foreach ($child->generics as $i => $generic)
                 {
-                    $compare = $this->compatible
-                    (
+                    $compare = $this->compatible(
                         $parent->generics[$i],
                         $generic
                     );
@@ -92,8 +87,7 @@ 
             return false;
         }
 
-        throw new \DomainException
-        (
+        throw new \DomainException(
             'Unknown type: ' . get_class($parent)
         );
     }

src/Factory/GenericCompositeGenerator.php

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;
 

Indentation +2 added lines, -2 removed lines

@@ -82,7 +82,7 @@ 
         $i = 0;
 
         $code =
-              'namespace ' . $class->namespace . ';'
+                '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

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

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/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/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/Model/Collection/ListCollection/ArrayList.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/ListCollection/FlexibleList.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/Map/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 =