Sqlserver limit #252

Merged
merged 2 commits into from Jan 30, 2013

Projects

None yet

3 participants

@norzechowicz

Unfortunately fix from my last commit it not enough to handle more complicated queries.
This change should fix points 2 and 3 from this list but it also removes adding TOP statement if $offset is not set.
I know it looks realy messy but I can't figure out any other solution to handel queries with subqueries and ordering (such thing is generated after using Doctrine Paginator and ordering).

@doctrinebot

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/DBAL-419

@beberlei beberlei merged commit 2e03129 into doctrine:master Jan 30, 2013

1 check passed

Details default The Travis build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment