diff --git a/plugins/elasticsearch/controllers/elasticsearch_plugin_controller.rb b/plugins/elasticsearch/controllers/elasticsearch_plugin_controller.rb index ae6cfdc..5bd551f 100644 --- a/plugins/elasticsearch/controllers/elasticsearch_plugin_controller.rb +++ b/plugins/elasticsearch/controllers/elasticsearch_plugin_controller.rb @@ -26,7 +26,8 @@ class ElasticsearchPluginController < ApplicationController end def define_search_fields_types - @search_filter_types = ElasticsearchHelper::search_filters - @selected_filter_field = (params[:selected_filter_field] || :relevance).to_sym + @filter_types = ElasticsearchHelper::filters + @selected_filter = (params[:filter] || :relevance).to_sym end + end diff --git a/plugins/elasticsearch/helpers/elasticsearch_helper.rb b/plugins/elasticsearch/helpers/elasticsearch_helper.rb index 9208927..cfad0bf 100644 --- a/plugins/elasticsearch/helpers/elasticsearch_helper.rb +++ b/plugins/elasticsearch/helpers/elasticsearch_helper.rb @@ -11,11 +11,11 @@ module ElasticsearchHelper } end - def self.search_filters + def self.filters { - :relevance => { label: _("Relevance")}, - :lexical => { label: _("Alphabetical")}, - :recent => { label: _("More Recent")}, + :relevance => { label: _("Relevance")}, + :lexical => { label: _("Alphabetical")}, + :more_recent => { label: _("More Recent")}, } end @@ -52,7 +52,7 @@ module ElasticsearchHelper end def search_from_all_models - query = get_query params[:query], sort_by: get_sort_by(params[:selected_filter_field]) + query = get_query params[:query], sort_by: get_sort_by(params[:selected_filter]) models = searchable_models Elasticsearch::Model.search(query, models, size: default_per_page(params[:per_page])).page(params[:page]).records end @@ -61,7 +61,7 @@ module ElasticsearchHelper begin klass = model.to_s.classify.constantize - query = get_query params[:query], klass: klass, sort_by: get_sort_by(params[:selected_filter_field]) + query = get_query params[:query], klass: klass, sort_by: get_sort_by(params[:selected_filter]) klass.search(query, size: default_per_page(params[:per_page])).page(params[:page]).records rescue [] diff --git a/plugins/elasticsearch/test/unit/controllers/elasticsearch_plugin_controller_test.rb b/plugins/elasticsearch/test/unit/controllers/elasticsearch_plugin_controller_test.rb index 6ab5f5a..b03dc70 100644 --- a/plugins/elasticsearch/test/unit/controllers/elasticsearch_plugin_controller_test.rb +++ b/plugins/elasticsearch/test/unit/controllers/elasticsearch_plugin_controller_test.rb @@ -30,7 +30,8 @@ class ElasticsearchPluginControllerTest < ActionController::TestCase assert_response :success assert_not_nil assigns(:searchable_types) assert_not_nil assigns(:selected_type) - assert_not_nil assigns(:search_filter_types) + assert_not_nil assigns(:filter_types) + assert_not_nil assigns(:selected_filter) end should 'return 10 results if selected_type is nil and query is nil' do diff --git a/plugins/elasticsearch/views/elasticsearch_plugin/search.html.erb b/plugins/elasticsearch/views/elasticsearch_plugin/search.html.erb index b5b06f5..736039d 100644 --- a/plugins/elasticsearch/views/elasticsearch_plugin/search.html.erb +++ b/plugins/elasticsearch/views/elasticsearch_plugin/search.html.erb @@ -5,7 +5,7 @@
- <%= @results.total %><%= _(" results for ") %><%= @query %> + <%= @results.count %><%= _(" results for ") %><%= @query %>
<% else %>- <%= @results.total %><%= _(" total results") %> + <%= @results.count %><%= _(" total results") %>
<% end %>