Skip to content

Fix DefaultMappingFor not consuming IdPropertyName and DisableIdInference#8877

Merged
flobernd merged 1 commit intomainfrom
fix/default-mapping-for-id-property
Apr 20, 2026
Merged

Fix DefaultMappingFor not consuming IdPropertyName and DisableIdInference#8877
flobernd merged 1 commit intomainfrom
fix/default-mapping-for-id-property

Conversation

@flobernd
Copy link
Copy Markdown
Member

Summary

  • The IEnumerable<ClrTypeMapping> overload of DefaultMappingFor was missing handling for IdPropertyName and DisableIdInference
  • The non-generic Action<ClrTypeMappingDescriptor> overload was also missing DisableIdInference handling
  • Both overloads now match the behavior of the generic DefaultMappingFor<TDocument> overload

Closes #8869

…ence (#8869)

The IEnumerable<ClrTypeMapping> overload was missing handling for
IdPropertyName and DisableIdInference.
@flobernd flobernd merged commit a8f5a6e into main Apr 20, 2026
31 of 33 checks passed
@flobernd flobernd deleted the fix/default-mapping-for-id-property branch April 20, 2026 08:42
github-actions Bot pushed a commit that referenced this pull request Apr 20, 2026
github-actions Bot pushed a commit that referenced this pull request Apr 20, 2026
github-actions Bot pushed a commit that referenced this pull request Apr 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
8.19
9.3
9.4

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

flobernd added a commit that referenced this pull request Apr 20, 2026
…IdInference (#8877) (#8878)

Co-authored-by: Florian Bernd <git@flobernd.de>
flobernd added a commit that referenced this pull request Apr 20, 2026
…dInference (#8877) (#8879)

Co-authored-by: Florian Bernd <git@flobernd.de>
flobernd added a commit that referenced this pull request Apr 20, 2026
…dInference (#8877) (#8880)

Co-authored-by: Florian Bernd <git@flobernd.de>
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.

DefaultMappingFor does not consume IdPropertyName when using IEnumerable<ClrTypeMapping> overload

1 participant