Operate on the instance variable

projectblacklight · Jan 4, 2016 · e120c98 · e120c98
1 parent 87dd1fd
commit e120c98
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/app/helpers/blacklight/url_helper_behavior.rb b/app/helpers/blacklight/url_helper_behavior.rb
@@ -92,8 +92,7 @@ def controller_tracking_method
   def link_to_query(query)
     p = search_state.to_h.except(:page, :action)
     p[:q] = query
-    link_url = search_action_path(p.to_h)
-    link_to(query, link_url)
+    link_to(query, search_action_path(p))
   end
 
   ##

diff --git a/lib/blacklight/search_state.rb b/lib/blacklight/search_state.rb
@@ -17,7 +17,7 @@ def initialize(params, blacklight_config)
         # This is the typical (not-ActionView::TestCase) code path.
         @params = params.to_unsafe_h
         # In Rails 5 to_unsafe_h returns a HashWithIndifferentAccess, in Rails 4 it returns Hash
-        @params = params.with_indifferent_access if @params.instance_of? Hash
+        @params = @params.with_indifferent_access if @params.instance_of? Hash
       end
       @blacklight_config = blacklight_config
     end

diff --git a/spec/support/controller_level_helpers.rb b/spec/support/controller_level_helpers.rb
@@ -14,4 +14,4 @@ def blacklight_configuration_context
   def initialize_controller_helpers(helper)
     helper.extend ControllerViewHelpers
   end
-end
+end