spiral / framework

src/Prototype/tests/InjectorTest.php

Spacing +10 added lines, -10 removed lines

@@ -20,7 +20,7 @@ 
 {
     public function setUp(): void
     {
-        if ((string)ini_get('zend.assertions') === 1) {
+        if ((string)ini_get('zend.assertions') === 1){
             ini_set('zend.assertions', 0);
         }
     }
@@ -33,7 +33,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])
@@ -46,7 +46,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])
@@ -66,7 +66,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),
@@ -84,7 +84,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])
@@ -109,7 +109,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])
@@ -127,7 +127,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])
@@ -143,7 +143,7 @@ 
      */
     public function testModifyConstructor(): void
     {
-        $filename = __DIR__ . '/Fixtures/WithConstructor.php';
+        $filename = __DIR__.'/Fixtures/WithConstructor.php';
         $extractor = new Extractor();
 
         $parameters = $extractor->extractFromFilename($filename);
@@ -169,7 +169,7 @@ 
      */
     public function testPriorOptionalConstructorParameters(): void
     {
-        $filename = __DIR__ . '/Fixtures/OptionalConstructorArgsClass.php';
+        $filename = __DIR__.'/Fixtures/OptionalConstructorArgsClass.php';
         $extractor = new Extractor();
 
         $parameters = $extractor->extractFromFilename($filename);
@@ -200,7 +200,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/NodeExtractor.php

Spacing +9 added lines, -9 removed lines

@@ -27,7 +27,7 @@ 
         private readonly Names $namesResolver,
         private readonly Namespaces $namespacesResolver,
         Parser $parser = null
-    ) {
+    ){
         $this->parser = $parser ?? (new ParserFactory())->create(ParserFactory::ONLY_PHP7);
     }
 
@@ -60,11 +60,11 @@ 
         $declarator = new DeclareClass();
         $this->traverse($filename, $declarator);
 
-        if (empty($declarator->getClass())) {
+        if (empty($declarator->getClass())){
             throw new ClassNotDeclaredException($filename);
         }
 
-        if ($declarator->getNamespace()) {
+        if ($declarator->getNamespace()){
             return ClassNode::createWithNamespace($declarator->getClass(), $declarator->getNamespace());
         }
 
@@ -75,7 +75,7 @@ 
     {
         $tr = new NodeTraverser();
 
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor){
             $tr->addVisitor($visitor);
         }
 
@@ -84,7 +84,7 @@ 
 
     private function fillStmts(ClassNode $definition, array $imports): void
     {
-        foreach ($imports as $import) {
+        foreach ($imports as $import){
             $definition->addImportUsage($import['name'], $import['alias']);
         }
     }
@@ -97,10 +97,10 @@ 
         $reflection = new \ReflectionClass(\sprintf('%s\%s', $definition->namespace, $definition->class));
 
         $constructor = $reflection->getConstructor();
-        if ($constructor !== null) {
+        if ($constructor !== null){
             $definition->hasConstructor = $constructor->getDeclaringClass()->getName() === $reflection->getName();
 
-            foreach ($reflection->getConstructor()->getParameters() as $parameter) {
+            foreach ($reflection->getConstructor()->getParameters() as $parameter){
                 $definition->addParam($parameter);
             }
         }
@@ -112,8 +112,8 @@ 
      */
     private function fillConstructorVars(array $vars, ClassNode $definition): void
     {
-        foreach ($vars as $k => $var) {
-            if (isset($definition->constructorParams[$var])) {
+        foreach ($vars as $k => $var){
+            if (isset($definition->constructorParams[$var])){
                 unset($vars[$k]);
             }
         }

src/Prototype/src/Injector.php

Spacing +4 added lines, -4 removed lines

@@ -30,7 +30,7 @@ 
     public function __construct(
         Lexer $lexer = null,
         private readonly PrettyPrinterAbstract $printer = new Standard()
-    ) {
+    ){
         $this->lexer = $lexer ?? new Emulative([
             'usedAttributes' => [
                 'comments',
@@ -53,8 +53,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());
@@ -68,7 +68,7 @@ 
         $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

Spacing +6 added lines, -6 removed lines

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

Braces +8 added lines, -4 removed lines

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

src/Prototype/src/NodeVisitors/UpdateConstructor.php

Spacing +17 added lines, -17 removed lines

@@ -25,18 +25,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 Class_) {
+        if (!$node instanceof 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);
         }
 
@@ -48,7 +48,7 @@ 
      */
     private function addDependencies(ClassMethod $constructor): void
     {
-        foreach ($this->definition->dependencies as $dependency) {
+        foreach ($this->definition->dependencies as $dependency){
             \array_unshift($constructor->params, $this->buildConstructorParam($dependency));
         }
     }
@@ -65,34 +65,34 @@ 
     private function addParentConstructorCall(ClassMethod $constructor): void
     {
         $parentConstructorDependencies = [];
-        foreach ($this->definition->constructorParams as $param) {
+        foreach ($this->definition->constructorParams as $param){
             $parentConstructorDependencies[] = new Arg(new 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 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 Expression(
@@ -113,8 +113,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;
             }
         }
@@ -124,13 +124,13 @@ 
 
     private function getParamType(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

@@ -30,13 +30,15 @@ 
 
     public function leaveNode(Node $node): int|Node|null
     {
-        if (!$node instanceof Class_) {
+        if (!$node instanceof 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);
         }
 
@@ -48,7 +50,8 @@ 
      */
     private function addDependencies(ClassMethod $constructor): void
     {
-        foreach ($this->definition->dependencies as $dependency) {
+        foreach ($this->definition->dependencies as $dependency)
+        {
             \array_unshift($constructor->params, $this->buildConstructorParam($dependency));
         }
     }
@@ -65,34 +68,41 @@ 
     private function addParentConstructorCall(ClassMethod $constructor): void
     {
         $parentConstructorDependencies = [];
-        foreach ($this->definition->constructorParams as $param) {
+        foreach ($this->definition->constructorParams as $param)
+        {
             $parentConstructorDependencies[] = new Arg(new 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 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 Expression(
@@ -113,8 +123,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;
             }
         }
@@ -124,13 +136,16 @@ 
 
     private function getParamType(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;
         }
 

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

Spacing +2 added lines, -2 removed lines

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

Braces +4 added lines, -2 removed lines

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

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

Spacing +2 added lines, -2 removed lines

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

Braces +4 added lines, -2 removed lines

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

src/Prototype/src/NodeVisitors/ClassNode/LocateVariables.php

Spacing +5 added lines, -5 removed lines

@@ -15,18 +15,18 @@
 {
     private array $vars = [];
 
-    public function enterNode(Node $node): int|ClassMethod|null
+    public function enterNode(Node $node): int | ClassMethod | null
     {
-        if ($node instanceof Class_) {
-            foreach ($node->stmts as $stmt) {
-                if ($stmt instanceof ClassMethod && $stmt->name === '__construct') {
+        if ($node instanceof Class_){
+            foreach ($node->stmts as $stmt){
+                if ($stmt instanceof ClassMethod && $stmt->name === '__construct'){
                     return $stmt;
                 }
             }
             return NodeTraverser::DONT_TRAVERSE_CHILDREN;
         }
 
-        if ($node instanceof Variable) {
+        if ($node instanceof Variable){
             $this->vars[] = $node->name;
         }
 

Braces +8 added lines, -4 removed lines

@@ -17,16 +17,20 @@
 
     public function enterNode(Node $node): int|ClassMethod|null
     {
-        if ($node instanceof Class_) {
-            foreach ($node->stmts as $stmt) {
-                if ($stmt instanceof ClassMethod && $stmt->name === '__construct') {
+        if ($node instanceof Class_)
+        {
+            foreach ($node->stmts as $stmt)
+            {
+                if ($stmt instanceof ClassMethod && $stmt->name === '__construct')
+                {
                     return $stmt;
                 }
             }
             return NodeTraverser::DONT_TRAVERSE_CHILDREN;
         }
 
-        if ($node instanceof Variable) {
+        if ($node instanceof Variable)
+        {
             $this->vars[] = $node->name;
         }
 

src/Prototype/src/NodeVisitors/RemoveUse.php

Spacing +5 added lines, -5 removed lines

@@ -15,19 +15,19 @@
  */
 final class RemoveUse extends NodeVisitorAbstract
 {
-    public function leaveNode(Node $node): int|Node|null
+    public function leaveNode(Node $node): int | Node | null
     {
-        if (!$node instanceof Use_) {
+        if (!$node instanceof Use_){
             return null;
         }
 
-        foreach ($node->uses as $index => $use) {
-            if ($use->name->toString() === PrototypeTrait::class) {
+        foreach ($node->uses as $index => $use){
+            if ($use->name->toString() === PrototypeTrait::class){
                 unset($node->uses[$index]);
             }
         }
 
-        if (empty($node->uses)) {
+        if (empty($node->uses)){
             return NodeTraverser::REMOVE_NODE;
         }
 

Braces +8 added lines, -4 removed lines

@@ -17,17 +17,21 @@
 {
     public function leaveNode(Node $node): int|Node|null
     {
-        if (!$node instanceof Use_) {
+        if (!$node instanceof Use_)
+        {
             return null;
         }
 
-        foreach ($node->uses as $index => $use) {
-            if ($use->name->toString() === PrototypeTrait::class) {
+        foreach ($node->uses as $index => $use)
+        {
+            if ($use->name->toString() === PrototypeTrait::class)
+            {
                 unset($node->uses[$index]);
             }
         }
 
-        if (empty($node->uses)) {
+        if (empty($node->uses))
+        {
             return NodeTraverser::REMOVE_NODE;
         }