We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Expression.some does not work properly.
Expression.some
Examples:
const e1 = Expression.parse("$main.countDistinct($user)"); const e2 = Expression.parse("$main.countDistinct($user) * 100");
Check:
e1.some(e => e instanceof CountDistinctExpression); // returns true e2.some(e => e instanceof CountDistinctExpression); // returns false
That's because here:
plywood/src/expressions/baseExpression.ts
Line 969 in 574c6aa
we always return some boolean, and inside:
Line 2076 in 574c6aa
we early return if we get non-null value. So everyHelper doesn't recurse and can't find correct expression nested.
everyHelper
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
Expression.some
does not work properly.Examples:
Check:
That's because here:
plywood/src/expressions/baseExpression.ts
Line 969 in 574c6aa
we always return some boolean, and inside:
plywood/src/expressions/baseExpression.ts
Line 2076 in 574c6aa
we early return if we get non-null value. So
everyHelper
doesn't recurse and can't find correct expression nested.The text was updated successfully, but these errors were encountered: