spiral / framework

src/Prototype/tests/Traverse/ConstructorParamsVisitor.php

Braces +17 added lines, -7 removed lines

@@ -16,19 +16,29 @@
      */
     public function leaveNode(Node $node): void
     {
-        if ($node instanceof Node\Stmt\ClassMethod && $node->name->name === '__construct') {
-            foreach ($node->params as $param) {
-                if ($param->type instanceof Node\NullableType) {
-                    if ($param->type->type instanceof Node\Identifier) {
+        if ($node instanceof Node\Stmt\ClassMethod && $node->name->name === '__construct')
+        {
+            foreach ($node->params as $param)
+            {
+                if ($param->type instanceof Node\NullableType)
+                {
+                    if ($param->type->type instanceof Node\Identifier)
+                    {
                         $type = $param->type->type->name;
-                    } else {
+                    }
+                    else
+                    {
                         $type = implode('\\', $param->type->type->getParts());
                     }
 
                     $type = "?$type";
-                } elseif ($param->type instanceof Node\Name) {
+                }
+                elseif ($param->type instanceof Node\Name)
+                {
                     $type = implode('\\', $param->type->getParts());
-                } else {
+                }
+                else
+                {
                     $type = $param->type->name ?? null;
                 }
 

src/Prototype/src/NodeVisitors/ClassNode/LocateStatements.php

Braces +4 added lines, -2 removed lines

@@ -16,8 +16,10 @@
 
     public function enterNode(Node $node): void
     {
-        if ($node instanceof Node\Stmt\Use_) {
-            foreach ($node->uses as $use) {
+        if ($node instanceof Node\Stmt\Use_)
+        {
+            foreach ($node->uses as $use)
+            {
                 $this->imports[] = [
                     'name'  => \implode('\\', $use->name->getParts()),
                     'alias' => $use->alias->name ?? null,

src/Prototype/src/NodeVisitors/ClassNode/DeclareClass.php

Braces +4 added lines, -2 removed lines

@@ -15,11 +15,13 @@
 
     public function enterNode(Node $node): ?int
     {
-        if ($node instanceof Node\Stmt\Namespace_) {
+        if ($node instanceof Node\Stmt\Namespace_)
+        {
             $this->namespace = \implode('\\', $node->name->getParts());
         }
 
-        if ($node instanceof Node\Stmt\Class_) {
+        if ($node instanceof Node\Stmt\Class_)
+        {
             $this->class = $node->name->name;
 
             return NodeTraverser::STOP_TRAVERSAL;

src/Prototype/src/NodeVisitors/RemoveTrait.php

Braces +8 added lines, -4 removed lines

@@ -17,12 +17,15 @@ 
 {
     public function leaveNode(Node $node): int|Node|null
     {
-        if (!$node instanceof Node\Stmt\TraitUse) {
+        if (!$node instanceof Node\Stmt\TraitUse)
+        {
             return null;
         }
 
-        foreach ($node->traits as $index => $use) {
-            if ($use instanceof Node\Name) {
+        foreach ($node->traits as $index => $use)
+        {
+            if ($use instanceof Node\Name)
+            {
                 $name = $this->trimSlashes(\implode('\\', $use->getParts()));
                 if (
                     \in_array($name, [
@@ -36,7 +39,8 @@ 
         }
 
         $node->traits = \array_values($node->traits);
-        if (empty($node->traits)) {
+        if (empty($node->traits))
+        {
             return NodeTraverser::REMOVE_NODE;
         }
 

src/Prototype/src/NodeVisitors/AddUse.php

Braces +36 added lines, -18 removed lines

@@ -20,20 +20,26 @@ 
 
     public function __construct(
         private readonly ClassNode $node,
-    ) {}
+    ) {
+}
 
     public function leaveNode(Node $node): ?Node
     {
-        if (!$node instanceof Node\Stmt\Namespace_) {
+        if (!$node instanceof Node\Stmt\Namespace_)
+        {
             return null;
         }
 
         $imported = [];
-        if (!$this->node->hasConstructor && $this->node->constructorParams) {
-            foreach ($this->node->constructorParams as $param) {
-                if (!empty($param->type) && $param->type->fullName) {
+        if (!$this->node->hasConstructor && $this->node->constructorParams)
+        {
+            foreach ($this->node->constructorParams as $param)
+            {
+                if (!empty($param->type) && $param->type->fullName)
+                {
                     $import = [$param->type->fullName, $param->type->alias];
-                    if (\in_array($import, $imported, true)) {
+                    if (\in_array($import, $imported, true))
+                    {
                         continue;
                     }
 
@@ -43,9 +49,11 @@ 
             }
         }
 
-        foreach ($this->node->dependencies as $dependency) {
+        foreach ($this->node->dependencies as $dependency)
+        {
             $import = [$dependency->type->fullName, $dependency->type->alias];
-            if (\in_array($import, $imported, true)) {
+            if (\in_array($import, $imported, true))
+            {
                 continue;
             }
 
@@ -68,8 +76,10 @@ 
 
     private function definePlacementID(Node\Stmt\Namespace_ $node): int
     {
-        foreach ($node->stmts as $index => $child) {
-            if ($child instanceof Node\Stmt\Class_) {
+        foreach ($node->stmts as $index => $child)
+        {
+            if ($child instanceof Node\Stmt\Class_)
+            {
                 return $index;
             }
         }
@@ -86,13 +96,17 @@ 
     {
         $uses = $this->getExistingUseParts($stmts);
 
-        foreach ($nodes as $i => $node) {
-            if (!$node instanceof Node\Stmt\Use_) {
+        foreach ($nodes as $i => $node)
+        {
+            if (!$node instanceof Node\Stmt\Use_)
+            {
                 continue;
             }
 
-            foreach ($node->uses as $use) {
-                if (\in_array($use->name->getParts(), $uses, true)) {
+            foreach ($node->uses as $use)
+            {
+                if (\in_array($use->name->getParts(), $uses, true))
+                {
                     unset($nodes[$i]);
                 }
             }
@@ -111,12 +125,15 @@ 
     private function getExistingUseParts(array $stmts): array
     {
         $uses = [];
-        foreach ($stmts as $stmt) {
-            if (!$stmt instanceof Node\Stmt\Use_) {
+        foreach ($stmts as $stmt)
+        {
+            if (!$stmt instanceof Node\Stmt\Use_)
+            {
                 continue;
             }
 
-            foreach ($stmt->uses as $use) {
+            foreach ($stmt->uses as $use)
+            {
                 $uses[] = $use->name->getParts();
             }
         }
@@ -127,7 +144,8 @@ 
     private function buildUse(string $type, ?string $alias = null): Node\Stmt\Use_
     {
         $b = new Use_(new Node\Name($type), Node\Stmt\Use_::TYPE_NORMAL);
-        if (!empty($alias)) {
+        if (!empty($alias))
+        {
             $b->as($alias);
         }
 

src/Prototype/src/NodeVisitors/UpdateConstructor.php

Braces +36 added lines, -18 removed lines

@@ -17,17 +17,21 @@ 
 {
     public function __construct(
         private readonly ClassNode $definition,
-    ) {}
+    ) {
+}
 
     public function leaveNode(Node $node): int|Node|null
     {
-        if (!$node instanceof Node\Stmt\Class_) {
+        if (!$node instanceof Node\Stmt\Class_)
+        {
             return null;
         }
 
         $constructor = $this->getConstructor($node);
-        if ($constructor === null) {
-            if ($this->definition->dependencies === []) {
+        if ($constructor === null)
+        {
+            if ($this->definition->dependencies === [])
+            {
                 return $node;
             }
 
@@ -38,7 +42,8 @@ 
 
         $this->addDependencies($constructor);
 
-        if (!$this->definition->hasConstructor && $this->definition->constructorParams) {
+        if (!$this->definition->hasConstructor && $this->definition->constructorParams)
+        {
             $this->addParentConstructorCall($constructor);
         }
 
@@ -50,7 +55,8 @@ 
      */
     private function addDependencies(Node\Stmt\ClassMethod $constructor): void
     {
-        foreach ($this->definition->dependencies as $dependency) {
+        foreach ($this->definition->dependencies as $dependency)
+        {
             \array_unshift($constructor->params, $this->buildConstructorParam($dependency));
         }
     }
@@ -67,34 +73,41 @@ 
     private function addParentConstructorCall(Node\Stmt\ClassMethod $constructor): void
     {
         $parentConstructorDependencies = [];
-        foreach ($this->definition->constructorParams as $param) {
+        foreach ($this->definition->constructorParams as $param)
+        {
             $parentConstructorDependencies[] = new Node\Arg(new Node\Expr\Variable($param->name));
 
             $cp = new Param($param->name);
-            if (!empty($param->type)) {
+            if (!empty($param->type))
+            {
                 $type = $this->getParamType($param);
-                if ($param->nullable) {
+                if ($param->nullable)
+                {
                     $type = \sprintf('?%s', $type);
                 }
 
                 $cp->setType(new Node\Name($type));
             }
 
-            if ($param->byRef) {
+            if ($param->byRef)
+            {
                 $cp->makeByRef();
             }
 
-            if ($param->isVariadic) {
+            if ($param->isVariadic)
+            {
                 $cp->makeVariadic();
             }
 
-            if ($param->hasDefault) {
+            if ($param->hasDefault)
+            {
                 $cp->setDefault($param->default);
             }
             $constructor->params[] = $cp->getNode();
         }
 
-        if ($parentConstructorDependencies !== []) {
+        if ($parentConstructorDependencies !== [])
+        {
             \array_unshift(
                 $constructor->stmts,
                 new Node\Stmt\Expression(
@@ -115,8 +128,10 @@ 
 
     private function getPropertyType(Dependency $dependency): string
     {
-        foreach ($this->definition->getStmts() as $stmt) {
-            if ($stmt->name === $dependency->type->fullName && $stmt->alias) {
+        foreach ($this->definition->getStmts() as $stmt)
+        {
+            if ($stmt->name === $dependency->type->fullName && $stmt->alias)
+            {
                 return $stmt->alias;
             }
         }
@@ -126,13 +141,16 @@ 
 
     private function getParamType(ClassNode\ConstructorParam $param): string
     {
-        foreach ($this->definition->getStmts() as $stmt) {
-            if ($stmt->name === $param->type->fullName && $stmt->alias) {
+        foreach ($this->definition->getStmts() as $stmt)
+        {
+            if ($stmt->name === $param->type->fullName && $stmt->alias)
+            {
                 return $stmt->alias;
             }
         }
 
-        if ($param->type->alias) {
+        if ($param->type->alias)
+        {
             return $param->type->alias;
         }