phpmyadmin / sql-parser

tools/TestGenerator.php

Spacing +3 added lines, -3 removed lines

@@ -27,15 +27,15 @@
 $debug = empty($argv[3]) ? null : rtrim($argv[3], '/');
 
 // Checking if all directories are valid.
-if (! is_dir($input)) {
+if (!is_dir($input)) {
     throw new Exception('The input directory does not exist.');
 }
 
-if (! is_dir($output)) {
+if (!is_dir($output)) {
     throw new Exception('The output directory does not exist.');
 }
 
-if (($debug !== null) && (! is_dir($debug))) {
+if (($debug !== null) && (!is_dir($debug))) {
     throw new Exception('The debug directory does not exist.');
 }
 

tools/ContextGenerator.php

Spacing +2 added lines, -2 removed lines

@@ -26,11 +26,11 @@
 $output = rtrim($argv[2], '/');
 
 // Checking if all directories are valid.
-if (! is_dir($input)) {
+if (!is_dir($input)) {
     throw new Exception('The input directory does not exist.');
 }
 
-if (! is_dir($output)) {
+if (!is_dir($output)) {
     throw new Exception('The output directory does not exist.');
 }
 

src/Context.php

Spacing +5 added lines, -5 removed lines

@@ -278,7 +278,7 @@ 
         $str = strtoupper($str);
 
         if (isset(static::$KEYWORDS[$str])) {
-            if ($isReserved && ! (static::$KEYWORDS[$str] & Token::FLAG_KEYWORD_RESERVED)) {
+            if ($isReserved && !(static::$KEYWORDS[$str] & Token::FLAG_KEYWORD_RESERVED)) {
                 return null;
             }
 
@@ -300,7 +300,7 @@ 
      */
     public static function isOperator($str)
     {
-        if (! isset(static::$OPERATORS[$str])) {
+        if (!isset(static::$OPERATORS[$str])) {
             return null;
         }
 
@@ -506,7 +506,7 @@ 
             $context = self::$contextPrefix . $context;
         }
 
-        if (! class_exists($context)) {
+        if (!class_exists($context)) {
             throw @new LoaderException('Specified context ("' . $context . '") does not exist.', $context);
         }
 
@@ -542,7 +542,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);
@@ -599,7 +599,7 @@ 
             return $str;
         }
 
-        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;
         }
 

src/Components/OptionsArray.php

Spacing +10 added lines, -10 removed lines

@@ -123,7 +123,7 @@ 
 
             if ($lastOption === null) {
                 $upper = strtoupper($token->token);
-                if (! isset($options[$upper])) {
+                if (!isset($options[$upper])) {
                     // There is no option to be processed.
                     break;
                 }
@@ -160,7 +160,7 @@ 
             }
 
             if ($state === 0) {
-                if (! is_array($lastOption)) {
+                if (!is_array($lastOption)) {
                     // This is a just keyword option without any value.
                     // This is the beginning and the end of it.
                     $ret->options[$lastOptionId] = $token->value;
@@ -234,7 +234,7 @@ 
                 $ret->options[$lastOptionId]['expr'] .= $token->token;
 
                 if (
-                    ! (($token->token === '(') && ($brackets === 1)
+                    !(($token->token === '(') && ($brackets === 1)
                     || (($token->token === ')') && ($brackets === 0)))
                 ) {
                     // First pair of brackets is being skipped.
@@ -292,12 +292,12 @@ 
 
         $options = [];
         foreach ($component->options as $option) {
-            if (! is_array($option)) {
+            if (!is_array($option)) {
                 $options[] = $option;
             } else {
                 $options[] = $option['name']
-                    . (! empty($option['equals']) && $option['equals'] ? '=' : ' ')
-                    . (! empty($option['expr']) ? $option['expr'] : $option['value']);
+                    . (!empty($option['equals']) && $option['equals'] ? '=' : ' ')
+                    . (!empty($option['expr']) ? $option['expr'] : $option['value']);
             }
         }
 
@@ -317,10 +317,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;
             }
         }
@@ -339,12 +339,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/Components/GroupKeyword.php

Spacing +2 added lines, -2 removed lines

@@ -94,7 +94,7 @@ 
                 ) {
                     $expr->type = $token->keyword;
                 } elseif (($token->type === Token::TYPE_OPERATOR) && ($token->value === ',')) {
-                    if (! empty($expr->expr)) {
+                    if (!empty($expr->expr)) {
                         $ret[] = $expr;
                     }
 
@@ -107,7 +107,7 @@ 
         }
 
         // Last iteration was not processed.
-        if (! empty($expr->expr)) {
+        if (!empty($expr->expr)) {
             $ret[] = $expr;
         }
 

src/Components/Expression.php

Spacing +17 added lines, -17 removed lines

@@ -201,7 +201,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'];
         }
@@ -229,14 +229,14 @@ 
             }
 
             if ($token->type === Token::TYPE_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)) {
@@ -248,7 +248,7 @@ 
                     }
 
                     if ($token->keyword === 'AS') {
-                        if (! empty($options['breakOnAlias'])) {
+                        if (!empty($options['breakOnAlias'])) {
                             break;
                         }
 
@@ -271,7 +271,7 @@ 
                     }
 
                     $isExpr = true;
-                } elseif ($brackets === 0 && strlen((string) $ret->expr) > 0 && ! $alias) {
+                } elseif ($brackets === 0 && strlen((string) $ret->expr) > 0 && !$alias) {
                     /* End of expression */
                     break;
                 }
@@ -287,7 +287,7 @@ 
                 || (($token->type === Token::TYPE_OPERATOR)
                 && ($token->value !== '.'))
             ) {
-                if (! empty($options['parseField'])) {
+                if (!empty($options['parseField'])) {
                     break;
                 }
 
@@ -297,7 +297,7 @@ 
             }
 
             if ($token->type === Token::TYPE_OPERATOR) {
-                if (! empty($options['breakOnParentheses']) && (($token->value === '(') || ($token->value === ')'))) {
+                if (!empty($options['breakOnParentheses']) && (($token->value === '(') || ($token->value === ')'))) {
                     // No brackets were expected.
                     break;
                 }
@@ -321,7 +321,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;
@@ -347,7 +347,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;
                 }
@@ -361,13 +361,13 @@ 
                     && ($prev[0] === null
                         || (($prev[0]->type !== Token::TYPE_OPERATOR || $prev[0]->token === ')')
                             && ($prev[0]->type !== Token::TYPE_KEYWORD
-                                || ! ($prev[0]->flags & Token::FLAG_KEYWORD_RESERVED))))
+                                || !($prev[0]->flags & Token::FLAG_KEYWORD_RESERVED))))
                     && (($prev[1]->type === Token::TYPE_STRING)
                         || ($prev[1]->type === Token::TYPE_SYMBOL
-                            && ! ($prev[1]->flags & Token::FLAG_SYMBOL_VARIABLE))
+                            && !($prev[1]->flags & Token::FLAG_SYMBOL_VARIABLE))
                         || ($prev[1]->type === Token::TYPE_NONE))
                 ) {
-                    if (! empty($ret->alias)) {
+                    if (!empty($ret->alias)) {
                         $parser->error('An alias was previously found.', $token);
                         break;
                     }
@@ -376,12 +376,12 @@ 
                 } else {
                     $ret->expr .= $token->token;
                 }
-            } elseif (! $isExpr) {
+            } elseif (!$isExpr) {
                 if (($token->type === Token::TYPE_OPERATOR) && ($token->value === '.')) {
                     // 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);
                     }
 
@@ -398,11 +398,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;
                         }
@@ -460,7 +460,7 @@ 
             $ret = implode('.', Context::escape($fields));
         }
 
-        if (! empty($component->alias)) {
+        if (!empty($component->alias)) {
             $ret .= ' AS ' . Context::escape($component->alias);
         }
 

src/Components/CaseExpression.php

Spacing +3 added lines, -3 removed lines

@@ -211,7 +211,7 @@ 
 
                 // Handle optional AS keyword before alias
                 if ($token->type === Token::TYPE_KEYWORD && $token->keyword === 'AS') {
-                    if ($asFound || ! empty($ret->alias)) {
+                    if ($asFound || !empty($ret->alias)) {
                         $parser->error('Potential duplicate alias of CASE expression.', $token);
                         break;
                     }
@@ -233,11 +233,11 @@ 
                 if (
                     $asFound
                     || $token->type === Token::TYPE_STRING
-                    || ($token->type === Token::TYPE_SYMBOL && ! $token->flags & Token::FLAG_SYMBOL_VARIABLE)
+                    || ($token->type === Token::TYPE_SYMBOL && !$token->flags & Token::FLAG_SYMBOL_VARIABLE)
                     || $token->type === Token::TYPE_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/Condition.php

Spacing +3 added lines, -3 removed lines

@@ -154,7 +154,7 @@ 
                 } else {
                     // The expression ended.
                     $expr->expr = trim($expr->expr);
-                    if (! empty($expr->expr)) {
+                    if (!empty($expr->expr)) {
                         $ret[] = $expr;
                     }
 
@@ -172,7 +172,7 @@ 
             if (
                 ($token->type === Token::TYPE_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;
@@ -215,7 +215,7 @@ 
 
         // Last iteration was not processed.
         $expr->expr = trim($expr->expr);
-        if (! empty($expr->expr)) {
+        if (!empty($expr->expr)) {
             $ret[] = $expr;
         }
 

src/Statement.php

Spacing +11 added lines, -11 removed lines

@@ -173,7 +173,7 @@ 
 
             // Checking if this field was already built.
             if ($type & 1) {
-                if (! empty($built[$field])) {
+                if (!empty($built[$field])) {
                     continue;
                 }
 
@@ -186,7 +186,7 @@ 
             }
 
             // Checking if the result of the builder should be added.
-            if (! ($type & 1)) {
+            if (!($type & 1)) {
                 continue;
             }
 
@@ -317,10 +317,10 @@ 
 
             // Looking for duplicated clauses.
             if (
-                ! empty(Parser::$KEYWORD_PARSERS[$token->value])
-                || ! empty(Parser::$STATEMENT_PARSERS[$token->value])
+                !empty(Parser::$KEYWORD_PARSERS[$token->value])
+                || !empty(Parser::$STATEMENT_PARSERS[$token->value])
             ) {
-                if (! empty($parsedClauses[$token->value])) {
+                if (!empty($parsedClauses[$token->value])) {
                     $parser->error('This type of clause was previously parsed.', $token);
                     break;
                 }
@@ -333,18 +333,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
@@ -359,7 +359,7 @@ 
                     break;
                 }
 
-                if (! $parsedOptions) {
+                if (!$parsedOptions) {
                     if (empty(static::$OPTIONS[$token->value])) {
                         // Skipping keyword because if it is not a option.
                         ++$list->idx;
@@ -529,7 +529,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')) {