From 75bbf39ec8428613dabdd7955e7f089f59e8c13f Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Fri, 16 Mar 2018 11:39:30 +0000 Subject: [PATCH] Bug 20114: (follow-up) Don't add sort in query cgi To test: 1 - Do a search for"*:*" 2 - Try to limit using a facet 3 - Get a search error 4 - Note the URL says kw=*:* 5 - Apply patch 6 - Repeate search 7 - Facet should work 8 - Note URL is better formed 9 - Test with advanced searches as well --- Koha/SearchEngine/Elasticsearch/QueryBuilder.pm | 3 --- 1 file changed, 3 deletions(-) diff --git a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm index 55b8d25..c156761 100644 --- a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm +++ b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm @@ -283,9 +283,6 @@ sub _build_query_query_string { $uri->query_param_append('q' => $operand) if $operand; $uri->query_param_append('op' => $operator) if $operator; } - if ($query_parts->{sort_by}) { - $uri->query_param('sort_by' => $query_parts->{sort_by}); - } return $uri->query; } -- 2.1.4