Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ALTER .. MODIFY ... ENUM failed! (Regression) #511

Closed
BorisValgo opened this issue Sep 18, 2023 · 2 comments
Closed

ALTER .. MODIFY ... ENUM failed! (Regression) #511

BorisValgo opened this issue Sep 18, 2023 · 2 comments
Assignees
@MauricioFauth
Labels
bug
Milestone
5.8.2

Comments

@BorisValgo
Copy link

BorisValgo commented Sep 18, 2023 •
edited
Loading

Hi,

From the version 5.8.1 ALTER ENUM FAILD.
For example :
ALTER TABLE users_type MODIFY limitations ENUM('tout', 'rien', 'bu', 'agence', 'agence_limite', 'n-1', 'agence_inactif'); UPDATE users_type SET limitations = 'agence_inactif' WHERE id_users_type = 19;
When we user the 5.8.0 version the "lint-query --query" say OK but in 5.8.1 we have exception :
#1: A new statement was found, but no delimiter between it and the previous one. (near "UPDATE" at position 122)

Thank you
MauricioFauth added a commit to MauricioFauth/sql-parser that referenced this issue Sep 18, 2023
@MauricioFauth
Fix ALTER regression
9c5fee3 
- Fixes phpmyadmin#511
- Reverts phpmyadmin#485

Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
@MauricioFauth MauricioFauth mentioned this issue Sep 18, 2023
Merged
MauricioFauth added a commit to MauricioFauth/sql-parser that referenced this issue Sep 19, 2023
@MauricioFauth
Fix ALTER regression
adfe314 
- Fixes phpmyadmin#511
- Reverts phpmyadmin#485

Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
@MauricioFauth MauricioFauth added this to the 5.8.2 milestone Sep 19, 2023
@MauricioFauth MauricioFauth self-assigned this Sep 19, 2023
@williamdes

This comment was marked as resolved.

Sign in to view
@williamdes
Copy link
Member

Released as 5.8.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MauricioFauth MauricioFauth

Labels
bug
Projects
None yet
Milestone
5.8.2
Development

No branches or pull requests
3 participants
@MauricioFauth @williamdes @BorisValgo