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

bug: positional params can get out of order #1

Closed
mightyguava opened this issue Nov 3, 2020 · 0 comments
Closed

bug: positional params can get out of order #1

mightyguava opened this issue Nov 3, 2020 · 0 comments
Assignees
@mightyguava

Comments

@mightyguava
Copy link
Owner

mightyguava commented Nov 3, 2020 •
edited

Key condition expressions are processed before filter condition expressions. When using positional placeholders, ordinals for key conditions always get assigned first. The ordering is wrong if key conditions actually appear later on the AST.
@mightyguava mightyguava self-assigned this Nov 3, 2020
mightyguava added a commit that referenced this issue Nov 6, 2020
@mightyguava
fix positional param ordering and add generic visitor
5377199 
Fixes #1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mightyguava mightyguava

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mightyguava