FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix AllData for inheritance, many-to-many and non-Id keys by SimonCropp · Pull Request #1071 · VerifyTests/Verify.EntityFramework · GitHub

Fix AllData for inheritance, many-to-many and non-Id keys - #1071

Merged
SimonCropp merged 1 commit into
mainfrom
fix-alldata
Sep 23, 2026
Merged

SimonCropp merged 1 commit into
mainfrom
fix-alldata

Conversation

Copy link
Copy Markdown
Member
  • Entities in a hierarchy were returned once for each type in it, since a query for a base type already includes derived entities. Only root types are queried.
  • An implicit many-to-many join entity is a shared-type entity, which Set() cannot access, so AllData threw. Shared-type entities are queried by name.
  • Rows were ordered in memory by a property named Id, leaving other keys in database order. Rows are now ordered in the query by the primary key.

- Entities in a hierarchy were returned once for each type in it, since a query
  for a base type already includes derived entities. Only root types are queried.
- An implicit many-to-many join entity is a shared-type entity, which Set<T>()
  cannot access, so AllData threw. Shared-type entities are queried by name.
- Rows were ordered in memory by a property named Id, leaving other keys in
  database order. Rows are now ordered in the query by the primary key.
SimonCropp added this to the 15.5.0 milestone Sep 23, 2026
SimonCropp merged commit 7e3554d into main Sep 23, 2026
4 of 5 checks passed
SimonCropp deleted the fix-alldata branch September 23, 2026 01:12
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL