blacklight 6.4.0 → 6.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 08a32b124f67c2eee4651aa298a0c04c6cc1b0a3
-  data.tar.gz: 99eb3c4e5a513755c0b44572d3f59e43ad46ae84
+  metadata.gz: 48b6e29df0426d17771b97c8cef4084c11e2398c
+  data.tar.gz: 6b1bd2738314e8b9966c31625455bd4cd6da99f5
 SHA512:
-  metadata.gz: 36debbcf121bb69692fd23a4f3183fa11a463cf33d754e2dd25589ef3fed03b2d89ef8c12a695bbc6666f42ecb67b2d6cc00600fbec0eeb1e2d2bd12970c993c
-  data.tar.gz: 4efcffee2bac9324aa344f2671a987e5628f1be7444c3517c4f8b63ea6917a7b8fee38dc796d616fb5a637dfca961382d89e6e0bca6512c49733f60a7c891874
+  metadata.gz: a930acfc5fb45d28e874e569f2efe499e4cdb46ee5bafa3370be6f9ec2c9840b52ac243a6844777418cd709b2337377543bf89dd74451b509bb5b4d90f1ee0b7
+  data.tar.gz: d2783d8e3661d95f9aa758b0b0ea6d3fbf3b80ac32320d53d8a37640d7b7dab5651f1251edd155eefa188084440d8ae73dc3ce4fb9e6c5866afbc02ee70f6e1d

data/VERSION

@@ -1 +1 @@
-6.4.0
+6.4.1

data/app/assets/stylesheets/blacklight/_twitter_typeahead.scss

@@ -1,7 +1,7 @@
 .twitter-typeahead {
   float: left;
   width: 100%;
-  z-index: 10000;
+  z-index: 500;
 
   input.tt-input.form-control {
     width: 100%;

data/lib/blacklight/search_state.rb

@@ -10,22 +10,25 @@ module Blacklight
     # @param [ActionController::Parameters] params
     # @param [Blacklight::Config] blacklight_config
     def initialize(params, blacklight_config)
-      if params.instance_of? Hash
-        # This is an ActionView::TestCase workaround. Will be resolved by
-        # https://github.com/rails/rails/pull/22913 (Rails > 4.2.5)
-        @params = params.with_indifferent_access
-      else
+      if params.respond_to?(:to_unsafe_h)
         # 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
+      elsif params.is_a? Hash
+        # This is an ActionView::TestCase workaround for Rails 4.2.
+        @params = params.dup.with_indifferent_access
+      else
+        @params = params.dup.to_h.with_indifferent_access
       end
+
       @blacklight_config = blacklight_config
     end
 
-    def to_h
+    def to_hash
       @params
     end
+    alias to_h to_hash
 
     def reset
       Blacklight::SearchState.new(ActionController::Parameters.new, blacklight_config)
@@ -112,7 +115,7 @@ module Blacklight
     # @yield [params] The merged parameters hash before being sanitized
     def params_for_search(params_to_merge={}, &block)
       # params hash we'll return
-      my_params = params.dup.merge(params_to_merge.dup)
+      my_params = params.dup.merge(Blacklight::SearchState.new(params_to_merge, blacklight_config))
 
       if block_given?
         yield my_params

data/spec/lib/blacklight/search_state_spec.rb

@@ -9,27 +9,61 @@ describe Blacklight::SearchState do
   end
 
   let(:parameter_class) { ActionController::Parameters }
-  let(:helper) { described_class.new(params, blacklight_config) }
+  let(:search_state) { described_class.new(params, blacklight_config) }
   let(:params) { parameter_class.new }
 
+  describe '#to_h' do
+    let(:data) { { a: '1'} }
+    let(:params) { parameter_class.new data }
+
+    it 'returns a copy of the original parameters' do
+      expect(search_state.to_h).to eq data.with_indifferent_access
+      expect(search_state.to_h.object_id).not_to eq params.object_id
+    end
+
+    context 'with AC::Parameters' do
+      let(:parameter_class) { ActionController::Parameters }
+
+      it 'returns the hash data' do
+        expect(search_state.to_h).to eq data.with_indifferent_access
+      end
+    end
+
+    context 'with HashWithIndifferentAccess' do
+      let(:parameter_class) { HashWithIndifferentAccess }
+
+      it 'returns the hash data' do
+        expect(search_state.to_h).to eq data.with_indifferent_access
+      end
+    end
+
+    context 'with Hash' do
+      let(:params) { data }
+
+      it 'returns the hash data' do
+        expect(search_state.to_h).to eq data.with_indifferent_access
+      end
+    end
+  end
+
   describe "params_for_search" do
     let(:params) { parameter_class.new 'default' => 'params' }
 
     it "takes original params" do
-      result = helper.params_for_search
+      result = search_state.params_for_search
       expect(result).to eq({ 'default' => 'params' })
       expect(params.object_id).to_not eq result.object_id
     end
 
     it "accepts params to merge into the controller's params" do
-      result = helper.params_for_search(q: 'query')
+      result = search_state.params_for_search(q: 'query')
       expect(result).to eq('q' => 'query', 'default' => 'params')
     end
 
     context "when params have blacklisted keys" do
       let(:params) { parameter_class.new action: 'action', controller: 'controller', id: "id", commit: 'commit' }
       it "removes them" do
-        result = helper.params_for_search
+        result = search_state.params_for_search
         expect(result.keys).to_not include(:action, :controller, :commit, :id)
       end
     end
@@ -38,7 +72,7 @@ describe Blacklight::SearchState do
       context "and per_page changed" do
         let(:params) { parameter_class.new per_page: 20, page: 5 }
         it "adjusts the current page" do
-          result = helper.params_for_search(per_page: 100)
+          result = search_state.params_for_search(per_page: 100)
           expect(result[:page]).to eq 1
         end
       end
@@ -46,7 +80,7 @@ describe Blacklight::SearchState do
       context "and per_page didn't change" do
         let(:params) { parameter_class.new per_page: 20, page: 5 }
         it "doesn't change the current page" do
-          result = helper.params_for_search(per_page: 20)
+          result = search_state.params_for_search(per_page: 20)
           expect(result[:page]).to eq 5
         end
       end
@@ -54,7 +88,7 @@ describe Blacklight::SearchState do
       context "and the sort changes" do
         let(:params) { parameter_class.new sort: 'field_a', page: 5 }
         it "adjusts the current page" do
-          result = helper.params_for_search(sort: 'field_b')
+          result = search_state.params_for_search(sort: 'field_b')
           expect(result[:page]).to eq 1
         end
       end
@@ -62,7 +96,7 @@ describe Blacklight::SearchState do
       context "and the sort didn't change" do
         let(:params) { parameter_class.new sort: 'field_a', page: 5 }
         it "doesn't change the current page" do
-          result = helper.params_for_search(sort: 'field_a')
+          result = search_state.params_for_search(sort: 'field_a')
           expect(result[:page]).to eq 5
         end
       end
@@ -71,7 +105,7 @@ describe Blacklight::SearchState do
     context "with a block" do
       let(:params) { parameter_class.new a: 1, b: 2 }
       it "evalutes the block and allow it to add or remove keys" do
-        result = helper.params_for_search(c: 3) do |params|
+        result = search_state.params_for_search(c: 3) do |params|
           params.extract! :a, :b
           params[:d] = 'd'
         end
@@ -100,14 +134,14 @@ describe Blacklight::SearchState do
     context "when there are no pre-existing facets" do
       let(:params) { params_no_existing_facet }
       it "adds facet value" do
-        result_params = helper.add_facet_params("facet_field", "facet_value")
+        result_params = search_state.add_facet_params("facet_field", "facet_value")
         expect(result_params[:f]).to be_a Hash
         expect(result_params[:f]["facet_field"]).to be_a_kind_of(Array)
         expect(result_params[:f]["facet_field"]).to eq ["facet_value"]
       end
 
       it "leaves non-facet params alone" do
-        result_params = helper.add_facet_params("facet_field_2", "new_facet_value")
+        result_params = search_state.add_facet_params("facet_field_2", "new_facet_value")
 
         params.each_pair do |key, value|
           next if key == :f
@@ -116,9 +150,9 @@ describe Blacklight::SearchState do
       end
 
       it "uses the facet's key in the url" do
-        allow(helper).to receive(:facet_configuration_for_field).with('some_field').and_return(double(single: true, field: "a_solr_field", key: "some_key"))
+        allow(search_state).to receive(:facet_configuration_for_field).with('some_field').and_return(double(single: true, field: "a_solr_field", key: "some_key"))
 
-        result_params = helper.add_facet_params('some_field', 'my_value')
+        result_params = search_state.add_facet_params('some_field', 'my_value')
 
         expect(result_params[:f]['some_key']).to have(1).item
         expect(result_params[:f]['some_key'].first).to eq 'my_value'
@@ -128,7 +162,7 @@ describe Blacklight::SearchState do
     context "when there are pre-existing facets" do
       let(:params) { params_existing_facets }
       it "adds a facet param to existing facet constraints" do
-        result_params = helper.add_facet_params("facet_field_2", "new_facet_value")
+        result_params = search_state.add_facet_params("facet_field_2", "new_facet_value")
 
         expect(result_params[:f]).to be_a Hash
 
@@ -143,7 +177,7 @@ describe Blacklight::SearchState do
       end
 
       it "leaves non-facet params alone" do
-        result_params = helper.add_facet_params("facet_field_2", "new_facet_value")
+        result_params = search_state.add_facet_params("facet_field_2", "new_facet_value")
 
         params.each_pair do |key, value|
           next if key == 'f'
@@ -155,8 +189,8 @@ describe Blacklight::SearchState do
     context "with a facet selected in the params" do
       let(:params) { parameter_class.new f: { 'single_value_facet_field' => 'other_value' } }
       it "replaces facets configured as single" do
-        allow(helper).to receive(:facet_configuration_for_field).with('single_value_facet_field').and_return(double(single: true, key: "single_value_facet_field"))
-        result_params = helper.add_facet_params('single_value_facet_field', 'my_value')
+        allow(search_state).to receive(:facet_configuration_for_field).with('single_value_facet_field').and_return(double(single: true, key: "single_value_facet_field"))
+        result_params = search_state.add_facet_params('single_value_facet_field', 'my_value')
 
         expect(result_params[:f]['single_value_facet_field']).to have(1).item
         expect(result_params[:f]['single_value_facet_field'].first).to eq 'my_value'
@@ -164,20 +198,20 @@ describe Blacklight::SearchState do
     end
 
     it "accepts a FacetItem instead of a plain facet value" do
-      result_params = helper.add_facet_params('facet_field_1', double(value: 123))
+      result_params = search_state.add_facet_params('facet_field_1', double(value: 123))
 
       expect(result_params[:f]['facet_field_1']).to include(123)
     end
 
     it "defers to the field set on a FacetItem" do
-      result_params = helper.add_facet_params('facet_field_1', double(:field => 'facet_field_2', :value => 123))
+      result_params = search_state.add_facet_params('facet_field_1', double(:field => 'facet_field_2', :value => 123))
 
       expect(result_params[:f]['facet_field_1']).to be_blank
       expect(result_params[:f]['facet_field_2']).to include(123)
     end
 
     it "adds any extra fq parameters from the FacetItem" do
-      result_params = helper.add_facet_params('facet_field_1', double(:value => 123, fq: { 'facet_field_2' => 'abc' }))
+      result_params = search_state.add_facet_params('facet_field_1', double(:value => 123, fq: { 'facet_field_2' => 'abc' }))
 
       expect(result_params[:f]['facet_field_1']).to include(123)
       expect(result_params[:f]['facet_field_2']).to include('abc')
@@ -197,7 +231,7 @@ describe Blacklight::SearchState do
     }
 
     it "does not include request parameters used by the facet paginator" do
-      params = helper.add_facet_params_and_redirect("facet_field_2", "facet_value")
+      params = search_state.add_facet_params_and_redirect("facet_field_2", "facet_value")
 
       bad_keys = Blacklight::Solr::FacetPaginator.request_keys.values + [:id]
       bad_keys.each do |paginator_key|
@@ -206,13 +240,13 @@ describe Blacklight::SearchState do
     end
 
     it 'removes :page request key' do
-      params = helper.add_facet_params_and_redirect("facet_field_2", "facet_value")
+      params = search_state.add_facet_params_and_redirect("facet_field_2", "facet_value")
       expect(params).to_not have_key(:page)
     end
 
     it "otherwise does the same thing as add_facet_params" do
-      added_facet_params = helper.add_facet_params("facet_field_2", "facet_value")
-      added_facet_params_from_facet_action = helper.add_facet_params_and_redirect("facet_field_2", "facet_value")
+      added_facet_params = search_state.add_facet_params("facet_field_2", "facet_value")
+      added_facet_params_from_facet_action = search_state.add_facet_params_and_redirect("facet_field_2", "facet_value")
 
       expect(added_facet_params_from_facet_action).to eq added_facet_params.except(Blacklight::Solr::FacetPaginator.request_keys[:page], Blacklight::Solr::FacetPaginator.request_keys[:sort])
     end
@@ -225,7 +259,7 @@ describe Blacklight::SearchState do
       let(:facet_params) { { 'some_field' => ['some_value', 'another_value'] } }
 
       it "removes the facet / value tuple from the query parameters" do
-        params = helper.remove_facet_params('some_field', 'some_value')
+        params = search_state.remove_facet_params('some_field', 'some_value')
         expect(params[:f]['some_field']).not_to include 'some_value'
         expect(params[:f]['some_field']).to include 'another_value'
       end
@@ -233,13 +267,13 @@ describe Blacklight::SearchState do
 
     context "when the facet has configuration" do
       before do
-        allow(helper).to receive(:facet_configuration_for_field).with('some_field').and_return(
+        allow(search_state).to receive(:facet_configuration_for_field).with('some_field').and_return(
           double(single: true, field: "a_solr_field", key: "some_key"))
       end
       let(:facet_params) { { 'some_key' => ['some_value', 'another_value'] } }
 
       it "uses the facet's key configuration" do
-        params = helper.remove_facet_params('some_field', 'some_value')
+        params = search_state.remove_facet_params('some_field', 'some_value')
         expect(params[:f]['some_key']).not_to eq 'some_value'
         expect(params[:f]['some_key']).to include 'another_value'
       end
@@ -249,7 +283,7 @@ describe Blacklight::SearchState do
       facet_params['another_field'] = ['some_value']
       facet_params['some_field'] = ['some_value']
 
-      params = helper.remove_facet_params('some_field', 'some_value')
+      params = search_state.remove_facet_params('some_field', 'some_value')
 
       expect(params[:f]).not_to have_key 'some_field'
     end
@@ -257,7 +291,7 @@ describe Blacklight::SearchState do
     it "removes the 'f' parameter entirely when no facets remain" do
       facet_params['some_field'] = ['some_value']
 
-      params = helper.remove_facet_params('some_field', 'some_value')
+      params = search_state.remove_facet_params('some_field', 'some_value')
 
       expect(params).not_to have_key :f
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: blacklight
 version: !ruby/object:Gem::Version
-  version: 6.4.0
+  version: 6.4.1
 platform: ruby
 authors:
 - Jonathan Rochkind
@@ -17,7 +17,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-07-28 00:00:00.000000000 Z
+date: 2016-08-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -688,7 +688,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.4
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Blacklight provides a discovery interface for any Solr (http://lucene.apache.org/solr)