phpmyadmin / sql-parser

src/Parsers/GroupKeywords.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/Parsers/Expressions.php

Spacing +18 added lines, -18 removed lines

@@ -80,7 +80,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 Expression();
 
@@ -113,7 +113,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'];
         }
@@ -140,18 +140,18 @@ 
             }
 
             if ($token->type === TokenType::Keyword) {
-                if (($brackets > 0) && empty($ret->subquery) && ! empty(Parser::STATEMENT_PARSERS[$token->keyword])) {
+                if (($brackets > 0) && empty($ret->subquery) && !empty(Parser::STATEMENT_PARSERS[$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)) {
-                    if (! in_array($token->keyword, self::ALLOWED_KEYWORDS, true)) {
+                    if (!in_array($token->keyword, self::ALLOWED_KEYWORDS, true)) {
                         // A reserved keyword that is not allowed in the
                         // expression was found so the expression must have
                         // ended and a new clause is starting.
@@ -159,7 +159,7 @@ 
                     }
 
                     if ($token->keyword === 'AS') {
-                        if (! empty($options['breakOnAlias'])) {
+                        if (!empty($options['breakOnAlias'])) {
                             break;
                         }
 
@@ -183,7 +183,7 @@ 
 
                     $isExpr = true;
                 } elseif (
-                    $brackets === 0 && strlen((string) $ret->expr) > 0 && ! $alias
+                    $brackets === 0 && strlen((string) $ret->expr) > 0 && !$alias
                     && ($ret->table === null || $ret->table === '')
                 ) {
                     /* End of expression */
@@ -201,7 +201,7 @@ 
                 || (($token->type === TokenType::Operator)
                 && ($token->value !== '.'))
             ) {
-                if (! empty($options['parseField'])) {
+                if (!empty($options['parseField'])) {
                     break;
                 }
 
@@ -211,7 +211,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;
                 }
@@ -235,7 +235,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;
@@ -261,7 +261,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;
                 }
@@ -275,15 +275,15 @@ 
                     && ($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_PARAMETER))
+                            && !($prev[1]->flags & Token::FLAG_SYMBOL_VARIABLE)
+                            && !($prev[1]->flags & Token::FLAG_SYMBOL_PARAMETER))
                         || ($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;
                     }
@@ -314,7 +314,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);
                     }
 
@@ -331,11 +331,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;
                         }

src/Parsers/OrderKeywords.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/Parsers/Limits.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
 
             // Skip if not a number or a bind parameter (?)
             if (
-                ! ($token->type === TokenType::Number
+                !($token->type === TokenType::Number
                     || ($token->type === TokenType::Symbol && ($token->flags & Token::FLAG_SYMBOL_PARAMETER)))
             ) {
                 break;

src/Parsers/Keys.php

Spacing +3 added lines, -3 removed lines

@@ -112,9 +112,9 @@ 
             } elseif ($state === 1) {
                 if (($token->type === TokenType::Operator) && ($token->value === '(')) {
                     $positionBeforeSearch = $list->idx;
-                    $list->idx++;// Ignore the current token "(" or the search condition will always be true
+                    $list->idx++; // Ignore the current token "(" or the search condition will always be true
                     $nextToken = $list->getNext();
-                    $list->idx = $positionBeforeSearch;// Restore the position
+                    $list->idx = $positionBeforeSearch; // Restore the position
 
                     if ($nextToken !== null && $nextToken->value === '(') {
                         // Switch to expression mode
@@ -163,7 +163,7 @@ 
                 if ($token->type === TokenType::Operator) {
                     // This got back to here and we reached the end of the expression
                     if ($token->value === ')') {
-                        $state = 4;// go back to state 4 to fetch options
+                        $state = 4; // go back to state 4 to fetch options
                         continue;
                     }
 

src/Components/OptionsArray.php

Spacing +7 added lines, -7 removed lines

@@ -33,12 +33,12 @@ 
 
         $options = [];
         foreach ($this->options as $option) {
-            if (! is_array($option)) {
+            if (!is_array($option)) {
                 $options[] = $option;
             } else {
                 $options[] = $option['name']
-                    . (! empty($option['equals']) ? '=' : ' ')
-                    . (! empty($option['expr']) ? $option['expr'] : $option['value']);
+                    . (!empty($option['equals']) ? '=' : ' ')
+                    . (!empty($option['expr']) ? $option['expr'] : $option['value']);
             }
         }
 
@@ -56,10 +56,10 @@ 
     {
         foreach ($this->options as $option) {
             if (is_array($option)) {
-                if (! strcasecmp($key, $option['name'])) {
+                if (!strcasecmp($key, $option['name'])) {
                     return $getExpr ? $option['expr'] : $option['value'];
                 }
-            } elseif (! strcasecmp($key, $option)) {
+            } elseif (!strcasecmp($key, $option)) {
                 return true;
             }
         }
@@ -78,12 +78,12 @@ 
     {
         foreach ($this->options as $idx => $option) {
             if (is_array($option)) {
-                if (! strcasecmp($key, $option['name'])) {
+                if (!strcasecmp($key, $option['name'])) {
                     unset($this->options[$idx]);
 
                     return true;
                 }
-            } elseif (! strcasecmp($key, $option)) {
+            } elseif (!strcasecmp($key, $option)) {
                 unset($this->options[$idx]);
 
                 return true;

src/Statements/CreateStatement.php

Spacing +21 added lines, -21 removed lines

@@ -276,7 +276,7 @@ 
      *
      * Used by all `CREATE` statements.
      */
-    public Expression|null $name = null;
+    public Expression | null $name = null;
 
     /**
      * The options of the entity (table, procedure, function, etc.).
@@ -287,7 +287,7 @@ 
      * @see CreateStatement::FUNCTION_OPTIONS
      * @see CreateStatement::TRIGGER_OPTIONS
      */
-    public OptionsArray|null $entityOptions = null;
+    public OptionsArray | null $entityOptions = null;
 
     /**
      * If `CREATE TABLE`, a list of columns and keys.
@@ -297,7 +297,7 @@ 
      *
      * @var CreateDefinition[]|ArrayObj|null
      */
-    public array|ArrayObj|null $fields = null;
+    public array | ArrayObj | null $fields = null;
 
     /**
      * If `CREATE TABLE WITH`.
@@ -306,7 +306,7 @@ 
      *
      * Used by `CREATE TABLE`, `CREATE VIEW`
      */
-    public WithStatement|null $with = null;
+    public WithStatement | null $with = null;
 
     /**
      * If `CREATE TABLE ... SELECT`.
@@ -314,55 +314,55 @@ 
      *
      * Used by `CREATE TABLE`, `CREATE VIEW`
      */
-    public SelectStatement|null $select = null;
+    public SelectStatement | null $select = null;
 
     /**
      * If `CREATE TABLE ... LIKE`.
      *
      * Used by `CREATE TABLE`
      */
-    public Expression|null $like = null;
+    public Expression | null $like = null;
 
     /**
      * Expression used for partitioning.
      */
-    public string|null $partitionBy = null;
+    public string | null $partitionBy = null;
 
     /**
      * The number of partitions.
      */
-    public int|null $partitionsNum = null;
+    public int | null $partitionsNum = null;
 
     /**
      * Expression used for subpartitioning.
      */
-    public string|null $subpartitionBy = null;
+    public string | null $subpartitionBy = null;
 
     /**
      * The number of subpartitions.
      */
-    public int|null $subpartitionsNum = null;
+    public int | null $subpartitionsNum = null;
 
     /**
      * The partition of the new table.
      *
      * @var PartitionDefinition[]|null
      */
-    public array|null $partitions = null;
+    public array | null $partitions = null;
 
     /**
      * If `CREATE TRIGGER` the name of the table.
      *
      * Used by `CREATE TRIGGER`.
      */
-    public Expression|null $table = null;
+    public Expression | null $table = null;
 
     /**
      * The return data type of this routine.
      *
      * Used by `CREATE FUNCTION`.
      */
-    public DataType|null $return = null;
+    public DataType | null $return = null;
 
     /**
      * The parameters of this routine.
@@ -371,7 +371,7 @@ 
      *
      * @var ParameterDefinition[]|null
      */
-    public array|null $parameters = null;
+    public array | null $parameters = null;
 
     /**
      * The body of this function or procedure.
@@ -424,23 +424,23 @@ 
 
             $partition = '';
 
-            if (! empty($this->partitionBy)) {
+            if (!empty($this->partitionBy)) {
                 $partition .= "\nPARTITION BY " . $this->partitionBy;
             }
 
-            if (! empty($this->partitionsNum)) {
+            if (!empty($this->partitionsNum)) {
                 $partition .= "\nPARTITIONS " . $this->partitionsNum;
             }
 
-            if (! empty($this->subpartitionBy)) {
+            if (!empty($this->subpartitionBy)) {
                 $partition .= "\nSUBPARTITION BY " . $this->subpartitionBy;
             }
 
-            if (! empty($this->subpartitionsNum)) {
+            if (!empty($this->subpartitionsNum)) {
                 $partition .= "\nSUBPARTITIONS " . $this->subpartitionsNum;
             }
 
-            if (! empty($this->partitions)) {
+            if (!empty($this->partitions)) {
                 $partition .= "\n" . PartitionDefinitions::buildAll($this->partitions);
             }
 
@@ -630,7 +630,7 @@ 
                         $token = $list->getNextOfType(TokenType::Number);
                         --$list->idx; // `getNextOfType` also advances one position.
                         $this->subpartitionsNum = $token->value;
-                    } elseif (! empty($field)) {
+                    } elseif (!empty($field)) {
                         /*
                          * Handling the content of `PARTITION BY` and `SUBPARTITION BY`.
                          */
@@ -659,7 +659,7 @@ 
                             $field = null;
                         }
                     } elseif (($token->type === TokenType::Operator) && ($token->value === '(')) {
-                        if (! empty($this->partitionBy)) {
+                        if (!empty($this->partitionBy)) {
                             $this->partitions = ArrayObjs::parse(
                                 $parser,
                                 $list,

tests/Parser/ParserLongExportsTest.php

Spacing +2 added lines, -2 removed lines

@@ -66,7 +66,7 @@ 
             if ($statement instanceof TransactionStatement) {
                 $this->assertNotNull($statement->statements);
                 foreach ($statement->statements as $transactionStatement) {
-                    if (! $transactionStatement instanceof SetStatement) {
+                    if (!$transactionStatement instanceof SetStatement) {
                         continue;
                     }
 
@@ -76,7 +76,7 @@ 
                 continue;
             }
 
-            if (! $statement instanceof SetStatement) {
+            if (!$statement instanceof SetStatement) {
                 continue;
             }
 

src/Token.php

Spacing +7 added lines, -7 removed lines

@@ -70,12 +70,12 @@ 
     /**
      * The value this token contains (i.e. token after some evaluation).
      */
-    public bool|float|int|string $value;
+    public bool | float | int | string $value;
 
     /**
      * The keyword value this token contains, always uppercase.
      */
-    public string|null $keyword = null;
+    public string | null $keyword = null;
 
     /**
      * The type of this token.
@@ -93,7 +93,7 @@ 
      * The position is counted in chars, not bytes, so you should
      * use mb_* functions to properly handle utf-8 multibyte chars.
      */
-    public int|null $position = null;
+    public int | null $position = null;
 
     /**
      * @param string    $token the value of the token
@@ -113,12 +113,12 @@ 
      *
      * If no processing can be done it will return the initial string.
      */
-    public function extract(): bool|float|int|string
+    public function extract(): bool | float | int | string
     {
         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;
@@ -143,7 +143,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;
                 }
 
@@ -178,7 +178,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',
                     );