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.

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

Make filter parsing case insensitive #39

Open
thomasbalstroem opened this issue Nov 21, 2024 · 1 comment
Open

Make filter parsing case insensitive #39

thomasbalstroem opened this issue Nov 21, 2024 · 1 comment

Comments

@thomasbalstroem
Copy link

Apparently, in Malstroem ver. 270924, the AND operator is not accepted for the Filter option. Example: -filter "volume > 1 AND maxdepth > 0.1".
The error message says:

Traceback (most recent call last):
File "malstroem\scripts\cli.py", line 89, in
File "click\core.py", line 1157, in call
File "click\core.py", line 1078, in main
File "click\core.py", line 1688, in invoke
File "click\core.py", line 1434, in invoke
File "click\core.py", line 783, in invoke
File "malstroem\scripts\complete.py", line 58, in process_all
File "malstroem\scripts_utils.py", line 33, in parse_filter
File "malstroem\scripts_utils.py", line 26, in check_filter
Exception: Unsupported filter statement. Illegal parts: AND
[PYI-16520:ERROR] Failed to execute script 'cli' due to unhandled exception!

Kind regards, Thomas Balstrøm, Univ. of Copenhagen

@AsgerPetersen
Copy link
Contributor

Hi @thomasbalstroem

Could you please try using lower case 'and' instead of upper case?

Best regards

@AsgerPetersen AsgerPetersen changed the title Filter operator issue Make filter parsing case insensitive Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants