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

Fix updating of embedded documents if their atomic collection is not dirty #1154

Merged
merged 2 commits into from
Jun 20, 2015

Conversation

malarzm
Copy link
Member

@malarzm malarzm commented Jun 20, 2015

Fixes #1153

For the record this PR can be considered as a hotfix for functionality that I've accidentally broken fixing #1141. I start to dislike the way we're managing atomic collection updates but I'm planning to fix it along with commit() flow I've described in #1128 (comment) so I'm willing to merge this now and rewrite later.

@malarzm malarzm added the Bug label Jun 20, 2015
@malarzm malarzm added this to the 1.0.0 milestone Jun 20, 2015
@jwage
Copy link
Member

jwage commented Jun 20, 2015

👍

malarzm added a commit that referenced this pull request Jun 20, 2015
Fix updating of embedded documents if their atomic collection is not dirty
@malarzm malarzm merged commit 21bffab into doctrine:master Jun 20, 2015
@malarzm malarzm deleted the gh1153 branch June 20, 2015 22:13
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

2 participants