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

Boolean operators raise filter error: Expected left to be defined. #26

Closed
MartinMa opened this issue Feb 14, 2023 · 6 comments · Fixed by #27
Closed

Boolean operators raise filter error: Expected left to be defined. #26

MartinMa opened this issue Feb 14, 2023 · 6 comments · Fixed by #27
Labels

Comments

@MartinMa
Copy link
Contributor

The README explains, that boolean expressions like the following one can be used. But the filter method raises the error Expected left to be defined., when using this exact query (straight from the README):

(name:"foo bar" AND bio:"quick fox") OR name:fox

The parser works fine. The filter methods seems to be flawed, from what I can see.

I also couldn't find any tests that cover this use case.

liqe version: 3.6.0

@MartinMa
Copy link
Contributor Author

The fix was fairly easy. I added a PR.

@connorjs
Copy link

What are the steps to get this merged? I just hit this same error.

@ronnyTodgers
Copy link

@gajus could you merge this? It's such a brilliant library!

@gajus gajus closed this as completed in #27 Oct 3, 2023
@github-actions
Copy link

github-actions bot commented Oct 3, 2023

🎉 This issue has been resolved in version 3.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ronnyTodgers
Copy link

Thank you @gajus !

@gajus
Copy link
Owner

gajus commented Oct 4, 2023

Of course

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants