LINQKit icon indicating copy to clipboard operation
LINQKit copied to clipboard

Published 20 hours ago • verified publisher icon scottksmith95

Simplify LeftJoin-extension

Open TheConstructor opened this issue 2 years ago • 1 comments

This should archive the same result, but would be way more readable. Also it doesn't require reflection.

It may also be possible to merge the remapping from a separate Select into the SelectMany, but I don't know, if this is something EF (Core) would always understand right.

TheConstructor avatar May 20 '22 09:05 TheConstructor

Added a test; with EF Core 5/6 my implementation works, while the original solution can not be translated by EF Core (Exception). Please verify against older targets, as I don't have them handy right now.

TheConstructor avatar May 21 '22 13:05 TheConstructor

@StefH I am pretty sure I did, but I opened this PR 5 months ago 😅

TheConstructor avatar Oct 23 '22 11:10 TheConstructor