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

Extended search doesn't work when combining several search queries #103

Closed
Pigeo opened this issue Jun 26, 2017 · 4 comments
Closed

Extended search doesn't work when combining several search queries #103

Pigeo opened this issue Jun 26, 2017 · 4 comments

Comments

@Pigeo
Copy link

Pigeo commented Jun 26, 2017

I'm trying to search all codepoints inside the "Common" script having the properties "Lowercase" AND "Changes When Uppercased (CWU)" both TRUE. It gives me 8849 codepoints found whereas a simple search on the "Lowercase" property alone gives only 2173 codepoints therefore I would expect to have less than 2173 in my extended search.
It seems that my 3 constraints (Script == Common ; Lowercase == true ; CWU == true) are combined using "OR" logical operators instead of "AND" logical operators as expected.

@Boldewyn
Copy link
Contributor

This issue is still unresolved. I made some progress with the fulltext search, and most of the time it does the right thing. Unfortunately it seems like searching for scripts doesn’t work at all at the moment.

By the way: With the switch to the fulltext index you can indeed DIY a search query that matches your request, just not with a nice GUI, and with the need for a bit of insider knowledge: The search term +Zyyy +prop_Lower_1 +prop_CWU_1 will match exactly the 27 characters with the given properties: https://codepoints.net/search?q=%2BZyyy+%2Bprop_Lower_1+%2Bprop_CWU_1

@Boldewyn
Copy link
Contributor

By the way: Thank you very much for reporting it! (Although this thanks is coming almost six years too late 😔 )

@Pigeo
Copy link
Author

Pigeo commented Jan 24, 2023

You're welcome, no problem! Better late than never 😂

@Boldewyn
Copy link
Contributor

Fixed: https://codepoints.net/search?sc=Zyyy&Lower=1&CWU=1

That is, the search is still an OR search, and the query results in ~10,000 matches. But the 27 code points that match all terms are the very first ones.

If I can figure out a user-friendly way to switch between AND and OR searches, I’ll add that to the search UI. But since it is already somewhat cluttered (due to the large number of possible parameters), I’m hesitant to just slap another checkbox in there.

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