Passed
Pull Request — master (#1115)
by Maxim
22:29
created
src/SendIt/tests/RenderTest.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -33,7 +33,8 @@
 block discarded – undo
33 33
 
34 34
     public function tearDown(): void
35 35
     {
36
-        foreach (glob(__DIR__ . '/App/runtime/cache/views/*.php') as $file) {
36
+        foreach (glob(__DIR__ . '/App/runtime/cache/views/*.php') as $file)
37
+        {
37 38
             @unlink($file);
38 39
         }
39 40
     }
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -19,14 +19,14 @@
 block discarded – undo
19 19
     public function setUp(): void
20 20
     {
21 21
         $this->app = App::create([
22
-            'root' => __DIR__ . '/App',
23
-            'app'  => __DIR__ . '/App'
22
+            'root' => __DIR__.'/App',
23
+            'app'  => __DIR__.'/App'
24 24
         ])->run();
25 25
     }
26 26
 
27 27
     public function tearDown(): void
28 28
     {
29
-        foreach (glob(__DIR__ . '/App/runtime/cache/views/*.php') as $file) {
29
+        foreach (glob(__DIR__.'/App/runtime/cache/views/*.php') as $file){
30 30
             @unlink($file);
31 31
         }
32 32
     }
Please login to merge, or discard this patch.
src/Storage/src/File/UriResolvableTrait.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@
 block discarded – undo
31 31
     {
32 32
         $resolver = $this->getResolver();
33 33
 
34
-        if ($resolver === null) {
34
+        if ($resolver === null){
35 35
             throw new \LogicException('Can not generate public url: File not accessible by HTTP');
36 36
         }
37 37
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,8 @@
 block discarded – undo
31 31
     {
32 32
         $resolver = $this->getResolver();
33 33
 
34
-        if ($resolver === null) {
34
+        if ($resolver === null)
35
+        {
35 36
             throw new \LogicException('Can not generate public url: File not accessible by HTTP');
36 37
         }
37 38
 
Please login to merge, or discard this patch.
src/Storage/tests/TestCase.php 2 patches
Braces   +11 added lines, -5 removed lines patch added patch discarded remove patch
@@ -63,20 +63,26 @@
 block discarded – undo
63 63
         );
64 64
 
65 65
         /** @var \SplFileInfo $file */
66
-        foreach($iterator as $file) {
67
-            if ($file->getFilename() === '.gitignore') {
66
+        foreach($iterator as $file)
67
+        {
68
+            if ($file->getFilename() === '.gitignore')
69
+            {
68 70
                 continue;
69 71
             }
70 72
 
71 73
             \error_clear_last();
72 74
 
73
-            if ($file->isDir()) {
75
+            if ($file->isDir())
76
+            {
74 77
                 @\rmdir($file->getPathname());
75
-            } else {
78
+            }
79
+            else
80
+            {
76 81
                 @\unlink($file->getPathname());
77 82
             }
78 83
 
79
-            if ($error = \error_get_last()) {
84
+            if ($error = \error_get_last())
85
+            {
80 86
                 $prefix = 'An error occurred while clear temporary local storage directory: ';
81 87
                 $this->addWarning($prefix . $error['message']);
82 88
             }
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
     /**
15 15
      * @var string
16 16
      */
17
-    protected const LOCAL_STORAGE_DIRECTORY = __DIR__ . '/storage';
17
+    protected const LOCAL_STORAGE_DIRECTORY = __DIR__.'/storage';
18 18
 
19 19
     /**
20 20
      * @var BucketInterface
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
         );
39 39
 
40 40
         $this->second = Bucket::fromAdapter(
41
-            new LocalFilesystemAdapter(self::LOCAL_STORAGE_DIRECTORY . '/second')
41
+            new LocalFilesystemAdapter(self::LOCAL_STORAGE_DIRECTORY.'/second')
42 42
         );
43 43
     }
44 44
 
@@ -63,22 +63,22 @@  discard block
 block discarded – undo
63 63
         );
64 64
 
65 65
         /** @var \SplFileInfo $file */
66
-        foreach($iterator as $file) {
67
-            if ($file->getFilename() === '.gitignore') {
66
+        foreach ($iterator as $file){
67
+            if ($file->getFilename() === '.gitignore'){
68 68
                 continue;
69 69
             }
70 70
 
71 71
             \error_clear_last();
72 72
 
73
-            if ($file->isDir()) {
73
+            if ($file->isDir()){
74 74
                 @\rmdir($file->getPathname());
75
-            } else {
75
+            }else{
76 76
                 @\unlink($file->getPathname());
77 77
             }
78 78
 
79
-            if ($error = \error_get_last()) {
79
+            if ($error = \error_get_last()){
80 80
                 $prefix = 'An error occurred while clear temporary local storage directory: ';
81
-                $this->addWarning($prefix . $error['message']);
81
+                $this->addWarning($prefix.$error['message']);
82 82
             }
83 83
         }
84 84
     }
Please login to merge, or discard this patch.
src/Storage/tests/StorageTestCase.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -151,7 +151,8 @@
 block discarded – undo
151 151
 
152 152
         $actual = '';
153 153
         $stream = $this->local->getStream('file.txt');
154
-        while (!\feof($stream)) {
154
+        while (!\feof($stream))
155
+        {
155 156
             $actual .= \fread($stream, 256);
156 157
         }
157 158
         \fclose($stream);
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -57,8 +57,8 @@  discard block
 block discarded – undo
57 57
     public function testVisibility(): void
58 58
     {
59 59
         $this->markTestSkipped(
60
-            'This test [' . __FUNCTION__ . '] returns incorrect visibility ' .
61
-                'of files on Windows OS. ' .
60
+            'This test ['.__FUNCTION__.'] returns incorrect visibility '.
61
+                'of files on Windows OS. '.
62 62
             'It is required to understand the situation'
63 63
         );
64 64
 
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
151 151
 
152 152
         $actual = '';
153 153
         $stream = $this->local->getStream('file.txt');
154
-        while (!\feof($stream)) {
154
+        while (!\feof($stream)){
155 155
             $actual .= \fread($stream, 256);
156 156
         }
157 157
         \fclose($stream);
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
      */
176 176
     public function testLastModified(): void
177 177
     {
178
-        $now = (int) \floor(\microtime(true));
178
+        $now = (int)\floor(\microtime(true));
179 179
 
180 180
         $this->local->create('file.txt');
181 181
         $before = $this->local->getLastModified('file.txt');
Please login to merge, or discard this patch.
src/Storage/tests/ManagerTestCase.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
     {
93 93
         $this->expectException(InvalidArgumentException::class);
94 94
         $this->expectExceptionMessage(
95
-            'URI argument must be a valid URI in ' .
95
+            'URI argument must be a valid URI in '.
96 96
             '"[STORAGE]://[PATH_TO_FILE]" format, but `test://` given'
97 97
         );
98 98
 
@@ -157,8 +157,8 @@  discard block
 block discarded – undo
157 157
     public function testVisibility(): void
158 158
     {
159 159
         $this->markTestSkipped(
160
-            'This test [' . __FUNCTION__ . '] returns incorrect visibility ' .
161
-                'of files on Windows OS. ' .
160
+            'This test ['.__FUNCTION__.'] returns incorrect visibility '.
161
+                'of files on Windows OS. '.
162 162
             'It is required to understand the situation'
163 163
         );
164 164
 
@@ -257,7 +257,7 @@  discard block
 block discarded – undo
257 257
 
258 258
         $actual = '';
259 259
         $stream = $this->manager->getStream('file.txt');
260
-        while (!\feof($stream)) {
260
+        while (!\feof($stream)){
261 261
             $actual .= \fread($stream, 256);
262 262
         }
263 263
         \fclose($stream);
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -257,7 +257,8 @@
 block discarded – undo
257 257
 
258 258
         $actual = '';
259 259
         $stream = $this->manager->getStream('file.txt');
260
-        while (!\feof($stream)) {
260
+        while (!\feof($stream))
261
+        {
261 262
             $actual .= \fread($stream, 256);
262 263
         }
263 264
         \fclose($stream);
Please login to merge, or discard this patch.
src/Storage/tests/FileTestCase.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -190,7 +190,8 @@
 block discarded – undo
190 190
 
191 191
         $actual = '';
192 192
         $stream = $source->getStream();
193
-        while (!\feof($stream)) {
193
+        while (!\feof($stream))
194
+        {
194 195
             $actual .= \fread($stream, 256);
195 196
         }
196 197
         \fclose($stream);
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -81,8 +81,8 @@  discard block
 block discarded – undo
81 81
     public function testVisibility(): void
82 82
     {
83 83
         $this->markTestSkipped(
84
-            'This test [' . __FUNCTION__ . '] returns incorrect visibility ' .
85
-                'of files on Windows OS. ' .
84
+            'This test ['.__FUNCTION__.'] returns incorrect visibility '.
85
+                'of files on Windows OS. '.
86 86
             'It is required to understand the situation'
87 87
         );
88 88
 
@@ -190,7 +190,7 @@  discard block
 block discarded – undo
190 190
 
191 191
         $actual = '';
192 192
         $stream = $source->getStream();
193
-        while (!\feof($stream)) {
193
+        while (!\feof($stream)){
194 194
             $actual .= \fread($stream, 256);
195 195
         }
196 196
         \fclose($stream);
@@ -216,7 +216,7 @@  discard block
 block discarded – undo
216 216
      */
217 217
     public function testLastModified(): void
218 218
     {
219
-        $now = (int) \floor(\microtime(true));
219
+        $now = (int)\floor(\microtime(true));
220 220
 
221 221
         $file = $this->local->file('file.txt')
222 222
             ->create()
Please login to merge, or discard this patch.
src/Prototype/src/ClassNode/ConstructorParam.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -56,13 +56,13 @@
 block discarded – undo
56 56
         $stmt->name = $parameter->getName();
57 57
 
58 58
         $type = $parameter->getType();
59
-        if ($type instanceof \ReflectionNamedType) {
59
+        if ($type instanceof \ReflectionNamedType){
60 60
             $stmt->type = Type::create($type->getName());
61 61
             $stmt->builtIn = $type->isBuiltin();
62 62
             $stmt->nullable = $type->allowsNull();
63 63
         }
64 64
 
65
-        if ($parameter->isDefaultValueAvailable()) {
65
+        if ($parameter->isDefaultValueAvailable()){
66 66
             $stmt->hasDefault = true;
67 67
             $stmt->default = $parameter->getDefaultValue();
68 68
         }
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -56,13 +56,15 @@
 block discarded – undo
56 56
         $stmt->name = $parameter->getName();
57 57
 
58 58
         $type = $parameter->getType();
59
-        if ($type instanceof \ReflectionNamedType) {
59
+        if ($type instanceof \ReflectionNamedType)
60
+        {
60 61
             $stmt->type = Type::create($type->getName());
61 62
             $stmt->builtIn = $type->isBuiltin();
62 63
             $stmt->nullable = $type->allowsNull();
63 64
         }
64 65
 
65
-        if ($parameter->isDefaultValueAvailable()) {
66
+        if ($parameter->isDefaultValueAvailable())
67
+        {
66 68
             $stmt->hasDefault = true;
67 69
             $stmt->default = $parameter->getDefaultValue();
68 70
         }
Please login to merge, or discard this patch.
src/Storage/src/File.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -64,7 +64,8 @@
 block discarded – undo
64 64
     {
65 65
         $name = $this->storage->getName();
66 66
 
67
-        if ($name === null) {
67
+        if ($name === null)
68
+        {
68 69
             return $this->getPathname();
69 70
         }
70 71
 
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
         private readonly BucketInterface $storage,
20 20
         private readonly string $pathname,
21 21
         private readonly ?UriResolverInterface $resolver = null
22
-    ) {
22
+    ){
23 23
     }
24 24
 
25 25
     public function __toString(): string
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
     {
32 32
         $name = $this->storage->getName();
33 33
 
34
-        if ($name === null) {
34
+        if ($name === null){
35 35
             return $this->getPathname();
36 36
         }
37 37
 
Please login to merge, or discard this patch.
src/Storage/src/Bucket/ReadableTrait.php 3 patches
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -136,7 +136,7 @@
 block discarded – undo
136 136
         #[ExpectedValues(valuesFromClass: \League\Flysystem\Visibility::class)]
137 137
         string $visibility
138 138
     ): string {
139
-        return $visibility === \League\Flysystem\Visibility::PUBLIC
139
+        return $visibility === \League\Flysystem\Visibility::public
140 140
             ? Visibility::VISIBILITY_PUBLIC
141 141
             : Visibility::VISIBILITY_PRIVATE;
142 142
     }
Please login to merge, or discard this patch.
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -29,9 +29,9 @@  discard block
 block discarded – undo
29 29
     {
30 30
         $fs = $this->getOperator();
31 31
 
32
-        try {
32
+        try{
33 33
             return $fs->fileExists($pathname);
34
-        } catch (FilesystemException $e) {
34
+        }catch (FilesystemException $e){
35 35
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
36 36
         }
37 37
     }
@@ -43,9 +43,9 @@  discard block
 block discarded – undo
43 43
     {
44 44
         $fs = $this->getOperator();
45 45
 
46
-        try {
46
+        try{
47 47
             return $fs->read($pathname);
48
-        } catch (FilesystemException $e) {
48
+        }catch (FilesystemException $e){
49 49
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
50 50
         }
51 51
     }
@@ -57,9 +57,9 @@  discard block
 block discarded – undo
57 57
     {
58 58
         $fs = $this->getOperator();
59 59
 
60
-        try {
60
+        try{
61 61
             return $fs->readStream($pathname);
62
-        } catch (FilesystemException $e) {
62
+        }catch (FilesystemException $e){
63 63
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
64 64
         }
65 65
     }
@@ -71,9 +71,9 @@  discard block
 block discarded – undo
71 71
     {
72 72
         $fs = $this->getOperator();
73 73
 
74
-        try {
74
+        try{
75 75
             return $fs->lastModified($pathname);
76
-        } catch (FilesystemException $e) {
76
+        }catch (FilesystemException $e){
77 77
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
78 78
         }
79 79
     }
@@ -85,9 +85,9 @@  discard block
 block discarded – undo
85 85
     {
86 86
         $fs = $this->getOperator();
87 87
 
88
-        try {
88
+        try{
89 89
             return $fs->fileSize($pathname);
90
-        } catch (FilesystemException $e) {
90
+        }catch (FilesystemException $e){
91 91
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
92 92
         }
93 93
     }
@@ -99,9 +99,9 @@  discard block
 block discarded – undo
99 99
     {
100 100
         $fs = $this->getOperator();
101 101
 
102
-        try {
102
+        try{
103 103
             return $fs->mimeType($pathname);
104
-        } catch (FilesystemException $e) {
104
+        }catch (FilesystemException $e){
105 105
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
106 106
         }
107 107
     }
@@ -114,11 +114,11 @@  discard block
 block discarded – undo
114 114
     {
115 115
         $fs = $this->getOperator();
116 116
 
117
-        try {
117
+        try{
118 118
             return $this->fromFlysystemVisibility(
119 119
                 $fs->visibility($pathname)
120 120
             );
121
-        } catch (FilesystemException $e) {
121
+        }catch (FilesystemException $e){
122 122
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
123 123
         }
124 124
     }
Please login to merge, or discard this patch.
Braces   +35 added lines, -14 removed lines patch added patch discarded remove patch
@@ -29,9 +29,12 @@  discard block
 block discarded – undo
29 29
     {
30 30
         $fs = $this->getOperator();
31 31
 
32
-        try {
32
+        try
33
+        {
33 34
             return $fs->fileExists($pathname);
34
-        } catch (FilesystemException $e) {
35
+        }
36
+        catch (FilesystemException $e)
37
+        {
35 38
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
36 39
         }
37 40
     }
@@ -43,9 +46,12 @@  discard block
 block discarded – undo
43 46
     {
44 47
         $fs = $this->getOperator();
45 48
 
46
-        try {
49
+        try
50
+        {
47 51
             return $fs->read($pathname);
48
-        } catch (FilesystemException $e) {
52
+        }
53
+        catch (FilesystemException $e)
54
+        {
49 55
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
50 56
         }
51 57
     }
@@ -57,9 +63,12 @@  discard block
 block discarded – undo
57 63
     {
58 64
         $fs = $this->getOperator();
59 65
 
60
-        try {
66
+        try
67
+        {
61 68
             return $fs->readStream($pathname);
62
-        } catch (FilesystemException $e) {
69
+        }
70
+        catch (FilesystemException $e)
71
+        {
63 72
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
64 73
         }
65 74
     }
@@ -71,9 +80,12 @@  discard block
 block discarded – undo
71 80
     {
72 81
         $fs = $this->getOperator();
73 82
 
74
-        try {
83
+        try
84
+        {
75 85
             return $fs->lastModified($pathname);
76
-        } catch (FilesystemException $e) {
86
+        }
87
+        catch (FilesystemException $e)
88
+        {
77 89
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
78 90
         }
79 91
     }
@@ -85,9 +97,12 @@  discard block
 block discarded – undo
85 97
     {
86 98
         $fs = $this->getOperator();
87 99
 
88
-        try {
100
+        try
101
+        {
89 102
             return $fs->fileSize($pathname);
90
-        } catch (FilesystemException $e) {
103
+        }
104
+        catch (FilesystemException $e)
105
+        {
91 106
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
92 107
         }
93 108
     }
@@ -99,9 +114,12 @@  discard block
 block discarded – undo
99 114
     {
100 115
         $fs = $this->getOperator();
101 116
 
102
-        try {
117
+        try
118
+        {
103 119
             return $fs->mimeType($pathname);
104
-        } catch (FilesystemException $e) {
120
+        }
121
+        catch (FilesystemException $e)
122
+        {
105 123
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
106 124
         }
107 125
     }
@@ -114,11 +132,14 @@  discard block
 block discarded – undo
114 132
     {
115 133
         $fs = $this->getOperator();
116 134
 
117
-        try {
135
+        try
136
+        {
118 137
             return $this->fromFlysystemVisibility(
119 138
                 $fs->visibility($pathname)
120 139
             );
121
-        } catch (FilesystemException $e) {
140
+        }
141
+        catch (FilesystemException $e)
142
+        {
122 143
             throw new FileOperationException($e->getMessage(), $e->getCode(), $e);
123 144
         }
124 145
     }
Please login to merge, or discard this patch.