spiral / framework

src/Distribution/src/Internal/AmazonUriFactory.php

Spacing +3 added lines, -3 removed lines

@@ -21,8 +21,8 @@ 
      * @var string
      */
     private const ERROR_NOT_AVAILABLE =
-        'The "aws/aws-sdk-php" package is supplied with the Guzzle PSR-7 ' .
-        'implementation, but it is not available. Please install the ' .
+        'The "aws/aws-sdk-php" package is supplied with the Guzzle PSR-7 '.
+        'implementation, but it is not available. Please install the '.
         '"aws/aws-sdk-php" package or use any other implementation of PSR-17 factories.'
     ;
 
@@ -41,7 +41,7 @@ 
 
     private function assertAvailable(): void
     {
-        if (\class_exists(Uri::class)) {
+        if (\class_exists(Uri::class)){
             return;
         }
 

Braces +2 added lines, -1 removed lines

@@ -41,7 +41,8 @@
 
     private function assertAvailable(): void
     {
-        if (\class_exists(Uri::class)) {
+        if (\class_exists(Uri::class))
+        {
             return;
         }
 

src/Queue/src/Driver/NullDriver.php

Spacing +1 added lines, -1 removed lines

@@ -17,6 +17,6 @@
     {
         // Do nothing
 
-        return (string) Uuid::uuid4();
+        return (string)Uuid::uuid4();
     }
 }

src/Framework/Console/CommandLocator.php

Braces +4 added lines, -2 removed lines

@@ -38,8 +38,10 @@
     public function locateCommands(): array
     {
         $commands = [];
-        foreach ($this->classes->getScopedClasses('consoleCommands', SymfonyCommand::class) as $class) {
-            if ($class->isAbstract()) {
+        foreach ($this->classes->getScopedClasses('consoleCommands', SymfonyCommand::class) as $class)
+        {
+            if ($class->isAbstract())
+            {
                 continue;
             }
 

Spacing +3 added lines, -3 removed lines

@@ -17,7 +17,7 @@ 
         private readonly ScopedClassesInterface $classes,
         ContainerInterface $container,
         array $interceptors = [],
-    ) {
+    ){
         $this->container = $container;
         $this->interceptors = $interceptors;
     }
@@ -25,8 +25,8 @@ 
     public function locateCommands(): array
     {
         $commands = [];
-        foreach ($this->classes->getScopedClasses('consoleCommands', SymfonyCommand::class) as $class) {
-            if ($class->isAbstract()) {
+        foreach ($this->classes->getScopedClasses('consoleCommands', SymfonyCommand::class) as $class){
+            if ($class->isAbstract()){
                 continue;
             }
 

src/Prototype/tests/LocatorTest.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
             'directories' => [],
             'scopes' => [
                 'prototypes' => [
-                    'directories' => [__DIR__ . '/Fixtures']
+                    'directories' => [__DIR__.'/Fixtures']
                 ]
             ]
         ])));

src/Files/src/Files.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     /**
      * Default file mode for this manager.
      */
-    public const DEFAULT_FILE_MODE = self::READONLY;
+    public const DEFAULT_FILE_MODE = self::readonly;
 
     /**
      * Files to be removed when component destructed.

Spacing +41 added lines, -41 removed lines

@@ -39,30 +39,30 @@ 
         ?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 +74,7 @@ 
 
     public function read(string $filename): string
     {
-        if (!$this->exists($filename)) {
+        if (!$this->exists($filename)){
             throw new FileNotFoundException($filename);
         }
 
@@ -96,12 +96,12 @@ 
     ): 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,12 +112,12 @@ 
                 $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) {
-            throw new WriteErrorException($e->getMessage(), (int) $e->getCode(), $e);
+        }catch (\Exception $e){
+            throw new WriteErrorException($e->getMessage(), (int)$e->getCode(), $e);
         }
 
         return $result !== false;
@@ -134,7 +134,7 @@ 
 
     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 +153,7 @@ 
      */
     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 +162,15 @@ 
             \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 +179,7 @@ 
 
     public function move(string $filename, string $destination): bool
     {
-        if (!$this->exists($filename)) {
+        if (!$this->exists($filename)){
             throw new FileNotFoundException($filename);
         }
 
@@ -188,7 +188,7 @@ 
 
     public function copy(string $filename, string $destination): bool
     {
-        if (!$this->exists($filename)) {
+        if (!$this->exists($filename)){
             throw new FileNotFoundException($filename);
         }
 
@@ -197,7 +197,7 @@ 
 
     public function touch(string $filename, ?int $mode = null): bool
     {
-        if (!\touch($filename)) {
+        if (!\touch($filename)){
             return false;
         }
 
@@ -213,7 +213,7 @@ 
     {
         $this->exists($filename) or throw new FileNotFoundException($filename);
 
-        return (int) \filesize($filename);
+        return (int)\filesize($filename);
     }
 
     public function extension(string $filename): string
@@ -223,7 +223,7 @@ 
 
     public function md5(string $filename): string
     {
-        if (!$this->exists($filename)) {
+        if (!$this->exists($filename)){
             throw new FileNotFoundException($filename);
         }
 
@@ -237,7 +237,7 @@ 
 
     public function time(string $filename): int
     {
-        if (!$this->exists($filename)) {
+        if (!$this->exists($filename)){
             throw new FileNotFoundException($filename);
         }
 
@@ -268,7 +268,7 @@ 
 
     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,14 +279,14 @@ 
     public function getFiles(string $location, ?string $pattern = null): array
     {
         $result = [];
-        foreach ($this->filesIterator($location, $pattern) as $filename) {
-            if ($this->isDirectory($filename->getPathname())) {
-                $result = \array_merge($result, $this->getFiles($filename . DIRECTORY_SEPARATOR));
+        foreach ($this->filesIterator($location, $pattern) as $filename){
+            if ($this->isDirectory($filename->getPathname())){
+                $result = \array_merge($result, $this->getFiles($filename.DIRECTORY_SEPARATOR));
 
                 continue;
             }
 
-            $result[] = $this->normalizePath((string) $filename);
+            $result[] = $this->normalizePath((string)$filename);
         }
 
         return $result;
@@ -294,14 +294,14 @@ 
 
     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 +314,7 @@ 
     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);
         }
@@ -322,7 +322,7 @@ 
         $path = \str_replace(['//', '\\'], '/', $path);
 
         //Potentially open links and ../ type directories?
-        return ($isUnc ? $leadingSlashes : '') . \rtrim($path, '/') . ($asDirectory ? '/' : '');
+        return ($isUnc ? $leadingSlashes : '').\rtrim($path, '/').($asDirectory ? '/' : '');
     }
 
     /**
@@ -337,21 +337,21 @@ 
         $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, '..');
                     break;
                 }
-                $relative[0] = './' . $relative[0];
+                $relative[0] = './'.$relative[0];
             }
         }
 
@@ -363,7 +363,7 @@ 
      */
     public function __destruct()
     {
-        foreach ($this->destructFiles as $filename) {
+        foreach ($this->destructFiles as $filename){
             $this->delete($filename);
         }
     }
@@ -371,7 +371,7 @@ 
     private function filesIterator(string $location, ?string $pattern = null): \GlobIterator
     {
         $pattern ??= '*';
-        $regexp = \rtrim($location, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR . \ltrim($pattern, DIRECTORY_SEPARATOR);
+        $regexp = \rtrim($location, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR.\ltrim($pattern, DIRECTORY_SEPARATOR);
 
         return new \GlobIterator($regexp);
     }

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/Files/src/FilesInterface.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     public const RUNTIME = 0666;
 
     //Only owner can write
-    public const READONLY = 0644;
+    public const readonly = 0644;
 
     /**
      * Few size constants for better size manipulations.

src/Http/src/Config/HttpConfig.php

Spacing +2 added lines, -2 removed lines

@@ -58,10 +58,10 @@
      */
     public function getChunkSize(): ?int
     {
-        if (\is_null($this->config['chunkSize']) || (int) $this->config['chunkSize'] < 0) {
+        if (\is_null($this->config['chunkSize']) || (int)$this->config['chunkSize'] < 0){
             return null;
         }
 
-        return (int) $this->config['chunkSize'];
+        return (int)$this->config['chunkSize'];
     }
 }

Braces +2 added lines, -1 removed lines

@@ -58,7 +58,8 @@
      */
     public function getChunkSize(): ?int
     {
-        if (\is_null($this->config['chunkSize']) || (int) $this->config['chunkSize'] < 0) {
+        if (\is_null($this->config['chunkSize']) || (int) $this->config['chunkSize'] < 0)
+        {
             return null;
         }
 

src/Session/src/Config/SessionConfig.php

Spacing +3 added lines, -3 removed lines

@@ -43,15 +43,15 @@
      */
     public function getHandler(): ?Autowire
     {
-        if (empty($this->config['handler'])) {
+        if (empty($this->config['handler'])){
             return null;
         }
 
-        if ($this->config['handler'] instanceof Autowire) {
+        if ($this->config['handler'] instanceof Autowire){
             return $this->config['handler'];
         }
 
-        if (\class_exists($this->config['handler'])) {
+        if (\class_exists($this->config['handler'])){
             return new Autowire($this->config['handler']);
         }
 

Braces +6 added lines, -3 removed lines

@@ -43,15 +43,18 @@
      */
     public function getHandler(): ?Autowire
     {
-        if (empty($this->config['handler'])) {
+        if (empty($this->config['handler']))
+        {
             return null;
         }
 
-        if ($this->config['handler'] instanceof Autowire) {
+        if ($this->config['handler'] instanceof Autowire)
+        {
             return $this->config['handler'];
         }
 
-        if (\class_exists($this->config['handler'])) {
+        if (\class_exists($this->config['handler']))
+        {
             return new Autowire($this->config['handler']);
         }
 

src/Serializer/src/Serializer/PhpSerializer.php

Spacing +8 added lines, -8 removed lines

@@ -10,33 +10,33 @@
 
 final class PhpSerializer implements SerializerInterface
 {
-    public function serialize(mixed $payload): string|\Stringable
+    public function serialize(mixed $payload): string | \Stringable
     {
         return \serialize($payload);
     }
 
-    public function unserialize(\Stringable|string $payload, object|string|null $type = null): mixed
+    public function unserialize(\Stringable | string $payload, object | string | null $type = null): mixed
     {
-        if (\is_object($type)) {
+        if (\is_object($type)){
             $type = $type::class;
         }
 
-        if (\is_string($type) && !\class_exists($type) && !\interface_exists($type)) {
+        if (\is_string($type) && !\class_exists($type) && !\interface_exists($type)){
             throw new InvalidArgumentException(\sprintf('Class or interface `%s` doesn\'t exist.', $type));
         }
 
         return $this->runUnserialize($payload, $type);
     }
 
-    private function runUnserialize(\Stringable|string $payload, ?string $type = null): mixed
+    private function runUnserialize(\Stringable | string $payload, ?string $type = null): mixed
     {
-        $result = \unserialize((string) $payload, $type ? [] : ['allowed_classes' => false]);
+        $result = \unserialize((string)$payload, $type ? [] : ['allowed_classes' => false]);
 
-        if ($result === false) {
+        if ($result === false){
             throw new UnserializeException('Failed to unserialize data.');
         }
 
-        if ($type !== null && !$result instanceof $type) {
+        if ($type !== null && !$result instanceof $type){
             throw new InvalidArgumentException(\sprintf(
                 'Data received after unserializing must be of type: `%s`, received `%s`',
                 $type,

Braces +8 added lines, -4 removed lines

@@ -17,11 +17,13 @@ 
 
     public function unserialize(\Stringable|string $payload, object|string|null $type = null): mixed
     {
-        if (\is_object($type)) {
+        if (\is_object($type))
+        {
             $type = $type::class;
         }
 
-        if (\is_string($type) && !\class_exists($type) && !\interface_exists($type)) {
+        if (\is_string($type) && !\class_exists($type) && !\interface_exists($type))
+        {
             throw new InvalidArgumentException(\sprintf('Class or interface `%s` doesn\'t exist.', $type));
         }
 
@@ -32,11 +34,13 @@ 
     {
         $result = \unserialize((string) $payload, $type ? [] : ['allowed_classes' => false]);
 
-        if ($result === false) {
+        if ($result === false)
+        {
             throw new UnserializeException('Failed to unserialize data.');
         }
 
-        if ($type !== null && !$result instanceof $type) {
+        if ($type !== null && !$result instanceof $type)
+        {
             throw new InvalidArgumentException(\sprintf(
                 'Data received after unserializing must be of type: `%s`, received `%s`',
                 $type,