The Stemmer processor should have a supportsIndex() which checks whether English is actually one of the site languages. If not, the processor shouldn't be available at all, it's just confusing.

CommentFileSizeAuthor
#3 3220988-3--hide_stemmer_on_non_english_sites.patch3.54 KBdrunken monkey
#3 3220988-3--hide_stemmer_on_non_english_sites--interdiff.txt3.32 KBdrunken monkey
#2 search_api-hide-stemmer-for-non-english-sites-3220988-2.patch1.08 KBvarshith

Comments

drunken monkey created an issue. See original summary.

varshith’s picture

varshith commented
Status: Active » Needs review
StatusFileSize
new search_api-hide-stemmer-for-non-english-sites-3220988-2.patch1.08 KB

Added supportsIndex() to Stemmer processor.
Works for custom language codes like en-GB too (Ref: https://www.w3.org/International/articles/language-tags/).

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
StatusFileSize
new 3220988-3--hide_stemmer_on_non_english_sites--interdiff.txt3.32 KB
new 3220988-3--hide_stemmer_on_non_english_sites.patch3.54 KB

Thanks, looks great!
I just reformatted the code a bit and added a short test. Please test/review!

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Needs review » Fixed

Committed. Thanks again!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.