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

v.db.addcolumn: fix enclosing column name with SQL standard double quotes #3633

Merged

Conversation

tmszi
Copy link
Member

@tmszi tmszi commented Apr 19, 2024

Enable to use column name which is reserved DB backend keyword.

…otes

Enable to use column name which is reserved DB backend keyword.
@tmszi tmszi added this to the 8.4.0 milestone Apr 19, 2024
@tmszi tmszi self-assigned this Apr 19, 2024
@github-actions github-actions bot added vector Related to vector data processing Python Related code is in Python module labels Apr 19, 2024
@tmszi tmszi added bug Something isn't working backport to 8.3 labels Apr 19, 2024
@github-actions github-actions bot added libraries tests Related to Test Suite labels Apr 20, 2024
@tmszi tmszi merged commit ef730b9 into OSGeo:main Apr 20, 2024
26 checks passed
@tmszi tmszi deleted the v_db_addcolumn-fix-enclose-col-name-with-double-quotes branch April 20, 2024 16:04
neteler pushed a commit that referenced this pull request Apr 23, 2024
Backport of: fix enclosing column name with SQL standard double quotes (#3633)
* Fix fatal message
* Fix maximum number of string split by whitespace string
* Fix add new column name type
lindakarlovska pushed a commit to lindakarlovska/grass that referenced this pull request Apr 29, 2024
…otes (OSGeo#3633)

* Fix fatal message

* Fix maximum number of string split by whitespace string

* Fix add new column name type
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working libraries module Python Related code is in Python tests Related to Test Suite vector Related to vector data processing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants