Skip to content

[DDC-2113] Surround WHERE clause with parens if using SQLFilter#495

Closed
vclayton wants to merge 735 commits intodoctrine:2.2from
vclayton:DDC-2113
Closed

[DDC-2113] Surround WHERE clause with parens if using SQLFilter#495
vclayton wants to merge 735 commits intodoctrine:2.2from
vclayton:DDC-2113

Conversation

@vclayton
Copy link

@vclayton vclayton commented Nov 1, 2012

Patch to address DDC-2113, by surrounding the WHERE clause conditional with parens if there are any filterClauses

FabioBatSilva and others added 30 commits June 18, 2012 14:42
Added an empty line feed preceding the return statement
SimplifiedXmlDriver and SimplifiedYamlDriver are still not valid after this commit
guilhermeblanco and others added 24 commits October 11, 2012 22:22
Fixed bug with comment option not being added to column.
fixed typo/bug and CS on use statements
While executing updates on an entity scheduled for update without
a change-set, an "Undefined index" notice is raised.
Prevented "Undefined index" notice when updating
Fixed empty namespace in generated code when repository class do not have namespace
check for false as a return value from get_parent_class(), not null
fix StatementMock bindParam parameters
@doctrinebot
Copy link

Hello,

thank you for positing this Pull Request. I have automatically opened an issue on our Jira Bug Tracker for you with the details of this Pull-Request. See the Link:

http://doctrine-project.org/jira/browse/DDC-2114

@doctrinebot
Copy link

Oh btw, I just (automatically) realized that you are not creating this pull request against the master branch.

Unless there are good reasons for this, i would suggest to close and rebase the Pull Request against master and then create it again. Sorry!

@vclayton
Copy link
Author

vclayton commented Nov 2, 2012

I'll recreate it in master.

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.