Skip to content

Commit 76dc54b

Browse files
author
Thomas Flori
committed
reset entities exists properties after delete
1 parent 9e5634e commit 76dc54b

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

src/Entity.php

+2-3
Original file line numberDiff line numberDiff line change
@@ -581,12 +581,11 @@ public function getData()
581581
/**
582582
* Set new original data
583583
*
584-
* @param array $data
585584
* @internal
586585
*/
587-
public function setOriginalData(array $data)
586+
public function setOriginalData(array $data, bool $exists = true)
588587
{
589-
$this->exists = true;
588+
$this->exists = $exists;
590589
$this->originalData = $data;
591590
}
592591

src/EntityManager.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -551,7 +551,7 @@ public function delete(Entity $entity)
551551
return false;
552552
}
553553
$this->getDbal()->deleteEntity($entity);
554-
$entity->setOriginalData([]);
554+
$entity->setOriginalData([], false);
555555
$this->fire(new Deleted($entity));
556556
return true;
557557
}

tests/EntityManager/DataModificationTest.php

+2-1
Original file line numberDiff line numberDiff line change
@@ -533,8 +533,9 @@ public function deleteRemovesOriginalData($class, $data, $statement)
533533
->andReturn($statement = m::mock(\PDOStatement::class));
534534
$statement->shouldReceive('rowCount')->andReturn(1);
535535

536-
$this->em->delete($entity);
536+
$entity->delete();
537537

538538
self::assertTrue($entity->isDirty());
539+
self::assertFalse($entity->exists());
539540
}
540541
}

0 commit comments

Comments
 (0)