phpmyadmin
diff --git a/‎CHANGELOG.md‎
Lines changed: 4 additions & 0 deletions b/‎CHANGELOG.md‎
Lines changed: 4 additions & 0 deletions
diff --git a/‎src/Components/AlterOperation.php‎
Lines changed: 7 additions & 14 deletions b/‎src/Components/AlterOperation.php‎
Lines changed: 7 additions & 14 deletions
diff --git a/‎tests/Misc/BugsTest.php‎
Lines changed: 3 additions & 0 deletions b/‎tests/Misc/BugsTest.php‎
Lines changed: 3 additions & 0 deletions
diff --git a/‎tests/data/bugs/gh234.in‎
Lines changed: 1 addition & 0 deletions b/‎tests/data/bugs/gh234.in‎
Lines changed: 1 addition & 0 deletions
@@ -8,6 +8,10 @@
 
 ## [5.9.x] - YYYY-MM-DD
 
+## [5.8.2] - 2023-09-19
+
+- Fix a regression with the ALTER operation (#511)
+
 ## [5.8.1] - 2023-09-15
 
 - Fix `:=` was not recognized as an operator just like `=` (#306)
 
@@ -400,33 +400,26 @@ public static function parse(Parser $parser, TokensList $list, array $options =
                     } elseif (($token->value === ',') && ($brackets === 0)) {
                         break;
                     }
-                } 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)) {
+                } elseif (! self::checkIfTokenQuotedSymbol($token) && $token->type !== Token::TYPE_STRING) {
+                    if (isset(Parser::STATEMENT_PARSERS[$arrayKey]) && Parser::STATEMENT_PARSERS[$arrayKey] !== '') {
                         $list->idx++; // Ignore the current token
                         $nextToken = $list->getNext();
 
-                        if ($nextToken !== null && $nextToken->value === '(') {
+                        if ($token->value === 'SET' && $nextToken !== null && $nextToken->value === '(') {
+                            // To avoid adding the tokens between the SET() parentheses to the unknown tokens
                             $list->getNextOfTypeAndValue(Token::TYPE_OPERATOR, ')');
-                        } elseif ($nextToken !== null && $nextToken->value === 'DEFAULT') {
+                        } elseif ($token->value === 'SET' && $nextToken !== null && $nextToken->value === 'DEFAULT') {
                             // to avoid adding the `DEFAULT` token to the unknown tokens.
                             ++$list->idx;
                         } else {
+                            // 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(
                                 'A new statement was found, but no delimiter between it and the previous one.',
                                 $token
                             );
                             break;
                         }
-                    } elseif (! empty(Parser::STATEMENT_PARSERS[$arrayKey])) {
-                        // 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(
-                            'A new statement was found, but no delimiter between it and the previous one.',
-                            $token
-                        );
-                        break;
                     } elseif (
                         (array_key_exists($arrayKey, self::DATABASE_OPTIONS)
                         || array_key_exists($arrayKey, self::TABLE_OPTIONS))
 
@@ -28,8 +28,11 @@ public static function bugProvider(): array
             ['bugs/gh9'],
             ['bugs/gh14'],
             ['bugs/gh16'],
+            ['bugs/gh234'],
             ['bugs/gh317'],
+            ['bugs/gh478'],
             ['bugs/gh508'],
+            ['bugs/gh511'],
             ['bugs/pma11800'],
             ['bugs/pma11836'],
             ['bugs/pma11843'],
 
@@ -0,0 +1 @@
+ALTER TABLE `mail_template` CHANGE COLUMN `mtpl_group` `mtpl_group` ENUM('ORDER') NULL DEFAULT NULL ;
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+ALTER TABLE `mail_template` CHANGE COLUMN `mtpl_group` `mtpl_group` ENUM('ORDER') NULL DEFAULT NULL ;