Skip to content

Fix for 20303. #20477

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
merged 1 commit into from
Apr 1, 2020
Merged

Fix for 20303. #20477

merged 1 commit into from
Apr 1, 2020

Conversation

lajones
Copy link
Contributor

@lajones lajones commented Mar 31, 2020

Throw if cannot set IsUnique because the NavProp has the opposite multiplicity.

Fixes #20303.

@lajones lajones self-assigned this Mar 31, 2020
@lajones lajones requested a review from AndriySvyryd March 31, 2020 22:12
Copy link
Member

@AndriySvyryd AndriySvyryd left a comment

Choose a reason for hiding this comment

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

Also throw in InternalForeignKeyBuilder.IsUnique

@lajones lajones force-pushed the 20200330_Issue20303_01 branch from 7f871d8 to 947fd9d Compare April 1, 2020 00:15
@lajones
Copy link
Contributor Author

lajones commented Apr 1, 2020

@AndriySvyryd Completely updated this PR to take the new approach.

@lajones lajones requested a review from AndriySvyryd April 1, 2020 00:18
@lajones lajones changed the title Partial fix for 20303. Fix for 20303. Apr 1, 2020
@lajones lajones force-pushed the 20200330_Issue20303_01 branch from 947fd9d to 0a38509 Compare April 1, 2020 00:32
@lajones lajones merged commit 7eac3aa into master Apr 1, 2020
@lajones lajones deleted the 20200330_Issue20303_01 branch April 1, 2020 01:37
# 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.

Throw early when user mistakenly uses OwnsOne() but points to a collection
2 participants