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/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/Utils/Query.php

Spacing +16 added lines, -16 removed lines

@@ -300,23 +300,23 @@ 
             $flags['distinct'] = true;
         }
 
-        if (! empty($statement->group) || ! empty($statement->having)) {
+        if (!empty($statement->group) || !empty($statement->having)) {
             $flags['is_group'] = true;
         }
 
-        if (! empty($statement->into) && ($statement->into->type === 'OUTFILE')) {
+        if (!empty($statement->into) && ($statement->into->type === 'OUTFILE')) {
             $flags['is_export'] = true;
         }
 
         $expressions = $statement->expr;
-        if (! empty($statement->join)) {
+        if (!empty($statement->join)) {
             foreach ($statement->join as $join) {
                 $expressions[] = $join->expr;
             }
         }
 
         foreach ($expressions as $expr) {
-            if (! empty($expr->function)) {
+            if (!empty($expr->function)) {
                 if ($expr->function === 'COUNT') {
                     $flags['is_count'] = true;
                 } elseif (in_array($expr->function, static::$functions)) {
@@ -331,15 +331,15 @@ 
             $flags['is_subquery'] = true;
         }
 
-        if (! empty($statement->procedure) && ($statement->procedure->name === 'ANALYSE')) {
+        if (!empty($statement->procedure) && ($statement->procedure->name === 'ANALYSE')) {
             $flags['is_analyse'] = true;
         }
 
-        if (! empty($statement->group)) {
+        if (!empty($statement->group)) {
             $flags['group'] = true;
         }
 
-        if (! empty($statement->having)) {
+        if (!empty($statement->having)) {
             $flags['having'] = true;
         }
 
@@ -347,7 +347,7 @@ 
             $flags['union'] = true;
         }
 
-        if (! empty($statement->join)) {
+        if (!empty($statement->join)) {
             $flags['join'] = true;
         }
 
@@ -438,11 +438,11 @@ 
             || ($statement instanceof UpdateStatement)
             || ($statement instanceof DeleteStatement)
         ) {
-            if (! empty($statement->limit)) {
+            if (!empty($statement->limit)) {
                 $flags['limit'] = true;
             }
 
-            if (! empty($statement->order)) {
+            if (!empty($statement->order)) {
                 $flags['order'] = true;
             }
         }
@@ -492,7 +492,7 @@ 
             // Finding tables' aliases and their associated real names.
             $tableAliases = [];
             foreach ($statement->from as $expr) {
-                if (! isset($expr->table, $expr->alias) || ($expr->table === '') || ($expr->alias === '')) {
+                if (!isset($expr->table, $expr->alias) || ($expr->table === '') || ($expr->alias === '')) {
                     continue;
                 }
 
@@ -517,7 +517,7 @@ 
                         ];
                     }
 
-                    if (! in_array($arr, $ret['select_tables'])) {
+                    if (!in_array($arr, $ret['select_tables'])) {
                         $ret['select_tables'][] = $arr;
                     }
                 } else {
@@ -530,7 +530,7 @@ 
             // extracted from the FROM clause.
             if ($ret['select_tables'] === []) {
                 foreach ($statement->from as $expr) {
-                    if (! isset($expr->table) || ($expr->table === '')) {
+                    if (!isset($expr->table) || ($expr->table === '')) {
                         continue;
                     }
 
@@ -571,7 +571,7 @@ 
         } elseif (($statement instanceof AlterStatement) || ($statement instanceof TruncateStatement)) {
             $expressions = [$statement->table];
         } elseif ($statement instanceof DropStatement) {
-            if (! $statement->options->has('TABLE')) {
+            if (!$statement->options->has('TABLE')) {
                 // No tables are dropped.
                 return [];
             }
@@ -845,7 +845,7 @@ 
 
             $statement .= $token->token;
 
-            if (($token->type === TokenType::Delimiter) && ! empty($token->token)) {
+            if (($token->type === TokenType::Delimiter) && !empty($token->token)) {
                 $delimiter = $token->token;
                 $fullStatement = true;
                 break;
@@ -854,7 +854,7 @@ 
 
         // No statement was found so we return the entire query as being the
         // remaining part.
-        if (! $fullStatement) {
+        if (!$fullStatement) {
             return [
                 null,
                 $query,

src/Components/JoinKeyword.php

Spacing +3 added lines, -3 removed lines

@@ -188,7 +188,7 @@ 
             }
         }
 
-        if (! empty($expr->type)) {
+        if (!empty($expr->type)) {
             $ret[] = $expr;
         }
 
@@ -210,9 +210,9 @@ 
         $ret = [];
         foreach ($component as $c) {
             $ret[] = array_search($c->type, self::JOINS) . ' ' . $c->expr
-                . (! empty($c->on)
+                . (!empty($c->on)
                     ? ' ON ' . Condition::buildAll($c->on) : '')
-                . (! empty($c->using)
+                . (!empty($c->using)
                     ? ' USING ' . $c->using->build() : '');
         }
 

src/Utils/Formatter.php

Spacing +6 added lines, -6 removed lines

@@ -435,7 +435,7 @@ 
                 // The options of a clause should stay on the same line and everything that follows.
                 if (
                     $this->options['parts_newline']
-                    && ! $formattedOptions
+                    && !$formattedOptions
                     && empty(self::$inlineClauses[$lastClause])
                     && (
                         $curr->type !== TokenType::Keyword
@@ -492,7 +492,7 @@ 
                         end($blocksLineEndings) === true
                         || (
                             empty(self::$inlineClauses[$lastClause])
-                            && ! $shortGroup
+                            && !$shortGroup
                             && $this->options['parts_newline']
                         )
                     ) {
@@ -528,7 +528,7 @@ 
                     $lineEnded = false;
                 } elseif (
                     $prev->keyword === 'DELIMITER'
-                    || ! (
+                    || !(
                     ($prev->type === TokenType::Operator && ($prev->value === '.' || $prev->value === '('))
                     // No space after . (
                     || ($curr->type === TokenType::Operator
@@ -643,13 +643,13 @@ 
 
         foreach ($this->options['formats'] as $format) {
             if (
-                $token->type->value !== $format['type'] || ! (($token->flags & $format['flags']) === $format['flags'])
+                $token->type->value !== $format['type'] || !(($token->flags & $format['flags']) === $format['flags'])
             ) {
                 continue;
             }
 
             // Running transformation function.
-            if (! empty($format['function'])) {
+            if (!empty($format['function'])) {
                 $func = $format['function'];
                 $text = $func($text);
             }
@@ -758,7 +758,7 @@ 
      *
      * @psalm-return 1|2|false
      */
-    public static function isClause($token): int|false
+    public static function isClause($token): int | false
     {
         if (
             ($token->type === TokenType::Keyword && isset(Parser::STATEMENT_PARSERS[$token->keyword]))