Passed
Pull Request — master (#1200)
by Abdul Malik
12:31
created
src/Exceptions/src/Renderer/Highlighter.php 2 patches
Braces   +12 added lines, -6 removed lines patch added patch discarded remove patch
@@ -24,9 +24,11 @@  discard block
 block discarded – undo
24 24
         $lines = \explode("\n", \str_replace("\r\n", "\n", $this->highlight($source)));
25 25
 
26 26
         $result = '';
27
-        foreach ($lines as $number => $code) {
27
+        foreach ($lines as $number => $code)
28
+        {
28 29
             $human = $number + 1;
29
-            if (!empty($around) && ($human < $line - $around || $human >= $line + $around + 1)) {
30
+            if (!empty($around) && ($human < $line - $around || $human >= $line + $around + 1))
31
+            {
30 32
                 //Not included in a range
31 33
                 continue;
32 34
             }
@@ -44,7 +46,8 @@  discard block
 block discarded – undo
44 46
     {
45 47
         $result = '';
46 48
         $previous = [];
47
-        foreach ($this->getTokens($source) as $token) {
49
+        foreach ($this->getTokens($source) as $token)
50
+        {
48 51
             $result .= $this->renderer->token($token, $previous);
49 52
             $previous = $token;
50 53
         }
@@ -60,12 +63,15 @@  discard block
 block discarded – undo
60 63
         $tokens = [];
61 64
         $line = 0;
62 65
 
63
-        foreach (\token_get_all($source) as $token) {
64
-            if (isset($token[2])) {
66
+        foreach (\token_get_all($source) as $token)
67
+        {
68
+            if (isset($token[2]))
69
+            {
65 70
                 $line = $token[2];
66 71
             }
67 72
 
68
-            if (!\is_array($token)) {
73
+            if (!\is_array($token))
74
+            {
69 75
                 $token = [$token, $token, $line];
70 76
             }
71 77
 
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 {
14 14
     public function __construct(
15 15
         private readonly StyleInterface $renderer
16
-    ) {
16
+    ){
17 17
     }
18 18
 
19 19
     /**
@@ -24,9 +24,9 @@  discard block
 block discarded – undo
24 24
         $lines = \explode("\n", \str_replace("\r\n", "\n", $this->highlight($source)));
25 25
 
26 26
         $result = '';
27
-        foreach ($lines as $number => $code) {
27
+        foreach ($lines as $number => $code){
28 28
             $human = $number + 1;
29
-            if (!empty($around) && ($human < $line - $around || $human >= $line + $around + 1)) {
29
+            if (!empty($around) && ($human < $line - $around || $human >= $line + $around + 1)){
30 30
                 //Not included in a range
31 31
                 continue;
32 32
             }
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
     {
45 45
         $result = '';
46 46
         $previous = [];
47
-        foreach ($this->getTokens($source) as $token) {
47
+        foreach ($this->getTokens($source) as $token){
48 48
             $result .= $this->renderer->token($token, $previous);
49 49
             $previous = $token;
50 50
         }
@@ -60,12 +60,12 @@  discard block
 block discarded – undo
60 60
         $tokens = [];
61 61
         $line = 0;
62 62
 
63
-        foreach (\token_get_all($source) as $token) {
64
-            if (isset($token[2])) {
63
+        foreach (\token_get_all($source) as $token){
64
+            if (isset($token[2])){
65 65
                 $line = $token[2];
66 66
             }
67 67
 
68
-            if (!\is_array($token)) {
68
+            if (!\is_array($token)){
69 69
                 $token = [$token, $token, $line];
70 70
             }
71 71
 
Please login to merge, or discard this patch.
src/Exceptions/src/Renderer/JsonRenderer.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,24 +30,24 @@
 block discarded – undo
30 30
 
31 31
     private function renderTrace(array $trace, Verbosity $verbosity): \Generator
32 32
     {
33
-        foreach ($trace as $item) {
33
+        foreach ($trace as $item){
34 34
             $result = [];
35 35
 
36
-            if (isset($item['class'])) {
36
+            if (isset($item['class'])){
37 37
                 $result['function'] = \sprintf(
38 38
                     '%s%s%s()',
39 39
                     $item['class'],
40 40
                     $item['type'],
41 41
                     $item['function']
42 42
                 );
43
-            } else {
43
+            }else{
44 44
                 $result['function'] = \sprintf(
45 45
                     '%s()',
46 46
                     $item['function']
47 47
                 );
48 48
             }
49 49
 
50
-            if ($verbosity->value >= Verbosity::VERBOSE->value && isset($item['file'])) {
50
+            if ($verbosity->value >= Verbosity::VERBOSE->value && isset($item['file'])){
51 51
                 $result['at'] = [
52 52
                     'file' => $item['file'] ?? null,
53 53
                     'line' => $item['line'] ?? null,
Please login to merge, or discard this patch.
Braces   +9 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,24 +30,29 @@
 block discarded – undo
30 30
 
31 31
     private function renderTrace(array $trace, Verbosity $verbosity): \Generator
32 32
     {
33
-        foreach ($trace as $item) {
33
+        foreach ($trace as $item)
34
+        {
34 35
             $result = [];
35 36
 
36
-            if (isset($item['class'])) {
37
+            if (isset($item['class']))
38
+            {
37 39
                 $result['function'] = \sprintf(
38 40
                     '%s%s%s()',
39 41
                     $item['class'],
40 42
                     $item['type'],
41 43
                     $item['function']
42 44
                 );
43
-            } else {
45
+            }
46
+            else
47
+            {
44 48
                 $result['function'] = \sprintf(
45 49
                     '%s()',
46 50
                     $item['function']
47 51
                 );
48 52
             }
49 53
 
50
-            if ($verbosity->value >= Verbosity::VERBOSE->value && isset($item['file'])) {
54
+            if ($verbosity->value >= Verbosity::VERBOSE->value && isset($item['file']))
55
+            {
51 56
                 $result['at'] = [
52 57
                     'file' => $item['file'] ?? null,
53 58
                     'line' => $item['line'] ?? null,
Please login to merge, or discard this patch.
src/Exceptions/src/Renderer/AbstractRenderer.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
     protected function getStacktrace(\Throwable $e): array
28 28
     {
29 29
         $stacktrace = $e->getTrace();
30
-        if (empty($stacktrace)) {
30
+        if (empty($stacktrace)){
31 31
             return [];
32 32
         }
33 33
 
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
             'line' => $e->getLine(),
38 38
         ] + $stacktrace[0];
39 39
 
40
-        if ($stacktrace[0] !== $header) {
40
+        if ($stacktrace[0] !== $header){
41 41
             \array_unshift($stacktrace, $header);
42 42
         }
43 43
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,7 +27,8 @@  discard block
 block discarded – undo
27 27
     protected function getStacktrace(\Throwable $e): array
28 28
     {
29 29
         $stacktrace = $e->getTrace();
30
-        if (empty($stacktrace)) {
30
+        if (empty($stacktrace))
31
+        {
31 32
             return [];
32 33
         }
33 34
 
@@ -37,7 +38,8 @@  discard block
 block discarded – undo
37 38
             'line' => $e->getLine(),
38 39
         ] + $stacktrace[0];
39 40
 
40
-        if ($stacktrace[0] !== $header) {
41
+        if ($stacktrace[0] !== $header)
42
+        {
41 43
             \array_unshift($stacktrace, $header);
42 44
         }
43 45
 
Please login to merge, or discard this patch.
src/Exceptions/src/Style/ConsoleStyle.php 2 patches
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -13,9 +13,9 @@  discard block
 block discarded – undo
13 13
 class ConsoleStyle implements StyleInterface
14 14
 {
15 15
     protected array $templates = [
16
-        'token'  => '%s%s' . Color::RESET,
17
-        'line'   => Color::LIGHT_CYAN . ' %s ' . Color::RESET . " %s\n",
18
-        'active' => Color::BG_RED . ' ' . Color::LIGHT_WHITE . '%s ' . Color::RESET . " %s\n",
16
+        'token'  => '%s%s'.Color::RESET,
17
+        'line'   => Color::LIGHT_CYAN.' %s '.Color::RESET." %s\n",
18
+        'active' => Color::BG_RED.' '.Color::LIGHT_WHITE.'%s '.Color::RESET." %s\n",
19 19
     ];
20 20
 
21 21
     protected array $style = [
@@ -78,8 +78,8 @@  discard block
 block discarded – undo
78 78
             T_VARIABLE,
79 79
         ],
80 80
         Color::LIGHT_YELLOW  => [
81
-            '->' . T_STRING,
82
-            '::' . T_STRING,
81
+            '->'.T_STRING,
82
+            '::'.T_STRING,
83 83
         ],
84 84
     ];
85 85
 
@@ -87,12 +87,12 @@  discard block
 block discarded – undo
87 87
     {
88 88
         $style = $this->getStyle($token, $previous);
89 89
 
90
-        if (!\str_contains((string) $token[1], "\n")) {
90
+        if (!\str_contains((string)$token[1], "\n")){
91 91
             return \sprintf($this->templates['token'], $style, $token[1]);
92 92
         }
93 93
 
94 94
         $lines = [];
95
-        foreach (\explode("\n", (string) $token[1]) as $line) {
95
+        foreach (\explode("\n", (string)$token[1]) as $line){
96 96
             $lines[] = \sprintf($this->templates['token'], $style, $line);
97 97
         }
98 98
 
@@ -113,16 +113,16 @@  discard block
 block discarded – undo
113 113
      */
114 114
     private function getStyle(array $token, array $previous): string
115 115
     {
116
-        if (!empty($previous)) {
117
-            foreach ($this->style as $style => $tokens) {
118
-                if (\in_array($previous[1] . $token[0], $tokens)) {
116
+        if (!empty($previous)){
117
+            foreach ($this->style as $style => $tokens){
118
+                if (\in_array($previous[1].$token[0], $tokens)){
119 119
                     return $style;
120 120
                 }
121 121
             }
122 122
         }
123 123
 
124
-        foreach ($this->style as $style => $tokens) {
125
-            if (\in_array($token[0], $tokens)) {
124
+        foreach ($this->style as $style => $tokens){
125
+            if (\in_array($token[0], $tokens)){
126 126
                 return $style;
127 127
             }
128 128
         }
Please login to merge, or discard this patch.
Braces   +14 added lines, -7 removed lines patch added patch discarded remove patch
@@ -87,12 +87,14 @@  discard block
 block discarded – undo
87 87
     {
88 88
         $style = $this->getStyle($token, $previous);
89 89
 
90
-        if (!\str_contains((string) $token[1], "\n")) {
90
+        if (!\str_contains((string) $token[1], "\n"))
91
+        {
91 92
             return \sprintf($this->templates['token'], $style, $token[1]);
92 93
         }
93 94
 
94 95
         $lines = [];
95
-        foreach (\explode("\n", (string) $token[1]) as $line) {
96
+        foreach (\explode("\n", (string) $token[1]) as $line)
97
+        {
96 98
             $lines[] = \sprintf($this->templates['token'], $style, $line);
97 99
         }
98 100
 
@@ -113,16 +115,21 @@  discard block
 block discarded – undo
113 115
      */
114 116
     private function getStyle(array $token, array $previous): string
115 117
     {
116
-        if (!empty($previous)) {
117
-            foreach ($this->style as $style => $tokens) {
118
-                if (\in_array($previous[1] . $token[0], $tokens)) {
118
+        if (!empty($previous))
119
+        {
120
+            foreach ($this->style as $style => $tokens)
121
+            {
122
+                if (\in_array($previous[1] . $token[0], $tokens))
123
+                {
119 124
                     return $style;
120 125
                 }
121 126
             }
122 127
         }
123 128
 
124
-        foreach ($this->style as $style => $tokens) {
125
-            if (\in_array($token[0], $tokens)) {
129
+        foreach ($this->style as $style => $tokens)
130
+        {
131
+            if (\in_array($token[0], $tokens))
132
+            {
126 133
                 return $style;
127 134
             }
128 135
         }
Please login to merge, or discard this patch.
src/Exceptions/src/Style/PlainStyle.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
 
19 19
     public function line(int $number, string $code, bool $target = false): string
20 20
     {
21
-        if ($target) {
21
+        if ($target){
22 22
             return \sprintf(
23 23
                 ">%s %s\n",
24 24
                 \str_pad((string)$number, 4, ' ', STR_PAD_LEFT),
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,8 @@
 block discarded – undo
18 18
 
19 19
     public function line(int $number, string $code, bool $target = false): string
20 20
     {
21
-        if ($target) {
21
+        if ($target)
22
+        {
22 23
             return \sprintf(
23 24
                 ">%s %s\n",
24 25
                 \str_pad((string)$number, 4, ' ', STR_PAD_LEFT),
Please login to merge, or discard this patch.
src/Reactor/tests/ClassDeclarationTest.php 2 patches
Upper-Lower-Casing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -42,15 +42,15 @@
 block discarded – undo
42 42
 
43 43
         $declaration
44 44
             ->addConstant('PRIVATE', 123)
45
-            ->setVisibility(Partial\Visibility::PRIVATE);
45
+            ->setVisibility(Partial\Visibility::private);
46 46
 
47 47
         $declaration
48 48
             ->addConstant('PROTECTED', 456)
49
-            ->setVisibility(Partial\Visibility::PROTECTED);
49
+            ->setVisibility(Partial\Visibility::protected);
50 50
 
51 51
         $declaration
52 52
             ->addConstant('PUBLIC', 789)
53
-            ->setVisibility(Partial\Visibility::PUBLIC);
53
+            ->setVisibility(Partial\Visibility::public);
54 54
 
55 55
         $declaration
56 56
             ->addConstant('WITH_COMMENT', 'foo')
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
             */
29 29
             final class MyClass extends Spiral\Tests\Reactor\ClassDeclarationTest implements Countable
30 30
             {
31
-            }'), preg_replace('/\s+/', '', (string) $declaration));
31
+            }'), preg_replace('/\s+/', '', (string)$declaration));
32 32
     }
33 33
 
34 34
     public function testClassDeclarationWithConstants(): void
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 
76 76
                 #[Foo\Cached(mode: true)]
77 77
                 public const WITH_ATTRIBUTE = \'attr\';
78
-            }'), preg_replace('/\s+/', '', (string) $declaration));
78
+            }'), preg_replace('/\s+/', '', (string)$declaration));
79 79
     }
80 80
 
81 81
     public function testClassDeclarationWithMethods(): void
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
             {
107 107
                 return count($items ?: $this->items);
108 108
             }
109
-        }'), preg_replace('/\s+/', '', (string) $declaration));
109
+        }'), preg_replace('/\s+/', '', (string)$declaration));
110 110
     }
111 111
 
112 112
     public function testName(): void
Please login to merge, or discard this patch.
src/Reactor/src/Traits/FunctionLike.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
     {
58 58
         $param = Parameter::fromElement($this->element->addParameter($name));
59 59
 
60
-        if (\func_num_args() > 1) {
60
+        if (\func_num_args() > 1){
61 61
             $param->setDefaultValue($defaultValue);
62 62
         }
63 63
 
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
     {
98 98
         $type = $this->element->getReturnType(false);
99 99
 
100
-        return $type === null ? null : (string) $type;
100
+        return $type === null ? null : (string)$type;
101 101
     }
102 102
 
103 103
     public function setReturnReference(bool $state = true): static
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -57,7 +57,8 @@
 block discarded – undo
57 57
     {
58 58
         $param = Parameter::fromElement($this->element->addParameter($name));
59 59
 
60
-        if (\func_num_args() > 1) {
60
+        if (\func_num_args() > 1)
61
+        {
61 62
             $param->setDefaultValue($defaultValue);
62 63
         }
63 64
 
Please login to merge, or discard this patch.
src/Reactor/src/Traits/CommentAware.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -9,10 +9,10 @@
 block discarded – undo
9 9
  */
10 10
 trait CommentAware
11 11
 {
12
-    public function setComment(array|string|null $comment): static
12
+    public function setComment(array | string | null $comment): static
13 13
     {
14
-        if (\is_array($comment)) {
15
-            foreach ($comment as $value) {
14
+        if (\is_array($comment)){
15
+            foreach ($comment as $value){
16 16
                 $this->element->addComment($value);
17 17
             }
18 18
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -11,8 +11,10 @@
 block discarded – undo
11 11
 {
12 12
     public function setComment(array|string|null $comment): static
13 13
     {
14
-        if (\is_array($comment)) {
15
-            foreach ($comment as $value) {
14
+        if (\is_array($comment))
15
+        {
16
+            foreach ($comment as $value)
17
+            {
16 18
                 $this->element->addComment($value);
17 19
             }
18 20
 
Please login to merge, or discard this patch.
src/Reactor/src/Traits/PropertiesAware.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
     /** @param string $name without $ */
40 40
     public function addProperty(string $name, mixed $value = null): Property
41 41
     {
42
-        if (\func_num_args() > 1) {
42
+        if (\func_num_args() > 1){
43 43
             return Property::fromElement($this->element->addProperty($name, $value));
44 44
         }
45 45
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,8 @@
 block discarded – undo
39 39
     /** @param string $name without $ */
40 40
     public function addProperty(string $name, mixed $value = null): Property
41 41
     {
42
-        if (\func_num_args() > 1) {
42
+        if (\func_num_args() > 1)
43
+        {
43 44
             return Property::fromElement($this->element->addProperty($name, $value));
44 45
         }
45 46
 
Please login to merge, or discard this patch.