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

src/Http/src/Request/InputManager.php

Braces +38 added lines, -18 removed lines

@@ -134,10 +134,13 @@ 
     {
         $input = clone $this;
 
-        if ($add) {
+        if ($add)
+        {
             $input->prefix .= '.' . $prefix;
             $input->prefix = \trim($input->prefix, '.');
-        } else {
+        }
+        else
+        {
             $input->prefix = $prefix;
         }
 
@@ -173,9 +176,12 @@ 
      */
     public function request(): Request
     {
-        try {
+        try
+        {
             $request = $this->container->get(Request::class);
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             $scope = \implode('.', \array_reverse(Introspector::scopeNames($this->container)));
             throw new ScopeException(
                 "Unable to get `ServerRequestInterface` in the `$scope` container scope",
@@ -185,7 +191,8 @@ 
         }
 
         // Flushing input state
-        if ($this->request !== $request) {
+        if ($this->request !== $request)
+        {
             $this->bags = [];
             $this->request = $request;
         }
@@ -202,7 +209,8 @@ 
 
         $position = \strrpos($header, 'Bearer ');
 
-        if ($position !== false) {
+        if ($position !== false)
+        {
             $header = \substr($header, $position + 7);
 
             /** @psalm-suppress FalsableReturnStatement */
@@ -257,16 +265,21 @@ 
     public function isJsonExpected(bool $softMatch = false): bool
     {
         $acceptHeader = AcceptHeader::fromString($this->request()->getHeaderLine('Accept'));
-        foreach ($this->jsonTypes as $jsonType) {
-            if ($acceptHeader->has($jsonType)) {
+        foreach ($this->jsonTypes as $jsonType)
+        {
+            if ($acceptHeader->has($jsonType))
+            {
                 return true;
             }
         }
 
-        if ($softMatch) {
-            foreach ($acceptHeader->getAll() as $item) {
+        if ($softMatch)
+        {
+            foreach ($acceptHeader->getAll() as $item)
+            {
                 $itemValue = \strtolower((string) $item->getValue());
-                if (\str_ends_with($itemValue, '/json') || \str_ends_with($itemValue, '+json')) {
+                if (\str_ends_with($itemValue, '/json') || \str_ends_with($itemValue, '+json'))
+                {
                     return true;
                 }
             }
@@ -305,19 +318,22 @@ 
         // ensure proper request association
         $this->request();
 
-        if (isset($this->bags[$name])) {
+        if (isset($this->bags[$name]))
+        {
             return $this->bags[$name];
         }
 
         $definition = $this->findBagDefinition($name);
-        if (!$definition) {
+        if (!$definition)
+        {
             throw new InputException(\sprintf("Undefined input bag '%s'", $name));
         }
 
         $class = $definition['class'];
         $data = \call_user_func([$this->request(), $definition['source']]);
 
-        if (!\is_array($data)) {
+        if (!\is_array($data))
+        {
             $data = (array) $data;
         }
 
@@ -326,7 +342,8 @@ 
 
     public function hasBag(string $name): bool
     {
-        if (isset($this->bags[$name])) {
+        if (isset($this->bags[$name]))
+        {
             return true;
         }
 
@@ -364,12 +381,15 @@ 
      */
     private function findBagDefinition(string $name): ?array
     {
-        if (isset($this->bagAssociations[$name])) {
+        if (isset($this->bagAssociations[$name]))
+        {
             return $this->bagAssociations[$name];
         }
 
-        foreach ($this->bagAssociations as $bag) {
-            if (isset($bag['alias']) && $bag['alias'] === $name) {
+        foreach ($this->bagAssociations as $bag)
+        {
+            if (isset($bag['alias']) && $bag['alias'] === $name)
+            {
                 return $bag;
             }
         }

src/Files/src/Files.php

Braces +71 added lines, -34 removed lines

@@ -39,30 +39,36 @@ 
         ?int $mode = null,
         bool $recursivePermissions = true,
     ): bool {
-        if (empty($mode)) {
+        if (empty($mode))
+        {
             $mode = self::DEFAULT_FILE_MODE;
         }
 
         //Directories always executable
         $mode |= 0o111;
-        if (\is_dir($directory)) {
+        if (\is_dir($directory))
+        {
             //Exists :(
             return $this->setPermissions($directory, $mode);
         }
 
-        if (!$recursivePermissions) {
+        if (!$recursivePermissions)
+        {
             return \mkdir($directory, $mode, true);
         }
 
         $directoryChain = [\basename($directory)];
 
         $baseDirectory = $directory;
-        while (!\is_dir($baseDirectory = \dirname($baseDirectory))) {
+        while (!\is_dir($baseDirectory = \dirname($baseDirectory)))
+        {
             $directoryChain[] = \basename($baseDirectory);
         }
 
-        foreach (\array_reverse($directoryChain) as $dir) {
-            if (!\mkdir($baseDirectory = \sprintf('%s/%s', $baseDirectory, $dir))) {
+        foreach (\array_reverse($directoryChain) as $dir)
+        {
+            if (!\mkdir($baseDirectory = \sprintf('%s/%s', $baseDirectory, $dir)))
+            {
                 return false;
             }
 
@@ -74,7 +80,8 @@ 
 
     public function read(string $filename): string
     {
-        if (!$this->exists($filename)) {
+        if (!$this->exists($filename))
+        {
             throw new FileNotFoundException($filename);
         }
 
@@ -96,12 +103,15 @@ 
     ): bool {
         $mode ??= self::DEFAULT_FILE_MODE;
 
-        try {
-            if ($ensureDirectory) {
+        try
+        {
+            if ($ensureDirectory)
+            {
                 $this->ensureDirectory(\dirname($filename), $mode);
             }
 
-            if ($this->exists($filename)) {
+            if ($this->exists($filename))
+            {
                 //Forcing mode for existed file
                 $this->setPermissions($filename, $mode);
             }
@@ -112,11 +122,14 @@ 
                 $append ? FILE_APPEND | LOCK_EX : LOCK_EX,
             );
 
-            if ($result !== false) {
+            if ($result !== false)
+            {
                 //Forcing mode after file creation
                 $this->setPermissions($filename, $mode);
             }
-        } catch (\Exception $e) {
+        }
+        catch (\Exception $e)
+        {
             throw new WriteErrorException($e->getMessage(), (int) $e->getCode(), $e);
         }
 
@@ -134,7 +147,8 @@ 
 
     public function delete(string $filename): bool
     {
-        if ($this->exists($filename)) {
+        if ($this->exists($filename))
+        {
             $result = \unlink($filename);
 
             //Wiping out changes in local file cache
@@ -153,7 +167,8 @@ 
      */
     public function deleteDirectory(string $directory, bool $contentOnly = false): bool
     {
-        if (!$this->isDirectory($directory)) {
+        if (!$this->isDirectory($directory))
+        {
             throw new FilesException(\sprintf('Undefined or invalid directory %s', $directory));
         }
 
@@ -162,15 +177,20 @@ 
             \RecursiveIteratorIterator::CHILD_FIRST,
         );
 
-        foreach ($files as $file) {
-            if ($file->isDir()) {
+        foreach ($files as $file)
+        {
+            if ($file->isDir())
+            {
                 \rmdir($file->getRealPath());
-            } else {
+            }
+            else
+            {
                 $this->delete($file->getRealPath());
             }
         }
 
-        if (!$contentOnly) {
+        if (!$contentOnly)
+        {
             return \rmdir($directory);
         }
 
@@ -179,7 +199,8 @@ 
 
     public function move(string $filename, string $destination): bool
     {
-        if (!$this->exists($filename)) {
+        if (!$this->exists($filename))
+        {
             throw new FileNotFoundException($filename);
         }
 
@@ -188,7 +209,8 @@ 
 
     public function copy(string $filename, string $destination): bool
     {
-        if (!$this->exists($filename)) {
+        if (!$this->exists($filename))
+        {
             throw new FileNotFoundException($filename);
         }
 
@@ -197,7 +219,8 @@ 
 
     public function touch(string $filename, ?int $mode = null): bool
     {
-        if (!\touch($filename)) {
+        if (!\touch($filename))
+        {
             return false;
         }
 
@@ -223,7 +246,8 @@ 
 
     public function md5(string $filename): string
     {
-        if (!$this->exists($filename)) {
+        if (!$this->exists($filename))
+        {
             throw new FileNotFoundException($filename);
         }
 
@@ -237,7 +261,8 @@ 
 
     public function time(string $filename): int
     {
-        if (!$this->exists($filename)) {
+        if (!$this->exists($filename))
+        {
             throw new FileNotFoundException($filename);
         }
 
@@ -268,7 +293,8 @@ 
 
     public function setPermissions(string $filename, int $mode): bool
     {
-        if (\is_dir($filename)) {
+        if (\is_dir($filename))
+        {
             //Directories must always be executable (i.e. 664 for dir => 775)
             $mode |= 0111;
         }
@@ -279,8 +305,10 @@ 
     public function getFiles(string $location, ?string $pattern = null): array
     {
         $result = [];
-        foreach ($this->filesIterator($location, $pattern) as $filename) {
-            if ($this->isDirectory($filename->getPathname())) {
+        foreach ($this->filesIterator($location, $pattern) as $filename)
+        {
+            if ($this->isDirectory($filename->getPathname()))
+            {
                 $result = \array_merge($result, $this->getFiles($filename . DIRECTORY_SEPARATOR));
 
                 continue;
@@ -294,14 +322,16 @@ 
 
     public function tempFilename(string $extension = '', ?string $location = null): string
     {
-        if (empty($location)) {
+        if (empty($location))
+        {
             $location = \sys_get_temp_dir();
         }
 
         $filename = \tempnam($location, 'spiral');
         $filename === false and throw new FilesException('Unable to create temporary file.');
 
-        if (!empty($extension)) {
+        if (!empty($extension))
+        {
             $old = $filename;
             $filename = \sprintf('%s.%s', $filename, $extension);
             \rename($old, $filename);
@@ -314,7 +344,8 @@ 
     public function normalizePath(string $path, bool $asDirectory = false): string
     {
         $isUnc = \str_starts_with($path, '\\\\') || \str_starts_with($path, '//');
-        if ($isUnc) {
+        if ($isUnc)
+        {
             $leadingSlashes = \substr($path, 0, 2);
             $path = \substr($path, 2);
         }
@@ -337,15 +368,20 @@ 
         $path = \explode('/', $path);
         $relative = $path;
 
-        foreach ($from as $depth => $dir) {
+        foreach ($from as $depth => $dir)
+        {
             //Find first non-matching dir
-            if ($dir === $path[$depth]) {
+            if ($dir === $path[$depth])
+            {
                 //Ignore this directory
                 \array_shift($relative);
-            } else {
+            }
+            else
+            {
                 //Get number of remaining dirs to $from
                 $remaining = \count($from) - $depth;
-                if ($remaining > 1) {
+                if ($remaining > 1)
+                {
                     //Add traversals up to first matching directory
                     $padLength = (\count($relative) + $remaining - 1) * -1;
                     $relative = \array_pad($relative, $padLength, '..');
@@ -363,7 +399,8 @@ 
      */
     public function __destruct()
     {
-        foreach ($this->destructFiles as $filename) {
+        foreach ($this->destructFiles as $filename)
+        {
             $this->delete($filename);
         }
     }

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

Braces +8 added lines, -4 removed lines

@@ -15,9 +15,12 @@ 
 
     public function enterNode(Node $node): int|Node\Stmt\ClassMethod|null
     {
-        if ($node instanceof Node\Stmt\Class_) {
-            foreach ($node->stmts as $stmt) {
-                if ($stmt instanceof Node\Stmt\ClassMethod && $stmt->name->toLowerString() === '__construct') {
+        if ($node instanceof Node\Stmt\Class_)
+        {
+            foreach ($node->stmts as $stmt)
+            {
+                if ($stmt instanceof Node\Stmt\ClassMethod && $stmt->name->toLowerString() === '__construct')
+                {
                     return $stmt;
                 }
             }
@@ -25,7 +28,8 @@ 
             return NodeVisitor::DONT_TRAVERSE_CHILDREN;
         }
 
-        if ($node instanceof Node\Expr\Variable) {
+        if ($node instanceof Node\Expr\Variable)
+        {
             $this->vars[] = $node->name;
         }