Skip to content

EXPOSED-690 fix Table name quotation#2356

Draft
SIMULATAN wants to merge 1 commit intoJetBrains:mainfrom
SIMULATAN:fix/table-name-quotation
Draft

EXPOSED-690 fix Table name quotation#2356
SIMULATAN wants to merge 1 commit intoJetBrains:mainfrom
SIMULATAN:fix/table-name-quotation

Conversation

@SIMULATAN
Copy link

Description

This PR should eventually fix a quotation issue encountered when using invalid characters in table names.
Tracked by https://youtrack.jetbrains.com/issue/EXPOSED-690, details are there

Status

Currently, just a (obviously failing) test case is implemented to demonstrate this issue. Should a maintainer be kind enough to tackle this problem (as I am not familiar with the codebase and potential blast radius of a change), they are free to update my branch (edits for maintainers are enabled).

Type of Change

Please mark the relevant options with an "X":

  • Bug fix
  • New feature
  • Documentation update

Updates/remove existing public API methods:

  • Is breaking change

Affected databases:

  • MariaDB
  • Mysql5
  • Mysql8
  • Oracle
  • Postgres
  • SqlServer
  • H2
  • SQLite

Checklist

  • Unit tests are in place
  • The build is green (including the Detekt check)
  • All public methods affected by my PR has up to date API docs
  • Documentation for my change is up to date

Related Issues

https://youtrack.jetbrains.com/issue/EXPOSED-690

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.

1 participant