phpmyadmin / sql-parser

src/Statement.php

Spacing +11 added lines, -11 removed lines

@@ -88,7 +88,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;
@@ -165,7 +165,7 @@ 
 
             // Checking if this field was already built.
             if ($type & 1) {
-                if (! empty($built[$field])) {
+                if (!empty($built[$field])) {
                     continue;
                 }
 
@@ -178,7 +178,7 @@ 
             }
 
             // Checking if the result of the builder should be added.
-            if (! ($type & 1)) {
+            if (!($type & 1)) {
                 continue;
             }
 
@@ -304,8 +304,8 @@ 
             $options = [];
 
             // Looking for duplicated clauses.
-            if (! empty(Parser::KEYWORD_PARSERS[$token->value]) || ! empty(Parser::STATEMENT_PARSERS[$token->value])) {
-                if (! empty($parsedClauses[$token->value])) {
+            if (!empty(Parser::KEYWORD_PARSERS[$token->value]) || !empty(Parser::STATEMENT_PARSERS[$token->value])) {
+                if (!empty($parsedClauses[$token->value])) {
                     $parser->error('This type of clause was previously parsed.', $token);
                     break;
                 }
@@ -318,18 +318,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::KEYWORD_PARSERS[$tokenValue]) && $list->idx < $list->count) {
+            if (!empty(Parser::KEYWORD_PARSERS[$tokenValue]) && $list->idx < $list->count) {
                 $class = Parser::KEYWORD_PARSERS[$tokenValue]['class'];
                 $field = Parser::KEYWORD_PARSERS[$tokenValue]['field'];
-                if (! empty(Parser::KEYWORD_PARSERS[$tokenValue]['options'])) {
+                if (!empty(Parser::KEYWORD_PARSERS[$tokenValue]['options'])) {
                     $options = Parser::KEYWORD_PARSERS[$tokenValue]['options'];
                 }
             }
 
             // Checking if this is the beginning of the statement.
-            if (! empty(Parser::STATEMENT_PARSERS[$token->keyword])) {
+            if (!empty(Parser::STATEMENT_PARSERS[$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
@@ -344,7 +344,7 @@ 
                     break;
                 }
 
-                if (! $parsedOptions) {
+                if (!$parsedOptions) {
                     if (empty(static::$statementOptions[$token->value])) {
                         // Skipping keyword because if it is not a option.
                         ++$list->idx;
@@ -517,7 +517,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')) {

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() : '');
         }