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

PostgreSQL: trim schema name. #1196

Closed
wants to merge 1 commit into from

Conversation

@murdos
Copy link
Contributor

@murdos murdos commented Feb 2, 2016

The issue came while trying to detect schema in PostgreSQLDbSupport.getOriginalSchema() with an from originalSchema set with value: '"$user", public'.
I was ending with a schema named " public" (with a leading space).
The trim could either be done in PostgreSQLDbSupport.getOriginalSchema(), but probably twice, or just once in getSchema().

The issue came while trying to dectect schema from string: '"$user",
public'.
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Feb 3, 2016

Thanks. I decided to fix it in PostgreSQLDbSupport.getOriginalSchema() instead as that was were the rest of the String manipulation was happening already.

@murdos
Copy link
Contributor Author

@murdos murdos commented Feb 3, 2016

Fine, thanks.

@murdos murdos deleted the postgresql-fix-schema-detection branch Mar 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants