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

Avoid considering unchanged properties as modified #30640

Merged
merged 1 commit into from
Apr 6, 2023

Conversation

AndriySvyryd
Copy link
Member

Fixes #30601

Description

Starting with 7.0 EF compares the original store value with the current one when persisting a modified property, but due to a bug the comparison almost always returned false even if the values should be treated as equal.

Customer impact

In the reported case it results in an exception, because the properties appear to create a dependency cycle when treated as if they are modified.

How found

Customer report on 7.0

Regression

Yes

Testing

Added a test for the impacted scenario.

Risk

Low. Added quirk to revert to old behavior if necessary.

@AndriySvyryd AndriySvyryd requested a review from a team April 5, 2023 23:56
@rbhanda rbhanda added this to the 7.0.6 milestone Apr 6, 2023
@AndriySvyryd AndriySvyryd changed the title Avoid updating unchanged properties Avoid considering unchanged properties as modified Apr 6, 2023
@AndriySvyryd AndriySvyryd merged commit 31f8721 into release/7.0 Apr 6, 2023
@AndriySvyryd AndriySvyryd deleted the Issue30601 branch April 6, 2023 18:27
@rbhanda rbhanda modified the milestones: 7.0.6, 7.0.7 Jun 1, 2023
@ajcvickers ajcvickers removed this from the 7.0.7 milestone Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants