spaark / composite-utils

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 +9 added lines, -18 removed lines

@@ -78,8 +78,7 @@ 
      * @param PHPNativeReflectionClass $reflect
      * @param ReflectionCompositeProviderInterface $provider
      */
-    public function __construct
-    (
+    public function __construct(
         PHPNativeReflectionClass $reflect,
         ReflectionCompositeProviderInterface $provider
     )
@@ -117,13 +116,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()]
         );
@@ -146,8 +143,7 @@ 
             $size = count($this->reflector->{'get' . $name}());
             foreach ($prefixes as $prefix)
             {
-                $this->accessor->setRawValue
-                (
+                $this->accessor->setRawValue(
                     $prefix . $name,
                     new FixedList($size)
                 );
@@ -177,8 +173,7 @@ 
      * @param bool $checkFile
      * @param string $singular
      */
-    protected function addItems
-    (
+    protected function addItems(
         string $name,
         bool $checkFile,
         string $signular
@@ -209,8 +204,7 @@ 
                     new $factory($item),
                     $item
                 );
-                $this->accessor->rawAddToValue
-                (
+                $this->accessor->rawAddToValue(
                     'local' . ucfirst($name),
                     $item
                 );
@@ -227,8 +221,7 @@ 
      * @param PHPNativeReflectionClass $reflect
      * @param string $method
      */
-    protected function addInheritance
-    (
+    protected function addInheritance(
         string $group,
         PHPNativeReflectionClass $reflect,
         string $method = 'rawAddToValue'
@@ -248,15 +241,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()]

src/Factory/Reflection/TypeParser.php

Doc Comments +6 added lines

@@ -47,6 +47,9 @@ 
         return $this->innerParse($value);
     }
 
+    /**
+     * @param string $value
+     */
     protected function innerParse($value)
     {
         $nullable = false;
@@ -120,6 +123,9 @@ 
         return $this->resolveName($current, $nullable, $collection);
     }
 
+    /**
+     * @param integer $i
+     */
     protected function checkCollectionClose($value, $i)
     {
         if ($i + 1 === strlen($value))

Spacing +12 added lines, -17 removed lines

@@ -63,8 +63,7 @@ 
                     $nullable = true;
                     break;
                 case '<':
-                    $stack->push($this->resolveGenericName
-                    (
+                    $stack->push($this->resolveGenericName(
                         $current,
                         $nullable,
                         $collection
@@ -72,8 +71,7 @@ 
                     break;
                 case ',':
                     $stack->top()->generics[] =
-                        $this->resolveName
-                        (
+                        $this->resolveName(
                             $current,
                             $nullable,
                             $collection
@@ -89,8 +87,7 @@ 
                 case '>':
                     $item = $stack->pop();
                     $item->generics[] =
-                        $this->resolveName
-                        (
+                        $this->resolveName(
                             $current,
                             $nullable,
                             $collection
@@ -132,18 +129,17 @@ 
         }
         elseif ($i + 2 !== strlen($value))
         {
-            if (!in_array($value{$i + 2}, ['>',',']))
+            if (!in_array($value{$i + 2}, ['>', ',']))
             {
                 throw new \Exception('Unexpected char after collection');
             }
         }
     }
 
-    protected function resolveGenericName
-    (
-        string &$value,
-        bool &$nullable,
-        bool &$collection
+    protected function resolveGenericName(
+        string & $value,
+        bool & $nullable,
+        bool & $collection
     )
     {
         $type = $this->resolveName($value, $nullable, $collection);
@@ -156,11 +152,10 @@ 
         return $type;
     }
 
-    protected function resolveName
-    (
-        string &$value,
-        bool &$nullable,
-        bool &$collection
+    protected function resolveName(
+        string & $value,
+        bool & $nullable,
+        bool & $collection
     )
     {
         switch ($value)

Braces +4 added lines, -8 removed lines

@@ -106,8 +106,7 @@ 
                     if ($stack->isEmpty())
                     {
                         return $item;
-                    }
-                    else
+                    } else
                     {
                         $stack->top()->types[] = $item;
                     }
@@ -125,12 +124,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}, ['>',',']))
             {
@@ -189,8 +186,7 @@ 
                     if ($useStatements->containsKey($value))
                     {
                         $value = $useStatements[$value]->classname;
-                    }
-                    else
+                    } else
                     {
                         $value = $this->context->namespace->namespace
                             . '\\' . $value;