phpmyadmin / sql-parser

src/Utils/Table.php

Spacing +6 added lines, -6 removed lines

@@ -26,7 +26,7 @@ 
      */
     public static function getForeignKeys($statement)
     {
-        if (empty($statement->fields) || (! is_array($statement->fields)) || (! $statement->options->has('TABLE'))) {
+        if (empty($statement->fields) || (!is_array($statement->fields)) || (!$statement->options->has('TABLE'))) {
             return [];
         }
 
@@ -39,7 +39,7 @@ 
 
             $columns = [];
             foreach ($field->key->columns as $column) {
-                if (! isset($column['name'])) {
+                if (!isset($column['name'])) {
                     continue;
                 }
 
@@ -51,7 +51,7 @@ 
                 'index_list' => $columns,
             ];
 
-            if (! empty($field->references)) {
+            if (!empty($field->references)) {
                 $tmp['ref_db_name'] = $field->references->table->database;
                 $tmp['ref_table_name'] = $field->references->table->table;
                 $tmp['ref_index_list'] = $field->references->columns;
@@ -84,7 +84,7 @@ 
      */
     public static function getFields($statement)
     {
-        if (empty($statement->fields) || (! is_array($statement->fields)) || (! $statement->options->has('TABLE'))) {
+        if (empty($statement->fields) || (!is_array($statement->fields)) || (!$statement->options->has('TABLE'))) {
             return [];
         }
 
@@ -101,7 +101,7 @@ 
                 'timestamp_not_null' => false,
             ];
 
-            if (! $field->options) {
+            if (!$field->options) {
                 continue;
             }
 
@@ -128,7 +128,7 @@ 
 
             $option = $field->options->has('AS');
 
-            if (! $option) {
+            if (!$option) {
                 continue;
             }
 

src/Components/Key.php

Spacing +5 added lines, -5 removed lines

@@ -163,9 +163,9 @@ 
             } elseif ($state === 1) {
                 if (($token->type === Token::TYPE_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
@@ -182,7 +182,7 @@ 
                         $state = 3;
                     } elseif (($token->value === ',') || ($token->value === ')')) {
                         $state = $token->value === ',' ? 2 : 4;
-                        if (! empty($lastColumn)) {
+                        if (!empty($lastColumn)) {
                             $ret->columns[] = $lastColumn;
                             $lastColumn = [];
                         }
@@ -214,7 +214,7 @@ 
                 if ($token->type === Token::TYPE_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;
                     }
 
@@ -256,7 +256,7 @@ 
     public static function build($component, array $options = [])
     {
         $ret = $component->type . ' ';
-        if (! empty($component->name)) {
+        if (!empty($component->name)) {
             $ret .= Context::escape($component->name) . ' ';
         }
 

src/UtfString.php

Spacing +2 added lines, -2 removed lines

@@ -190,7 +190,7 @@ 
         $this->byteIdx = 0;
         $this->charIdx = 0;
         $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');
@@ -298,7 +298,7 @@ 
         // Use the default ASCII map as queries are mostly ASCII chars
         // ord($byte) has a performance cost
 
-        if (! isset(static::$asciiMap[$byte])) {
+        if (!isset(static::$asciiMap[$byte])) {
             // Complete the cache with missing items
             static::$asciiMap[$byte] = ord($byte);
         }

src/Statements/CreateStatement.php

Spacing +10 added lines, -10 removed lines

@@ -417,7 +417,7 @@ 
     public function build()
     {
         $fields = '';
-        if (! empty($this->fields)) {
+        if (!empty($this->fields)) {
             if (is_array($this->fields)) {
                 $fields = CreateDefinition::build($this->fields) . ' ';
             } elseif ($this->fields instanceof ArrayObj) {
@@ -449,23 +449,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" . PartitionDefinition::build($this->partitions);
             }
 
@@ -487,7 +487,7 @@ 
                 . OptionsArray::build($this->options) . ' '
                 . Expression::build($this->name) . ' '
                 . $fields . ' AS ' . $builtStatement
-                . (! empty($this->body) ? TokensList::build($this->body) : '') . ' '
+                . (!empty($this->body) ? TokensList::build($this->body) : '') . ' '
                 . OptionsArray::build($this->entityOptions);
         } elseif ($this->options->has('TRIGGER')) {
             return 'CREATE '
@@ -541,7 +541,7 @@ 
             ]
         );
 
-        if (! isset($this->name) || ($this->name === '')) {
+        if (!isset($this->name) || ($this->name === '')) {
             $parser->error('The name of the entity was expected.', $list->tokens[$list->idx]);
         } else {
             ++$list->idx; // Skipping field.
@@ -657,7 +657,7 @@ 
                         $token = $list->getNextOfType(Token::TYPE_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`.
                          */
@@ -686,7 +686,7 @@ 
                             $field = null;
                         }
                     } elseif (($token->type === Token::TYPE_OPERATOR) && ($token->value === '(')) {
-                        if (! empty($this->partitionBy)) {
+                        if (!empty($this->partitionBy)) {
                             $this->partitions = ArrayObj::parse(
                                 $parser,
                                 $list,

src/Statements/WithStatement.php

Indentation +3 added lines, -4 removed lines

@@ -22,7 +22,6 @@ 
 
 /**
  * `WITH` statement.
-
  *  WITH [RECURSIVE] query_name [ (column_name [,...]) ] AS (SELECT ...) [, ...]
  */
 final class WithStatement extends Statement
@@ -263,9 +262,9 @@ 
 
         // 5 is the only valid end state
         if ($state !== 5) {
-             /**
-             * Token parsed at this moment.
-             */
+                /**
+                 * Token parsed at this moment.
+                 */
             $token = $list->tokens[$list->idx];
 
             $parser->error('Unexpected end of the WITH CTE.', $token);

Spacing +3 added lines, -3 removed lines

@@ -133,7 +133,7 @@ 
                     break;
                 }
             } elseif ($state === 2) {
-                if (! ($token->type === Token::TYPE_KEYWORD && $token->keyword === 'AS')) {
+                if (!($token->type === Token::TYPE_KEYWORD && $token->keyword === 'AS')) {
                     $parser->error('AS keyword was expected.', $token);
                     break;
                 }
@@ -145,7 +145,7 @@ 
                 $list->idx++; // Ignore the current token
                 $nextKeyword = $list->getNext();
 
-                if (! ($token->value === '(' && ($nextKeyword && $nextKeyword->value === 'SELECT'))) {
+                if (!($token->value === '(' && ($nextKeyword && $nextKeyword->value === 'SELECT'))) {
                     $parser->error('Subquery of the CTE was expected.', $token);
                     $list->idx = $idxBeforeGetNext;
                     break;
@@ -313,7 +313,7 @@ 
             }
 
             ++$list->idx;
-            if (! isset($list->tokens[$list->idx])) {
+            if (!isset($list->tokens[$list->idx])) {
                 break;
             }
 

src/Tools/TestGenerator.php

Spacing +6 added lines, -6 removed lines

@@ -82,7 +82,7 @@ 
         // The code below extracts only the relevant information.
 
         // Extracting lexer's errors.
-        if (! empty($lexer->errors)) {
+        if (!empty($lexer->errors)) {
             /** @var LexerException $err */
             foreach ($lexer->errors as $err) {
                 $lexerErrors[] = [
@@ -97,7 +97,7 @@ 
         }
 
         // Extracting parser's errors.
-        if (! empty($parser->errors)) {
+        if (!empty($parser->errors)) {
             /** @var ParserException $err */
             foreach ($parser->errors as $err) {
                 $parserErrors[] = [
@@ -137,7 +137,7 @@ 
     public static function build($type, $input, $output, $debug = null, $ansi = false)
     {
         // Support query types: `lexer` / `parser`.
-        if (! in_array($type, ['lexer', 'parser'])) {
+        if (!in_array($type, ['lexer', 'parser'])) {
             throw new Exception('Unknown test type (expected `lexer` or `parser`).');
         }
 
@@ -223,11 +223,11 @@ 
             if (is_dir($inputFile)) {
                 // Creating required directories to maintain the structure.
                 // Ignoring errors if the folder structure exists already.
-                if (! is_dir($outputFile)) {
+                if (!is_dir($outputFile)) {
                     mkdir($outputFile);
                 }
 
-                if (($debug !== null) && (! is_dir($debugFile))) {
+                if (($debug !== null) && (!is_dir($debugFile))) {
                     mkdir($debugFile);
                 }
 
@@ -242,7 +242,7 @@ 
                 }
 
                 // Building the test.
-                if (! file_exists($outputFile)) {
+                if (!file_exists($outputFile)) {
                     echo sprintf("Building test for %s...\n", $inputFile);
                     static::build(
                         str_contains($inputFile, 'lex') ? 'lexer' : 'parser',

src/Parser.php

Spacing +2 added lines, -2 removed lines

@@ -489,7 +489,7 @@ 
             } else {
                 // Checking if it is a known statement that can be parsed.
                 if (empty(static::$STATEMENT_PARSERS[$token->keyword])) {
-                    if (! isset(static::$STATEMENT_PARSERS[$token->keyword])) {
+                    if (!isset(static::$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.
@@ -530,7 +530,7 @@ 
 
             // Handles unions.
             if (
-                ! empty($unionType)
+                !empty($unionType)
                 && ($lastStatement instanceof SelectStatement)
                 && ($statement instanceof SelectStatement)
             ) {

src/Lexer.php

Spacing +15 added lines, -15 removed lines

@@ -15,7 +15,7 @@ 
 use function strlen;
 use function substr;
 
-if (! defined('USE_UTF_STRINGS')) {
+if (!defined('USE_UTF_STRINGS')) {
     // NOTE: In previous versions of PHP (5.5 and older) the default
     // internal encoding is "ISO-8859-1".
     // All `mb_` functions must specify the correct encoding, which is
@@ -199,7 +199,7 @@ 
 
         // For multi-byte strings, a new instance of `UtfString` is
         // initialized (only if `UtfString` usage is forced.
-        if (! $str instanceof UtfString && USE_UTF_STRINGS && $len !== mb_strlen($str, 'UTF-8')) {
+        if (!$str instanceof UtfString && USE_UTF_STRINGS && $len !== mb_strlen($str, 'UTF-8')) {
             $str = new UtfString($str);
         }
 
@@ -209,7 +209,7 @@ 
         $this->strict = $strict;
 
         // Setting the delimiter.
-        $this->setDelimiter(! empty($delimiter) ? $delimiter : static::$DEFAULT_DELIMITER);
+        $this->setDelimiter(!empty($delimiter) ? $delimiter : static::$DEFAULT_DELIMITER);
 
         $this->lex();
     }
@@ -337,7 +337,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];
@@ -396,8 +396,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;
             }
@@ -435,10 +435,10 @@ 
             $next = $this->list->getNext();
             if (
                 ($next->type !== Token::TYPE_KEYWORD
-                    || ! in_array($next->value, $this->KEYWORD_NAME_INDICATORS, true)
+                    || !in_array($next->value, $this->KEYWORD_NAME_INDICATORS, true)
                 )
                 && ($next->type !== Token::TYPE_OPERATOR
-                    || ! in_array($next->value, $this->OPERATOR_NAME_INDICATORS, true)
+                    || !in_array($next->value, $this->OPERATOR_NAME_INDICATORS, true)
                 )
                 && ($next->value !== null)
             ) {
@@ -521,7 +521,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;
             }
 
@@ -609,7 +609,7 @@ 
             $token .= $this->str[$this->last];
             $flags = Context::isOperator($token);
 
-            if (! $flags) {
+            if (!$flags) {
                 continue;
             }
 
@@ -631,7 +631,7 @@ 
     {
         $token = $this->str[$this->last];
 
-        if (! Context::isWhitespace($token)) {
+        if (!Context::isWhitespace($token)) {
             return null;
         }
 
@@ -871,7 +871,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')
@@ -974,7 +974,7 @@ 
         $token = $this->str[$this->last];
         $flags = Context::isString($token);
 
-        if (! $flags && $token !== $quote) {
+        if (!$flags && $token !== $quote) {
             return null;
         }
 
@@ -1026,7 +1026,7 @@ 
         $token = $this->str[$this->last];
         $flags = Context::isSymbol($token);
 
-        if (! $flags) {
+        if (!$flags) {
             return null;
         }
 
@@ -1077,7 +1077,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.

tests/Tools/ContextGeneratorTest.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
         $this->assertEquals([
             Token::TYPE_KEYWORD | Token::FLAG_KEYWORD_RESERVED => [
                 8 => ['RESERVED'],
-                9 => ['RESERVED2','RESERVED3','RESERVED4','RESERVED5'],
+                9 => ['RESERVED2', 'RESERVED3', 'RESERVED4', 'RESERVED5'],
             ],
             Token::TYPE_KEYWORD | Token::FLAG_KEYWORD_FUNCTION => [8 => ['FUNCTION']],
             Token::TYPE_KEYWORD | Token::FLAG_KEYWORD_DATA_TYPE => [8 => ['DATATYPE']],