Bugzilla – Attachment 42067 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: Display the correct number of facets (5 instead of 6)
Bug-12478-Display-the-correct-number-of-facets-5-i.patch (text/plain), 1.06 KB, created by
Jonathan Druart
on 2015-08-28 11:38:56 UTC
(
hide
)
Description:
Bug 12478: Display the correct number of facets (5 instead of 6)
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2015-08-28 11:38:56 UTC
Size:
1.06 KB
patch
obsolete
>From 1eadb1a86bba985d0b5b4f432254cdf1531ac73e Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Fri, 28 Aug 2015 11:24:24 +0100 >Subject: [PATCH] Bug 12478: Display the correct number of facets (5 instead of > 6) > >The loop on facet entries was wrong, it should stop after 5 >iterations. >--- > 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 51e485f..56dec5a 100644 >--- a/Koha/SearchEngine/Elasticsearch/Search.pm >+++ b/Koha/SearchEngine/Elasticsearch/Search.pm >@@ -415,7 +415,7 @@ sub _convert_facets { > "type_label_$type_to_label{$type}" => 1, > type_link_value => $type, > }; >- foreach my $term ( @{ $data->{terms} }[ 0 .. $limit ] ) { >+ foreach my $term ( @{ $data->{terms} }[ 0 .. $limit - 1 ] ) { > my $t = $term->{term}; > my $c = $term->{count}; > if ( exists( $special{$type} ) ) { >-- >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