Skip to content

correction for single quote inside double quote#835

Closed
ndebeiss wants to merge 1 commit into
spring-projects:masterfrom
ndebeiss:master
Closed

correction for single quote inside double quote#835
ndebeiss wants to merge 1 commit into
spring-projects:masterfrom
ndebeiss:master

Conversation

@ndebeiss

Copy link
Copy Markdown
Contributor

Some databases (Oracle for example) allow alias of columns inside double
quote in order to have case sensitive and special characters, including
single quote.
For example :

select address as "programmer’s address" from programmer

So those single quotes must not be taken as literal beginning.

I have signed and agree to the terms of the SpringSource Individual
Contributor License Agreement.

Some databases (Oracle for example) allow alias of columns inside double
quote in order to have case sensitive and special characters, including
single quote. So those single quotes must not be taken as literals.
@sbrannen sbrannen self-assigned this Jul 10, 2015
@sbrannen

Copy link
Copy Markdown
Member

@ndebeiss, can you please create a JIRA issue so that this request can be officially tracked?

Thanks!

Sam

@ndebeiss

Copy link
Copy Markdown
Contributor Author

I created the Jira : https://jira.spring.io/browse/SPR-13218
Many thanks for all the work on Spring, and following of issues.

@sbrannen

Copy link
Copy Markdown
Member

Great. Thanks! :)

@sbrannen

Copy link
Copy Markdown
Member

Have you signed the SpringSource CLA?

@ndebeiss

Copy link
Copy Markdown
Contributor Author

It is done :-) ! You want the confirmation number ?

@sbrannen

Copy link
Copy Markdown
Member

The confirmation number is for yourself.

However, you need to make the following statement as outlined here:

I have signed and agree to the terms of the SpringSource Individual Contributor License Agreement.

sbrannen added a commit that referenced this pull request Jul 14, 2015
* ndebeiss-master:
  Support single quotes nested in double quotes in SQL scripts
@sbrannen

Copy link
Copy Markdown
Member

Merged in 649d145

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