-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Nav expansion visitor does not visit the non-lambda arguments (Skip, Take, ElementAt) #32312
Labels
area-query
closed-fixed
The issue has been fixed and is/will be included in the release indicated by the issue milestone.
Servicing-approved
type-bug
Milestone
Comments
maumar
added a commit
that referenced
this issue
Nov 16, 2023
…m argument (and possibly other non-lambda arguments) This was always a problem but was uncovered by change in how we process Contains. We were going through the full process of nav expansion on the argument (only on the source), which could lead to untranslatable expression tree in some cases. Fixes #32217 Fixes #32312
maumar
added a commit
that referenced
this issue
Nov 16, 2023
…m argument (and possibly other non-lambda arguments) This was always a problem but was uncovered by change in how we process Contains. We were going through the full process of nav expansion on the argument (only on the source), which could lead to untranslatable expression tree in some cases. Fixes #32217 Fixes #32312
maumar
added a commit
that referenced
this issue
Nov 16, 2023
…m argument (and possibly other non-lambda arguments) This was always a problem but was uncovered by change in how we process Contains. We were going through the full process of nav expansion on the argument (only on the source), which could lead to untranslatable expression tree in some cases. Fixes #32217 Fixes #32312
ajcvickers
added
the
closed-fixed
The issue has been fixed and is/will be included in the release indicated by the issue milestone.
label
Nov 17, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area-query
closed-fixed
The issue has been fixed and is/will be included in the release indicated by the issue milestone.
Servicing-approved
type-bug
This tracks having nav expansion properly visit non-lambda arguments. It is the same as #32217, but tracks all operators other than Contains; this is because we want to have separate quirks for the 8.0.x patch, to allow users to keep visiting Contains (clear, known regression), but avoid visiting other operators in case that causes a problem.
The text was updated successfully, but these errors were encountered: