Skip to content

Escape index id properly#103

Merged
Meldiron merged 2 commits intomainfrom
fix-index-id-escape
Jan 25, 2022
Merged

Escape index id properly#103
Meldiron merged 2 commits intomainfrom
fix-index-id-escape

Conversation

@Meldiron
Copy link
Copy Markdown
Contributor

@Meldiron Meldiron commented Jan 25, 2022

Before, attempt to create index with id order would fail:

CREATE INDEX order ON `appwrite`.`_project_dev_collection_posts` ( `title`(255) ASC );
appwrite-worker-database        | SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'order ON `appwrite`.`_project_dev_collection_posts` ( `title`(255) ASC )' at line 1

With this PR, order is a valid ID of the key.

@Meldiron
Copy link
Copy Markdown
Contributor Author

@Meldiron Meldiron merged commit ecc143f into main Jan 25, 2022
@Meldiron Meldiron deleted the fix-index-id-escape branch January 25, 2022 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants