Attachment 108177 Details for Bug 25872 – Bug 25872: Fix scoping of $limits

[patch] Bug 25872: Fix scoping of $limits

Bug-25872-Fix-scoping-of-limits.patch (text/plain), 1.32 KB, created by Nick Clemens (kidclamp) on 2020-08-13 12:58:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2020-08-13 12:58:50 UTC

Size: 1.32 KB

patch

obsolete

>From 0b569f71a56c41589de943a6f5671f2bdebc5b75 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Thu, 13 Aug 2020 12:56:33 +0000
>Subject: [PATCH] Bug 25872: Fix scoping of $limits
>
>To test:
>1 - Enable Elasticsearch
>2 - On OPAC perform advanced search, selecting only an itype, ccode, or LOC limit
>3 - Attempt to sort results
>4 - You are returned to Advanced search
>5 - Apply patch
>6 - Repeat
>7 - It sorts!
>---
> Koha/SearchEngine/Elasticsearch/QueryBuilder.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
>index 8f3aea9445..8e2adf8136 100644
>--- a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
>+++ b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
>@@ -176,7 +176,7 @@ sub build_query_compat {
>     } else {
>         my @sort_params  = $self->_convert_sort_fields(@$sort_by);
>         my @index_params = $self->_convert_index_fields(@$indexes);
>-        my $limits       = $self->_fix_limit_special_cases($orig_limits);
>+        $limits       = $self->_fix_limit_special_cases($orig_limits);
>         if ( $params->{suppress} ) { push @$limits, "suppress:false"; }
>         # Merge the indexes in with the search terms and the operands so that
>         # each search thing is a handy unit.
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 25872: 108176 | 108177 | 108211 | 108212 | 108257 | 108258 | 108507