Closed (fixed)
Project:
Search API
Version:
8.x-1.x-dev
Component:
Views integration
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
7 May 2017 at 18:08 UTC
Updated:
2 Jul 2017 at 08:45 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
drunken monkeyIt's not an error (it would be in red and (usually) at the top of the form in that case), it's just a description. But it is ambiguously phrased, I guess, now that I look at it again. Also, as you note, we got the name of the parse mode wrong, it's supposed to be "Multiple words".
The attached patch changes the description to this:
Would you agree that that makes it clearer?
Comment #3
tjtj commentedThat is clearer, but I did use "Multiple words", and the text remains in thew View setup, and still appears on my pages. It just confuses my users.
Comment #4
drunken monkeyAh, so you expose the operator, too, and the description is also displayed in the exposed form?
That's of course a bad idea, yes, and shouldn't be the case. The attached revision of the patch should fix that, too, and only show the description in the admin UI.
Comment #5
drunken monkeyOops, wrong parent method.
Anyways, could you please test the patch and verify that it solves your problem?
Comment #6
borisson_Looks good to me, waiting for @tjtj to rtbc this.
Comment #8
drunken monkeyTired of waiting, and pretty sure this does what it should.
Committed.
Thanks for reviewing, Joris!