Though search api provides a way to change parse mode conjunction. It is not possible to select conjunction with Multiple words parse mode, which is default to AND. Provide a setting with parse mode to be able to change default AND conjunction to OR as well.

CommentFileSizeAuthor
#9 search_api_page-3108496-5.patch4.67 KBash2303
#4 search_api_page-3108496-4.patch4.47 KBash2303
#3 search_api_page-3108496-3.patch4.39 KBash2303
#2 search_api_page-3108496-2.patch4.2 KBash2303

Comments

ash2303 created an issue. See original summary.

ash2303’s picture

ash2303 commented
StatusFileSize
new search_api_page-3108496-2.patch4.2 KB
ash2303’s picture

ash2303 commented
StatusFileSize
new search_api_page-3108496-3.patch4.39 KB

Resolved failing tests.

ash2303’s picture

ash2303 commented
StatusFileSize
new search_api_page-3108496-4.patch4.47 KB
ash2303’s picture

ash2303 commented
Status: Active » Needs review
ash2303’s picture

ash2303 commented
Assigned: ash2303 » Unassigned
weseze’s picture

weseze commented

Is there a reason why the keyword OR is not recognized in any of the parse modes? I would expect that keyword to just work since it is also indicated in the "no-results" text.

Is this an issue with search_api or search_api_page?

ash2303’s picture

ash2303 commented

The issue is related to search_api_page as search_api already has support. Just that there is no option in search_api_page form to add OR support with Parse Mode.

ash2303’s picture

ash2303 commented
StatusFileSize
new search_api_page-3108496-5.patch4.67 KB

Correct config export miss.

The last submitted patch, 4: search_api_page-3108496-4.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

crtlf’s picture

crtlf commented

Thanks a lot, @ash2303 !

I'm a bit surprised that this issue hasn't been tested in four years, as it seems like a pretty common feature.
I can confirm that the patch is still working on 8.x-1.38

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Needs review » Needs work

#11 sounds like RTBC. Still I think it would make sense to turn this into a MR.

It also adds a new test, so this is great! Would someone like to finish this?