@@ -, +, @@ --- Koha/SearchEngine/Elasticsearch/QueryBuilder.pm | 4 ++-- t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t | 4 +++- 2 files changed, 5 insertions(+), 3 deletions(-) --- a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm +++ a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm @@ -132,11 +132,11 @@ sub build_query { my $display_library_facets = C4::Context->preference('DisplayLibraryFacets'); if ( $display_library_facets eq 'both' or $display_library_facets eq 'home' ) { - $res->{aggregations}{homebranch} = { terms => { field => "homebranch__facet" } }; + $res->{aggregations}{homebranch} = { terms => { field => "homebranch__facet", size => $size } }; } if ( $display_library_facets eq 'both' or $display_library_facets eq 'holding' ) { - $res->{aggregations}{holdingbranch} = { terms => { field => "holdingbranch__facet" } }; + $res->{aggregations}{holdingbranch} = { terms => { field => "holdingbranch__facet", size => $size } }; } return $res; } --- a/t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t +++ a/t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t @@ -216,7 +216,7 @@ subtest 'build_authorities_query_compat() tests' => sub { }; subtest 'build_query tests' => sub { - plan tests => 50; + plan tests => 51; my $qb; @@ -244,9 +244,11 @@ subtest 'build_query tests' => sub { ); t::lib::Mocks::mock_preference('FacetMaxCount','37'); + t::lib::Mocks::mock_preference('DisplayLibraryFacets','both'); $query = $qb->build_query('test', %options); ok( defined $query->{aggregations}{ccode}{terms}{size},'we need to ask for a size or we get only 5 facet' ); is( $query->{aggregations}{ccode}{terms}{size}, 37,'we ask for the size as defined by the syspref FacetMaxCount'); + is( $query->{aggregations}{homebranch}{terms}{size}, 37,'we ask for the size as defined by the syspref FacetMaxCount fir homebranch'); t::lib::Mocks::mock_preference('DisplayLibraryFacets','both'); $query = $qb->build_query(); --