Bugzilla – Attachment 43128 Details for
Bug 12478
Elasticsearch support for Koha
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 12478: (follow-up) Display the correct number of facets
Bug-12478-follow-up-Display-the-correct-number-of-.patch (text/plain), 1.25 KB, created by
Jonathan Druart
on 2015-10-05 13:42:06 UTC
(
hide
)
Description:
Bug 12478: (follow-up) Display the correct number of facets
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2015-10-05 13:42:06 UTC
Size:
1.25 KB
patch
obsolete
>From a81a9abe701388e3742361011b707c1f3f6aa1e3 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Mon, 5 Oct 2015 10:58:50 +0100 >Subject: [PATCH] Bug 12478: (follow-up) Display the correct number of facets > >This patch fixes a side-effect of the previous "Display the correct >number of facets (5 instead of 6)" patch. > >If you ask for more facets ('show more' link) and that the number of >facets is > 5 but < 10, 1 facet will be missing. >--- > Koha/SearchEngine/Elasticsearch/Search.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/Koha/SearchEngine/Elasticsearch/Search.pm b/Koha/SearchEngine/Elasticsearch/Search.pm >index 520baca..aa9da8b 100644 >--- a/Koha/SearchEngine/Elasticsearch/Search.pm >+++ b/Koha/SearchEngine/Elasticsearch/Search.pm >@@ -416,7 +416,7 @@ sub _convert_facets { > "type_label_$type_to_label{$type}" => 1, > type_link_value => $type, > }; >- $limit = $#{ $data->{terms} } if ( $limit > @{ $data->{terms} } ); >+ $limit = @{ $data->{terms} } if ( $limit > @{ $data->{terms} } ); > foreach my $term ( @{ $data->{terms} }[ 0 .. $limit - 1 ] ) { > my $t = $term->{term}; > my $c = $term->{count}; >-- >2.1.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 12478
:
29204
|
31128
|
31129
|
33136
|
33137
|
33138
|
33139
|
33140
|
33141
|
35654
|
40903
|
42030
|
42031
|
42032
|
42033
|
42062
|
42063
|
42064
|
42065
|
42066
|
42067
|
42068
|
42069
|
42392
|
42393
|
43127
|
43128
|
43129
|
43130
|
43131
|
43132
|
43133
|
43134
|
43140
|
43141
|
43142
|
43148
|
43345
|
43346
|
43347
|
43348
|
43349
|
43350
|
43351
|
43352
|
43373
|
47767
|
47768
|
50115
|
50159
|
50211
|
50212
|
50213
|
50214
|
50245
|
50246
|
50291
|
50292