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

Clarify what happens in relationship fixup #1235

Closed
adisazhar123 opened this issue Jan 15, 2019 · 1 comment · Fixed by #2972
Closed

Clarify what happens in relationship fixup #1235

adisazhar123 opened this issue Jan 15, 2019 · 1 comment · Fixed by #2972

Comments

@adisazhar123
Copy link

adisazhar123 commented Jan 15, 2019

In reference to this sentence

Note that blog will also be inserted into the database because it is a new entity that is referenced by the navigation property of an entity that is already tracked by the context (post).

What happens to the old Blog entity? Let's say that only the current post is in the collection of Blog.Posts, and I change the current post's blog to a new Blog entity. Will the old blog entity be removed from DB? Do I have to explicitly call Remove function?


Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

Copy link

I guess the rational behaviour is to not deleting the previous blog (because e.g., it may have other posts), though I'm not sure.

@ajcvickers ajcvickers changed the title Changing relationships Clarify what happens in relationship fixup Sep 1, 2019
@ajcvickers ajcvickers added this to the Backlog milestone Sep 1, 2019
@ajcvickers ajcvickers modified the milestones: Backlog, 5.0.0 Dec 30, 2020
@ajcvickers ajcvickers self-assigned this Dec 30, 2020
ajcvickers added a commit that referenced this issue Dec 30, 2020
Fixes #2725
Fixes #2707
Fixes #2525
Fixes #2524
Fixes #2331
Fixes #2076
Fixes #1489
Fixes #1453
Fixes #1398
Fixes #1339
Fixes #1235
Fixes #1007
Fixes #863
Fixes #786
Fixes #531
Fixes #505
ajcvickers added a commit that referenced this issue Dec 30, 2020
Fixes #2725
Fixes #2707
Fixes #2525
Fixes #2524
Fixes #2331
Fixes #2076
Fixes #1489
Fixes #1453
Fixes #1398
Fixes #1339
Fixes #1235
Fixes #1007
Fixes #863
Fixes #786
Fixes #531
Fixes #505
ajcvickers added a commit that referenced this issue Dec 31, 2020
Fixes #2725
Fixes #2707
Fixes #2525
Fixes #2524
Fixes #2331
Fixes #2105
Fixes #2076
Fixes #1541
Fixes #1489
Fixes #1453
Fixes #1398
Fixes #1339
Fixes #1235
Fixes #1007
Fixes #863
Fixes #815
Fixes #786
Fixes #531
Fixes #505
ajcvickers added a commit that referenced this issue Jan 6, 2021
Fixes #2725
Fixes #2707
Fixes #2525
Fixes #2524
Fixes #2331
Fixes #2076
Fixes #1489
Fixes #1453
Fixes #1398
Fixes #1339
Fixes #1235
Fixes #1007
Fixes #863
Fixes #786
Fixes #531
Fixes #505
ajcvickers added a commit that referenced this issue Jan 6, 2021
Fixes #2725
Fixes #2707
Fixes #2525
Fixes #2524
Fixes #2331
Fixes #2105
Fixes #2076
Fixes #1541
Fixes #1489
Fixes #1453
Fixes #1398
Fixes #1339
Fixes #1235
Fixes #1007
Fixes #863
Fixes #815
Fixes #786
Fixes #531
Fixes #505
ajcvickers added a commit that referenced this issue Jan 6, 2021
Fixes #2725
Fixes #2707
Fixes #2525
Fixes #2524
Fixes #2331
Fixes #2105
Fixes #2076
Fixes #1541
Fixes #1489
Fixes #1453
Fixes #1398
Fixes #1339
Fixes #1235
Fixes #1007
Fixes #863
Fixes #815
Fixes #786
Fixes #531
Fixes #505
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants