RetailCRM-Mirror/doctrine2
1
0
Fork 0
mirror of synced 2025-02-09 00:39:25 +03:00
Code Issues Projects Releases Wiki Activity

Clear $this->collection even when empty, to reset indexes

Browse Source
This commit is contained in:
Steevan BARBOYON 2016-10-31 12:03:19 +01:00 committed by Marco Pivetta
parent 9b36947a48
commit 3dadfa49d5
2 changed files with 27 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/Doctrine/ORM/PersistentCollection.php
View File

@ -536,6 +536,8 @@ final class PersistentCollection extends AbstractLazyCollection implements Selec
public function clear() public function clear()
{ {
if ($this->initialized && $this->isEmpty()) { if ($this->initialized && $this->isEmpty()) {
$this->collection->clear();
return; return;
} }

25
tests/Doctrine/Tests/ORM/PersistentCollectionTest.php
View File

@ -83,4 +83,29 @@ class PersistentCollectionTest extends OrmTestCase
$this->collection->next(); $this->collection->next();
$this->assertTrue($this->collection->isInitialized()); $this->assertTrue($this->collection->isInitialized());
} }
/**
* Test that PersistentCollection::clear() clear elements, and reset keys
*/
public function testClear()
{
$this->setUpPersistentCollection();
$this->collection->add('dummy');
$this->assertEquals([0], array_keys($this->collection->toArray()));
$this->collection->removeElement('dummy');
$this->assertEquals([], array_keys($this->collection->toArray()));
$this->collection->add('dummy');
$this->collection->clear();
$this->assertEquals([], array_keys($this->collection->toArray()));
// test fix clear doesn't reset collection keys when collection is empty
$this->collection->add('dummy');
$this->collection->removeElement('dummy');
$this->collection->clear();
$this->collection->add('dummy');
$this->assertEquals([0], array_keys($this->collection->toArray()));
}
} }