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.

Sign up for GitHub

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

Jump to bottom

Using ArrayCollection->clear over an entity marked to be persisted, doesn't create corresponding DELETE statements #7862

Closed
ferrastas opened this issue Oct 9, 2019 · 4 comments
Closed

Using ArrayCollection->clear over an entity marked to be persisted, doesn't create corresponding DELETE statements #7862

ferrastas opened this issue Oct 9, 2019 · 4 comments
Assignees
@ostrolucky
Labels
Bug Regression
Milestone
2.6.5

Comments

@ferrastas
Copy link
Contributor

Bug Report

Q A
BC Break yes/no
Version 2.6.4

Summary

Since 2.6.4 using clear over a collection using DEFERRED_EXPLICIT will not create the corresponding DELETE statements even when persisting the entity.
Introduced in #7758

Current behavior

Using clear over a collection using DEFERRED_EXPLICIT will not create the corresponding DELETE statements even when persisting the entity.

How to reproduce

A failing test available in #7861

Expected behavior

Collection items should be removed for entities marked to be persisted.
@SenseException
Copy link
Member

Thank you for providing the failing test case 👍

@lcobucci lcobucci added this to the 2.6.5 milestone Nov 12, 2019
@lcobucci
Copy link
Member

Handled by #7861

@paxal
Copy link
Sponsor Contributor

paxal commented Nov 19, 2019

Yes I'm the one to blame and I feed very sorry 🤦‍♂️
Hopefully everything is fine now 😅
Thanks !

@Ocramius
Copy link
Member

@paxal take it easy: regressions are normal, we all do our best to avoid them, but can't have everything in mind.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ostrolucky ostrolucky

Labels
Bug Regression
Projects
None yet
Milestone
2.6.5
Development

No branches or pull requests
6 participants
@ferrastas @Ocramius @lcobucci @ostrolucky @SenseException @paxal