-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Use the correct MemberInfo for collection include #16287
Labels
closed-fixed
The issue has been fixed and is/will be included in the release indicated by the issue milestone.
type-bug
Milestone
Comments
@AndriySvyryd - How should I construct my model to add a regression test for this? Is this only applicable to collection include (as opposed to naked collection navigation projection)? |
Something like modelbuilder.Entity<CoolEntity>().SetNavigationAccessMode(PropertyAccessMode.Field); private class CoolEntity
{
private ICollection<CoolEntity> _otherEntities;
public int Id { get; set;}
public ICollection<CoolEntity> OtherEntities
{
get => _otherEntities;
set => throw new InvalidOperationException();
}
} This applies to all collections populated directly from query. |
ajcvickers
added a commit
that referenced
this issue
Jul 28, 2019
Fixes #16287 Ended up being quite a can of worms--cleaned up some CLR accessor code that is no longer being used (it was there for old pipeline) and found several additional bugs in the same area.
ajcvickers
added a commit
that referenced
this issue
Jul 28, 2019
Fixes #16287 Ended up being quite a can of worms--cleaned up some CLR accessor code that is no longer being used (it was there for old pipeline) and found several additional bugs in the same area.
ajcvickers
added a commit
that referenced
this issue
Jul 28, 2019
Fixes #16287 Ended up being quite a can of worms--cleaned up some CLR accessor code that is no longer being used (it was there for old pipeline) and found several additional bugs in the same area.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
closed-fixed
The issue has been fixed and is/will be included in the release indicated by the issue milestone.
type-bug
Usually this should be
navigation.GetMemberInfo(forConstruction: true, forSet: true)
The text was updated successfully, but these errors were encountered: