Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove ability to merge detached entities #9488

Merged
merged 1 commit into from Feb 8, 2022

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Feb 8, 2022

The EntityManager::merge() method has been deprecated for a long time (#7901). This PR removes this feature from the 3.0 branch.

@derrabus derrabus added this to the 3.0.0 milestone Feb 8, 2022
*/
public function merge($entity): object
public function merge($object): object
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to keep the method for compatibility with Persistence 2. We can drop it as soon as we move to Persistence >= 3.

@derrabus derrabus mentioned this pull request Feb 8, 2022
@derrabus derrabus merged commit d13d0f5 into doctrine:3.0.x Feb 8, 2022
@derrabus derrabus deleted the remove/merge branch February 8, 2022 23:43
mpdude added a commit to mpdude/doctrine2 that referenced this pull request Jun 23, 2023
`EntityManager::merge()` has been deprecated in doctrine#8461 and removed in doctrine#9488.

This PR removes a few remaining references and artefacts that - to my understanding - refer to it.
mpdude added a commit to mpdude/doctrine2 that referenced this pull request Jun 24, 2023
`EntityManager::merge()` has been deprecated in doctrine#8461 and removed in doctrine#9488.

This PR removes a few remaining references and artefacts that - to my understanding - refer to it.
mpdude added a commit to mpdude/doctrine2 that referenced this pull request Jun 24, 2023
`EntityManager::merge()` has been deprecated in doctrine#8461 and removed in doctrine#9488.

This PR removes a few remaining references and artefacts that - to my understanding - refer to it.
mpdude added a commit to mpdude/doctrine2 that referenced this pull request Jun 25, 2023
`EntityManager::merge()` has been deprecated in doctrine#8461 and removed in doctrine#9488.

This PR removes a few remaining references and artefacts that - to my understanding - refer to it.
derrabus pushed a commit that referenced this pull request Jun 27, 2023
`EntityManager::merge()` has been deprecated in #8461 and removed in #9488.

This PR removes a few remaining references and artefacts that - to my understanding - refer to it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants