Remove internal tag from computeChangeSet

[VincentLanglet]

In the doctrine https://www.doctrine-project.org/projects/doctrine-orm/en/2.8/reference/events.html#onflush

The computeChangeSet is recommended to be used

If you create and persist a new entity in onFlush, then calling EntityManager#persist() is not enough. You have to execute an additional call to $unitOfWork->computeChangeSet($classMetadata, $entity).
Changing primitive fields or associations requires you to explicitly trigger a re-computation of the changeset of the affected entity. This can be done by calling $unitOfWork->recomputeSingleEntityChangeSet($classMetadata, $entity).

So the method shouldn't be flag as internal:

• To provide a BC-promise
• To avoid psalm error with internal method

[VincentLanglet]

Friendly ping @greg0ire, what do you think about this PR ? :)

[greg0ire]

It makes sense to me 👍
Can you find the author of this comment and ping them if they are still part of the team?

[VincentLanglet]

It was @beberlei 6f36672

[greg0ire]

Ok, I think the doc you linked to was already present, so this must be a mistake.

[greg0ire]

Thanks @VincentLanglet !