[PR #6794] Remove copy() method from EntityManager #10144

New Issue

admin · 2026-01-22T16:06:34+01:00

admin commented

2026-01-22 16:06:34 +01:00

Original Pull Request: https://github.com/doctrine/orm/pull/6794

State: closed
Merged: Yes

The method EntityManager::copy() is a pretty old addition to Doctrine and never got any functionality, which is why I created this PR to remove this method from the EntityManager.

Because I had to remove this method from EntityManagerInterface too, I consider this a BC break. Even though the original EntityManager never got any functionality for copy, a user may have created a custom entity manager for a project, that uses a custom implementation of copy.

Questions:

Do we need tests for the removal of a method?
Do we need a deprecation PR as an in between step?

**Original Pull Request:** https://github.com/doctrine/orm/pull/6794 **State:** closed **Merged:** Yes --- The method `EntityManager::copy()` is a pretty old addition to Doctrine and never got any functionality, which is why I created this PR to remove this method from the `EntityManager`. Because I had to remove this method from `EntityManagerInterface` too, I consider this a BC break. Even though the original `EntityManager` never got any functionality for copy, a user may have created a custom entity manager for a project, that uses a custom implementation of copy. Questions: * Do we need tests for the removal of a method? * Do we need a deprecation PR as an in between step?

admin added the pull-request label 2026-01-22 16:06:34 +01:00

admin closed this issue

2026-01-22 16:06:34 +01:00

admin referenced this issue

2026-01-22 16:13:00 +01:00

[PR #10144] Fix FieldMapping for generated key #12127

Sign in to join this conversation.