Bug 23521: (follow-up) Add quotes
authorNick Clemens <nick@bywatersolutions.com>
Fri, 25 Oct 2019 13:49:05 +0000 (13:49 +0000)
committerJoy Nelson <joy@bywatersolutions.com>
Fri, 3 Apr 2020 17:21:48 +0000 (17:21 +0000)
commit120547748d4fe6497512169b4a7a8be895128f87
treec352bb7a2df0bb3949fbde82effb47a6710fa77f
parentbfe923de3e57631ac59515c65e90e9253988de23
Bug 23521: (follow-up) Add quotes

Not only should terms from facets/limits be grouped, but order should probably matter
We build a different facet for "Dillinger Girl" or "Girl Dillinger"

The parens are possibly overkill now, but I think it makes it very clear and does not hurt

To test:
1 - Follow original plan
2 - Create a new record with 'Girl Dillinger' as author
3 - Search for 'Dill*'
4 - You get all three records (and maybe others that match)
5 - Limit by 'Girl Dillinger' - you get two records
6 - Same for 'Dillinger Girl'
7 - Apply patch
8 - Limits/facets for 'Dillinger Girl' and 'Girl Dillinger' now match a isngle record

Signed-off-by: Ere Maijala <ere.maijala@helsinki.fi>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t