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

Compare with dbname with existing databases needs unquoted dbname #401

Merged
merged 1 commit into from Mar 15, 2015
Merged

Compare with dbname with existing databases needs unquoted dbname #401

merged 1 commit into from Mar 15, 2015

Conversation

kingcrunch
Copy link
Contributor

listDatabases() returns the unquoted database names and therefore
the command must use the unquoted database name for comparison

@mobula9
Copy link

mobula9 commented Mar 6, 2015

👍

@bmancone
Copy link

bmancone commented Mar 9, 2015

👍

@stof : Could you take a look at this PR ? 😄
Thanks!

@cordoval
Copy link
Contributor

👍 nice catch @kingcrunch !

I ran into this today and this was the solution!

guilhermeblanco added a commit that referenced this pull request Mar 15, 2015
Compare with dbname with existing databases needs unquoted dbname
@guilhermeblanco guilhermeblanco merged commit 3bf42bf into doctrine:master Mar 15, 2015
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.

None yet

5 participants