@@ -, +, @@ --- Koha/SearchEngine/Elasticsearch/QueryBuilder.pm | 2 +- .../Koha_SearchEngine_Elasticsearch_Search.t | 20 ++++++++++---------- 2 files changed, 11 insertions(+), 11 deletions(-) --- a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm +++ a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm @@ -487,7 +487,7 @@ sub _convert_sort_fields { pubdate => 'pubdate', ); my %sort_order_convert = - ( qw( dsc desc ), qw( asc asc ), qw( az asc ), qw( za desc ) ); + ( qw( desc desc ), qw( dsc desc ), qw( asc asc ), qw( az asc ), qw( za desc ) ); # Convert the fields and orders, drop anything we don't know about. grep { $_->{field} } map { --- a/t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t +++ a/t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t @@ -230,7 +230,7 @@ subtest 'build_query tests' => sub { }; subtest "_convert_sort_fields" => sub { - plan tests => 1; + plan tests => 2; my @sort_by = $builder->_convert_sort_fields(qw( call_number_asc author_dsc )); is_deeply( \@sort_by, @@ -242,13 +242,13 @@ subtest "_convert_sort_fields" => sub { ); # We could expect this to pass, but direction is undef instead of 'desc' - #@sort_by = $builder->_convert_sort_fields(qw( call_number_asc author_desc )); - #is_deeply( - # \@sort_by, - # [ - # { field => 'callnum', direction => 'asc' }, - # { field => 'author', direction => 'desc' } - # ], - # 'sort fields should have been split correctly' - #); + @sort_by = $builder->_convert_sort_fields(qw( call_number_asc author_desc )); + is_deeply( + \@sort_by, + [ + { field => 'callnum', direction => 'asc' }, + { field => 'author', direction => 'desc' } + ], + 'sort fields should have been split correctly' + ); }; --