phpmyadmin / sql-parser

src/UtfString.php

Spacing +2 added lines, -2 removed lines

@@ -190,7 +190,7 @@ 
         $this->byteIdx = 0;
         $this->charIdx = 0;
         $this->byteLen = mb_strlen($str, '8bit');
-        if (! mb_check_encoding($str, 'UTF-8')) {
+        if (!mb_check_encoding($str, 'UTF-8')) {
             $this->charLen = 0;
         } else {
             $this->charLen = mb_strlen($str, 'UTF-8');
@@ -298,7 +298,7 @@ 
         // Use the default ASCII map as queries are mostly ASCII chars
         // ord($byte) has a performance cost
 
-        if (! isset(static::$asciiMap[$byte])) {
+        if (!isset(static::$asciiMap[$byte])) {
             // Complete the cache with missing items
             static::$asciiMap[$byte] = ord($byte);
         }

src/Statements/CreateStatement.php

Spacing +10 added lines, -10 removed lines

@@ -417,7 +417,7 @@ 
     public function build()
     {
         $fields = '';
-        if (! empty($this->fields)) {
+        if (!empty($this->fields)) {
             if (is_array($this->fields)) {
                 $fields = CreateDefinition::build($this->fields) . ' ';
             } elseif ($this->fields instanceof ArrayObj) {
@@ -449,23 +449,23 @@ 
 
             $partition = '';
 
-            if (! empty($this->partitionBy)) {
+            if (!empty($this->partitionBy)) {
                 $partition .= "\nPARTITION BY " . $this->partitionBy;
             }
 
-            if (! empty($this->partitionsNum)) {
+            if (!empty($this->partitionsNum)) {
                 $partition .= "\nPARTITIONS " . $this->partitionsNum;
             }
 
-            if (! empty($this->subpartitionBy)) {
+            if (!empty($this->subpartitionBy)) {
                 $partition .= "\nSUBPARTITION BY " . $this->subpartitionBy;
             }
 
-            if (! empty($this->subpartitionsNum)) {
+            if (!empty($this->subpartitionsNum)) {
                 $partition .= "\nSUBPARTITIONS " . $this->subpartitionsNum;
             }
 
-            if (! empty($this->partitions)) {
+            if (!empty($this->partitions)) {
                 $partition .= "\n" . PartitionDefinition::build($this->partitions);
             }
 
@@ -487,7 +487,7 @@ 
                 . OptionsArray::build($this->options) . ' '
                 . Expression::build($this->name) . ' '
                 . $fields . ' AS ' . $builtStatement
-                . (! empty($this->body) ? TokensList::build($this->body) : '') . ' '
+                . (!empty($this->body) ? TokensList::build($this->body) : '') . ' '
                 . OptionsArray::build($this->entityOptions);
         } elseif ($this->options->has('TRIGGER')) {
             return 'CREATE '
@@ -541,7 +541,7 @@ 
             ]
         );
 
-        if (! isset($this->name) || ($this->name === '')) {
+        if (!isset($this->name) || ($this->name === '')) {
             $parser->error('The name of the entity was expected.', $list->tokens[$list->idx]);
         } else {
             ++$list->idx; // Skipping field.
@@ -657,7 +657,7 @@ 
                         $token = $list->getNextOfType(Token::TYPE_NUMBER);
                         --$list->idx; // `getNextOfType` also advances one position.
                         $this->subpartitionsNum = $token->value;
-                    } elseif (! empty($field)) {
+                    } elseif (!empty($field)) {
                         /*
                          * Handling the content of `PARTITION BY` and `SUBPARTITION BY`.
                          */
@@ -686,7 +686,7 @@ 
                             $field = null;
                         }
                     } elseif (($token->type === Token::TYPE_OPERATOR) && ($token->value === '(')) {
-                        if (! empty($this->partitionBy)) {
+                        if (!empty($this->partitionBy)) {
                             $this->partitions = ArrayObj::parse(
                                 $parser,
                                 $list,

src/Statements/WithStatement.php

Indentation +3 added lines, -4 removed lines

@@ -22,7 +22,6 @@ 
 
 /**
  * `WITH` statement.
-
  *  WITH [RECURSIVE] query_name [ (column_name [,...]) ] AS (SELECT ...) [, ...]
  */
 final class WithStatement extends Statement
@@ -263,9 +262,9 @@ 
 
         // 5 is the only valid end state
         if ($state !== 5) {
-             /**
-             * Token parsed at this moment.
-             */
+                /**
+                 * Token parsed at this moment.
+                 */
             $token = $list->tokens[$list->idx];
 
             $parser->error('Unexpected end of the WITH CTE.', $token);

Spacing +4 added lines, -4 removed lines

@@ -133,7 +133,7 @@ 
                     break;
                 }
             } elseif ($state === 2) {
-                if (! ($token->type === Token::TYPE_KEYWORD && $token->keyword === 'AS')) {
+                if (!($token->type === Token::TYPE_KEYWORD && $token->keyword === 'AS')) {
                     $parser->error('AS keyword was expected.', $token);
                     break;
                 }
@@ -145,7 +145,7 @@ 
                 $list->idx++; // Ignore the current token
                 $nextKeyword = $list->getNext();
 
-                if (! ($token->value === '(' && ($nextKeyword && $nextKeyword->value === 'SELECT'))) {
+                if (!($token->value === '(' && ($nextKeyword && $nextKeyword->value === 'SELECT'))) {
                     $parser->error('Subquery of the CTE was expected.', $token);
                     $list->idx = $idxBeforeGetNext;
                     break;
@@ -295,7 +295,7 @@ 
     /**
      * Get tokens within the WITH expression to use them in another parser
      */
-    private function getSubTokenList(TokensList $list): ParserException|TokensList
+    private function getSubTokenList(TokensList $list): ParserException | TokensList
     {
         $idx = $list->idx;
         $token = $list->tokens[$list->idx];
@@ -311,7 +311,7 @@ 
             }
 
             ++$list->idx;
-            if (! isset($list->tokens[$list->idx])) {
+            if (!isset($list->tokens[$list->idx])) {
                 break;
             }
 

tests/Tools/ContextGeneratorTest.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
         $this->assertEquals([
             Token::TYPE_KEYWORD | Token::FLAG_KEYWORD_RESERVED => [
                 8 => ['RESERVED'],
-                9 => ['RESERVED2','RESERVED3','RESERVED4','RESERVED5'],
+                9 => ['RESERVED2', 'RESERVED3', 'RESERVED4', 'RESERVED5'],
             ],
             Token::TYPE_KEYWORD | Token::FLAG_KEYWORD_FUNCTION => [8 => ['FUNCTION']],
             Token::TYPE_KEYWORD | Token::FLAG_KEYWORD_DATA_TYPE => [8 => ['DATATYPE']],

tests/benchmarks/UtfStringBench.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
     {
         $str1 = new UtfString($this->testContents);
         for ($i = 0; $i < $str1->length(); $i++) {
-            $str1[$i];// Make offset offsetGet work
+            $str1[$i]; // Make offset offsetGet work
         }
     }
 

tests/Parser/ParserLongExportsTest.php

Spacing +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
         foreach ($parser->statements as $statement) {
             if ($statement instanceof TransactionStatement) {
                 foreach ($statement->statements as $transactionStatement) {
-                    if (! $transactionStatement instanceof SetStatement) {
+                    if (!$transactionStatement instanceof SetStatement) {
                         continue;
                     }
 
@@ -74,7 +74,7 @@ 
                 continue;
             }
 
-            if (! $statement instanceof SetStatement) {
+            if (!$statement instanceof SetStatement) {
                 continue;
             }
 

src/Tools/TestGenerator.php

Spacing +7 added lines, -7 removed lines

@@ -83,7 +83,7 @@ 
         // The code below extracts only the relevant information.
 
         // Extracting lexer's errors.
-        if (! empty($lexer->errors)) {
+        if (!empty($lexer->errors)) {
             /** @var LexerException $err */
             foreach ($lexer->errors as $err) {
                 $lexerErrors[] = [
@@ -98,7 +98,7 @@ 
         }
 
         // Extracting parser's errors.
-        if (! empty($parser->errors)) {
+        if (!empty($parser->errors)) {
             /** @var ParserException $err */
             foreach ($parser->errors as $err) {
                 $parserErrors[] = [
@@ -138,7 +138,7 @@ 
     public static function build($type, $input, $output, $debug = null, $ansi = false)
     {
         // Support query types: `lexer` / `parser`.
-        if (! in_array($type, ['lexer', 'parser'])) {
+        if (!in_array($type, ['lexer', 'parser'])) {
             throw new Exception('Unknown test type (expected `lexer` or `parser`).');
         }
 
@@ -183,7 +183,7 @@ 
         );
 
         // Remove the project path from .out file, it changes for each dev
-        $projectFolder = dirname(__DIR__, 2);// Jump to root
+        $projectFolder = dirname(__DIR__, 2); // Jump to root
         $encoded = str_replace($projectFolder, '<project-root>', $encoded);
 
         file_put_contents($output, $encoded);
@@ -223,11 +223,11 @@ 
             if (is_dir($inputFile)) {
                 // Creating required directories to maintain the structure.
                 // Ignoring errors if the folder structure exists already.
-                if (! is_dir($outputFile)) {
+                if (!is_dir($outputFile)) {
                     mkdir($outputFile);
                 }
 
-                if (($debug !== null) && (! is_dir($debugFile))) {
+                if (($debug !== null) && (!is_dir($debugFile))) {
                     mkdir($debugFile);
                 }
 
@@ -242,7 +242,7 @@ 
                 }
 
                 // Building the test.
-                if (! file_exists($outputFile)) {
+                if (!file_exists($outputFile)) {
                     echo sprintf("Building test for %s...\n", $inputFile);
                     static::build(
                         str_contains($inputFile, 'lex') ? 'lexer' : 'parser',

src/Translator.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
      */
     public static function gettext($msgid)
     {
-        if (! class_exists(Loader::class, true)) {
+        if (!class_exists(Loader::class, true)) {
             return $msgid;
         }
 

src/Lexer.php

Spacing +15 added lines, -15 removed lines

@@ -15,7 +15,7 @@ 
 use function strlen;
 use function substr;
 
-if (! defined('USE_UTF_STRINGS')) {
+if (!defined('USE_UTF_STRINGS')) {
     // NOTE: In previous versions of PHP (5.5 and older) the default
     // internal encoding is "ISO-8859-1".
     // All `mb_` functions must specify the correct encoding, which is
@@ -201,7 +201,7 @@ 
 
         // For multi-byte strings, a new instance of `UtfString` is
         // initialized (only if `UtfString` usage is forced.
-        if (! $str instanceof UtfString && USE_UTF_STRINGS && $len !== mb_strlen($str, 'UTF-8')) {
+        if (!$str instanceof UtfString && USE_UTF_STRINGS && $len !== mb_strlen($str, 'UTF-8')) {
             $str = new UtfString($str);
         }
 
@@ -211,7 +211,7 @@ 
         $this->strict = $strict;
 
         // Setting the delimiter.
-        $this->setDelimiter(! empty($delimiter) ? $delimiter : static::$defaultDelimiter);
+        $this->setDelimiter(!empty($delimiter) ? $delimiter : static::$defaultDelimiter);
 
         $this->lex();
     }
@@ -339,7 +339,7 @@ 
                 $delimiterLen = 0;
                 while (
                     ++$this->last < $this->len
-                    && ! Context::isWhitespace($this->str[$this->last])
+                    && !Context::isWhitespace($this->str[$this->last])
                     && $delimiterLen < 15
                 ) {
                     $this->delimiter .= $this->str[$this->last];
@@ -398,8 +398,8 @@ 
             }
 
             if (
-                ($next->type !== Token::TYPE_KEYWORD || ! in_array($next->value, ['FROM', 'USING'], true))
-                && ($next->type !== Token::TYPE_OPERATOR || ! in_array($next->value, [',', ')'], true))
+                ($next->type !== Token::TYPE_KEYWORD || !in_array($next->value, ['FROM', 'USING'], true))
+                && ($next->type !== Token::TYPE_OPERATOR || !in_array($next->value, [',', ')'], true))
             ) {
                 continue;
             }
@@ -437,10 +437,10 @@ 
             $next = $this->list->getNext();
             if (
                 ($next->type !== Token::TYPE_KEYWORD
-                    || ! in_array($next->value, $this->keywordNameIndicators, true)
+                    || !in_array($next->value, $this->keywordNameIndicators, true)
                 )
                 && ($next->type !== Token::TYPE_OPERATOR
-                    || ! in_array($next->value, $this->operatorNameIndicators, true)
+                    || !in_array($next->value, $this->operatorNameIndicators, true)
                 )
                 && ($next->value !== null)
             ) {
@@ -523,7 +523,7 @@ 
             $token .= $this->str[$this->last];
             $flags = Context::isKeyword($token);
 
-            if (($this->last + 1 !== $this->len && ! Context::isSeparator($this->str[$this->last + 1])) || ! $flags) {
+            if (($this->last + 1 !== $this->len && !Context::isSeparator($this->str[$this->last + 1])) || !$flags) {
                 continue;
             }
 
@@ -611,7 +611,7 @@ 
             $token .= $this->str[$this->last];
             $flags = Context::isOperator($token);
 
-            if (! $flags) {
+            if (!$flags) {
                 continue;
             }
 
@@ -633,7 +633,7 @@ 
     {
         $token = $this->str[$this->last];
 
-        if (! Context::isWhitespace($token)) {
+        if (!Context::isWhitespace($token)) {
             return null;
         }
 
@@ -873,7 +873,7 @@ 
             } elseif ($state === 2) {
                 $flags |= Token::FLAG_NUMBER_HEX;
                 if (
-                    ! (
+                    !(
                         ($this->str[$this->last] >= '0' && $this->str[$this->last] <= '9')
                         || ($this->str[$this->last] >= 'A' && $this->str[$this->last] <= 'F')
                         || ($this->str[$this->last] >= 'a' && $this->str[$this->last] <= 'f')
@@ -976,7 +976,7 @@ 
         $token = $this->str[$this->last];
         $flags = Context::isString($token);
 
-        if (! $flags && $token !== $quote) {
+        if (!$flags && $token !== $quote) {
             return null;
         }
 
@@ -1028,7 +1028,7 @@ 
         $token = $this->str[$this->last];
         $flags = Context::isSymbol($token);
 
-        if (! $flags) {
+        if (!$flags) {
             return null;
         }
 
@@ -1079,7 +1079,7 @@ 
             return null;
         }
 
-        while (++$this->last < $this->len && ! Context::isSeparator($this->str[$this->last])) {
+        while (++$this->last < $this->len && !Context::isSeparator($this->str[$this->last])) {
             $token .= $this->str[$this->last];
 
             // Test if end of token equals the current delimiter. If so, remove it from the token.