phpmyadmin / sql-parser

src/Parser.php

Spacing +3 added lines, -3 removed lines

@@ -488,7 +488,7 @@ 
                 $list->idx = $lastIdx;
             } elseif (empty(self::STATEMENT_PARSERS[$token->keyword])) {
                 // Checking if it is a known statement that can be parsed.
-                if (! isset(self::STATEMENT_PARSERS[$token->keyword])) {
+                if (!isset(self::STATEMENT_PARSERS[$token->keyword])) {
                     // A statement is considered recognized if the parser
                     // is aware that it is a statement, but it does not have
                     // a parser for it yet.
@@ -528,7 +528,7 @@ 
 
             // Handles unions.
             if (
-                ! empty($unionType)
+                !empty($unionType)
                 && ($lastStatement instanceof SelectStatement)
                 && ($statement instanceof SelectStatement)
             ) {
@@ -615,7 +615,7 @@ 
      *
      * @throws ParserException throws the exception, if strict mode is enabled.
      */
-    public function error($msg, Token|null $token = null, $code = 0): void
+    public function error($msg, Token | null $token = null, $code = 0): void
     {
         $error = new ParserException(
             Translator::gettext($msg),

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')) {

src/UtfString.php

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
     {
         $this->str = $str;
         $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');
@@ -100,7 +100,7 @@ 
      *
      * @param int $offset the offset to be returned
      */
-    public function offsetGet($offset): string|null
+    public function offsetGet($offset): string | null
     {
         // This function moves the internal byte and character pointer to the requested offset.
         // This function is part of hot code so the aim is to do the following

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;
                     }