spiral / framework

src/Storage/tests/TestCase.php

Braces +11 added lines, -5 removed lines

@@ -63,20 +63,26 @@
         );
 
         /** @var \SplFileInfo $file */
-        foreach($iterator as $file) {
-            if ($file->getFilename() === '.gitignore') {
+        foreach($iterator as $file)
+        {
+            if ($file->getFilename() === '.gitignore')
+            {
                 continue;
             }
 
             \error_clear_last();
 
-            if ($file->isDir()) {
+            if ($file->isDir())
+            {
                 @\rmdir($file->getPathname());
-            } else {
+            }
+            else
+            {
                 @\unlink($file->getPathname());
             }
 
-            if ($error = \error_get_last()) {
+            if ($error = \error_get_last())
+            {
                 $prefix = 'An error occurred while clear temporary local storage directory: ';
                 $this->addWarning($prefix . $error['message']);
             }

Spacing +8 added lines, -8 removed lines

@@ -14,7 +14,7 @@ 
     /**
      * @var string
      */
-    protected const LOCAL_STORAGE_DIRECTORY = __DIR__ . '/storage';
+    protected const LOCAL_STORAGE_DIRECTORY = __DIR__.'/storage';
 
     /**
      * @var BucketInterface
@@ -38,7 +38,7 @@ 
         );
 
         $this->second = Bucket::fromAdapter(
-            new LocalFilesystemAdapter(self::LOCAL_STORAGE_DIRECTORY . '/second')
+            new LocalFilesystemAdapter(self::LOCAL_STORAGE_DIRECTORY.'/second')
         );
     }
 
@@ -63,22 +63,22 @@ 
         );
 
         /** @var \SplFileInfo $file */
-        foreach($iterator as $file) {
-            if ($file->getFilename() === '.gitignore') {
+        foreach ($iterator as $file){
+            if ($file->getFilename() === '.gitignore'){
                 continue;
             }
 
             \error_clear_last();
 
-            if ($file->isDir()) {
+            if ($file->isDir()){
                 @\rmdir($file->getPathname());
-            } else {
+            }else{
                 @\unlink($file->getPathname());
             }
 
-            if ($error = \error_get_last()) {
+            if ($error = \error_get_last()){
                 $prefix = 'An error occurred while clear temporary local storage directory: ';
-                $this->addWarning($prefix . $error['message']);
+                $this->addWarning($prefix.$error['message']);
             }
         }
     }

src/Storage/tests/StorageTestCase.php

Braces +2 added lines, -1 removed lines

@@ -151,7 +151,8 @@
 
         $actual = '';
         $stream = $this->local->getStream('file.txt');
-        while (!\feof($stream)) {
+        while (!\feof($stream))
+        {
             $actual .= \fread($stream, 256);
         }
         \fclose($stream);

Spacing +4 added lines, -4 removed lines

@@ -57,8 +57,8 @@ 
     public function testVisibility(): void
     {
         $this->markTestSkipped(
-            'This test [' . __FUNCTION__ . '] returns incorrect visibility ' .
-                'of files on Windows OS. ' .
+            'This test ['.__FUNCTION__.'] returns incorrect visibility '.
+                'of files on Windows OS. '.
             'It is required to understand the situation'
         );
 
@@ -151,7 +151,7 @@ 
 
         $actual = '';
         $stream = $this->local->getStream('file.txt');
-        while (!\feof($stream)) {
+        while (!\feof($stream)){
             $actual .= \fread($stream, 256);
         }
         \fclose($stream);
@@ -175,7 +175,7 @@ 
      */
     public function testLastModified(): void
     {
-        $now = (int) \floor(\microtime(true));
+        $now = (int)\floor(\microtime(true));
 
         $this->local->create('file.txt');
         $before = $this->local->getLastModified('file.txt');

src/Storage/tests/ManagerTestCase.php

Spacing +4 added lines, -4 removed lines

@@ -92,7 +92,7 @@ 
     {
         $this->expectException(InvalidArgumentException::class);
         $this->expectExceptionMessage(
-            'URI argument must be a valid URI in ' .
+            'URI argument must be a valid URI in '.
             '"[STORAGE]://[PATH_TO_FILE]" format, but `test://` given'
         );
 
@@ -157,8 +157,8 @@ 
     public function testVisibility(): void
     {
         $this->markTestSkipped(
-            'This test [' . __FUNCTION__ . '] returns incorrect visibility ' .
-                'of files on Windows OS. ' .
+            'This test ['.__FUNCTION__.'] returns incorrect visibility '.
+                'of files on Windows OS. '.
             'It is required to understand the situation'
         );
 
@@ -257,7 +257,7 @@ 
 
         $actual = '';
         $stream = $this->manager->getStream('file.txt');
-        while (!\feof($stream)) {
+        while (!\feof($stream)){
             $actual .= \fread($stream, 256);
         }
         \fclose($stream);

Braces +2 added lines, -1 removed lines

@@ -257,7 +257,8 @@
 
         $actual = '';
         $stream = $this->manager->getStream('file.txt');
-        while (!\feof($stream)) {
+        while (!\feof($stream))
+        {
             $actual .= \fread($stream, 256);
         }
         \fclose($stream);

src/Storage/tests/FileTestCase.php

Braces +2 added lines, -1 removed lines

@@ -190,7 +190,8 @@
 
         $actual = '';
         $stream = $source->getStream();
-        while (!\feof($stream)) {
+        while (!\feof($stream))
+        {
             $actual .= \fread($stream, 256);
         }
         \fclose($stream);

Spacing +4 added lines, -4 removed lines

@@ -81,8 +81,8 @@ 
     public function testVisibility(): void
     {
         $this->markTestSkipped(
-            'This test [' . __FUNCTION__ . '] returns incorrect visibility ' .
-                'of files on Windows OS. ' .
+            'This test ['.__FUNCTION__.'] returns incorrect visibility '.
+                'of files on Windows OS. '.
             'It is required to understand the situation'
         );
 
@@ -190,7 +190,7 @@ 
 
         $actual = '';
         $stream = $source->getStream();
-        while (!\feof($stream)) {
+        while (!\feof($stream)){
             $actual .= \fread($stream, 256);
         }
         \fclose($stream);
@@ -216,7 +216,7 @@ 
      */
     public function testLastModified(): void
     {
-        $now = (int) \floor(\microtime(true));
+        $now = (int)\floor(\microtime(true));
 
         $file = $this->local->file('file.txt')
             ->create()

src/Prototype/src/ClassNode/ConstructorParam.php

Spacing +2 added lines, -2 removed lines

@@ -56,13 +56,13 @@
         $stmt->name = $parameter->getName();
 
         $type = $parameter->getType();
-        if ($type instanceof \ReflectionNamedType) {
+        if ($type instanceof \ReflectionNamedType){
             $stmt->type = Type::create($type->getName());
             $stmt->builtIn = $type->isBuiltin();
             $stmt->nullable = $type->allowsNull();
         }
 
-        if ($parameter->isDefaultValueAvailable()) {
+        if ($parameter->isDefaultValueAvailable()){
             $stmt->hasDefault = true;
             $stmt->default = $parameter->getDefaultValue();
         }

Braces +4 added lines, -2 removed lines

@@ -56,13 +56,15 @@
         $stmt->name = $parameter->getName();
 
         $type = $parameter->getType();
-        if ($type instanceof \ReflectionNamedType) {
+        if ($type instanceof \ReflectionNamedType)
+        {
             $stmt->type = Type::create($type->getName());
             $stmt->builtIn = $type->isBuiltin();
             $stmt->nullable = $type->allowsNull();
         }
 
-        if ($parameter->isDefaultValueAvailable()) {
+        if ($parameter->isDefaultValueAvailable())
+        {
             $stmt->hasDefault = true;
             $stmt->default = $parameter->getDefaultValue();
         }

src/Attributes/src/polyfill.php

Braces +6 added lines, -3 removed lines

@@ -13,7 +13,8 @@ 
 
     use Doctrine\Common\Annotations\Annotation\NamedArgumentConstructor;
 
-    if (!\interface_exists(NamedArgumentConstructorAnnotation::class)) {
+    if (!\interface_exists(NamedArgumentConstructorAnnotation::class))
+    {
         /**
          * Marker interface for PHP7/PHP8 compatible support for named
          * arguments (and constructor property promotion).
@@ -31,7 +32,8 @@ 
 
 namespace Doctrine\Common\Annotations\Annotation {
 
-    if (!\class_exists(NamedArgumentConstructor::class, false)) {
+    if (!\class_exists(NamedArgumentConstructor::class, false))
+    {
         /**
          * Annotation that indicates that the annotated class should be
          * constructed with a named argument call.
@@ -51,7 +53,8 @@ 
 
     use JetBrains\PhpStorm\ExpectedValues;
 
-    if (!\class_exists(Attribute::class, false)) {
+    if (!\class_exists(Attribute::class, false))
+    {
         #[Attribute(Attribute::TARGET_CLASS)]
         final class Attribute
         {

Spacing +3 added lines, -3 removed lines

@@ -13,7 +13,7 @@ 
 
     use Doctrine\Common\Annotations\Annotation\NamedArgumentConstructor;
 
-    if (!\interface_exists(NamedArgumentConstructorAnnotation::class)) {
+    if (!\interface_exists(NamedArgumentConstructorAnnotation::class)){
         /**
          * Marker interface for PHP7/PHP8 compatible support for named
          * arguments (and constructor property promotion).
@@ -31,7 +31,7 @@ 
 
 namespace Doctrine\Common\Annotations\Annotation {
 
-    if (!\class_exists(NamedArgumentConstructor::class, false)) {
+    if (!\class_exists(NamedArgumentConstructor::class, false)){
         /**
          * Annotation that indicates that the annotated class should be
          * constructed with a named argument call.
@@ -51,7 +51,7 @@ 
 
     use JetBrains\PhpStorm\ExpectedValues;
 
-    if (!\class_exists(Attribute::class, false)) {
+    if (!\class_exists(Attribute::class, false)){
         #[Attribute(Attribute::TARGET_CLASS)]
         final class Attribute
         {

src/Storage/src/File.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
     {
         $name = $this->storage->getName();
 
-        if ($name === null) {
+        if ($name === null){
             return $this->getPathname();
         }
 

Braces +2 added lines, -1 removed lines

@@ -64,7 +64,8 @@
     {
         $name = $this->storage->getName();
 
-        if ($name === null) {
+        if ($name === null)
+        {
             return $this->getPathname();
         }
 

src/Storage/src/Bucket/ReadableTrait.php

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

@@ -136,7 +136,7 @@
         #[ExpectedValues(valuesFromClass: \League\Flysystem\Visibility::class)]
         string $visibility
     ): string {
-        return $visibility === \League\Flysystem\Visibility::PUBLIC
+        return $visibility === \League\Flysystem\Visibility::public
             ? Visibility::VISIBILITY_PUBLIC
             : Visibility::VISIBILITY_PRIVATE;
     }

Spacing +14 added lines, -14 removed lines

@@ -29,9 +29,9 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try{
             return $fs->fileExists($pathname);
-        } catch (FilesystemException $e) {
+        }catch (FilesystemException $e){
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -43,9 +43,9 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try{
             return $fs->read($pathname);
-        } catch (FilesystemException $e) {
+        }catch (FilesystemException $e){
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -57,9 +57,9 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try{
             return $fs->readStream($pathname);
-        } catch (FilesystemException $e) {
+        }catch (FilesystemException $e){
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -71,9 +71,9 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try{
             return $fs->lastModified($pathname);
-        } catch (FilesystemException $e) {
+        }catch (FilesystemException $e){
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -85,9 +85,9 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try{
             return $fs->fileSize($pathname);
-        } catch (FilesystemException $e) {
+        }catch (FilesystemException $e){
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -99,9 +99,9 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try{
             return $fs->mimeType($pathname);
-        } catch (FilesystemException $e) {
+        }catch (FilesystemException $e){
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -114,11 +114,11 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try{
             return $this->fromFlysystemVisibility(
                 $fs->visibility($pathname)
             );
-        } catch (FilesystemException $e) {
+        }catch (FilesystemException $e){
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }

Braces +35 added lines, -14 removed lines

@@ -29,9 +29,12 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try
+        {
             return $fs->fileExists($pathname);
-        } catch (FilesystemException $e) {
+        }
+        catch (FilesystemException $e)
+        {
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -43,9 +46,12 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try
+        {
             return $fs->read($pathname);
-        } catch (FilesystemException $e) {
+        }
+        catch (FilesystemException $e)
+        {
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -57,9 +63,12 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try
+        {
             return $fs->readStream($pathname);
-        } catch (FilesystemException $e) {
+        }
+        catch (FilesystemException $e)
+        {
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -71,9 +80,12 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try
+        {
             return $fs->lastModified($pathname);
-        } catch (FilesystemException $e) {
+        }
+        catch (FilesystemException $e)
+        {
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -85,9 +97,12 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try
+        {
             return $fs->fileSize($pathname);
-        } catch (FilesystemException $e) {
+        }
+        catch (FilesystemException $e)
+        {
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -99,9 +114,12 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try
+        {
             return $fs->mimeType($pathname);
-        } catch (FilesystemException $e) {
+        }
+        catch (FilesystemException $e)
+        {
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -114,11 +132,14 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try
+        {
             return $this->fromFlysystemVisibility(
                 $fs->visibility($pathname)
             );
-        } catch (FilesystemException $e) {
+        }
+        catch (FilesystemException $e)
+        {
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }

src/Storage/src/Bucket/WritableTrait.php

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

@@ -166,8 +166,8 @@
         string $visibility
     ): string {
         return ($visibility === Visibility::VISIBILITY_PUBLIC)
-            ? \League\Flysystem\Visibility::PUBLIC
-            : \League\Flysystem\Visibility::PRIVATE;
+            ? \League\Flysystem\Visibility::public
+            : \League\Flysystem\Visibility::private;
     }
 
     /**

Spacing +23 added lines, -23 removed lines

@@ -29,7 +29,7 @@ 
      */
     public function create(string $pathname, array $config = []): FileInterface
     {
-        if ($this instanceof ReadableInterface && !$this->exists($pathname)) {
+        if ($this instanceof ReadableInterface && !$this->exists($pathname)){
             return $this->write($pathname, '', $config);
         }
 
@@ -45,8 +45,8 @@ 
 
         $fs = $this->getOperator();
 
-        try {
-            switch (true) {
+        try{
+            switch (true){
                 case \is_object($content):
                 case \is_string($content):
                     $fs->write($pathname, (string)$content, $config);
@@ -60,7 +60,7 @@ 
                     $message = 'Content must be a resource stream or stringable type, but %s passed';
                     throw new \InvalidArgumentException(\sprintf($message, \get_debug_type($content)));
             }
-        } catch (FilesystemException $e) {
+        }catch (FilesystemException $e){
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
 
@@ -77,9 +77,9 @@ 
     ): FileInterface {
         $fs = $this->getOperator();
 
-        try {
+        try{
             $fs->setVisibility($pathname, $this->toFlysystemVisibility($visibility));
-        } catch (FilesystemException $e) {
+        }catch (FilesystemException $e){
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
 
@@ -97,10 +97,10 @@ 
     ): FileInterface {
         $fs = $this->getOperator();
 
-        if ($storage === null || $storage === $this) {
-            try {
+        if ($storage === null || $storage === $this){
+            try{
                 $fs->copy($source, $destination, $config);
-            } catch (FilesystemException $e) {
+            }catch (FilesystemException $e){
                 throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
             }
 
@@ -121,10 +121,10 @@ 
     ): FileInterface {
         $fs = $this->getOperator();
 
-        if ($storage === null || $storage === $this) {
-            try {
+        if ($storage === null || $storage === $this){
+            try{
                 $fs->move($source, $destination, $config);
-            } catch (FilesystemException $e) {
+            }catch (FilesystemException $e){
                 throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
             }
 
@@ -145,13 +145,13 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try{
             $fs->delete($pathname);
 
-            if ($clean) {
+            if ($clean){
                 $this->deleteEmptyDirectories($this->getParentDirectory($pathname));
             }
-        } catch (FilesystemException $e) {
+        }catch (FilesystemException $e){
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -207,19 +207,19 @@ 
      */
     private function deleteEmptyDirectories(string $directory): void
     {
-        if (!$this->hasParentDirectory($directory)) {
+        if (!$this->hasParentDirectory($directory)){
             return;
         }
 
         $fs = $this->getOperator();
 
-        try {
-            if (!$this->hasFiles($directory)) {
+        try{
+            if (!$this->hasFiles($directory)){
                 $fs->deleteDirectory($directory);
 
                 $this->deleteEmptyDirectories($this->getParentDirectory($directory));
             }
-        } catch (FilesystemException $e) {
+        }catch (FilesystemException $e){
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -241,7 +241,7 @@ 
     {
         $fs = $this->getOperator();
 
-        foreach ($fs->listContents($directory) as $_) {
+        foreach ($fs->listContents($directory) as $_){
             return true;
         }
 
@@ -256,15 +256,15 @@ 
      */
     private function isStringable($value): bool
     {
-        if (\is_string($value)) {
+        if (\is_string($value)){
             return true;
         }
 
-        if (!\is_object($value)) {
+        if (!\is_object($value)){
             return false;
         }
 
-        if (\PHP_VERSION_ID >= 80000) {
+        if (\PHP_VERSION_ID >= 80000){
             return $value instanceof \Stringable;
         }
 

Braces +52 added lines, -23 removed lines

@@ -29,7 +29,8 @@ 
      */
     public function create(string $pathname, array $config = []): FileInterface
     {
-        if ($this instanceof ReadableInterface && !$this->exists($pathname)) {
+        if ($this instanceof ReadableInterface && !$this->exists($pathname))
+        {
             return $this->write($pathname, '', $config);
         }
 
@@ -45,8 +46,10 @@ 
 
         $fs = $this->getOperator();
 
-        try {
-            switch (true) {
+        try
+        {
+            switch (true)
+            {
                 case \is_object($content):
                 case \is_string($content):
                     $fs->write($pathname, (string)$content, $config);
@@ -60,7 +63,9 @@ 
                     $message = 'Content must be a resource stream or stringable type, but %s passed';
                     throw new \InvalidArgumentException(\sprintf($message, \get_debug_type($content)));
             }
-        } catch (FilesystemException $e) {
+        }
+        catch (FilesystemException $e)
+        {
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
 
@@ -77,9 +82,12 @@ 
     ): FileInterface {
         $fs = $this->getOperator();
 
-        try {
+        try
+        {
             $fs->setVisibility($pathname, $this->toFlysystemVisibility($visibility));
-        } catch (FilesystemException $e) {
+        }
+        catch (FilesystemException $e)
+        {
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
 
@@ -97,10 +105,14 @@ 
     ): FileInterface {
         $fs = $this->getOperator();
 
-        if ($storage === null || $storage === $this) {
-            try {
+        if ($storage === null || $storage === $this)
+        {
+            try
+            {
                 $fs->copy($source, $destination, $config);
-            } catch (FilesystemException $e) {
+            }
+            catch (FilesystemException $e)
+            {
                 throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
             }
 
@@ -121,10 +133,14 @@ 
     ): FileInterface {
         $fs = $this->getOperator();
 
-        if ($storage === null || $storage === $this) {
-            try {
+        if ($storage === null || $storage === $this)
+        {
+            try
+            {
                 $fs->move($source, $destination, $config);
-            } catch (FilesystemException $e) {
+            }
+            catch (FilesystemException $e)
+            {
                 throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
             }
 
@@ -145,13 +161,17 @@ 
     {
         $fs = $this->getOperator();
 
-        try {
+        try
+        {
             $fs->delete($pathname);
 
-            if ($clean) {
+            if ($clean)
+            {
                 $this->deleteEmptyDirectories($this->getParentDirectory($pathname));
             }
-        } catch (FilesystemException $e) {
+        }
+        catch (FilesystemException $e)
+        {
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -207,19 +227,24 @@ 
      */
     private function deleteEmptyDirectories(string $directory): void
     {
-        if (!$this->hasParentDirectory($directory)) {
+        if (!$this->hasParentDirectory($directory))
+        {
             return;
         }
 
         $fs = $this->getOperator();
 
-        try {
-            if (!$this->hasFiles($directory)) {
+        try
+        {
+            if (!$this->hasFiles($directory))
+            {
                 $fs->deleteDirectory($directory);
 
                 $this->deleteEmptyDirectories($this->getParentDirectory($directory));
             }
-        } catch (FilesystemException $e) {
+        }
+        catch (FilesystemException $e)
+        {
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -241,7 +266,8 @@ 
     {
         $fs = $this->getOperator();
 
-        foreach ($fs->listContents($directory) as $_) {
+        foreach ($fs->listContents($directory) as $_)
+        {
             return true;
         }
 
@@ -256,15 +282,18 @@ 
      */
     private function isStringable($value): bool
     {
-        if (\is_string($value)) {
+        if (\is_string($value))
+        {
             return true;
         }
 
-        if (!\is_object($value)) {
+        if (!\is_object($value))
+        {
             return false;
         }
 
-        if (\PHP_VERSION_ID >= 80000) {
+        if (\PHP_VERSION_ID >= 80000)
+        {
             return $value instanceof \Stringable;
         }