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.

Sign up for GitHub

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

Jump to bottom

Query: include over SelectMany with reference navigation projection doesn't work #11053

Closed
maumar opened this issue Feb 24, 2018 · 1 comment
Closed

Query: include over SelectMany with reference navigation projection doesn't work #11053

maumar opened this issue Feb 24, 2018 · 1 comment
Assignees
@maumar
Labels
closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-bug
Milestone
2.1.0

Comments

@maumar
Copy link
Contributor

maumar commented Feb 24, 2018

Reported by @LordMike in #10561:

db.Reservations
  .SelectMany(s => s.Applicants)
  .Select(s => s.Applicant)
  .Include(s => s.Properties)
  .ToList();

does not register the include
@maumar maumar self-assigned this Feb 24, 2018
@maumar maumar added this to the 2.1.0 milestone Feb 24, 2018
maumar added a commit that referenced this issue Feb 24, 2018
@maumar
Fix to #11053 - Query: include over SelectMany with reference navigat…
ef5cfee 
…ion projection doesn't work

Problem was that we were not properly updating PathFromQuerySource and QuerySource properties on IncludeResultOperator when processing SelectMany in nav rewrite. This issue is similar to #10101
maumar added a commit that referenced this issue Feb 24, 2018
@maumar
Fix to #11053 - Query: include over SelectMany with reference navigat…
46e09c0 
…ion projection doesn't work

Problem was that we were not properly updating PathFromQuerySource and QuerySource properties on IncludeResultOperator when processing SelectMany in nav rewrite. This issue is similar to #10101
maumar added a commit that referenced this issue Feb 27, 2018
@maumar
Fix to #11053 - Query: include over SelectMany with reference navigat…
44ecf7a 
…ion projection doesn't work

Problem was that we were not properly updating PathFromQuerySource and QuerySource properties on IncludeResultOperator when processing SelectMany in nav rewrite. This issue is similar to #10101
@maumar
Copy link
Contributor Author

maumar commented Feb 28, 2018

fixed in 44ecf7a

@maumar maumar closed this as completed Feb 28, 2018
@maumar maumar added the closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. label Feb 28, 2018
@maumar maumar mentioned this issue Jun 1, 2018
Closed
@ajcvickers ajcvickers modified the milestones: 2.1.0-preview2, 2.1.0 Nov 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maumar maumar

Labels
closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-bug
Projects
None yet
Milestone
2.1.0
Development

No branches or pull requests
2 participants
@ajcvickers @maumar