spiral / framework

src/Prototype/tests/InjectorTest.php

Braces +2 added lines, -1 removed lines

@@ -24,7 +24,8 @@
 {
     public function setUp(): void
     {
-        if ((string)ini_get('zend.assertions') === 1) {
+        if ((string)ini_get('zend.assertions') === 1)
+        {
             ini_set('zend.assertions', 0);
         }
     }

Spacing +10 added lines, -10 removed lines

@@ -19,7 +19,7 @@ 
 {
     public function setUp(): void
     {
-        if ((string)ini_get('zend.assertions') === 1) {
+        if ((string)ini_get('zend.assertions') === 1){
             ini_set('zend.assertions', 0);
         }
     }
@@ -32,7 +32,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/TestClass.php';
+        $filename = __DIR__.'/Fixtures/TestClass.php';
         $printed = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition($filename, ['testClass' => TestClass::class])
@@ -45,7 +45,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/WithPromotedProperty.php';
+        $filename = __DIR__.'/Fixtures/WithPromotedProperty.php';
         $printed = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition($filename, ['two' => InjectionTwo::class])
@@ -65,7 +65,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/TestEmptyClass.php';
+        $filename = __DIR__.'/Fixtures/TestEmptyClass.php';
         $content = file_get_contents($filename);
         $printed = $i->injectDependencies(
             file_get_contents($filename),
@@ -83,7 +83,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/TestClass.php';
+        $filename = __DIR__.'/Fixtures/TestClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition($filename, ['testClass' => TestClass::class])
@@ -108,7 +108,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/ChildClass.php';
+        $filename = __DIR__.'/Fixtures/ChildClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition($filename, ['testClass' => TestClass::class])
@@ -126,7 +126,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/ChildWithConstructorClass.php';
+        $filename = __DIR__.'/Fixtures/ChildWithConstructorClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition($filename, ['testClass' => TestClass::class])
@@ -142,7 +142,7 @@ 
      */
     public function testModifyConstructor(): void
     {
-        $filename = __DIR__ . '/Fixtures/WithConstructor.php';
+        $filename = __DIR__.'/Fixtures/WithConstructor.php';
         $extractor = new Traverse\Extractor();
 
         $parameters = $extractor->extractFromFilename($filename);
@@ -168,7 +168,7 @@ 
      */
     public function testPriorOptionalConstructorParameters(): void
     {
-        $filename = __DIR__ . '/Fixtures/OptionalConstructorArgsClass.php';
+        $filename = __DIR__.'/Fixtures/OptionalConstructorArgsClass.php';
         $extractor = new Traverse\Extractor();
 
         $parameters = $extractor->extractFromFilename($filename);
@@ -199,7 +199,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/ClassNode/ConflictResolver/Fixtures/ChildClass.php';
+        $filename = __DIR__.'/ClassNode/ConflictResolver/Fixtures/ChildClass.php';
         $printed = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition(

src/Prototype/src/Injector.php

Spacing +4 added lines, -4 removed lines

@@ -28,7 +28,7 @@ 
     public function __construct(
         Lexer $lexer = null,
         private readonly PrettyPrinterAbstract $printer = new Standard()
-    ) {
+    ){
         $this->lexer = $lexer ?? new Lexer\Emulative([
             'usedAttributes' => [
                 'comments',
@@ -51,8 +51,8 @@ 
      */
     public function injectDependencies(string $code, ClassNode $node, bool $removeTrait = false): string
     {
-        if (empty($node->dependencies)) {
-            if ($removeTrait) {
+        if (empty($node->dependencies)){
+            if ($removeTrait){
                 $tr = new NodeTraverser();
                 $tr->addVisitor(new RemoveUse());
                 $tr->addVisitor(new RemoveTrait());
@@ -66,7 +66,7 @@ 
         $tr = new NodeTraverser();
         $tr->addVisitor(new AddUse($node));
 
-        if ($removeTrait) {
+        if ($removeTrait){
             $tr->addVisitor(new RemoveUse());
             $tr->addVisitor(new RemoveTrait());
         }

Braces +6 added lines, -3 removed lines

@@ -51,8 +51,10 @@ 
      */
     public function injectDependencies(string $code, ClassNode $node, bool $removeTrait = false): string
     {
-        if (empty($node->dependencies)) {
-            if ($removeTrait) {
+        if (empty($node->dependencies))
+        {
+            if ($removeTrait)
+            {
                 $tr = new NodeTraverser();
                 $tr->addVisitor(new RemoveUse());
                 $tr->addVisitor(new RemoveTrait());
@@ -66,7 +68,8 @@ 
         $tr = new NodeTraverser();
         $tr->addVisitor(new AddUse($node));
 
-        if ($removeTrait) {
+        if ($removeTrait)
+        {
             $tr->addVisitor(new RemoveUse());
             $tr->addVisitor(new RemoveTrait());
         }

src/Prototype/src/NodeVisitors/LocateProperties.php

Braces +8 added lines, -4 removed lines

@@ -43,9 +43,12 @@ 
             $this->requested[$node->name->name] = $node->name->name;
         }
 
-        if ($node instanceof Node\Stmt\Property) {
-            foreach ($node->props as $prop) {
-                if ($prop instanceof Node\Stmt\PropertyProperty) {
+        if ($node instanceof Node\Stmt\Property)
+        {
+            foreach ($node->props as $prop)
+            {
+                if ($prop instanceof Node\Stmt\PropertyProperty)
+                {
                     $this->properties[$prop->name->name] = $prop->name->name;
                 }
             }
@@ -56,7 +59,8 @@ 
 
     private function promotedProperties(Node $node): void
     {
-        if (!$node instanceof Node\Param || !$node->var instanceof Node\Expr\Variable) {
+        if (!$node instanceof Node\Param || !$node->var instanceof Node\Expr\Variable)
+        {
             return;
         }
 

Spacing +6 added lines, -6 removed lines

@@ -38,13 +38,13 @@ 
             $node->var instanceof Node\Expr\Variable &&
             $node->var->name === 'this' &&
             $node->name instanceof Node\Identifier
-        ) {
+        ){
             $this->requested[$node->name->name] = $node->name->name;
         }
 
-        if ($node instanceof Node\Stmt\Property) {
-            foreach ($node->props as $prop) {
-                if ($prop instanceof Node\Stmt\PropertyProperty) {
+        if ($node instanceof Node\Stmt\Property){
+            foreach ($node->props as $prop){
+                if ($prop instanceof Node\Stmt\PropertyProperty){
                     $this->properties[$prop->name->name] = $prop->name->name;
                 }
             }
@@ -55,7 +55,7 @@ 
 
     private function promotedProperties(Node $node): void
     {
-        if (!$node instanceof Node\Param || !$node->var instanceof Node\Expr\Variable) {
+        if (!$node instanceof Node\Param || !$node->var instanceof Node\Expr\Variable){
             return;
         }
 
@@ -63,7 +63,7 @@ 
             $node->flags & Node\Stmt\Class_::MODIFIER_PUBLIC ||
             $node->flags & Node\Stmt\Class_::MODIFIER_PROTECTED ||
             $node->flags & Node\Stmt\Class_::MODIFIER_PRIVATE
-        ) {
+        ){
             $this->properties[$node->var->name] = $node->var->name;
         }
     }

src/Prototype/tests/ClassNode/ConflictResolver/ConflictResolverTest.php

Spacing +20 added lines, -20 removed lines

@@ -21,7 +21,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/TestClass.php';
+        $filename = __DIR__.'/Fixtures/TestClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition(
@@ -34,10 +34,10 @@ 
             )
         );
 
-        $this->assertStringContainsString(Fixtures\Some::class . ';', $r);
-        $this->assertStringContainsString(Fixtures\SubFolder\Some::class . ' as Some2;', $r);
-        $this->assertStringNotContainsString(Fixtures\SubFolder\Some::class . ';', $r);
-        $this->assertStringContainsString(Fixtures\ATest3::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\Some::class.';', $r);
+        $this->assertStringContainsString(Fixtures\SubFolder\Some::class.' as Some2;', $r);
+        $this->assertStringNotContainsString(Fixtures\SubFolder\Some::class.';', $r);
+        $this->assertStringContainsString(Fixtures\ATest3::class.';', $r);
     }
 
     /**
@@ -47,7 +47,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/TestClassWithImports.php';
+        $filename = __DIR__.'/Fixtures/TestClassWithImports.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition(
@@ -60,12 +60,12 @@ 
             )
         );
 
-        $this->assertStringContainsString(Fixtures\Some::class . ' as FTest;', $r);
-        $this->assertStringNotContainsString(Fixtures\Some::class . ';', $r);
-        $this->assertStringContainsString(Fixtures\SubFolder\Some::class . ' as TestAlias;', $r);
-        $this->assertStringNotContainsString(Fixtures\SubFolder\Some::class . ';', $r);
-        $this->assertStringContainsString(Fixtures\ATest3::class . ' as ATest;', $r);
-        $this->assertStringNotContainsString(Fixtures\ATest3::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\Some::class.' as FTest;', $r);
+        $this->assertStringNotContainsString(Fixtures\Some::class.';', $r);
+        $this->assertStringContainsString(Fixtures\SubFolder\Some::class.' as TestAlias;', $r);
+        $this->assertStringNotContainsString(Fixtures\SubFolder\Some::class.';', $r);
+        $this->assertStringContainsString(Fixtures\ATest3::class.' as ATest;', $r);
+        $this->assertStringNotContainsString(Fixtures\ATest3::class.';', $r);
     }
 
     /**
@@ -75,7 +75,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/ChildClass.php';
+        $filename = __DIR__.'/Fixtures/ChildClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition(
@@ -88,18 +88,18 @@ 
             )
         );
 
-        $this->assertStringContainsString(Fixtures\Some::class . ';', $r);
-        $this->assertStringContainsString(Fixtures\SubFolder\Some::class . ' as Some2;', $r);
-        $this->assertStringNotContainsString(Fixtures\SubFolder\Some::class . ';', $r);
-        $this->assertStringContainsString(Fixtures\ATest3::class . ' as ATestAlias;', $r);
-        $this->assertStringNotContainsString(Fixtures\ATest3::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\Some::class.';', $r);
+        $this->assertStringContainsString(Fixtures\SubFolder\Some::class.' as Some2;', $r);
+        $this->assertStringNotContainsString(Fixtures\SubFolder\Some::class.';', $r);
+        $this->assertStringContainsString(Fixtures\ATest3::class.' as ATestAlias;', $r);
+        $this->assertStringNotContainsString(Fixtures\ATest3::class.';', $r);
     }
 
     public function testDuplicateProperty(): void
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/DuplicatePropertyClass.php';
+        $filename = __DIR__.'/Fixtures/DuplicatePropertyClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition(
@@ -110,7 +110,7 @@ 
             )
         );
 
-        $this->assertStringContainsString(Fixtures\Some::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\Some::class.';', $r);
         $this->assertStringContainsString('__construct(private readonly Some $test)', $r);
     }
 

src/Prototype/src/NodeVisitors/UpdateConstructor.php

Spacing +17 added lines, -17 removed lines

@@ -17,18 +17,18 @@ 
 {
     public function __construct(
         private readonly ClassNode $definition
-    ) {
+    ){
     }
 
-    public function leaveNode(Node $node): int|Node|null
+    public function leaveNode(Node $node): int | Node | null
     {
-        if (!$node instanceof Node\Stmt\Class_) {
+        if (!$node instanceof Node\Stmt\Class_){
             return null;
         }
 
         $constructor = $this->getConstructorAttribute($node);
         $this->addDependencies($constructor);
-        if (!$this->definition->hasConstructor && $this->definition->constructorParams) {
+        if (!$this->definition->hasConstructor && $this->definition->constructorParams){
             $this->addParentConstructorCall($constructor);
         }
 
@@ -40,7 +40,7 @@ 
      */
     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));
         }
     }
@@ -57,34 +57,34 @@ 
     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(
@@ -105,8 +105,8 @@ 
 
     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;
             }
         }
@@ -116,13 +116,13 @@ 
 
     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;
         }
 

Braces +30 added lines, -15 removed lines

@@ -22,13 +22,15 @@ 
 
     public function leaveNode(Node $node): int|Node|null
     {
-        if (!$node instanceof Node\Stmt\Class_) {
+        if (!$node instanceof Node\Stmt\Class_)
+        {
             return null;
         }
 
         $constructor = $this->getConstructorAttribute($node);
         $this->addDependencies($constructor);
-        if (!$this->definition->hasConstructor && $this->definition->constructorParams) {
+        if (!$this->definition->hasConstructor && $this->definition->constructorParams)
+        {
             $this->addParentConstructorCall($constructor);
         }
 
@@ -40,7 +42,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));
         }
     }
@@ -57,34 +60,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(
@@ -105,8 +115,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;
             }
         }
@@ -116,13 +128,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;
         }