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

Method to get EntityType properties ordered by key members first #1634

Closed
bricelam opened this issue Feb 16, 2015 · 3 comments
Closed

Method to get EntityType properties ordered by key members first #1634

bricelam opened this issue Feb 16, 2015 · 3 comments
Assignees
Labels
closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement
Milestone

Comments

@bricelam
Copy link
Contributor

There are a few different places in the code where we want to get a list of an entity type's properties with the key members first. These currently include:

  • Migrations
    • CreateTable calls
    • Model snapshot
  • Reverse engineer

We should have one place to do this. For example, an extension method on IEntityType.

@bricelam
Copy link
Contributor Author

See also this discussion.

@AndriySvyryd
Copy link
Member

The order should be configurable in the model. The default order would be set by a convention that the providers can replace.

@mikary
Copy link
Contributor

mikary commented Mar 27, 2015

Properties are now ordered with properties that are part of the primary key first (in the order specified on the key for composite keys) in PR #1896

@mikary mikary closed this as completed Mar 27, 2015
@mikary mikary modified the milestones: 7.0.0-beta5, 7.0.0 Mar 27, 2015
@ajcvickers ajcvickers added the closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. label Oct 15, 2022
@ajcvickers ajcvickers modified the milestones: 1.0.0-beta5, 1.0.0 Oct 15, 2022
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-enhancement
Projects
None yet
Development

No branches or pull requests

5 participants