Passed
Pull Request — master (#355)
by Valentin
04:57
created
src/SendIt/src/MailJob.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -56,15 +56,15 @@  discard block
 block discarded – undo
56 56
 
57 57
         $email = $this->renderer->render($message);
58 58
 
59
-        if ($email->getFrom() === []) {
59
+        if ($email->getFrom() === []){
60 60
             $email->from(Address::create($this->config->getFromAddress()));
61 61
         }
62 62
 
63 63
         $recipients = $this->getRecipients($email);
64 64
 
65
-        try {
65
+        try{
66 66
             $this->mailer->send($email);
67
-        } catch (TransportExceptionInterface $e) {
67
+        }catch (TransportExceptionInterface $e){
68 68
             $this->getLogger()->error(
69 69
                 sprintf(
70 70
                     'Failed to send `%s` to "%s": %s',
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
 
99 99
         $addresses = array_merge($message->getTo(), $message->getCc(), $message->getBcc());
100 100
 
101
-        foreach ($addresses as $address) {
101
+        foreach ($addresses as $address){
102 102
             $emails[] = $address->toString();
103 103
         }
104 104
 
Please login to merge, or discard this patch.
Braces   +9 added lines, -4 removed lines patch added patch discarded remove patch
@@ -56,15 +56,19 @@  discard block
 block discarded – undo
56 56
 
57 57
         $email = $this->renderer->render($message);
58 58
 
59
-        if ($email->getFrom() === []) {
59
+        if ($email->getFrom() === [])
60
+        {
60 61
             $email->from(Address::create($this->config->getFromAddress()));
61 62
         }
62 63
 
63 64
         $recipients = $this->getRecipients($email);
64 65
 
65
-        try {
66
+        try
67
+        {
66 68
             $this->mailer->send($email);
67
-        } catch (TransportExceptionInterface $e) {
69
+        }
70
+        catch (TransportExceptionInterface $e)
71
+        {
68 72
             $this->getLogger()->error(
69 73
                 sprintf(
70 74
                     'Failed to send `%s` to "%s": %s',
@@ -98,7 +102,8 @@  discard block
 block discarded – undo
98 102
 
99 103
         $addresses = array_merge($message->getTo(), $message->getCc(), $message->getBcc());
100 104
 
101
-        foreach ($addresses as $address) {
105
+        foreach ($addresses as $address)
106
+        {
102 107
             $emails[] = $address->toString();
103 108
         }
104 109
 
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/Config/tests/fixtures/broken.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -7,4 +7,4 @@
 block discarded – undo
7 7
  * @author    Anton Titov (Wolfy-J)
8 8
  */
9 9
 
10
-return ;
10
+return;
Please login to merge, or discard this patch.
src/Attributes/src/Internal/Instantiator/NamedArgumentsInstantiator.php 2 patches
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -38,17 +38,17 @@  discard block
 block discarded – undo
38 38
      */
39 39
     public function instantiate(\ReflectionClass $attr, array $arguments, string $context): object
40 40
     {
41
-        if ($this->isNamedArgumentsSupported()) {
42
-            try {
41
+        if ($this->isNamedArgumentsSupported()){
42
+            try{
43 43
                 return $attr->newInstanceArgs($arguments);
44
-            } catch (\Throwable $e) {
44
+            }catch (\Throwable $e){
45 45
                 throw Exception::withLocation($e, $attr->getFileName(), $attr->getStartLine());
46 46
             }
47 47
         }
48 48
 
49 49
         $constructor = $this->getConstructor($attr);
50 50
 
51
-        if ($constructor === null) {
51
+        if ($constructor === null){
52 52
             return $attr->newInstanceWithoutConstructor();
53 53
         }
54 54
 
@@ -76,16 +76,16 @@  discard block
 block discarded – undo
76 76
     {
77 77
         $passed = [];
78 78
 
79
-        try {
80
-            foreach ($constructor->getParameters() as $parameter) {
79
+        try{
80
+            foreach ($constructor->getParameters() as $parameter){
81 81
                 $passed[] = $this->resolveParameter($ctx, $parameter, $arguments);
82 82
             }
83 83
 
84
-            if (\count($arguments)) {
84
+            if (\count($arguments)){
85 85
                 $message = \sprintf(self::ERROR_UNKNOWN_ARGUMENT, \array_key_first($arguments));
86 86
                 throw new \BadMethodCallException($message);
87 87
             }
88
-        } catch (\Throwable $e) {
88
+        }catch (\Throwable $e){
89 89
             throw Exception::withLocation($e, $constructor->getFileName(), $constructor->getStartLine());
90 90
         }
91 91
 
@@ -101,14 +101,14 @@  discard block
 block discarded – undo
101 101
      */
102 102
     private function resolveParameter(\ReflectionClass $ctx, \ReflectionParameter $param, array &$arguments)
103 103
     {
104
-        switch (true) {
104
+        switch (true){
105 105
             case \array_key_exists($param->getName(), $arguments):
106
-                $argument =  $arguments[$param->getName()];
106
+                $argument = $arguments[$param->getName()];
107 107
                 unset($arguments[$param->getName()]);
108 108
                 return $argument;
109 109
 
110 110
             case \array_key_exists($param->getPosition(), $arguments):
111
-                $argument =  $arguments[$param->getPosition()];
111
+                $argument = $arguments[$param->getPosition()];
112 112
                 unset($arguments[$param->getPosition()]);
113 113
                 return $argument;
114 114
 
Please login to merge, or discard this patch.
Braces   +20 added lines, -9 removed lines patch added patch discarded remove patch
@@ -38,17 +38,22 @@  discard block
 block discarded – undo
38 38
      */
39 39
     public function instantiate(\ReflectionClass $attr, array $arguments, string $context): object
40 40
     {
41
-        if ($this->isNamedArgumentsSupported()) {
42
-            try {
41
+        if ($this->isNamedArgumentsSupported())
42
+        {
43
+            try
44
+            {
43 45
                 return $attr->newInstanceArgs($arguments);
44
-            } catch (\Throwable $e) {
46
+            }
47
+            catch (\Throwable $e)
48
+            {
45 49
                 throw Exception::withLocation($e, $attr->getFileName(), $attr->getStartLine());
46 50
             }
47 51
         }
48 52
 
49 53
         $constructor = $this->getConstructor($attr);
50 54
 
51
-        if ($constructor === null) {
55
+        if ($constructor === null)
56
+        {
52 57
             return $attr->newInstanceWithoutConstructor();
53 58
         }
54 59
 
@@ -76,16 +81,21 @@  discard block
 block discarded – undo
76 81
     {
77 82
         $passed = [];
78 83
 
79
-        try {
80
-            foreach ($constructor->getParameters() as $parameter) {
84
+        try
85
+        {
86
+            foreach ($constructor->getParameters() as $parameter)
87
+            {
81 88
                 $passed[] = $this->resolveParameter($ctx, $parameter, $arguments);
82 89
             }
83 90
 
84
-            if (\count($arguments)) {
91
+            if (\count($arguments))
92
+            {
85 93
                 $message = \sprintf(self::ERROR_UNKNOWN_ARGUMENT, \array_key_first($arguments));
86 94
                 throw new \BadMethodCallException($message);
87 95
             }
88
-        } catch (\Throwable $e) {
96
+        }
97
+        catch (\Throwable $e)
98
+        {
89 99
             throw Exception::withLocation($e, $constructor->getFileName(), $constructor->getStartLine());
90 100
         }
91 101
 
@@ -101,7 +111,8 @@  discard block
 block discarded – undo
101 111
      */
102 112
     private function resolveParameter(\ReflectionClass $ctx, \ReflectionParameter $param, array &$arguments)
103 113
     {
104
-        switch (true) {
114
+        switch (true)
115
+        {
105 116
             case \array_key_exists($param->getName(), $arguments):
106 117
                 $argument =  $arguments[$param->getName()];
107 118
                 unset($arguments[$param->getName()]);
Please login to merge, or discard this patch.
src/Validation/tests/Checkers/DatetimeTest.php 2 patches
Braces   +8 added lines, -4 removed lines patch added patch discarded remove patch
@@ -203,7 +203,8 @@  discard block
 block discarded – undo
203 203
     {
204 204
         $checker = new DatetimeChecker();
205 205
 
206
-        foreach (\DateTimeZone::listIdentifiers() as $identifier) {
206
+        foreach (\DateTimeZone::listIdentifiers() as $identifier)
207
+        {
207 208
             $this->assertTrue($checker->timezone($identifier));
208 209
             $this->assertFalse($checker->timezone(str_rot13($identifier)));
209 210
         }
@@ -338,21 +339,24 @@  discard block
 block discarded – undo
338 339
 
339 340
     private function now(): \Closure
340 341
     {
341
-        return static function () {
342
+        return static function ()
343
+        {
342 344
             return \time();
343 345
         };
344 346
     }
345 347
 
346 348
     private function inFuture(int $seconds): \Closure
347 349
     {
348
-        return static function () use ($seconds) {
350
+        return static function () use ($seconds)
351
+        {
349 352
             return \time() + $seconds;
350 353
         };
351 354
     }
352 355
 
353 356
     private function inPast(int $seconds): \Closure
354 357
     {
355
-        return static function () use ($seconds) {
358
+        return static function () use ($seconds)
359
+        {
356 360
             return \time() - $seconds;
357 361
         };
358 362
     }
Please login to merge, or discard this patch.
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -178,24 +178,24 @@  discard block
 block discarded – undo
178 178
     public function validProvider(): array
179 179
     {
180 180
         return [
181
-            [true, time() - 1000,],
182
-            [true, time(),],
183
-            [true, date('u'),],
184
-            [true, time() + 1000,],
185
-            [true, '',],
186
-            [true, 'tomorrow +2hours',],
187
-            [true, 'yesterday -2hours',],
188
-            [true, 'now',],
189
-            [true, 'now + 1000 seconds',],
190
-            [true, 'now - 1000 seconds',],
191
-            [true, 0,],
192
-            [true, 1.1,],
193
-            [false, [],],
194
-            [false, false,],
195
-            [false, true,],
196
-            [false, null,],
197
-            [false, [],],
198
-            [false, new \stdClass(),],
181
+            [true, time() - 1000, ],
182
+            [true, time(), ],
183
+            [true, date('u'), ],
184
+            [true, time() + 1000, ],
185
+            [true, '', ],
186
+            [true, 'tomorrow +2hours', ],
187
+            [true, 'yesterday -2hours', ],
188
+            [true, 'now', ],
189
+            [true, 'now + 1000 seconds', ],
190
+            [true, 'now - 1000 seconds', ],
191
+            [true, 0, ],
192
+            [true, 1.1, ],
193
+            [false, [], ],
194
+            [false, false, ],
195
+            [false, true, ],
196
+            [false, null, ],
197
+            [false, [], ],
198
+            [false, new \stdClass(), ],
199 199
         ];
200 200
     }
201 201
 
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
     {
204 204
         $checker = new DatetimeChecker();
205 205
 
206
-        foreach (\DateTimeZone::listIdentifiers() as $identifier) {
206
+        foreach (\DateTimeZone::listIdentifiers() as $identifier){
207 207
             $this->assertTrue($checker->timezone($identifier));
208 208
             $this->assertFalse($checker->timezone(str_rot13($identifier)));
209 209
         }
@@ -338,7 +338,7 @@  discard block
 block discarded – undo
338 338
 
339 339
     private function now(): \Closure
340 340
     {
341
-        return static function () {
341
+        return static function (){
342 342
             return \time();
343 343
         };
344 344
     }
Please login to merge, or discard this patch.