For now stemmer_search.module.patch doesn't respect the module status. I mean it contains direct calls to RussianStemmer class without checking whether rustemmer module is enabled or not. This will lead to WSOD when module is disabled and someone tries to use search. It should be fixed by adding if (module_exists('rustemmer') {} clause. But I didn't look through the fix yet so it could be more complicated than that.

Anyway, if #870908: Remove patch for search module won't work out, we should elaborate a fix for this.

Comments

andypost’s picture

andypost
he/him
Primary language Russian
commented

Patch make search module dependable on rustemmer

graker’s picture

graker commented

Yes and I think that's not good at all :)

andypost’s picture

andypost
he/him
Primary language Russian
commented

This could be fixed for D7 while there's a work on search_excerpt #269911-65: Search result trimming should not fall inside HTML entities/tags

No way, pushed for D8

andypost’s picture

andypost
he/him
Primary language Russian
commented

Issue to track #493270: search_excerpt() doesn't work well with stemming