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.

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

lib/db: set SQL ANSI_QUOTES MODE in mysql driver #3612

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

tmszi
Copy link
Member

@tmszi tmszi commented Apr 17, 2024

Which allow to use SQL standard double quotes instead of backticks for escaping column name.

Successfully tested with MySQL DB backend.

Example for testing purpose.

Which allow to use SQL standard double quotes instead of backticks for
escaping column name.
@tmszi tmszi added this to the 8.4.0 milestone Apr 17, 2024
@tmszi tmszi self-assigned this Apr 17, 2024
@github-actions github-actions bot added C Related code is in C database Related to database management module labels Apr 17, 2024
@tmszi tmszi added the bug Something isn't working label Apr 17, 2024
@tmszi tmszi merged commit efb595f into OSGeo:main Apr 17, 2024
26 checks passed
@tmszi tmszi deleted the set-mysql-db-driver-ansi-mode branch April 17, 2024 12:24
tmszi added a commit to tmszi/grass that referenced this pull request Apr 18, 2024
Which allow to use SQL standard double quotes instead of backticks for
escaping column name.
neteler pushed a commit that referenced this pull request Apr 23, 2024
Which allow to use SQL standard double quotes instead of backticks for
escaping column name.
@neteler neteler changed the title db/drivers/mysql: set SQL ANSI_QUOTES MODE lib/db: set SQL ANSI_QUOTES MODE in mysql driver Jun 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working C Related code is in C database Related to database management module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants