Bug 37231: (Bug 34940 follow-up) Highlight logged-in library in facets does not work with ES
The highlight logged-in library in facets feature does not work with
Elasticsearch. This is because facet_title_value contains an additional
" (count)" information and so gives negative results in comparisons made
in facets.inc (line 107).
Test plan:
==========
1. Have a test installation with Elastiscsearch.
2. Set system preference 'DisplayLibraryFacets' to 'both home and holding libraries'
3. Perform a seach that will find items of your logged-in library.
In KTD, search for 'branch:CPL'
4. Check the facet (home and holding libraries) of your logged-in
library is not highlighted
5. Apply the patch ; restart_all.
6. Repeat p. 3. Check the facet (home and holding libraries)--you
should see your logged-in branch highlighted now.
Sponsored-by: Ignatianum University in Cracow
Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit 1d2b1dff3a)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
parent
47791c8868
commit
70f3a604c0
|
|
@ -497,7 +497,7 @@ sub _convert_facets {
|
|||
push @{ $facet->{facets} }, {
|
||||
facet_count => $c,
|
||||
facet_link_value => $t,
|
||||
facet_title_value => $t . " ($c)",
|
||||
facet_title_value => $t,
|
||||
facet_label_value => $label, # TODO either truncate this,
|
||||
# or make the template do it like it should anyway
|
||||
type_link_value => $type,
|
||||
|
|
|
|||
Loading…
Reference in New Issue