Passed
Pull Request — master (#523)
by Maurício
13:38
created
src/Components/Expression.php 1 patch
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -112,10 +112,10 @@  discard block
 block discarded – undo
112 112
      * @param string|null $alias    the name of the alias
113 113
      */
114 114
     public function __construct(
115
-        string|null $database = null,
116
-        string|null $table = null,
117
-        string|null $column = null,
118
-        string|null $alias = null,
115
+        string | null $database = null,
116
+        string | null $table = null,
117
+        string | null $column = null,
118
+        string | null $alias = null,
119 119
     ) {
120 120
         if (($column === null) && ($alias === null)) {
121 121
             $this->expr = $database; // case 1
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
      *
161 161
      * @throws ParserException
162 162
      */
163
-    public static function parse(Parser $parser, TokensList $list, array $options = []): Expression|null
163
+    public static function parse(Parser $parser, TokensList $list, array $options = []): Expression | null
164 164
     {
165 165
         $ret = new static();
166 166
 
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
         ];
204 204
 
205 205
         // When a field is parsed, no parentheses are expected.
206
-        if (! empty($options['parseField'])) {
206
+        if (!empty($options['parseField'])) {
207 207
             $options['breakOnParentheses'] = true;
208 208
             $options['field'] = $options['parseField'];
209 209
         }
@@ -230,14 +230,14 @@  discard block
 block discarded – undo
230 230
             }
231 231
 
232 232
             if ($token->type === TokenType::Keyword) {
233
-                if (($brackets > 0) && empty($ret->subquery) && ! empty(Parser::STATEMENT_PARSERS[$token->keyword])) {
233
+                if (($brackets > 0) && empty($ret->subquery) && !empty(Parser::STATEMENT_PARSERS[$token->keyword])) {
234 234
                     // A `(` was previously found and this keyword is the
235 235
                     // beginning of a statement, so this is a subquery.
236 236
                     $ret->subquery = $token->keyword;
237 237
                 } elseif (
238 238
                     ($token->flags & Token::FLAG_KEYWORD_FUNCTION)
239 239
                     && (empty($options['parseField'])
240
-                    && ! $alias)
240
+                    && !$alias)
241 241
                 ) {
242 242
                     $isExpr = true;
243 243
                 } elseif (($token->flags & Token::FLAG_KEYWORD_RESERVED) && ($brackets === 0)) {
@@ -249,7 +249,7 @@  discard block
 block discarded – undo
249 249
                     }
250 250
 
251 251
                     if ($token->keyword === 'AS') {
252
-                        if (! empty($options['breakOnAlias'])) {
252
+                        if (!empty($options['breakOnAlias'])) {
253 253
                             break;
254 254
                         }
255 255
 
@@ -272,7 +272,7 @@  discard block
 block discarded – undo
272 272
                     }
273 273
 
274 274
                     $isExpr = true;
275
-                } elseif ($brackets === 0 && strlen((string) $ret->expr) > 0 && ! $alias) {
275
+                } elseif ($brackets === 0 && strlen((string) $ret->expr) > 0 && !$alias) {
276 276
                     /* End of expression */
277 277
                     break;
278 278
                 }
@@ -288,7 +288,7 @@  discard block
 block discarded – undo
288 288
                 || (($token->type === TokenType::Operator)
289 289
                 && ($token->value !== '.'))
290 290
             ) {
291
-                if (! empty($options['parseField'])) {
291
+                if (!empty($options['parseField'])) {
292 292
                     break;
293 293
                 }
294 294
 
@@ -298,7 +298,7 @@  discard block
 block discarded – undo
298 298
             }
299 299
 
300 300
             if ($token->type === TokenType::Operator) {
301
-                if (! empty($options['breakOnParentheses']) && (($token->value === '(') || ($token->value === ')'))) {
301
+                if (!empty($options['breakOnParentheses']) && (($token->value === '(') || ($token->value === ')'))) {
302 302
                     // No brackets were expected.
303 303
                     break;
304 304
                 }
@@ -322,7 +322,7 @@  discard block
 block discarded – undo
322 322
 
323 323
                     --$brackets;
324 324
                     if ($brackets === 0) {
325
-                        if (! empty($options['parenthesesDelimited'])) {
325
+                        if (!empty($options['parenthesesDelimited'])) {
326 326
                             // The current token is the last bracket, the next
327 327
                             // one will be outside the expression.
328 328
                             $ret->expr .= $token->token;
@@ -348,7 +348,7 @@  discard block
 block discarded – undo
348 348
 
349 349
             if ($alias) {
350 350
                 // An alias is expected (the keyword `AS` was previously found).
351
-                if (! empty($ret->alias)) {
351
+                if (!empty($ret->alias)) {
352 352
                     $parser->error('An alias was previously found.', $token);
353 353
                     break;
354 354
                 }
@@ -362,15 +362,15 @@  discard block
 block discarded – undo
362 362
                     && ($prev[0] === null
363 363
                         || (($prev[0]->type !== TokenType::Operator || $prev[0]->token === ')')
364 364
                             && ($prev[0]->type !== TokenType::Keyword
365
-                                || ! ($prev[0]->flags & Token::FLAG_KEYWORD_RESERVED))))
365
+                                || !($prev[0]->flags & Token::FLAG_KEYWORD_RESERVED))))
366 366
                     && (($prev[1]->type === TokenType::String)
367 367
                         || ($prev[1]->type === TokenType::Symbol
368
-                            && ! ($prev[1]->flags & Token::FLAG_SYMBOL_VARIABLE)
369
-                            && ! ($prev[1]->flags & Token::FLAG_SYMBOL_PARAMETER))
368
+                            && !($prev[1]->flags & Token::FLAG_SYMBOL_VARIABLE)
369
+                            && !($prev[1]->flags & Token::FLAG_SYMBOL_PARAMETER))
370 370
                         || ($prev[1]->type === TokenType::None
371 371
                             && $prev[1]->token !== 'OVER'))
372 372
                 ) {
373
-                    if (! empty($ret->alias)) {
373
+                    if (!empty($ret->alias)) {
374 374
                         $parser->error('An alias was previously found.', $token);
375 375
                         break;
376 376
                     }
@@ -401,7 +401,7 @@  discard block
 block discarded – undo
401 401
                     // Found a `.` which means we expect a column name and
402 402
                     // the column name we parsed is actually the table name
403 403
                     // and the table name is actually a database name.
404
-                    if (! empty($ret->database) || $dot) {
404
+                    if (!empty($ret->database) || $dot) {
405 405
                         $parser->error('Unexpected dot.', $token);
406 406
                     }
407 407
 
@@ -418,11 +418,11 @@  discard block
 block discarded – undo
418 418
                         $dot = false;
419 419
                     } else {
420 420
                         // No alias is expected.
421
-                        if (! empty($options['breakOnAlias'])) {
421
+                        if (!empty($options['breakOnAlias'])) {
422 422
                             break;
423 423
                         }
424 424
 
425
-                        if (! empty($ret->alias)) {
425
+                        if (!empty($ret->alias)) {
426 426
                             $parser->error('An alias was previously found.', $token);
427 427
                             break;
428 428
                         }
@@ -470,7 +470,7 @@  discard block
 block discarded – undo
470 470
             $ret = implode('.', Context::escapeAll($fields));
471 471
         }
472 472
 
473
-        if (! empty($this->alias)) {
473
+        if (!empty($this->alias)) {
474 474
             $ret .= ' AS ' . Context::escape($this->alias);
475 475
         }
476 476
 
Please login to merge, or discard this patch.
src/Components/Key.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -105,10 +105,10 @@  discard block
 block discarded – undo
105 105
      * @phpstan-param array{name?: string, length?: int, order?: string}[] $columns
106 106
      */
107 107
     public function __construct(
108
-        string|null $name = null,
108
+        string | null $name = null,
109 109
         array $columns = [],
110
-        string|null $type = null,
111
-        OptionsArray|null $options = null,
110
+        string | null $type = null,
111
+        OptionsArray | null $options = null,
112 112
     ) {
113 113
         $this->name = $name;
114 114
         $this->columns = $columns;
@@ -174,9 +174,9 @@  discard block
 block discarded – undo
174 174
             } elseif ($state === 1) {
175 175
                 if (($token->type === TokenType::Operator) && ($token->value === '(')) {
176 176
                     $positionBeforeSearch = $list->idx;
177
-                    $list->idx++;// Ignore the current token "(" or the search condition will always be true
177
+                    $list->idx++; // Ignore the current token "(" or the search condition will always be true
178 178
                     $nextToken = $list->getNext();
179
-                    $list->idx = $positionBeforeSearch;// Restore the position
179
+                    $list->idx = $positionBeforeSearch; // Restore the position
180 180
 
181 181
                     if ($nextToken !== null && $nextToken->value === '(') {
182 182
                         // Switch to expression mode
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
                 if ($token->type === TokenType::Operator) {
226 226
                     // This got back to here and we reached the end of the expression
227 227
                     if ($token->value === ')') {
228
-                        $state = 4;// go back to state 4 to fetch options
228
+                        $state = 4; // go back to state 4 to fetch options
229 229
                         continue;
230 230
                     }
231 231
 
@@ -261,7 +261,7 @@  discard block
 block discarded – undo
261 261
     public function build(): string
262 262
     {
263 263
         $ret = $this->type . ' ';
264
-        if (! empty($this->name)) {
264
+        if (!empty($this->name)) {
265 265
             $ret .= Context::escape($this->name) . ' ';
266 266
         }
267 267
 
Please login to merge, or discard this patch.
src/Components/PartitionDefinition.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -231,7 +231,7 @@
 block discarded – undo
231 231
         return trim(
232 232
             'PARTITION ' . $this->name
233 233
             . (empty($this->type) ? '' : ' VALUES ' . $this->type . ' ' . $this->expr . ' ')
234
-            . (! empty($this->options) && ! empty($this->type) ? '' : ' ')
234
+            . (!empty($this->options) && !empty($this->type) ? '' : ' ')
235 235
             . $this->options . $subpartitions,
236 236
         );
237 237
     }
Please login to merge, or discard this patch.
src/Components/GroupKeyword.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
 final class GroupKeyword implements Component
19 19
 {
20 20
     /** @var 'ASC'|'DESC'|null */
21
-    public string|null $type = null;
21
+    public string | null $type = null;
22 22
 
23 23
     /**
24 24
      * The expression that is used for grouping.
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
     public $expr;
29 29
 
30 30
     /** @param Expression $expr the expression that we are sorting by */
31
-    public function __construct(Expression|null $expr = null)
31
+    public function __construct(Expression | null $expr = null)
32 32
     {
33 33
         $this->expr = $expr;
34 34
     }
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
                 ) {
87 87
                     $expr->type = $token->keyword;
88 88
                 } elseif (($token->type === TokenType::Operator) && ($token->value === ',')) {
89
-                    if (! empty($expr->expr)) {
89
+                    if (!empty($expr->expr)) {
90 90
                         $ret[] = $expr;
91 91
                     }
92 92
 
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
         }
100 100
 
101 101
         // Last iteration was not processed.
102
-        if (! empty($expr->expr)) {
102
+        if (!empty($expr->expr)) {
103 103
             $ret[] = $expr;
104 104
         }
105 105
 
Please login to merge, or discard this patch.
src/Components/CreateDefinition.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -150,11 +150,11 @@  discard block
 block discarded – undo
150 150
      * @param Reference|null    $references   references
151 151
      */
152 152
     public function __construct(
153
-        string|null $name = null,
154
-        OptionsArray|null $options = null,
155
-        DataType|Key|null $type = null,
153
+        string | null $name = null,
154
+        OptionsArray | null $options = null,
155
+        DataType | Key | null $type = null,
156 156
         bool $isConstraint = false,
157
-        Reference|null $references = null,
157
+        Reference | null $references = null,
158 158
     ) {
159 159
         $this->name = $name;
160 160
         $this->options = $options;
@@ -236,7 +236,7 @@  discard block
 block discarded – undo
236 236
                     $state = 4;
237 237
                 } elseif ($token->type === TokenType::Symbol || $token->type === TokenType::None) {
238 238
                     $expr->name = $token->value;
239
-                    if (! $expr->isConstraint) {
239
+                    if (!$expr->isConstraint) {
240 240
                         $state = 2;
241 241
                     }
242 242
                 } elseif ($token->type === TokenType::Keyword) {
@@ -277,7 +277,7 @@  discard block
 block discarded – undo
277 277
 
278 278
                 $state = 5;
279 279
             } elseif ($state === 5) {
280
-                if (! empty($expr->type) || ! empty($expr->key)) {
280
+                if (!empty($expr->type) || !empty($expr->key)) {
281 281
                     $ret[] = $expr;
282 282
                 }
283 283
 
@@ -297,7 +297,7 @@  discard block
 block discarded – undo
297 297
         }
298 298
 
299 299
         // Last iteration was not saved.
300
-        if (! empty($expr->type) || ! empty($expr->key)) {
300
+        if (!empty($expr->type) || !empty($expr->key)) {
301 301
             $ret[] = $expr;
302 302
         }
303 303
 
@@ -322,16 +322,16 @@  discard block
 block discarded – undo
322 322
             $tmp .= Context::escape($this->name) . ' ';
323 323
         }
324 324
 
325
-        if (! empty($this->type)) {
325
+        if (!empty($this->type)) {
326 326
             $this->type->lowercase = true;
327 327
             $tmp .= $this->type->build() . ' ';
328 328
         }
329 329
 
330
-        if (! empty($this->key)) {
330
+        if (!empty($this->key)) {
331 331
             $tmp .= $this->key . ' ';
332 332
         }
333 333
 
334
-        if (! empty($this->references)) {
334
+        if (!empty($this->references)) {
335 335
             $tmp .= 'REFERENCES ' . $this->references . ' ';
336 336
         }
337 337
 
Please login to merge, or discard this patch.
src/Components/AlterOperation.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -264,9 +264,9 @@  discard block
 block discarded – undo
264 264
      * @param Token[]                    $unknown    unparsed tokens found at the end of operation
265 265
      */
266 266
     public function __construct(
267
-        OptionsArray|null $options = null,
268
-        Expression|string|null $field = null,
269
-        array|null $partitions = null,
267
+        OptionsArray | null $options = null,
268
+        Expression | string | null $field = null,
269
+        array | null $partitions = null,
270 270
         public array $unknown = [],
271 271
     ) {
272 272
         $this->partitions = $partitions;
@@ -401,7 +401,7 @@  discard block
 block discarded – undo
401 401
                     } elseif (($token->value === ',') && ($brackets === 0)) {
402 402
                         break;
403 403
                     }
404
-                } elseif (! self::checkIfTokenQuotedSymbol($token) && $token->type !== TokenType::String) {
404
+                } elseif (!self::checkIfTokenQuotedSymbol($token) && $token->type !== TokenType::String) {
405 405
                     if (isset(Parser::STATEMENT_PARSERS[$arrayKey]) && Parser::STATEMENT_PARSERS[$arrayKey] !== '') {
406 406
                         $list->idx++; // Ignore the current token
407 407
                         $nextToken = $list->getNext();
@@ -424,7 +424,7 @@  discard block
 block discarded – undo
424 424
                     } elseif (
425 425
                         (array_key_exists($arrayKey, self::DATABASE_OPTIONS)
426 426
                         || array_key_exists($arrayKey, self::TABLE_OPTIONS))
427
-                        && ! self::checkIfColumnDefinitionKeyword($arrayKey)
427
+                        && !self::checkIfColumnDefinitionKeyword($arrayKey)
428 428
                     ) {
429 429
                         // This alter operation has finished, which means a comma
430 430
                         // was missing before start of new alter operation
Please login to merge, or discard this patch.