gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 24506: Multibranch limit does not work with ElasticSearch 

Test plan
1. Set up a system with ElasticSearch
2. Set up or make sure ther are at least two branches
3. create a group of branches
4. make sure there are items associated with any library in the group
5. make a search in opac limited by the library group
6. make an advanced search in intra limited by the library group
7. change SearchEngine system preference to Zebra and repeat 5. and 6.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
This commit is contained in:
Andreas Jonsson 2020-01-24 10:13:05 +00:00 committed by Joy Nelson
parent 3b3a84d272
commit 947db3b6cb
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
catalogue/search.pl
View File

@ -415,7 +415,7 @@ my %is_nolimit = map { $_ => 1 } @nolimits;
if($params->{'multibranchlimit'}) {
my $search_group = Koha::Library::Groups->find( $params->{multibranchlimit} );
my @libraries = $search_group->all_libraries;
my $multibranch = '('.join( " or ", map { 'branch: ' . $_->branchcode } @libraries ) .')';
my $multibranch = '('.join( " OR ", map { '(homebranch: ' . $_->branchcode .')' } @libraries ) .')';
push @limits, $multibranch if ($multibranch ne '()');
}

2
opac/opac-search.pl
View File

@ -503,7 +503,7 @@ if (@searchCategories > 0) {
if($params->{'multibranchlimit'}) {
my $search_group = Koha::Library::Groups->find( $params->{multibranchlimit} );
my @libraries = $search_group->all_libraries;
my $multibranch = '('.join( " or ", map { 'branch: ' . $_->branchcode } @libraries ) .')';
my $multibranch = '('.join( " OR ", map { 'homebranch: ' . $_->branchcode } @libraries ) .')';
push @limits, $multibranch if ($multibranch ne '()');
}