Passed
Pull Request — master (#471)
by
unknown
09:17
created
src/Statements/CreateStatement.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -417,7 +417,7 @@  discard block
 block discarded – undo
417 417
     public function build()
418 418
     {
419 419
         $fields = '';
420
-        if (! empty($this->fields)) {
420
+        if (!empty($this->fields)) {
421 421
             if (is_array($this->fields)) {
422 422
                 $fields = CreateDefinition::build($this->fields) . ' ';
423 423
             } elseif ($this->fields instanceof ArrayObj) {
@@ -449,23 +449,23 @@  discard block
 block discarded – undo
449 449
 
450 450
             $partition = '';
451 451
 
452
-            if (! empty($this->partitionBy)) {
452
+            if (!empty($this->partitionBy)) {
453 453
                 $partition .= "\nPARTITION BY " . $this->partitionBy;
454 454
             }
455 455
 
456
-            if (! empty($this->partitionsNum)) {
456
+            if (!empty($this->partitionsNum)) {
457 457
                 $partition .= "\nPARTITIONS " . $this->partitionsNum;
458 458
             }
459 459
 
460
-            if (! empty($this->subpartitionBy)) {
460
+            if (!empty($this->subpartitionBy)) {
461 461
                 $partition .= "\nSUBPARTITION BY " . $this->subpartitionBy;
462 462
             }
463 463
 
464
-            if (! empty($this->subpartitionsNum)) {
464
+            if (!empty($this->subpartitionsNum)) {
465 465
                 $partition .= "\nSUBPARTITIONS " . $this->subpartitionsNum;
466 466
             }
467 467
 
468
-            if (! empty($this->partitions)) {
468
+            if (!empty($this->partitions)) {
469 469
                 $partition .= "\n" . PartitionDefinition::build($this->partitions);
470 470
             }
471 471
 
@@ -487,7 +487,7 @@  discard block
 block discarded – undo
487 487
                 . OptionsArray::build($this->options) . ' '
488 488
                 . Expression::build($this->name) . ' '
489 489
                 . $fields . ' AS ' . $builtStatement
490
-                . (! empty($this->body) ? TokensList::build($this->body) : '') . ' '
490
+                . (!empty($this->body) ? TokensList::build($this->body) : '') . ' '
491 491
                 . OptionsArray::build($this->entityOptions);
492 492
         } elseif ($this->options->has('TRIGGER')) {
493 493
             return 'CREATE '
@@ -541,7 +541,7 @@  discard block
 block discarded – undo
541 541
             ]
542 542
         );
543 543
 
544
-        if (! isset($this->name) || ($this->name === '')) {
544
+        if (!isset($this->name) || ($this->name === '')) {
545 545
             $parser->error('The name of the entity was expected.', $list->tokens[$list->idx]);
546 546
         } else {
547 547
             ++$list->idx; // Skipping field.
@@ -657,7 +657,7 @@  discard block
 block discarded – undo
657 657
                         $token = $list->getNextOfType(Token::TYPE_NUMBER);
658 658
                         --$list->idx; // `getNextOfType` also advances one position.
659 659
                         $this->subpartitionsNum = $token->value;
660
-                    } elseif (! empty($field)) {
660
+                    } elseif (!empty($field)) {
661 661
                         /*
662 662
                          * Handling the content of `PARTITION BY` and `SUBPARTITION BY`.
663 663
                          */
@@ -686,7 +686,7 @@  discard block
 block discarded – undo
686 686
                             $field = null;
687 687
                         }
688 688
                     } elseif (($token->type === Token::TYPE_OPERATOR) && ($token->value === '(')) {
689
-                        if (! empty($this->partitionBy)) {
689
+                        if (!empty($this->partitionBy)) {
690 690
                             $this->partitions = ArrayObj::parse(
691 691
                                 $parser,
692 692
                                 $list,
Please login to merge, or discard this patch.
src/Statements/WithStatement.php 2 patches
Indentation   +3 added lines, -4 removed lines patch added patch discarded remove patch
@@ -22,7 +22,6 @@  discard block
 block discarded – undo
22 22
 
23 23
 /**
24 24
  * `WITH` statement.
25
-
26 25
  *  WITH [RECURSIVE] query_name [ (column_name [,...]) ] AS (SELECT ...) [, ...]
27 26
  */
28 27
 final class WithStatement extends Statement
@@ -263,9 +262,9 @@  discard block
 block discarded – undo
263 262
 
264 263
         // 5 is the only valid end state
265 264
         if ($state !== 5) {
266
-             /**
267
-             * Token parsed at this moment.
268
-             */
265
+                /**
266
+                 * Token parsed at this moment.
267
+                 */
269 268
             $token = $list->tokens[$list->idx];
270 269
 
271 270
             $parser->error('Unexpected end of the WITH CTE.', $token);
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
                     break;
134 134
                 }
135 135
             } elseif ($state === 2) {
136
-                if (! ($token->type === Token::TYPE_KEYWORD && $token->keyword === 'AS')) {
136
+                if (!($token->type === Token::TYPE_KEYWORD && $token->keyword === 'AS')) {
137 137
                     $parser->error('AS keyword was expected.', $token);
138 138
                     break;
139 139
                 }
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
                 $list->idx++; // Ignore the current token
146 146
                 $nextKeyword = $list->getNext();
147 147
 
148
-                if (! ($token->value === '(' && ($nextKeyword && $nextKeyword->value === 'SELECT'))) {
148
+                if (!($token->value === '(' && ($nextKeyword && $nextKeyword->value === 'SELECT'))) {
149 149
                     $parser->error('Subquery of the CTE was expected.', $token);
150 150
                     $list->idx = $idxBeforeGetNext;
151 151
                     break;
@@ -295,7 +295,7 @@  discard block
 block discarded – undo
295 295
     /**
296 296
      * Get tokens within the WITH expression to use them in another parser
297 297
      */
298
-    private function getSubTokenList(TokensList $list): ParserException|TokensList
298
+    private function getSubTokenList(TokensList $list): ParserException | TokensList
299 299
     {
300 300
         $idx = $list->idx;
301 301
         $token = $list->tokens[$list->idx];
@@ -311,7 +311,7 @@  discard block
 block discarded – undo
311 311
             }
312 312
 
313 313
             ++$list->idx;
314
-            if (! isset($list->tokens[$list->idx])) {
314
+            if (!isset($list->tokens[$list->idx])) {
315 315
                 break;
316 316
             }
317 317
 
Please login to merge, or discard this patch.
tests/Tools/ContextGeneratorTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@
 block discarded – undo
42 42
         $this->assertEquals([
43 43
             Token::TYPE_KEYWORD | Token::FLAG_KEYWORD_RESERVED => [
44 44
                 8 => ['RESERVED'],
45
-                9 => ['RESERVED2','RESERVED3','RESERVED4','RESERVED5'],
45
+                9 => ['RESERVED2', 'RESERVED3', 'RESERVED4', 'RESERVED5'],
46 46
             ],
47 47
             Token::TYPE_KEYWORD | Token::FLAG_KEYWORD_FUNCTION => [8 => ['FUNCTION']],
48 48
             Token::TYPE_KEYWORD | Token::FLAG_KEYWORD_DATA_TYPE => [8 => ['DATATYPE']],
Please login to merge, or discard this patch.
tests/benchmarks/UtfStringBench.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@
 block discarded – undo
26 26
     {
27 27
         $str1 = new UtfString($this->testContents);
28 28
         for ($i = 0; $i < $str1->length(); $i++) {
29
-            $str1[$i];// Make offset offsetGet work
29
+            $str1[$i]; // Make offset offsetGet work
30 30
         }
31 31
     }
32 32
 
Please login to merge, or discard this patch.
tests/Parser/ParserLongExportsTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
         foreach ($parser->statements as $statement) {
65 65
             if ($statement instanceof TransactionStatement) {
66 66
                 foreach ($statement->statements as $transactionStatement) {
67
-                    if (! $transactionStatement instanceof SetStatement) {
67
+                    if (!$transactionStatement instanceof SetStatement) {
68 68
                         continue;
69 69
                     }
70 70
 
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
                 continue;
75 75
             }
76 76
 
77
-            if (! $statement instanceof SetStatement) {
77
+            if (!$statement instanceof SetStatement) {
78 78
                 continue;
79 79
             }
80 80
 
Please login to merge, or discard this patch.
src/Tools/TestGenerator.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
         // The code below extracts only the relevant information.
84 84
 
85 85
         // Extracting lexer's errors.
86
-        if (! empty($lexer->errors)) {
86
+        if (!empty($lexer->errors)) {
87 87
             /** @var LexerException $err */
88 88
             foreach ($lexer->errors as $err) {
89 89
                 $lexerErrors[] = [
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
         }
99 99
 
100 100
         // Extracting parser's errors.
101
-        if (! empty($parser->errors)) {
101
+        if (!empty($parser->errors)) {
102 102
             /** @var ParserException $err */
103 103
             foreach ($parser->errors as $err) {
104 104
                 $parserErrors[] = [
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
     public static function build($type, $input, $output, $debug = null, $ansi = false)
139 139
     {
140 140
         // Support query types: `lexer` / `parser`.
141
-        if (! in_array($type, ['lexer', 'parser'])) {
141
+        if (!in_array($type, ['lexer', 'parser'])) {
142 142
             throw new Exception('Unknown test type (expected `lexer` or `parser`).');
143 143
         }
144 144
 
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
         );
184 184
 
185 185
         // Remove the project path from .out file, it changes for each dev
186
-        $projectFolder = dirname(__DIR__, 2);// Jump to root
186
+        $projectFolder = dirname(__DIR__, 2); // Jump to root
187 187
         $encoded = str_replace($projectFolder, '<project-root>', $encoded);
188 188
 
189 189
         file_put_contents($output, $encoded);
@@ -223,11 +223,11 @@  discard block
 block discarded – undo
223 223
             if (is_dir($inputFile)) {
224 224
                 // Creating required directories to maintain the structure.
225 225
                 // Ignoring errors if the folder structure exists already.
226
-                if (! is_dir($outputFile)) {
226
+                if (!is_dir($outputFile)) {
227 227
                     mkdir($outputFile);
228 228
                 }
229 229
 
230
-                if (($debug !== null) && (! is_dir($debugFile))) {
230
+                if (($debug !== null) && (!is_dir($debugFile))) {
231 231
                     mkdir($debugFile);
232 232
                 }
233 233
 
@@ -242,7 +242,7 @@  discard block
 block discarded – undo
242 242
                 }
243 243
 
244 244
                 // Building the test.
245
-                if (! file_exists($outputFile)) {
245
+                if (!file_exists($outputFile)) {
246 246
                     echo sprintf("Building test for %s...\n", $inputFile);
247 247
                     static::build(
248 248
                         str_contains($inputFile, 'lex') ? 'lexer' : 'parser',
Please login to merge, or discard this patch.
src/Translator.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@
 block discarded – undo
67 67
      */
68 68
     public static function gettext($msgid)
69 69
     {
70
-        if (! class_exists(Loader::class, true)) {
70
+        if (!class_exists(Loader::class, true)) {
71 71
             return $msgid;
72 72
         }
73 73
 
Please login to merge, or discard this patch.
src/Utils/Query.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -292,7 +292,7 @@  discard block
 block discarded – undo
292 292
         $flags['querytype'] = 'SELECT';
293 293
         $flags['is_select'] = true;
294 294
 
295
-        if (! empty($statement->from)) {
295
+        if (!empty($statement->from)) {
296 296
             $flags['select_from'] = true;
297 297
         }
298 298
 
@@ -300,23 +300,23 @@  discard block
 block discarded – undo
300 300
             $flags['distinct'] = true;
301 301
         }
302 302
 
303
-        if (! empty($statement->group) || ! empty($statement->having)) {
303
+        if (!empty($statement->group) || !empty($statement->having)) {
304 304
             $flags['is_group'] = true;
305 305
         }
306 306
 
307
-        if (! empty($statement->into) && ($statement->into->type === 'OUTFILE')) {
307
+        if (!empty($statement->into) && ($statement->into->type === 'OUTFILE')) {
308 308
             $flags['is_export'] = true;
309 309
         }
310 310
 
311 311
         $expressions = $statement->expr;
312
-        if (! empty($statement->join)) {
312
+        if (!empty($statement->join)) {
313 313
             foreach ($statement->join as $join) {
314 314
                 $expressions[] = $join->expr;
315 315
             }
316 316
         }
317 317
 
318 318
         foreach ($expressions as $expr) {
319
-            if (! empty($expr->function)) {
319
+            if (!empty($expr->function)) {
320 320
                 if ($expr->function === 'COUNT') {
321 321
                     $flags['is_count'] = true;
322 322
                 } elseif (in_array($expr->function, static::$functions)) {
@@ -331,23 +331,23 @@  discard block
 block discarded – undo
331 331
             $flags['is_subquery'] = true;
332 332
         }
333 333
 
334
-        if (! empty($statement->procedure) && ($statement->procedure->name === 'ANALYSE')) {
334
+        if (!empty($statement->procedure) && ($statement->procedure->name === 'ANALYSE')) {
335 335
             $flags['is_analyse'] = true;
336 336
         }
337 337
 
338
-        if (! empty($statement->group)) {
338
+        if (!empty($statement->group)) {
339 339
             $flags['group'] = true;
340 340
         }
341 341
 
342
-        if (! empty($statement->having)) {
342
+        if (!empty($statement->having)) {
343 343
             $flags['having'] = true;
344 344
         }
345 345
 
346
-        if (! empty($statement->union)) {
346
+        if (!empty($statement->union)) {
347 347
             $flags['union'] = true;
348 348
         }
349 349
 
350
-        if (! empty($statement->join)) {
350
+        if (!empty($statement->join)) {
351 351
             $flags['join'] = true;
352 352
         }
353 353
 
@@ -438,11 +438,11 @@  discard block
 block discarded – undo
438 438
             || ($statement instanceof UpdateStatement)
439 439
             || ($statement instanceof DeleteStatement)
440 440
         ) {
441
-            if (! empty($statement->limit)) {
441
+            if (!empty($statement->limit)) {
442 442
                 $flags['limit'] = true;
443 443
             }
444 444
 
445
-            if (! empty($statement->order)) {
445
+            if (!empty($statement->order)) {
446 446
                 $flags['order'] = true;
447 447
             }
448 448
         }
@@ -492,7 +492,7 @@  discard block
 block discarded – undo
492 492
             // Finding tables' aliases and their associated real names.
493 493
             $tableAliases = [];
494 494
             foreach ($statement->from as $expr) {
495
-                if (! isset($expr->table, $expr->alias) || ($expr->table === '') || ($expr->alias === '')) {
495
+                if (!isset($expr->table, $expr->alias) || ($expr->table === '') || ($expr->alias === '')) {
496 496
                     continue;
497 497
                 }
498 498
 
@@ -517,7 +517,7 @@  discard block
 block discarded – undo
517 517
                         ];
518 518
                     }
519 519
 
520
-                    if (! in_array($arr, $ret['select_tables'])) {
520
+                    if (!in_array($arr, $ret['select_tables'])) {
521 521
                         $ret['select_tables'][] = $arr;
522 522
                     }
523 523
                 } else {
@@ -530,7 +530,7 @@  discard block
 block discarded – undo
530 530
             // extracted from the FROM clause.
531 531
             if (empty($ret['select_tables'])) {
532 532
                 foreach ($statement->from as $expr) {
533
-                    if (! isset($expr->table) || ($expr->table === '')) {
533
+                    if (!isset($expr->table) || ($expr->table === '')) {
534 534
                         continue;
535 535
                     }
536 536
 
@@ -571,7 +571,7 @@  discard block
 block discarded – undo
571 571
         } elseif (($statement instanceof AlterStatement) || ($statement instanceof TruncateStatement)) {
572 572
             $expressions = [$statement->table];
573 573
         } elseif ($statement instanceof DropStatement) {
574
-            if (! $statement->options->has('TABLE')) {
574
+            if (!$statement->options->has('TABLE')) {
575 575
                 // No tables are dropped.
576 576
                 return [];
577 577
             }
@@ -851,7 +851,7 @@  discard block
 block discarded – undo
851 851
 
852 852
             $statement .= $token->token;
853 853
 
854
-            if (($token->type === Token::TYPE_DELIMITER) && ! empty($token->token)) {
854
+            if (($token->type === Token::TYPE_DELIMITER) && !empty($token->token)) {
855 855
                 $delimiter = $token->token;
856 856
                 $fullStatement = true;
857 857
                 break;
@@ -860,7 +860,7 @@  discard block
 block discarded – undo
860 860
 
861 861
         // No statement was found so we return the entire query as being the
862 862
         // remaining part.
863
-        if (! $fullStatement) {
863
+        if (!$fullStatement) {
864 864
             return [
865 865
                 null,
866 866
                 $query,
Please login to merge, or discard this patch.
src/Utils/Formatter.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -436,7 +436,7 @@  discard block
 block discarded – undo
436 436
                 // The options of a clause should stay on the same line and everything that follows.
437 437
                 if (
438 438
                     $this->options['parts_newline']
439
-                    && ! $formattedOptions
439
+                    && !$formattedOptions
440 440
                     && empty(self::$inlineClauses[$lastClause])
441 441
                     && (
442 442
                         $curr->type !== Token::TYPE_KEYWORD
@@ -496,7 +496,7 @@  discard block
 block discarded – undo
496 496
                         end($blocksLineEndings) === true
497 497
                         || (
498 498
                             empty(self::$inlineClauses[$lastClause])
499
-                            && ! $shortGroup
499
+                            && !$shortGroup
500 500
                             && $this->options['parts_newline']
501 501
                         )
502 502
                     ) {
@@ -538,7 +538,7 @@  discard block
 block discarded – undo
538 538
                     if (
539 539
                         // A space after delimiters that are longer than 2 characters.
540 540
                         $prev->keyword === 'DELIMITER'
541
-                        || ! (
541
+                        || !(
542 542
                             ($prev->type === Token::TYPE_OPERATOR && ($prev->value === '.' || $prev->value === '('))
543 543
                             // No space after . (
544 544
                             || ($curr->type === Token::TYPE_OPERATOR
@@ -652,12 +652,12 @@  discard block
 block discarded – undo
652 652
         static $prev;
653 653
 
654 654
         foreach ($this->options['formats'] as $format) {
655
-            if ($token->type !== $format['type'] || ! (($token->flags & $format['flags']) === $format['flags'])) {
655
+            if ($token->type !== $format['type'] || !(($token->flags & $format['flags']) === $format['flags'])) {
656 656
                 continue;
657 657
             }
658 658
 
659 659
             // Running transformation function.
660
-            if (! empty($format['function'])) {
660
+            if (!empty($format['function'])) {
661 661
                 $func = $format['function'];
662 662
                 $text = $func($text);
663 663
             }
Please login to merge, or discard this patch.