Passed
Pull Request — master (#1163)
by Abdul Malik
18:37 queued 07:49
created
monorepo-builder.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -10,8 +10,8 @@  discard block
 block discarded – undo
10 10
  *
11 11
  * @see https://github.com/symplify/symplify/issues/2061
12 12
  */
13
-\register_shutdown_function(static function () {
14
-    $dest = \json_decode(\file_get_contents(__DIR__ . '/composer.json'), true);
13
+\register_shutdown_function(static function (){
14
+    $dest = \json_decode(\file_get_contents(__DIR__.'/composer.json'), true);
15 15
 
16 16
     $result = [
17 17
         'name'              => 'spiral/framework',
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 
47 47
     $json = \json_encode($result, \JSON_PRETTY_PRINT | \JSON_UNESCAPED_SLASHES);
48 48
 
49
-    \file_put_contents(__DIR__ . '/composer.json', $json . "\n");
49
+    \file_put_contents(__DIR__.'/composer.json', $json."\n");
50 50
 });
51 51
 
52 52
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,8 @@
 block discarded – undo
10 10
  *
11 11
  * @see https://github.com/symplify/symplify/issues/2061
12 12
  */
13
-\register_shutdown_function(static function () {
13
+\register_shutdown_function(static function ()
14
+{
14 15
     $dest = \json_decode(\file_get_contents(__DIR__ . '/composer.json'), true);
15 16
 
16 17
     $result = [
Please login to merge, or discard this patch.
src/Streams/tests/StreamsTest.php 2 patches
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 
18 18
 class StreamsTest extends TestCase
19 19
 {
20
-    private const FIXTURE_DIRECTORY = __DIR__ . '/fixtures';
20
+    private const FIXTURE_DIRECTORY = __DIR__.'/fixtures';
21 21
 
22 22
     public function setUp(): void
23 23
     {
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
         $this->assertSame(strlen('sample text'), filesize($filename));
43 43
         $this->assertSame(md5('sample text'), md5_file($filename));
44 44
 
45
-        $newFilename = self::FIXTURE_DIRECTORY . '/test.txt';
45
+        $newFilename = self::FIXTURE_DIRECTORY.'/test.txt';
46 46
         copy($filename, $newFilename);
47 47
 
48 48
         $this->assertFileExists($newFilename);
@@ -83,15 +83,15 @@  discard block
 block discarded – undo
83 83
      */
84 84
     public function testException()
85 85
     {
86
-        try {
86
+        try{
87 87
             fopen('spiral://non-exists', 'rb');
88
-        } catch (\Throwable $e) {
88
+        }catch (\Throwable $e){
89 89
             $this->assertStringContainsString('failed to open stream', $e->getMessage());
90 90
         }
91 91
 
92
-        try {
92
+        try{
93 93
             filemtime('spiral://non-exists');
94
-        } catch (\Throwable $e) {
94
+        }catch (\Throwable $e){
95 95
             $this->assertStringContainsString('stat failed', $e->getMessage());
96 96
         }
97 97
     }
@@ -101,15 +101,15 @@  discard block
 block discarded – undo
101 101
      */
102 102
     public function testExceptionPHP8()
103 103
     {
104
-        try {
104
+        try{
105 105
             fopen('spiral://non-exists', 'rb');
106
-        } catch (\Throwable $e) {
106
+        }catch (\Throwable $e){
107 107
             $this->assertStringContainsString('Failed to open stream', $e->getMessage());
108 108
         }
109 109
 
110
-        try {
110
+        try{
111 111
             filemtime('spiral://non-exists');
112
-        } catch (\Throwable $e) {
112
+        }catch (\Throwable $e){
113 113
             $this->assertStringContainsString('stat failed', $e->getMessage());
114 114
         }
115 115
     }
Please login to merge, or discard this patch.
Braces   +20 added lines, -8 removed lines patch added patch discarded remove patch
@@ -83,15 +83,21 @@  discard block
 block discarded – undo
83 83
      */
84 84
     public function testException()
85 85
     {
86
-        try {
86
+        try
87
+        {
87 88
             fopen('spiral://non-exists', 'rb');
88
-        } catch (\Throwable $e) {
89
+        }
90
+        catch (\Throwable $e)
91
+        {
89 92
             $this->assertStringContainsString('failed to open stream', $e->getMessage());
90 93
         }
91 94
 
92
-        try {
95
+        try
96
+        {
93 97
             filemtime('spiral://non-exists');
94
-        } catch (\Throwable $e) {
98
+        }
99
+        catch (\Throwable $e)
100
+        {
95 101
             $this->assertStringContainsString('stat failed', $e->getMessage());
96 102
         }
97 103
     }
@@ -101,15 +107,21 @@  discard block
 block discarded – undo
101 107
      */
102 108
     public function testExceptionPHP8()
103 109
     {
104
-        try {
110
+        try
111
+        {
105 112
             fopen('spiral://non-exists', 'rb');
106
-        } catch (\Throwable $e) {
113
+        }
114
+        catch (\Throwable $e)
115
+        {
107 116
             $this->assertStringContainsString('Failed to open stream', $e->getMessage());
108 117
         }
109 118
 
110
-        try {
119
+        try
120
+        {
111 121
             filemtime('spiral://non-exists');
112
-        } catch (\Throwable $e) {
122
+        }
123
+        catch (\Throwable $e)
124
+        {
113 125
             $this->assertStringContainsString('stat failed', $e->getMessage());
114 126
         }
115 127
     }
Please login to merge, or discard this patch.
src/SendIt/src/MessageSerializer.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -51,11 +51,11 @@
 block discarded – undo
51 51
     public static function unpack(array $payload): MessageInterface
52 52
     {
53 53
         $message = new Message($payload['subject'], $payload['to'], $payload['data']);
54
-        if ($payload['from'] !== null) {
54
+        if ($payload['from'] !== null){
55 55
             $message->setFrom($payload['from']);
56 56
         }
57 57
 
58
-        if ($payload['replyTo'] !== null) {
58
+        if ($payload['replyTo'] !== null){
59 59
             $message->setReplyTo($payload['replyTo']);
60 60
         }
61 61
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -51,11 +51,13 @@
 block discarded – undo
51 51
     public static function unpack(array $payload): MessageInterface
52 52
     {
53 53
         $message = new Message($payload['subject'], $payload['to'], $payload['data']);
54
-        if ($payload['from'] !== null) {
54
+        if ($payload['from'] !== null)
55
+        {
55 56
             $message->setFrom($payload['from']);
56 57
         }
57 58
 
58
-        if ($payload['replyTo'] !== null) {
59
+        if ($payload['replyTo'] !== null)
60
+        {
59 61
             $message->setReplyTo($payload['replyTo']);
60 62
         }
61 63
 
Please login to merge, or discard this patch.
src/Prototype/tests/Commands/InjectCommandTest.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
         $source = file_get_contents($filename);
30 30
         $this->assertStringContainsString('use PrototypeTrait;', $source);
31 31
 
32
-        try {
32
+        try{
33 33
             $this->app->bindApp();
34 34
 
35 35
             $inp = new ArrayInput(['--remove' => true]);
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
             $this->app->get(Console::class)->run('prototype:inject', $inp, $out);
38 38
 
39 39
             $this->assertStringNotContainsString('use PrototypeTrait;', file_get_contents($filename));
40
-        } finally {
40
+        }finally{
41 41
             file_put_contents($filename, $source);
42 42
         }
43 43
     }
Please login to merge, or discard this patch.
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,7 +29,8 @@  discard block
 block discarded – undo
29 29
         $source = file_get_contents($filename);
30 30
         $this->assertStringContainsString('use PrototypeTrait;', $source);
31 31
 
32
-        try {
32
+        try
33
+        {
33 34
             $this->app->bindApp();
34 35
 
35 36
             $inp = new ArrayInput(['--remove' => true]);
@@ -37,7 +38,9 @@  discard block
 block discarded – undo
37 38
             $this->app->get(Console::class)->run('prototype:inject', $inp, $out);
38 39
 
39 40
             $this->assertStringNotContainsString('use PrototypeTrait;', file_get_contents($filename));
40
-        } finally {
41
+        }
42
+        finally
43
+        {
41 44
             file_put_contents($filename, $source);
42 45
         }
43 46
     }
Please login to merge, or discard this patch.
src/SendIt/views/builder.dark.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@  discard block
 block discarded – undo
7 7
  */
8 8
 $_msg_->subject(\Spiral\SendIt\Renderer\ViewRenderer::escapeSubject(inject('subject')));
9 9
 
10
-if (injected('from')) {
10
+if (injected('from')){
11 11
     $_msg_->from(\Symfony\Component\Mime\Address::create(inject('from')));
12 12
 }
13 13
 
@@ -19,13 +19,13 @@  discard block
 block discarded – undo
19 19
 
20 20
 ob_start(); ?>${html}<?php $_html_ = ob_get_clean();
21 21
 
22
-if (!empty($_html_)) {
22
+if (!empty($_html_)){
23 23
     $_msg_->html($_html_);
24 24
 }
25 25
 
26 26
 ob_start(); ?>${text}<?php $_text_ = ob_get_clean();
27 27
 
28
-if (!empty($_text_)) {
28
+if (!empty($_text_)){
29 29
     $_msg_->text($_text_);
30 30
 }
31 31
 ?>
Please login to merge, or discard this patch.
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -7,7 +7,8 @@  discard block
 block discarded – undo
7 7
  */
8 8
 $_msg_->subject(\Spiral\SendIt\Renderer\ViewRenderer::escapeSubject(inject('subject')));
9 9
 
10
-if (injected('from')) {
10
+if (injected('from'))
11
+{
11 12
     $_msg_->from(\Symfony\Component\Mime\Address::create(inject('from')));
12 13
 }
13 14
 
@@ -19,13 +20,15 @@  discard block
 block discarded – undo
19 20
 
20 21
 ob_start(); ?>${html}<?php $_html_ = ob_get_clean();
21 22
 
22
-if (!empty($_html_)) {
23
+if (!empty($_html_))
24
+{
23 25
     $_msg_->html($_html_);
24 26
 }
25 27
 
26 28
 ob_start(); ?>${text}<?php $_text_ = ob_get_clean();
27 29
 
28
-if (!empty($_text_)) {
30
+if (!empty($_text_))
31
+{
29 32
     $_msg_->text($_text_);
30 33
 }
31 34
 ?>
Please login to merge, or discard this patch.
src/Bridge/Stempler/src/ExceptionMapper.php 2 patches
Braces   +10 added lines, -5 removed lines patch added patch discarded remove patch
@@ -45,9 +45,12 @@  discard block
 block discarded – undo
45 45
         $classFilename = (new \ReflectionClass($class))->getFileName();
46 46
 
47 47
         // looking for proper view error location
48
-        if ($e->getFile() !== $classFilename) {
49
-            foreach ($e->getTrace() as $trace) {
50
-                if (isset($trace['file']) && $trace['file'] === $classFilename) {
48
+        if ($e->getFile() !== $classFilename)
49
+        {
50
+            foreach ($e->getTrace() as $trace)
51
+            {
52
+                if (isset($trace['file']) && $trace['file'] === $classFilename)
53
+                {
51 54
                     $line = $trace['line'];
52 55
                     break;
53 56
                 }
@@ -55,7 +58,8 @@  discard block
 block discarded – undo
55 58
         }
56 59
 
57 60
         $userStack = [];
58
-        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack) {
61
+        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack)
62
+        {
59 63
             $userStack[] = [
60 64
                 'file'     => $stack['file'],
61 65
                 'line'     => $stack['line'],
@@ -65,7 +69,8 @@  discard block
 block discarded – undo
65 69
                 'args'     => [$data],
66 70
             ];
67 71
 
68
-            if ($stack['file'] === $filename) {
72
+            if ($stack['file'] === $filename)
73
+            {
69 74
                 // no need to jump over root template
70 75
                 break;
71 76
             }
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
     public function __construct(
13 13
         private readonly SourceMap $sourcemap,
14 14
         private readonly int $lineOffset
15
-    ) {
15
+    ){
16 16
     }
17 17
 
18 18
     public function mapException(\Throwable $e, string $class, string $filename, array $data): \Throwable
@@ -21,9 +21,9 @@  discard block
 block discarded – undo
21 21
         $classFilename = (new \ReflectionClass($class))->getFileName();
22 22
 
23 23
         // looking for proper view error location
24
-        if ($e->getFile() !== $classFilename) {
25
-            foreach ($e->getTrace() as $trace) {
26
-                if (isset($trace['file']) && $trace['file'] === $classFilename) {
24
+        if ($e->getFile() !== $classFilename){
25
+            foreach ($e->getTrace() as $trace){
26
+                if (isset($trace['file']) && $trace['file'] === $classFilename){
27 27
                     $line = $trace['line'];
28 28
                     break;
29 29
                 }
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
         }
32 32
 
33 33
         $userStack = [];
34
-        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack) {
34
+        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack){
35 35
             $userStack[] = [
36 36
                 'file'     => $stack['file'],
37 37
                 'line'     => $stack['line'],
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
                 'args'     => [$data],
42 42
             ];
43 43
 
44
-            if ($stack['file'] === $filename) {
44
+            if ($stack['file'] === $filename){
45 45
                 // no need to jump over root template
46 46
                 break;
47 47
             }
Please login to merge, or discard this patch.
src/Bridge/Stempler/src/StemplerLoader.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -56,7 +56,8 @@
 block discarded – undo
56 56
      */
57 57
     public function load(string $path): Source
58 58
     {
59
-        if ($this->context === null) {
59
+        if ($this->context === null)
60
+        {
60 61
             throw new EngineException('Unable to use StemplerLoader without given context.');
61 62
         }
62 63
 
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
     public function __construct(
24 24
         private readonly LoaderInterface $loader,
25 25
         array $processors
26
-    ) {
26
+    ){
27 27
         $this->processors = $processors;
28 28
     }
29 29
 
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
      */
41 41
     public function load(string $path): Source
42 42
     {
43
-        if ($this->context === null) {
43
+        if ($this->context === null){
44 44
             throw new EngineException('Unable to use StemplerLoader without given context.');
45 45
         }
46 46
 
Please login to merge, or discard this patch.
src/Bridge/Stempler/src/StemplerView.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -59,7 +59,8 @@
 block discarded – undo
59 59
             $this->context
60 60
         );
61 61
 
62
-        if ($sourcemap === null) {
62
+        if ($sourcemap === null)
63
+        {
63 64
             return $e;
64 65
         }
65 66
 
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
         protected StemplerEngine $engine,
21 21
         protected ViewSource $view,
22 22
         protected ContextInterface $context
23
-    ) {
23
+    ){
24 24
         $this->container = $engine->getContainer();
25 25
     }
26 26
 
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
             $this->context
35 35
         );
36 36
 
37
-        if ($sourcemap === null) {
37
+        if ($sourcemap === null){
38 38
             return $e;
39 39
         }
40 40
 
Please login to merge, or discard this patch.
src/Bridge/Stempler/src/StemplerCache.php 2 patches
Braces   +12 added lines, -6 removed lines patch added patch discarded remove patch
@@ -67,15 +67,18 @@  discard block
 block discarded – undo
67 67
     public function isFresh(string $key): bool
68 68
     {
69 69
         $mapFilename = $this->mapFilename($key);
70
-        if (!$this->files->exists($mapFilename)) {
70
+        if (!$this->files->exists($mapFilename))
71
+        {
71 72
             return false;
72 73
         }
73 74
 
74 75
         $time = $this->files->time($this->filename($key));
75 76
 
76 77
         $files = (array)include $mapFilename;
77
-        foreach ($files as $file) {
78
-            if ($this->files->time($file) > $time) {
78
+        foreach ($files as $file)
79
+        {
80
+            if ($this->files->time($file) > $time)
81
+            {
79 82
                 // some partial has changed
80 83
                 return false;
81 84
             }
@@ -92,12 +95,14 @@  discard block
 block discarded – undo
92 95
     public function delete(string $key): void
93 96
     {
94 97
         $filename = $this->filename($key);
95
-        if ($this->files->exists($filename)) {
98
+        if ($this->files->exists($filename))
99
+        {
96 100
             $this->files->delete($filename);
97 101
         }
98 102
 
99 103
         $mapFilename = $this->mapFilename($key);
100
-        if ($this->files->exists($mapFilename)) {
104
+        if ($this->files->exists($mapFilename))
105
+        {
101 106
             $this->files->delete($mapFilename);
102 107
         }
103 108
     }
@@ -110,7 +115,8 @@  discard block
 block discarded – undo
110 115
     public function load(string $key): void
111 116
     {
112 117
         $filename = $this->filename($key);
113
-        if ($this->files->exists($filename)) {
118
+        if ($this->files->exists($filename))
119
+        {
114 120
             include_once $filename;
115 121
         }
116 122
     }
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
     public function __construct(
13 13
         private readonly string $directory,
14 14
         private readonly FilesInterface $files = new Files()
15
-    ) {
15
+    ){
16 16
     }
17 17
 
18 18
     /**
@@ -43,15 +43,15 @@  discard block
 block discarded – undo
43 43
     public function isFresh(string $key): bool
44 44
     {
45 45
         $mapFilename = $this->mapFilename($key);
46
-        if (!$this->files->exists($mapFilename)) {
46
+        if (!$this->files->exists($mapFilename)){
47 47
             return false;
48 48
         }
49 49
 
50 50
         $time = $this->files->time($this->filename($key));
51 51
 
52 52
         $files = (array)include $mapFilename;
53
-        foreach ($files as $file) {
54
-            if ($this->files->time($file) > $time) {
53
+        foreach ($files as $file){
54
+            if ($this->files->time($file) > $time){
55 55
                 // some partial has changed
56 56
                 return false;
57 57
             }
@@ -66,12 +66,12 @@  discard block
 block discarded – undo
66 66
     public function delete(string $key): void
67 67
     {
68 68
         $filename = $this->filename($key);
69
-        if ($this->files->exists($filename)) {
69
+        if ($this->files->exists($filename)){
70 70
             $this->files->delete($filename);
71 71
         }
72 72
 
73 73
         $mapFilename = $this->mapFilename($key);
74
-        if ($this->files->exists($mapFilename)) {
74
+        if ($this->files->exists($mapFilename)){
75 75
             $this->files->delete($mapFilename);
76 76
         }
77 77
     }
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
     public function load(string $key): void
83 83
     {
84 84
         $filename = $this->filename($key);
85
-        if ($this->files->exists($filename)) {
85
+        if ($this->files->exists($filename)){
86 86
             include_once $filename;
87 87
         }
88 88
     }
Please login to merge, or discard this patch.