phpmyadmin / sql-parser

src/Token.php

Spacing +3 added lines, -3 removed lines

@@ -222,7 +222,7 @@ 
         switch ($this->type) {
             case self::TYPE_KEYWORD:
                 $this->keyword = strtoupper($this->token);
-                if (! ($this->flags & self::FLAG_KEYWORD_RESERVED)) {
+                if (!($this->flags & self::FLAG_KEYWORD_RESERVED)) {
                     // Unreserved keywords should stay the way they are because they
                     // might represent field names.
                     return $this->token;
@@ -247,7 +247,7 @@ 
                     }
                 } elseif (($this->flags & self::FLAG_NUMBER_APPROXIMATE) || ($this->flags & self::FLAG_NUMBER_FLOAT)) {
                     $ret = (float) $ret;
-                } elseif (! ($this->flags & self::FLAG_NUMBER_BINARY)) {
+                } elseif (!($this->flags & self::FLAG_NUMBER_BINARY)) {
                     $ret = (int) $ret;
                 }
 
@@ -282,7 +282,7 @@ 
                     // in PHP 5.3- the `null` parameter isn't handled correctly.
                     $str = mb_substr(
                         $str,
-                        ! empty($str[1]) && ($str[1] === '@') ? 2 : 1,
+                        !empty($str[1]) && ($str[1] === '@') ? 2 : 1,
                         mb_strlen($str),
                         'UTF-8'
                     );

src/Components/OrderKeyword.php

Spacing +2 added lines, -2 removed lines

@@ -94,7 +94,7 @@ 
                 ) {
                     $expr->type = $token->keyword;
                 } elseif (($token->type === Token::TYPE_OPERATOR) && ($token->value === ',')) {
-                    if (! empty($expr->expr)) {
+                    if (!empty($expr->expr)) {
                         $ret[] = $expr;
                     }
 
@@ -107,7 +107,7 @@ 
         }
 
         // Last iteration was not processed.
-        if (! empty($expr->expr)) {
+        if (!empty($expr->expr)) {
             $ret[] = $expr;
         }
 

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/Components/FunctionCall.php

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
                 if ($token->type === Token::TYPE_OPERATOR && $token->value === '(') {
                     --$list->idx; // ArrayObj needs to start with `(`
                     $state = 1;
-                    continue;// do not add this token to the name
+                    continue; // do not add this token to the name
                 }
 
                 $ret->name .= $token->value;

src/Utils/Tokens.php

Spacing +6 added lines, -6 removed lines

@@ -47,8 +47,8 @@ 
         }
 
         // Flags.
-        return ! isset($pattern['flags'])
-            || (! (($pattern['flags'] & $token->flags) === 0));
+        return !isset($pattern['flags'])
+            || (!(($pattern['flags'] & $token->flags) === 0));
     }
 
     /**
@@ -56,17 +56,17 @@ 
      * @param Token[] $replace
      */
     public static function replaceTokens(
-        TokensList|string|UtfString $list,
+        TokensList | string | UtfString $list,
         array $find,
         array $replace
-    ): TokensList|string {
+    ): TokensList | string {
         /**
          * Whether the first parameter is a list.
          */
         $isList = $list instanceof TokensList;
 
         // Parsing the tokens.
-        if (! $isList) {
+        if (!$isList) {
             $list = Lexer::getTokens($list);
         }
 
@@ -124,7 +124,7 @@ 
                     ++$j;
                 }
 
-                if (! static::match($list->tokens[$j], $find[$k])) {
+                if (!static::match($list->tokens[$j], $find[$k])) {
                     // This token does not match the pattern.
                     break;
                 }

src/Statement.php

Spacing +11 added lines, -11 removed lines

@@ -102,7 +102,7 @@ 
      * @param Parser|null     $parser the instance that requests parsing
      * @param TokensList|null $list   the list of tokens to be parsed
      */
-    public function __construct(Parser|null $parser = null, TokensList|null $list = null)
+    public function __construct(Parser | null $parser = null, TokensList | null $list = null)
     {
         if (($parser === null) || ($list === null)) {
             return;
@@ -181,7 +181,7 @@ 
 
             // Checking if this field was already built.
             if ($type & 1) {
-                if (! empty($built[$field])) {
+                if (!empty($built[$field])) {
                     continue;
                 }
 
@@ -194,7 +194,7 @@ 
             }
 
             // Checking if the result of the builder should be added.
-            if (! ($type & 1)) {
+            if (!($type & 1)) {
                 continue;
             }
 
@@ -318,8 +318,8 @@ 
             $options = [];
 
             // Looking for duplicated clauses.
-            if (! empty(Parser::$keywordParsers[$token->value]) || ! empty(Parser::$statementParsers[$token->value])) {
-                if (! empty($parsedClauses[$token->value])) {
+            if (!empty(Parser::$keywordParsers[$token->value]) || !empty(Parser::$statementParsers[$token->value])) {
+                if (!empty($parsedClauses[$token->value])) {
                     $parser->error('This type of clause was previously parsed.', $token);
                     break;
                 }
@@ -332,18 +332,18 @@ 
             // but it might be the beginning of a statement of truncate,
             // so let the value use the keyword field for truncate type.
             $tokenValue = in_array($token->keyword, ['TRUNCATE']) ? $token->keyword : $token->value;
-            if (! empty(Parser::$keywordParsers[$tokenValue]) && $list->idx < $list->count) {
+            if (!empty(Parser::$keywordParsers[$tokenValue]) && $list->idx < $list->count) {
                 $class = Parser::$keywordParsers[$tokenValue]['class'];
                 $field = Parser::$keywordParsers[$tokenValue]['field'];
-                if (! empty(Parser::$keywordParsers[$tokenValue]['options'])) {
+                if (!empty(Parser::$keywordParsers[$tokenValue]['options'])) {
                     $options = Parser::$keywordParsers[$tokenValue]['options'];
                 }
             }
 
             // Checking if this is the beginning of the statement.
-            if (! empty(Parser::$statementParsers[$token->keyword])) {
+            if (!empty(Parser::$statementParsers[$token->keyword])) {
                 if (
-                    ! empty(static::$clauses) // Undefined for some statements.
+                    !empty(static::$clauses) // Undefined for some statements.
                     && empty(static::$clauses[$token->value])
                 ) {
                     // Some keywords (e.g. `SET`) may be the beginning of a
@@ -358,7 +358,7 @@ 
                     break;
                 }
 
-                if (! $parsedOptions) {
+                if (!$parsedOptions) {
                     if (empty(static::$statementOptions[$token->value])) {
                         // Skipping keyword because if it is not a option.
                         ++$list->idx;
@@ -530,7 +530,7 @@ 
                 if ($minJoin === 0 && stripos($clauseType, 'JOIN')) {
                     // First JOIN clause is detected
                     $minJoin = $maxJoin = $clauseStartIdx;
-                } elseif ($minJoin !== 0 && ! stripos($clauseType, 'JOIN')) {
+                } elseif ($minJoin !== 0 && !stripos($clauseType, 'JOIN')) {
                     // After a previous JOIN clause, a non-JOIN clause has been detected
                     $maxJoin = $lastIdx;
                 } elseif ($maxJoin < $clauseStartIdx && stripos($clauseType, 'JOIN')) {