spaark / composite-utils

src/Service/TypeComparator.php

Braces +2 added lines, -4 removed lines

@@ -41,12 +41,10 @@
         if ($parent instanceof MixedType)
         {
             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)
         {
             return
                 $child instanceof ObjectType && 

Spacing +2 added lines, -4 removed lines

@@ -31,8 +31,7 @@ 
      * @param AbstractType $child
      * @return boolean
      */
-    public function compatible
-    (
+    public function compatible(
         AbstractType $parent,
         AbstractType $child
     )
@@ -53,8 +52,7 @@ 
                 is_a($child->classname, $parent->classname, true);
         }
 
-        throw new \DomainException
-        (
+        throw new \DomainException(
             'Unknown type: ' . get_class($parent)
         );
     }

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;

src/Traits/HasReflectorTrait.php

Spacing +2 added lines, -4 removed lines

@@ -41,8 +41,7 @@ 
         if (!static::$reflectionComposite)
         {
             static::$reflectionComposite =
-                ReflectionCompositeFactory::fromClassName
-                (
+                ReflectionCompositeFactory::fromClassName(
                     get_called_class()
                 )
                 ->build();
@@ -54,8 +53,7 @@ 
     /**
      * Set the default ReflectionComposite for this class
      */
-    protected static function setDefaultReflectionComposite
-    (
+    protected static function setDefaultReflectionComposite(
         ReflectionComposite $defaultReflectionComposite
     )
     {

src/Factory/Reflection/GenericCompositeGenerator.php

Indentation +2 added lines, -2 removed lines

@@ -65,7 +65,7 @@ 
         $i = 0;
 
         $code =
-              '<?php namespace ' . $namespace . ';'
+                '<?php namespace ' . $namespace . ';'
             . 'class ' . $baseClass . ' extends ' . $originalClass
             . '{';
 
@@ -107,7 +107,7 @@ 
         }
 
         return
-              ($method->scope === 'static' ? 'static ' : '')
+                ($method->scope === 'static' ? 'static ' : '')
             . 'function ' . $method->name
             . '(' . implode(',', $params) . '){'
             . '__generic_' . $method->name

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

src/Model/Common/Equatable.php

Doc Comments +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
     /**
      * Checks if the given class is equal to this one
      *
-     * @param Type $oject
+     * @param Type $object
      * @return boolean
      */
     public function equals($object) : bool;

src/Factory/Reflection/ReflectionMethodFactory.php

Indentation +2 added lines, -2 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
             );
         }

Spacing +7 added lines, -13 removed lines

@@ -63,8 +63,7 @@ 
      */
     public static function fromName(string $class, string $method)
     {
-        return new static(new PHPNativeReflectionMethod
-        (
+        return new static(new PHPNativeReflectionMethod(
             $class, $method
         ));
     }
@@ -84,12 +83,11 @@ 
      *     this method belongs to
      * @return ReflectionMethod
      */
-    public function build(?ReflectionComposite $parent = null)
+    public function build(? ReflectionComposite $parent = null)
     {
         $this->typeParser = new TypeParser($parent);
         $this->accessor->setRawValue('owner', $parent);
-        $this->accessor->setRawValue
-        (
+        $this->accessor->setRawValue(
             'name',
             $this->reflector->getName()
         );
@@ -125,13 +123,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()))
         );
@@ -152,8 +148,7 @@ 
 
         if (!$this->parameters->containsKey($param))
         {
-            throw new \Exception
-            (
+            throw new \Exception(
                   'Tried to set param annotation for non existant '
                 . 'parameter: ' . $param
             );
@@ -178,8 +173,7 @@ 
      *
      * @param PHPNativeReflectionParameter $reflect
      */
-    protected function addParameter
-    (
+    protected function addParameter(
         PHPNativeReflectionParameter $reflect
     )
     : void

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