phpmyadmin / sql-parser

src/Lexer.php

Spacing +14 added lines, -14 removed lines

@@ -183,7 +183,7 @@ 
         $len = $str instanceof UtfString ? $str->length() : strlen($str);
 
         // For multi-byte strings, a new instance of `UtfString` is initialized.
-        if (! $str instanceof UtfString && $len !== mb_strlen($str, 'UTF-8')) {
+        if (!$str instanceof UtfString && $len !== mb_strlen($str, 'UTF-8')) {
             $str = new UtfString($str);
         }
 
@@ -193,7 +193,7 @@ 
         $this->strict = $strict;
 
         // Setting the delimiter.
-        $this->setDelimiter(! empty($delimiter) ? $delimiter : static::$defaultDelimiter);
+        $this->setDelimiter(!empty($delimiter) ? $delimiter : static::$defaultDelimiter);
 
         $this->lex();
     }
@@ -321,7 +321,7 @@ 
                 $delimiterLen = 0;
                 while (
                     ++$this->last < $this->len
-                    && ! Context::isWhitespace($this->str[$this->last])
+                    && !Context::isWhitespace($this->str[$this->last])
                     && $delimiterLen < 15
                 ) {
                     $this->delimiter .= $this->str[$this->last];
@@ -380,8 +380,8 @@ 
             }
 
             if (
-                ($next->type !== Token::TYPE_KEYWORD || ! in_array($next->value, ['FROM', 'USING'], true))
-                && ($next->type !== Token::TYPE_OPERATOR || ! in_array($next->value, [',', ')'], true))
+                ($next->type !== Token::TYPE_KEYWORD || !in_array($next->value, ['FROM', 'USING'], true))
+                && ($next->type !== Token::TYPE_OPERATOR || !in_array($next->value, [',', ')'], true))
             ) {
                 continue;
             }
@@ -419,10 +419,10 @@ 
             $next = $this->list->getNext();
             if (
                 ($next->type !== Token::TYPE_KEYWORD
-                    || ! in_array($next->value, $this->keywordNameIndicators, true)
+                    || !in_array($next->value, $this->keywordNameIndicators, true)
                 )
                 && ($next->type !== Token::TYPE_OPERATOR
-                    || ! in_array($next->value, $this->operatorNameIndicators, true)
+                    || !in_array($next->value, $this->operatorNameIndicators, true)
                 )
                 && ($next->value !== null)
             ) {
@@ -505,7 +505,7 @@ 
             $token .= $this->str[$this->last];
             $flags = Context::isKeyword($token);
 
-            if (($this->last + 1 !== $this->len && ! Context::isSeparator($this->str[$this->last + 1])) || ! $flags) {
+            if (($this->last + 1 !== $this->len && !Context::isSeparator($this->str[$this->last + 1])) || !$flags) {
                 continue;
             }
 
@@ -593,7 +593,7 @@ 
             $token .= $this->str[$this->last];
             $flags = Context::isOperator($token);
 
-            if (! $flags) {
+            if (!$flags) {
                 continue;
             }
 
@@ -615,7 +615,7 @@ 
     {
         $token = $this->str[$this->last];
 
-        if (! Context::isWhitespace($token)) {
+        if (!Context::isWhitespace($token)) {
             return null;
         }
 
@@ -855,7 +855,7 @@ 
             } elseif ($state === 2) {
                 $flags |= Token::FLAG_NUMBER_HEX;
                 if (
-                    ! (
+                    !(
                         ($this->str[$this->last] >= '0' && $this->str[$this->last] <= '9')
                         || ($this->str[$this->last] >= 'A' && $this->str[$this->last] <= 'F')
                         || ($this->str[$this->last] >= 'a' && $this->str[$this->last] <= 'f')
@@ -958,7 +958,7 @@ 
         $token = $this->str[$this->last];
         $flags = Context::isString($token);
 
-        if (! $flags && $token !== $quote) {
+        if (!$flags && $token !== $quote) {
             return null;
         }
 
@@ -1010,7 +1010,7 @@ 
         $token = $this->str[$this->last];
         $flags = Context::isSymbol($token);
 
-        if (! $flags) {
+        if (!$flags) {
             return null;
         }
 
@@ -1061,7 +1061,7 @@ 
             return null;
         }
 
-        while (++$this->last < $this->len && ! Context::isSeparator($this->str[$this->last])) {
+        while (++$this->last < $this->len && !Context::isSeparator($this->str[$this->last])) {
             $token .= $this->str[$this->last];
 
             // Test if end of token equals the current delimiter. If so, remove it from the token.

src/Parser.php

Spacing +3 added lines, -3 removed lines

@@ -496,7 +496,7 @@ 
                 $list->idx = $lastIdx;
             } elseif (empty(static::$statementParsers[$token->keyword])) {
                 // Checking if it is a known statement that can be parsed.
-                if (! isset(static::$statementParsers[$token->keyword])) {
+                if (!isset(static::$statementParsers[$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.
@@ -536,7 +536,7 @@ 
 
             // Handles unions.
             if (
-                ! empty($unionType)
+                !empty($unionType)
                 && ($lastStatement instanceof SelectStatement)
                 && ($statement instanceof SelectStatement)
             ) {
@@ -623,7 +623,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/Utils/Formatter.php

Spacing +6 added lines, -6 removed lines

@@ -436,7 +436,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 !== Token::TYPE_KEYWORD
@@ -496,7 +496,7 @@ 
                         end($blocksLineEndings) === true
                         || (
                             empty(self::$inlineClauses[$lastClause])
-                            && ! $shortGroup
+                            && !$shortGroup
                             && $this->options['parts_newline']
                         )
                     ) {
@@ -532,7 +532,7 @@ 
                     $lineEnded = false;
                 } elseif (
                     $prev->keyword === 'DELIMITER'
-                    || ! (
+                    || !(
                     ($prev->type === Token::TYPE_OPERATOR && ($prev->value === '.' || $prev->value === '('))
                     // No space after . (
                     || ($curr->type === Token::TYPE_OPERATOR
@@ -648,12 +648,12 @@ 
         static $prev;
 
         foreach ($this->options['formats'] as $format) {
-            if ($token->type !== $format['type'] || ! (($token->flags & $format['flags']) === $format['flags'])) {
+            if ($token->type !== $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);
             }
@@ -764,7 +764,7 @@ 
      *
      * @psalm-return 1|2|false
      */
-    public static function isClause($token): int|false
+    public static function isClause($token): int | false
     {
         if (
             ($token->type === Token::TYPE_KEYWORD && isset(Parser::$statementParsers[$token->keyword]))

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 [];
             }
@@ -847,7 +847,7 @@ 
 
             $statement .= $token->token;
 
-            if (($token->type === Token::TYPE_DELIMITER) && ! empty($token->token)) {
+            if (($token->type === Token::TYPE_DELIMITER) && !empty($token->token)) {
                 $delimiter = $token->token;
                 $fullStatement = true;
                 break;
@@ -856,7 +856,7 @@ 
 
         // No statement was found so we return the entire query as being the
         // remaining part.
-        if (! $fullStatement) {
+        if (!$fullStatement) {
             return [
                 null,
                 $query,

src/Context.php

Spacing +12 added lines, -12 removed lines

@@ -357,13 +357,13 @@ 
      *
      * @param bool $isReserved checks if the keyword is reserved
      */
-    public static function isKeyword(string $string, bool $isReserved = false): int|null
+    public static function isKeyword(string $string, bool $isReserved = false): int | null
     {
         $upperString = strtoupper($string);
 
         if (
-            ! isset(static::$keywords[$upperString])
-            || ($isReserved && ! (static::$keywords[$upperString] & Token::FLAG_KEYWORD_RESERVED))
+            !isset(static::$keywords[$upperString])
+            || ($isReserved && !(static::$keywords[$upperString] & Token::FLAG_KEYWORD_RESERVED))
         ) {
             return null;
         }
@@ -374,7 +374,7 @@ 
     /**
      * Checks if the given string is an operator and returns the appropriate flag for the operator.
      */
-    public static function isOperator(string $string): int|null
+    public static function isOperator(string $string): int | null
     {
         return static::$operators[$string] ?? null;
     }
@@ -392,7 +392,7 @@ 
      *
      * @return int|null the appropriate flag for the comment type
      */
-    public static function isComment(string $string, bool $end = false): int|null
+    public static function isComment(string $string, bool $end = false): int | null
     {
         if ($string === '') {
             return null;
@@ -455,7 +455,7 @@ 
      *
      * @return int|null the appropriate flag for the symbol type
      */
-    public static function isSymbol(string $string): int|null
+    public static function isSymbol(string $string): int | null
     {
         if ($string === '') {
             return null;
@@ -483,7 +483,7 @@ 
      *
      * @return int|null the appropriate flag for the string type
      */
-    public static function isString(string $string): int|null
+    public static function isString(string $string): int | null
     {
         if ($string === '') {
             return null;
@@ -537,7 +537,7 @@ 
             $context = self::$contextPrefix . $context;
         }
 
-        if (! class_exists($context)) {
+        if (!class_exists($context)) {
             return false;
         }
 
@@ -559,7 +559,7 @@ 
      *
      * @return string|null The loaded context. `null` if no context was loaded.
      */
-    public static function loadClosest(string $context = ''): string|null
+    public static function loadClosest(string $context = ''): string | null
     {
         $length = strlen($context);
         for ($i = $length; $i > 0;) {
@@ -573,7 +573,7 @@ 
                 $i -= 2;
                 $part = substr($context, $i, 2);
                 /* No more numeric parts to strip */
-                if (! is_numeric($part)) {
+                if (!is_numeric($part)) {
                     break 2;
                 }
             } while (intval($part) === 0 && $i > 0);
@@ -675,7 +675,7 @@ 
      */
     public static function escape(string $str, string $quote = '`')
     {
-        if ((static::$mode & self::SQL_MODE_NO_ENCLOSING_QUOTES) && (! static::isKeyword($str, true))) {
+        if ((static::$mode & self::SQL_MODE_NO_ENCLOSING_QUOTES) && (!static::isKeyword($str, true))) {
             return $str;
         }
 
@@ -719,7 +719,7 @@ 
      *
      * @return bool false on empty param, true/false on given constant/int value
      */
-    public static function hasMode(int|null $flag = null): bool
+    public static function hasMode(int | null $flag = null): bool
     {
         if (empty($flag)) {
             return false;

src/Statements/ReplaceStatement.php

Spacing +2 added lines, -2 removed lines

@@ -63,7 +63,7 @@ 
      *
      * @var ArrayObj[]|null
      */
-    public array|null $values = null;
+    public array | null $values = null;
 
     /**
      * If SET clause is present
@@ -71,7 +71,7 @@ 
      *
      * @var SetOperation[]|null
      */
-    public array|null $set = null;
+    public array | null $set = null;
 
     /**
      * If SELECT clause is present

src/Components/AlterOperation.php

Spacing +3 added lines, -3 removed lines

@@ -424,7 +424,7 @@ 
                     } elseif (($token->value === ',') && ($brackets === 0)) {
                         break;
                     }
-                } elseif (! self::checkIfTokenQuotedSymbol($token)) {
+                } elseif (!self::checkIfTokenQuotedSymbol($token)) {
                     // If the current token is "SET" or "ENUM", we want to avoid the token between their parenthesis in
                     // the unknown tokens.
                     if (in_array($token->value, ['SET', 'ENUM'], true)) {
@@ -443,7 +443,7 @@ 
                             );
                             break;
                         }
-                    } elseif (! empty(Parser::$statementParsers[$token->value])) {
+                    } elseif (!empty(Parser::$statementParsers[$token->value])) {
                         // We have reached the end of ALTER operation and suddenly found
                         // a start to new statement, but have not found a delimiter between them
                         $parser->error(
@@ -454,7 +454,7 @@ 
                     } elseif (
                         (array_key_exists($arrayKey, self::$databaseOptions)
                         || array_key_exists($arrayKey, self::$tableOptions))
-                        && ! self::checkIfColumnDefinitionKeyword($arrayKey)
+                        && !self::checkIfColumnDefinitionKeyword($arrayKey)
                     ) {
                         // This alter operation has finished, which means a comma
                         // was missing before start of new alter operation

tests/Lexer/ContextTest.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
      * Test for loading closest SQL context
      */
     #[DataProvider('contextLoadingProvider')]
-    public function testLoadClosest(string $context, string|null $expected): void
+    public function testLoadClosest(string $context, string | null $expected): void
     {
         $this->assertEquals($expected, Context::loadClosest($context));
         if ($expected !== null) {

tests/Misc/UtfStringTest.php

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
      * Test access to string.
      */
     #[DataProvider('utf8StringsProvider')]
-    public function testAccess(string $text, string|null $pos10, string|null $pos20): void
+    public function testAccess(string $text, string | null $pos10, string | null $pos20): void
     {
         $str = new UtfString($text);
         $this->assertEquals($pos10, $str->offsetGet(10));