Skip to content

Use local DetectChanges when cascade-deleting #22079

New issue

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

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

Already on GitHub? # to your account

Merged
1 commit merged into from
Aug 17, 2020
Merged

Use local DetectChanges when cascade-deleting #22079

1 commit merged into from
Aug 17, 2020

Conversation

ajcvickers
Copy link
Contributor

Fixes #17828

@ajcvickers ajcvickers requested a review from a team August 15, 2020 22:42
@ghost
Copy link

ghost commented Aug 15, 2020

Hello @ajcvickers!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

[InlineData(CascadeTiming.Immediate)]
[InlineData(CascadeTiming.Never)]
[InlineData(CascadeTiming.OnSaveChanges)]
public void Can_re_parent_optional_without_DetectChanges(CascadeTiming cascadeTiming)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
public void Can_re_parent_optional_without_DetectChanges(CascadeTiming cascadeTiming)
public void Can_reparent_optional_without_DetectChanges(CascadeTiming cascadeTiming)

[InlineData(CascadeTiming.Immediate)]
[InlineData(CascadeTiming.Never)]
[InlineData(CascadeTiming.OnSaveChanges)]
public void Can_re_parent_required_without_DetectChanges(CascadeTiming cascadeTiming)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
public void Can_re_parent_required_without_DetectChanges(CascadeTiming cascadeTiming)
public void Can_reparent_required_without_DetectChanges(CascadeTiming cascadeTiming)

@ghost ghost merged commit bbde459 into main Aug 17, 2020
@ghost ghost deleted the NewBranchName0815 branch August 17, 2020 18:42
This pull request was closed.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cascade deletions ignore current state of entities resulting in unexpected data loss
2 participants