Apologies if I'm not following correct procedure. I commented on another issue here but its marked as fixed, so I thought maybe I should start a new issue.
It's the same problem mentioned in that thread: the facets don't display correctly when searching for multiple words. Not only are the counts wrong (all of them are "1"), but some facets don't display all options that have results.
I can't help with coding, but I'm happy to do whatever I can to diagnose the problem!
Thanks!
| Comment | File | Size | Author |
|---|---|---|---|
| #20 | interdiff-18-20.txt | 491 bytes | rosinegrean |
| #20 | 2286329-20--multi_word_searches_with_facets.patch | 1.22 KB | rosinegrean |
| #18 | interdiff-15-18.txt | 1.02 KB | rosinegrean |
| #18 | 2286329-18--multi_word_searches_with_facets.patch | 1.19 KB | rosinegrean |
| #15 | 2286329-15--multi_word_searches_with_facets.patch | 1.08 KB | rosinegrean |
Comments
Comment #1
drunken monkeyFiling a new issue here was the right thing to do, I think. In general, posting into an existing issue is preferred, but if it's already closed you need to re-open it (set the status to "Active" again), otherwise it's unlikely that anyone would see your new comment.
However, in this case the issue is also rather old, so creating a new one is probably better.
Yes, the linked issue was committed and the problem fixed. However, it seems we inadverdently reverted that fix in #1299238: Add option for partial matching. I posted there to discuss why this was done or whether it was just a mistake.
In any case, the attached patch should fix this. I also included a regression test to make sure this doesn't happen again.
Please test!
Comment #3
drunken monkeyComment #4
drunken monkeyI hate this "feature".
Comment #5
lesliewagner1165 commentedThanks so much for the fix!
Now I just need some help on how to get the code into my site. (I know next to nothing about coding, and am terrified of messing things up!) Where might I go for explicit instructions? (I'm talking "explain this to me like I'm a fourth grader" instructions!)
Thanks
Comment #6
jibize commentedThanks drunken monkey! working great for me.
@lesliewagner1165 checkout https://www.drupal.org/patch/apply for info on how to apply a path.
Comment #7
jibize commentedHumm sorry spoke to fast, there are still some discrepancies between the facets and the result number. I haven't figure out why it is not always working as expected but I will keep digging.
Comment #8
jibize commentedActually there was a problem with my search index. Patch working great! ;-)
Comment #9
Johnny vd Laar commentedAs far as I can see it doesn't break anything for me.
Comment #10
drunken monkeyOK, great to hear. Thank you all for testing!
Committed.
Comment #13
drunken monkeyNeeds to be ported to D8.
Comment #14
drunken monkeyComment #15
rosinegrean commentedAs I see it, only the regression test needs to be ported.
Comment #18
rosinegrean commentedComment #20
rosinegrean commentedComment #21
drunken monkeyLooks good, thanks a lot!
Committed.