phpmyadmin / sql-parser

tests/Tools/ContextGeneratorTest.php

Spacing +1 added lines, -1 removed lines

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

src/Token.php

Spacing +3 added lines, -3 removed lines

@@ -128,7 +128,7 @@ 
         switch ($this->type) {
             case TokenType::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;
@@ -153,7 +153,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;
                 }
 
@@ -188,7 +188,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/GroupKeyword.php

Spacing +2 added lines, -2 removed lines

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

src/Components/Condition.php

Spacing +3 added lines, -3 removed lines

@@ -151,7 +151,7 @@ 
                 } else {
                     // The expression ended.
                     $expr->expr = trim($expr->expr);
-                    if (! empty($expr->expr)) {
+                    if (!empty($expr->expr)) {
                         $ret[] = $expr;
                     }
 
@@ -169,7 +169,7 @@ 
             if (
                 ($token->type === TokenType::Keyword)
                 && ($token->flags & Token::FLAG_KEYWORD_RESERVED)
-                && ! ($token->flags & Token::FLAG_KEYWORD_FUNCTION)
+                && !($token->flags & Token::FLAG_KEYWORD_FUNCTION)
             ) {
                 if ($token->value === 'BETWEEN') {
                     $betweenBefore = true;
@@ -212,7 +212,7 @@ 
 
         // Last iteration was not processed.
         $expr->expr = trim($expr->expr);
-        if (! empty($expr->expr)) {
+        if (!empty($expr->expr)) {
             $ret[] = $expr;
         }
 

src/Components/DataType.php

Spacing +2 added lines, -2 removed lines

@@ -95,7 +95,7 @@ 
      * @param TokensList           $list    the list of tokens that are being parsed
      * @param array<string, mixed> $options parameters for parsing
      */
-    public static function parse(Parser $parser, TokensList $list, array $options = []): DataType|null
+    public static function parse(Parser $parser, TokensList $list, array $options = []): DataType | null
     {
         $ret = new static();
 
@@ -125,7 +125,7 @@ 
 
             if ($state === 0) {
                 $ret->name = strtoupper((string) $token->value);
-                if (($token->type !== TokenType::Keyword) || (! ($token->flags & Token::FLAG_KEYWORD_DATA_TYPE))) {
+                if (($token->type !== TokenType::Keyword) || (!($token->flags & Token::FLAG_KEYWORD_DATA_TYPE))) {
                     $parser->error('Unrecognized data type.', $token);
                 }
 

src/Components/CaseExpression.php

Spacing +3 added lines, -3 removed lines

@@ -199,7 +199,7 @@ 
 
                 // Handle optional AS keyword before alias
                 if ($token->type === TokenType::Keyword && $token->keyword === 'AS') {
-                    if ($asFound || ! empty($ret->alias)) {
+                    if ($asFound || !empty($ret->alias)) {
                         $parser->error('Potential duplicate alias of CASE expression.', $token);
                         break;
                     }
@@ -221,11 +221,11 @@ 
                 if (
                     $asFound
                     || $token->type === TokenType::String
-                    || ($token->type === TokenType::Symbol && ! $token->flags & Token::FLAG_SYMBOL_VARIABLE)
+                    || ($token->type === TokenType::Symbol && !$token->flags & Token::FLAG_SYMBOL_VARIABLE)
                     || $token->type === TokenType::None
                 ) {
                     // An alias is expected (the keyword `AS` was previously found).
-                    if (! empty($ret->alias)) {
+                    if (!empty($ret->alias)) {
                         $parser->error('An alias was previously found.', $token);
                         break;
                     }

src/Components/FunctionCall.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
                 if ($token->type === TokenType::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/Components/CreateDefinition.php

Spacing +6 added lines, -6 removed lines

@@ -243,7 +243,7 @@ 
                     $state = 4;
                 } elseif ($token->type === TokenType::Symbol || $token->type === TokenType::None) {
                     $expr->name = $token->value;
-                    if (! $expr->isConstraint) {
+                    if (!$expr->isConstraint) {
                         $state = 2;
                     }
                 } elseif ($token->type === TokenType::Keyword) {
@@ -284,7 +284,7 @@ 
 
                 $state = 5;
             } elseif ($state === 5) {
-                if (! empty($expr->type) || ! empty($expr->key)) {
+                if (!empty($expr->type) || !empty($expr->key)) {
                     $ret[] = $expr;
                 }
 
@@ -304,7 +304,7 @@ 
         }
 
         // Last iteration was not saved.
-        if (! empty($expr->type) || ! empty($expr->key)) {
+        if (!empty($expr->type) || !empty($expr->key)) {
             $ret[] = $expr;
         }
 
@@ -332,16 +332,16 @@ 
             $tmp .= Context::escape($component->name) . ' ';
         }
 
-        if (! empty($component->type)) {
+        if (!empty($component->type)) {
             $component->type->lowercase = true;
             $tmp .= DataType::build($component->type) . ' ';
         }
 
-        if (! empty($component->key)) {
+        if (!empty($component->key)) {
             $tmp .= $component->key . ' ';
         }
 
-        if (! empty($component->references)) {
+        if (!empty($component->references)) {
             $tmp .= 'REFERENCES ' . $component->references . ' ';
         }
 

src/Components/Expression.php

Spacing +17 added lines, -17 removed lines

@@ -157,7 +157,7 @@ 
      *
      * @throws ParserException
      */
-    public static function parse(Parser $parser, TokensList $list, array $options = []): Expression|null
+    public static function parse(Parser $parser, TokensList $list, array $options = []): Expression | null
     {
         $ret = new static();
 
@@ -200,7 +200,7 @@ 
         ];
 
         // When a field is parsed, no parentheses are expected.
-        if (! empty($options['parseField'])) {
+        if (!empty($options['parseField'])) {
             $options['breakOnParentheses'] = true;
             $options['field'] = $options['parseField'];
         }
@@ -227,14 +227,14 @@ 
             }
 
             if ($token->type === TokenType::Keyword) {
-                if (($brackets > 0) && empty($ret->subquery) && ! empty(Parser::$statementParsers[$token->keyword])) {
+                if (($brackets > 0) && empty($ret->subquery) && !empty(Parser::$statementParsers[$token->keyword])) {
                     // A `(` was previously found and this keyword is the
                     // beginning of a statement, so this is a subquery.
                     $ret->subquery = $token->keyword;
                 } elseif (
                     ($token->flags & Token::FLAG_KEYWORD_FUNCTION)
                     && (empty($options['parseField'])
-                    && ! $alias)
+                    && !$alias)
                 ) {
                     $isExpr = true;
                 } elseif (($token->flags & Token::FLAG_KEYWORD_RESERVED) && ($brackets === 0)) {
@@ -246,7 +246,7 @@ 
                     }
 
                     if ($token->keyword === 'AS') {
-                        if (! empty($options['breakOnAlias'])) {
+                        if (!empty($options['breakOnAlias'])) {
                             break;
                         }
 
@@ -269,7 +269,7 @@ 
                     }
 
                     $isExpr = true;
-                } elseif ($brackets === 0 && strlen((string) $ret->expr) > 0 && ! $alias) {
+                } elseif ($brackets === 0 && strlen((string) $ret->expr) > 0 && !$alias) {
                     /* End of expression */
                     break;
                 }
@@ -285,7 +285,7 @@ 
                 || (($token->type === TokenType::Operator)
                 && ($token->value !== '.'))
             ) {
-                if (! empty($options['parseField'])) {
+                if (!empty($options['parseField'])) {
                     break;
                 }
 
@@ -295,7 +295,7 @@ 
             }
 
             if ($token->type === TokenType::Operator) {
-                if (! empty($options['breakOnParentheses']) && (($token->value === '(') || ($token->value === ')'))) {
+                if (!empty($options['breakOnParentheses']) && (($token->value === '(') || ($token->value === ')'))) {
                     // No brackets were expected.
                     break;
                 }
@@ -319,7 +319,7 @@ 
 
                     --$brackets;
                     if ($brackets === 0) {
-                        if (! empty($options['parenthesesDelimited'])) {
+                        if (!empty($options['parenthesesDelimited'])) {
                             // The current token is the last bracket, the next
                             // one will be outside the expression.
                             $ret->expr .= $token->token;
@@ -345,7 +345,7 @@ 
 
             if ($alias) {
                 // An alias is expected (the keyword `AS` was previously found).
-                if (! empty($ret->alias)) {
+                if (!empty($ret->alias)) {
                     $parser->error('An alias was previously found.', $token);
                     break;
                 }
@@ -359,14 +359,14 @@ 
                     && ($prev[0] === null
                         || (($prev[0]->type !== TokenType::Operator || $prev[0]->token === ')')
                             && ($prev[0]->type !== TokenType::Keyword
-                                || ! ($prev[0]->flags & Token::FLAG_KEYWORD_RESERVED))))
+                                || !($prev[0]->flags & Token::FLAG_KEYWORD_RESERVED))))
                     && (($prev[1]->type === TokenType::String)
                         || ($prev[1]->type === TokenType::Symbol
-                            && ! ($prev[1]->flags & Token::FLAG_SYMBOL_VARIABLE))
+                            && !($prev[1]->flags & Token::FLAG_SYMBOL_VARIABLE))
                         || ($prev[1]->type === TokenType::None
                             && $prev[1]->token !== 'OVER'))
                 ) {
-                    if (! empty($ret->alias)) {
+                    if (!empty($ret->alias)) {
                         $parser->error('An alias was previously found.', $token);
                         break;
                     }
@@ -397,7 +397,7 @@ 
                     // Found a `.` which means we expect a column name and
                     // the column name we parsed is actually the table name
                     // and the table name is actually a database name.
-                    if (! empty($ret->database) || $dot) {
+                    if (!empty($ret->database) || $dot) {
                         $parser->error('Unexpected dot.', $token);
                     }
 
@@ -414,11 +414,11 @@ 
                         $dot = false;
                     } else {
                         // No alias is expected.
-                        if (! empty($options['breakOnAlias'])) {
+                        if (!empty($options['breakOnAlias'])) {
                             break;
                         }
 
-                        if (! empty($ret->alias)) {
+                        if (!empty($ret->alias)) {
                             $parser->error('An alias was previously found.', $token);
                             break;
                         }
@@ -469,7 +469,7 @@ 
             $ret = implode('.', Context::escapeAll($fields));
         }
 
-        if (! empty($component->alias)) {
+        if (!empty($component->alias)) {
             $ret .= ' AS ' . Context::escape($component->alias);
         }