Merge pull request #392 from widop/patch-1
Invalid paginator query cloning
This commit is contained in:
commit
fe292cc503
@ -208,7 +208,9 @@ class Paginator implements \Countable, \IteratorAggregate
|
|||||||
{
|
{
|
||||||
/* @var $cloneQuery Query */
|
/* @var $cloneQuery Query */
|
||||||
$cloneQuery = clone $query;
|
$cloneQuery = clone $query;
|
||||||
$cloneQuery->setParameters($query->getParameters());
|
|
||||||
|
$cloneQuery->setParameters(clone $query->getParameters());
|
||||||
|
|
||||||
foreach ($query->getHints() as $name => $value) {
|
foreach ($query->getHints() as $name => $value) {
|
||||||
$cloneQuery->setHint($name, $value);
|
$cloneQuery->setHint($name, $value);
|
||||||
}
|
}
|
||||||
|
@ -128,6 +128,17 @@ class PaginationTest extends \Doctrine\Tests\OrmFunctionalTestCase
|
|||||||
count($paginator);
|
count($paginator);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function testCloneQuery()
|
||||||
|
{
|
||||||
|
$dql = "SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u";
|
||||||
|
$query = $this->_em->createQuery($dql);
|
||||||
|
|
||||||
|
$paginator = new Paginator($query);
|
||||||
|
$paginator->getIterator();
|
||||||
|
|
||||||
|
$this->assertTrue($query->getParameters()->isEmpty());
|
||||||
|
}
|
||||||
|
|
||||||
public function populate()
|
public function populate()
|
||||||
{
|
{
|
||||||
for ($i = 0; $i < 3; $i++) {
|
for ($i = 0; $i < 3; $i++) {
|
||||||
|
Loading…
x
Reference in New Issue
Block a user