punkave/symfony2-doctrine-pager — pull request #1 by boutell (2012-07-13 21:00:24+00:00)
Title: Directory structure now deps compatible; proper count query used

Hi Wes,

Two changes:

1. Your extra directory level inside the repository (DoctrinePagerBundle) was creating a redundant directory level when I tried to install this in vendor in the usual way via deps. If you look at other repositories for bundles, people put the bundle class etc. right in the root of the repository. I moved everything else up to the same level with the README and removed the DoctrinePagerBundle folder. Now I am able to use it successfully via deps.

2. I modified computeResults() to use a proper count query when computing numResults. This resulted in a 2x speedup of the page with just 150 total items (20 shown on the page). With a larger number of items it will make the difference between success and an out of memory error.

Thanks for the bundle sirrah!