Closed magicmoux closed 1 year ago
Thanks Max. I'll take a look sometime next week (famous last words before ghosting).
As for now I think this can be fixed as part of #184
Just as an afterthought (and before you ghost), these could be directly included in the Optimize Visitor (at the least to avoid the long extension chain and to get rid of the awkward names)
these could be directly included in the Optimize Visitor
Yes, please
these could be directly included in the Optimize Visitor
Yes, please
OK, creating a new PR from latest with those changes included into OptimizeExpressionVisitor
@hazzik Alex,
Working on subsequent tests upon #152 and its fixes, I tried more ways to handle sub/referenced queries than just the DbContext extension method issue.
The tests are provided within the first commit of this PR (you can fork this to see how and why they fail).