phpmyadmin / sql-parser

tests/Utils/CLITest.php

Spacing +6 added lines, -6 removed lines

@@ -50,7 +50,7 @@ 
 
     /** @param array<string, bool|string>|false $getopt */
     #[DataProvider('highlightParamsProvider')]
-    public function testRunHighlight(array|false $getopt, string $output, int $result): void
+    public function testRunHighlight(array | false $getopt, string $output, int $result): void
     {
         $cli = $this->getCLI($getopt);
         $this->expectOutputString($output);
@@ -122,7 +122,7 @@ 
 
     /** @param array<string, bool|string>|false $getopt */
     #[DataProvider('highlightParamsStdInProvider')]
-    public function testRunHighlightStdIn(string $input, array|false $getopt, string $output, int $result): void
+    public function testRunHighlightStdIn(string $input, array | false $getopt, string $output, int $result): void
     {
         $cli = $this->getCLIStdIn($input, $getopt);
         $this->expectOutputString($output);
@@ -187,7 +187,7 @@ 
 
     /** @param array<string, bool|string>|false $getopt */
     #[DataProvider('lintParamsStdInProvider')]
-    public function testRunLintFromStdIn(string $input, array|false $getopt, string $output, int $result): void
+    public function testRunLintFromStdIn(string $input, array | false $getopt, string $output, int $result): void
     {
         $cli = $this->getCLIStdIn($input, $getopt);
         $this->expectOutputString($output);
@@ -249,7 +249,7 @@ 
 
     /** @param array<string, bool|string>|false $getopt */
     #[DataProvider('lintParamsProvider')]
-    public function testRunLint(array|false $getopt, string $output, int $result): void
+    public function testRunLint(array | false $getopt, string $output, int $result): void
     {
         $cli = $this->getCLI($getopt);
         $this->expectOutputString($output);
@@ -313,7 +313,7 @@ 
 
     /** @param array<string, bool|string>|false $getopt */
     #[DataProvider('tokenizeParamsProvider')]
-    public function testRunTokenize(array|false $getopt, string $output, int $result): void
+    public function testRunTokenize(array | false $getopt, string $output, int $result): void
     {
         $cli = $this->getCLI($getopt);
         $this->expectOutputString($output);
@@ -365,7 +365,7 @@ 
 
     /** @param array<string, bool|string>|false $getopt */
     #[DataProvider('tokenizeParamsStdInProvider')]
-    public function testRunTokenizeStdIn(string $input, array|false $getopt, string $output, int $result): void
+    public function testRunTokenizeStdIn(string $input, array | false $getopt, string $output, int $result): void
     {
         $cli = $this->getCLIStdIn($input, $getopt);
         $this->expectOutputString($output);

tests/Lexer/ContextTest.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,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) {
@@ -118,7 +118,7 @@ 
     }
 
     #[DataProvider('providerForTestMode')]
-    public function testMode(int|string $mode, int $expected): void
+    public function testMode(int | string $mode, int $expected): void
     {
         Context::setMode($mode);
         $this->assertSame($expected, Context::getMode());

src/Lexer.php

Spacing +29 added lines, -29 removed lines

@@ -67,7 +67,7 @@ 
     /**
      * The string to be parsed.
      */
-    public string|UtfString $str = '';
+    public string | UtfString $str = '';
 
     /**
      * The length of `$str`.
@@ -113,7 +113,7 @@ 
      *                                    enabled or not
      * @param string           $delimiter the delimiter to be used
      */
-    public function __construct(string|UtfString $str, bool $strict = false, string|null $delimiter = null)
+    public function __construct(string | UtfString $str, bool $strict = false, string | null $delimiter = null)
     {
         if (Context::$keywords === []) {
             Context::load();
@@ -124,7 +124,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);
         }
 
@@ -134,7 +134,7 @@ 
         $this->strict = $strict;
 
         // Setting the delimiter.
-        $this->setDelimiter(! empty($delimiter) ? $delimiter : static::$defaultDelimiter);
+        $this->setDelimiter(!empty($delimiter) ? $delimiter : static::$defaultDelimiter);
 
         $this->lex();
     }
@@ -243,7 +243,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];
@@ -302,8 +302,8 @@ 
             }
 
             if (
-                ($next->type !== TokenType::Keyword || ! in_array($next->value, ['FROM', 'USING'], true))
-                && ($next->type !== TokenType::Operator || ! in_array($next->value, [',', ')'], true))
+                ($next->type !== TokenType::Keyword || !in_array($next->value, ['FROM', 'USING'], true))
+                && ($next->type !== TokenType::Operator || !in_array($next->value, [',', ')'], true))
             ) {
                 continue;
             }
@@ -341,10 +341,10 @@ 
             $next = $this->list->getNext();
             if (
                 ($next->type !== TokenType::Keyword
-                    || ! in_array($next->value, self::KEYWORD_NAME_INDICATORS, true)
+                    || !in_array($next->value, self::KEYWORD_NAME_INDICATORS, true)
                 )
                 && ($next->type !== TokenType::Operator
-                    || ! in_array($next->value, self::OPERATOR_NAME_INDICATORS, true)
+                    || !in_array($next->value, self::OPERATOR_NAME_INDICATORS, true)
                 )
                 && ($next->value !== '')
             ) {
@@ -388,7 +388,7 @@ 
     /**
      * Parses a keyword.
      */
-    public function parseKeyword(): Token|null
+    public function parseKeyword(): Token | null
     {
         $token = '';
 
@@ -428,7 +428,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;
             }
 
@@ -448,7 +448,7 @@ 
     /**
      * Parses a label.
      */
-    public function parseLabel(): Token|null
+    public function parseLabel(): Token | null
     {
         $token = '';
 
@@ -492,7 +492,7 @@ 
     /**
      * Parses an operator.
      */
-    public function parseOperator(): Token|null
+    public function parseOperator(): Token | null
     {
         $token = '';
 
@@ -512,7 +512,7 @@ 
             $token .= $this->str[$this->last];
             $flags = Context::isOperator($token);
 
-            if (! $flags) {
+            if (!$flags) {
                 continue;
             }
 
@@ -528,11 +528,11 @@ 
     /**
      * Parses a whitespace.
      */
-    public function parseWhitespace(): Token|null
+    public function parseWhitespace(): Token | null
     {
         $token = $this->str[$this->last];
 
-        if (! Context::isWhitespace($token)) {
+        if (!Context::isWhitespace($token)) {
             return null;
         }
 
@@ -548,7 +548,7 @@ 
     /**
      * Parses a comment.
      */
-    public function parseComment(): Token|null
+    public function parseComment(): Token | null
     {
         $iBak = $this->last;
         $token = $this->str[$this->last];
@@ -665,7 +665,7 @@ 
     /**
      * Parses a boolean.
      */
-    public function parseBool(): Token|null
+    public function parseBool(): Token | null
     {
         if ($this->last + 3 >= $this->len) {
             // At least `min(strlen('TRUE'), strlen('FALSE'))` characters are
@@ -696,7 +696,7 @@ 
     /**
      * Parses a number.
      */
-    public function parseNumber(): Token|null
+    public function parseNumber(): Token | null
     {
         // A rudimentary state machine is being used to parse numbers due to
         // the various forms of their notation.
@@ -763,7 +763,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')
@@ -859,12 +859,12 @@ 
      *
      * @throws LexerException
      */
-    public function parseString(string $quote = ''): Token|null
+    public function parseString(string $quote = ''): Token | null
     {
         $token = $this->str[$this->last];
         $flags = Context::isString($token);
 
-        if (! $flags && $token !== $quote) {
+        if (!$flags && $token !== $quote) {
             return null;
         }
 
@@ -909,12 +909,12 @@ 
      *
      * @throws LexerException
      */
-    public function parseSymbol(): Token|null
+    public function parseSymbol(): Token | null
     {
         $token = $this->str[$this->last];
         $flags = Context::isSymbol($token);
 
-        if (! $flags) {
+        if (!$flags) {
             return null;
         }
 
@@ -940,7 +940,7 @@ 
             if ($str === null) {
                 $str = $this->parseUnknown();
 
-                if ($str === null && ! ($flags & Token::FLAG_SYMBOL_PARAMETER)) {
+                if ($str === null && !($flags & Token::FLAG_SYMBOL_PARAMETER)) {
                     $this->error('Variable name was expected.', $this->str[$this->last], $this->last);
                 }
             }
@@ -956,14 +956,14 @@ 
     /**
      * Parses unknown parts of the query.
      */
-    public function parseUnknown(): Token|null
+    public function parseUnknown(): Token | null
     {
         $token = $this->str[$this->last];
         if (Context::isSeparator($token)) {
             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.
@@ -982,7 +982,7 @@ 
     /**
      * Parses the delimiter of the query.
      */
-    public function parseDelimiter(): Token|null
+    public function parseDelimiter(): Token | null
     {
         $idx = 0;
 
@@ -999,7 +999,7 @@ 
         return new Token($this->delimiter, TokenType::Delimiter);
     }
 
-    private function parse(): Token|null
+    private function parse(): Token | null
     {
         // It is best to put the parsers in order of their complexity
         // (ascending) and their occurrence rate (descending).

src/Statements/OptimizeStatement.php

Spacing +1 added lines, -1 removed lines

@@ -33,5 +33,5 @@
      *
      * @var Expression[]|null
      */
-    public array|null $tables = null;
+    public array | null $tables = null;
 }

tests/TestCase.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
      *     : list<array{string, Token|null, int}>
      * )
      */
-    public function getErrorsAsArray(Lexer|Parser $obj): array
+    public function getErrorsAsArray(Lexer | Parser $obj): array
     {
         $ret = [];
         if ($obj instanceof Lexer) {

src/Token.php

Spacing +4 added lines, -4 removed lines

@@ -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
@@ -118,7 +118,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;
@@ -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',
                     );

src/Components/Condition.php

Spacing +5 added lines, -5 removed lines

@@ -78,7 +78,7 @@ 
     public $expr;
 
     /** @param string $expr the condition or the operator */
-    public function __construct(string|null $expr = null)
+    public function __construct(string | null $expr = null)
     {
         $this->expr = trim((string) $expr);
     }
@@ -141,7 +141,7 @@ 
                 } else {
                     // The expression ended.
                     $expr->expr = trim($expr->expr);
-                    if (! empty($expr->expr)) {
+                    if (!empty($expr->expr)) {
                         $ret[] = $expr;
                     }
 
@@ -159,13 +159,13 @@ 
             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;
                 }
 
-                if ($brackets === 0 && ! in_array($token->value, self::ALLOWED_KEYWORDS, true)) {
+                if ($brackets === 0 && !in_array($token->value, self::ALLOWED_KEYWORDS, true)) {
                     break;
                 }
             }
@@ -202,7 +202,7 @@ 
 
         // Last iteration was not processed.
         $expr->expr = trim($expr->expr);
-        if (! empty($expr->expr)) {
+        if (!empty($expr->expr)) {
             $ret[] = $expr;
         }
 

src/Components/Expression.php

Spacing +23 added lines, -23 removed lines

@@ -113,10 +113,10 @@ 
      * @param string|null $alias    the name of the alias
      */
     public function __construct(
-        string|null $database = null,
-        string|null $table = null,
-        string|null $column = null,
-        string|null $alias = null,
+        string | null $database = null,
+        string | null $table = null,
+        string | null $column = null,
+        string | null $alias = null,
     ) {
         if (($column === null) && ($alias === null)) {
             $this->expr = $database; // case 1
@@ -161,7 +161,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 static();
 
@@ -194,7 +194,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'];
         }
@@ -221,18 +221,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.
@@ -240,7 +240,7 @@ 
                     }
 
                     if ($token->keyword === 'AS') {
-                        if (! empty($options['breakOnAlias'])) {
+                        if (!empty($options['breakOnAlias'])) {
                             break;
                         }
 
@@ -263,7 +263,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;
                 }
@@ -279,7 +279,7 @@ 
                 || (($token->type === TokenType::Operator)
                 && ($token->value !== '.'))
             ) {
-                if (! empty($options['parseField'])) {
+                if (!empty($options['parseField'])) {
                     break;
                 }
 
@@ -289,7 +289,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;
                 }
@@ -313,7 +313,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;
@@ -339,7 +339,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;
                 }
@@ -353,15 +353,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;
                     }
@@ -392,7 +392,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);
                     }
 
@@ -409,11 +409,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;
                         }
@@ -461,7 +461,7 @@ 
             $ret = implode('.', Context::escapeAll($fields));
         }
 
-        if (! empty($this->alias)) {
+        if (!empty($this->alias)) {
             $ret .= ' AS ' . Context::escape($this->alias);
         }
 

src/Statement.php

Spacing +14 added lines, -14 removed lines

@@ -63,23 +63,23 @@ 
      *
      * @see Statement::$statementOptions
      */
-    public OptionsArray|null $options = null;
+    public OptionsArray | null $options = null;
 
     /**
      * The index of the first token used in this statement.
      */
-    public int|null $first = null;
+    public int | null $first = null;
 
     /**
      * The index of the last token used in this statement.
      */
-    public int|null $last = null;
+    public int | null $last = null;
 
     /**
      * @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;
@@ -124,7 +124,7 @@ 
 
             // Checking if this field was already built.
             if ($type & self::ADD_CLAUSE) {
-                if (! empty($built[$field])) {
+                if (!empty($built[$field])) {
                     continue;
                 }
 
@@ -137,7 +137,7 @@ 
             }
 
             // Checking if the result of the builder should be added.
-            if (! ($type & self::ADD_CLAUSE)) {
+            if (!($type & self::ADD_CLAUSE)) {
                 continue;
             }
 
@@ -260,8 +260,8 @@ 
             $options = [];
 
             // Looking for duplicated clauses.
-            if (isset(Parser::KEYWORD_PARSERS[$token->value]) || ! empty(Parser::STATEMENT_PARSERS[$token->value])) {
-                if (! empty($parsedClauses[$token->value])) {
+            if (isset(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;
                 }
@@ -277,15 +277,15 @@ 
             if (isset(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
@@ -300,8 +300,8 @@ 
                     break;
                 }
 
-                if (! $parsedOptions) {
-                    if (! array_key_exists((string) $token->value, static::$statementOptions)) {
+                if (!$parsedOptions) {
+                    if (!array_key_exists((string) $token->value, static::$statementOptions)) {
                         // Skipping keyword because if it is not a option.
                         ++$list->idx;
                     }
@@ -468,7 +468,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')) {