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

Column names or parts of column names may be incorrectly converted to lowercase #1

Closed
ianmcook opened this issue Aug 31, 2019 · 1 comment
Closed

Column names or parts of column names may be incorrectly converted to lowercase #1

ianmcook opened this issue Aug 31, 2019 · 1 comment
Labels
bug Something isn't working

Comments

@ianmcook
Copy link
Owner

ianmcook commented Aug 31, 2019 •
edited
Loading

When a column name matches the name of a SQL function that queryparser translates, then that column name is converted to lowercase. This also occurs for parts of columns names that are separated from other parts by periods. For example:

parse_expression("Length > 2")
#length > 2

parse_expression("Petal.Length > 2")
#Petal.length > 2

As a workaround until this is resolved, enclose variable names in backticks:

parse_expression("`Length` > 2")
#Length > 2

parse_expression("`Petal.Length` > 2")
#Petal.Length > 2
@ianmcook ianmcook changed the title Periods in names are sometimes treated like word boundaries Variable names or parts of variable names may be incorrectly converted to lowercase Aug 31, 2019
@ianmcook ianmcook changed the title Variable names or parts of variable names may be incorrectly converted to lowercase Column names or parts of column names may be incorrectly converted to lowercase Aug 31, 2019
ianmcook pushed a commit that referenced this issue Aug 31, 2019
Fix issue with column names being converted to lowercase (#1)
e6fb07d
@ianmcook
Copy link
Owner Author

Fixed in e6fb07d

@ianmcook ianmcook added the bug Something isn't working label Aug 31, 2019
ianmcook pushed a commit that referenced this issue Mar 29, 2020
@StevenHibble
Merge pull request #1 from ianmcook/master
eb44382 
Rebase
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ianmcook