DDC-1138: Remove Entity and Transactional

[doctrinebot]

Jira issue originally created by user mitmaro:

I am getting an odd bug when I am removing an entity using the entity manager. An exception is being thrown saying "A new entity was found through a relationship that was not configured to cascade persist operations" but there are no new entities explicitly created.

This is the relevant piece of code and I have attached a full reduced test case.

$qb = $em
    ->getRepository('Bar')
    ->createQueryBuilder('bar1')
    ->select('bar1, foo1, bar2')
    ->join('bar1.foo', 'foo1')
    ->join('foo1.bar', 'bar2')
;

$qb->getQuery()->getResult();

$bar = $em->getRepository('Bar')->findOneBy(array('id' => 20));

$em->transactional(function($em) use ($bar) {
    $em->remove($bar);
    $em->flush();
});

Exception:

Fatal error: Uncaught exception 'InvalidArgumentException' with message 'A new entity was found through a relationship
that was not configured to cascade persist operations: Bar@0000000049dcca970000000025d8d6f9. Explicitly persist
the new entity or configure cascading persist operations on the relationship.' in doctrine2-orm/lib/Doctrine/ORM/UnitOfWork.php:576
Stack trace:
  #0 doctrine2-orm/lib/Doctrine/ORM/UnitOfWork.php(495): Doctrine\ORM\UnitOfWork >computeAssociationChanges(Array, Object(Doctrine\ORM\PersistentCollection))
  #1 doctrine2-orm/lib/Doctrine/ORM/UnitOfWork.php(537): Doctrine\ORM\UnitOfWork->computeChangeSet(Object(Doctrine\ORM\Mapping\ClassMetadata), Object(Foo))
  #2 doctrine2-orm/lib/Doctrine/ORM/UnitOfWork.php(256): Doctrine\ORM\UnitOfWork->computeChangeSets()
  #3 doctrine2-orm/lib/Doctrine/ORM/EntityManager.php(334): Doctrine\ORM\UnitO in doctrine2-orm/lib/Doctrine/ORM/UnitOfWork.php on line 576

A few other things I have noticed:

• Removing the transactional wrapper and using the DBAL connection transaction fixes the problem
• Removing the query before the update fixes the problem.
• The remove works without using an explicit transaction

[doctrinebot]

Comment created by mitmaro:

Uploaded the wrong test file.

[doctrinebot]

Comment created by @beberlei:

transactional() calls flush for you. You don't have to do it yourself.

Your code probably fails because of this double flush, can you verify?

[doctrinebot]

Issue was closed with resolution "Invalid"

[doctrinebot]

Comment created by mitmaro:

Verified. That does seem to be the issue. Thanks.

[doctrinebot]

Imported 2 attachments from Jira into https://gist.github.com/b8de79dc495a9035ce52

• 10988_Test.php
• 10989_ReducedTestCase.php