-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
There is the fail "UNIQUE constraint failed" with a LAZY collection #6613
Comments
@Sysaninster can you put this into a test case like the ones in https://github.com/doctrine/doctrine2/tree/bbe005837e51442e0d6037bafe70dc4b7712dac1/tests/Doctrine/Tests/ORM/Functional/Ticket please? |
Closing here, thanks for sending a failing test case! |
Ref: #6614 |
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
…g identifiers to reduce moving parts
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
…ng`), segregating phone creation away
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
…are restored in the dirty state of the collection
Reopening, since this will be closed once #6616 is merged. |
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
…toreNewObjectsInDirtyCollection` implementation
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
…ng identifiers to reduce moving parts
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
…ng`), segregating phone creation away
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
… are restored in the dirty state of the collection
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
…erifies basic dirty collection initialization semantics
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
…that are also coming from initialization data de-duplicates new and persisted items
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
… if all new items are contained in the initialized ones
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
…toreNewObjectsInDirtyCollection` implementation
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
Ocramius
added a commit
that referenced
this issue
Aug 11, 2017
@Sysaninster thanks for the bug report - this is fixed and will go into |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have the User entity with the LAZY collection field. If I do this code:
I will get the error:
If I remove the $user->phones->toArray(); line then it will work without a problem.
The text was updated successfully, but these errors were encountered: