-
Notifications
You must be signed in to change notification settings - Fork 22
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
Query for multiple indexes with similar names broken #78
Comments
…were interpreted as options. Fixes #78.
It still has to be decided whether the error handling should be different depending on the definition type of the option (via dict or index id with suffix separated by underscore character). I would be happy to receive your feedback. |
…were interpreted as options. Fixes #78.
…were interpreted as options. (#79) * Fix case where multiple indexes with similar name seperated by ``_`` were interpreted as options. Fixes #78. * fix ambiguity * raise ValueError again, when operator is not found ... this is intended behaviour * add tests for ambiguity fix * catch possible unpack error Co-authored-by: Michael Howitz <[email protected]> Co-authored-by: Peter Mathis <[email protected]> Co-authored-by: agitator <[email protected]>
Here we have still an open edge case: Given these two similar indexes
and searching by results in |
/cc @andbag
When constructing a query with similar named indexes like
workspace
andworkspace_root
, the query is broken since: #67Since
workspace_root
is available in the request,_root
is interpreted as an option for theworkspace
index. This fails with:This issue is related to: eea/eea.facetednavigation#174
The text was updated successfully, but these errors were encountered: