DDC-2332: Ensure managed entities are always tracked by UOW

[bigfoot90]

This is a rebase of MR #7407 on 2.9.x branch.
Fixes #3037

[greg0ire]

vendor/bin/phpcbf can help with the CS issues

[bigfoot90]

Thanks, fixed all cs

[bigfoot90]

@greg0ire Friendly ping

[qkdreyer]

Any updates on this ?

[bigfoot90]

Rebased on 2.10.x

[beberlei]

I am skeptical. Can you explain the test a little better via comments? Its doing s lot, not sure where exactly and what fails. Is the merge necessary to cause this?

[bigfoot90]

Rebased on 2.10.x again and added some comments on test as suggested.

This same test testObjectIdCollision copy-pasted on branch 2.10.x is failing on iteration 60, but your could be different.

[flack]

just in case it helps: A long time ago, I also made a failing test case for this issue: #843. Not sure if it is more understandable, but just wanted to mention it if someone is looking for a different approach

[carlos-granados]

Instead of saving the whole document, just save a reference to it. This will also stop it from being garbage collected and uses much less memory

$this->oidMap[$oid] = &$entity;

[mpdude]

@bigfoot90 maybe you want to join #10791