Skip to content

[DBAL-1016] Fix explicitly quoted table identifiers in ALTER TABLE statements#700

Merged
Ocramius merged 1 commit intodoctrine:masterfrom
deeky666:DBAL-1016
Oct 21, 2014
Merged

[DBAL-1016] Fix explicitly quoted table identifiers in ALTER TABLE statements#700
Ocramius merged 1 commit intodoctrine:masterfrom
deeky666:DBAL-1016

Conversation

@deeky666
Copy link
Copy Markdown
Member

Another improvement to the neverending quotation issues.
This patch fixes explicitly quoted table identifiers in ALTER TABLE statements.
Additionally some minor fixes had to be applied such as misc fixing of foreign key constraint statements order in the sequence of statements necessary to alter a table.

@doctrinebot
Copy link
Copy Markdown

Hello,

thank you for creating this pull request. I have automatically opened an issue
on our Jira Bug Tracker for you. See the issue link:

http://www.doctrine-project.org/jira/browse/DBAL-1016

We use Jira to track the state of pull requests and the versions they got
included in.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@deeky666 this is a minor BC break and should be documented

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nvm, it's not in 2.4, therefore not a BC break.

@Ocramius
Copy link
Copy Markdown
Member

Overall 👍, but needs docs on the minor BC break

@Ocramius Ocramius self-assigned this Oct 20, 2014
Ocramius added a commit that referenced this pull request Oct 21, 2014
[DBAL-1016] Fix explicitly quoted table identifiers in ALTER TABLE statements
@Ocramius Ocramius merged commit fbd4984 into doctrine:master Oct 21, 2014
@morozov morozov mentioned this pull request Jun 6, 2019
1 task
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants