blacklight 5.11.0 → 5.11.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 306445f3c792d11626c44500a5b30d974c01f5df
-  data.tar.gz: da7aa72df29cc2dd70ec8779d72928e8916d340a
+  metadata.gz: f5d4dba8182fcbbaa71a0f1e263c992b5ead7a54
+  data.tar.gz: a2ad75e6e2097d1881685a1f3e35414c885a70c0
 SHA512:
-  metadata.gz: 01e02b3b3605264f5bb0517a6abeec089e4b2eb8d96e604572808e6f805e59dfd448cedd6c35a311d436e2753ef6cc780e0fdf4d3c64e86101a6bab38fae1173
-  data.tar.gz: b95b6dc21666c74b2ba5c6103fc555300a559340efcc5e0b9315cf5f019c8de0d098e33d72cde8bfce43a0330fbab87d226697f6530783ebbd98dc1aa79ef586
+  metadata.gz: 2ec8a493c2c4e80ce27b9c901f6fb9de931d058d6a81f6dbc57ee2b54d4f1a321a13626d9df1342e430903cdf655ee1a380f38506f3e31b0af5708b23a5d2d96
+  data.tar.gz: 02343ff43095be1ffe760572b3cb1737dd14a38ab08178f5c3b9997fd46695895c627321dce537cc027f1c1059805ad37b5b268595077866cbad1b771f7212bf

data/VERSION

@@ -1 +1 @@
-5.11.0
+5.11.1

data/app/helpers/blacklight/render_constraints_helper_behavior.rb

@@ -69,7 +69,7 @@ module Blacklight::RenderConstraintsHelperBehavior
 
     safe_join(values.map do |val|
       next if val.blank? # skip empty string
-      render_constraint_element( facet_field_label(facet), facet_display_value(facet, val),
+      render_constraint_element( facet_field_label(facet_config.key), facet_display_value(facet, val),
                   :remove => search_action_path(remove_facet_params(facet, val, localized_params)),
                   :classes => ["filter", "filter-" + facet.parameterize]
                 )

data/app/helpers/blacklight/url_helper_behavior.rb

@@ -215,17 +215,19 @@ module Blacklight::UrlHelperBehavior
 
     facet_config = facet_configuration_for_field(field)
 
+    url_field = facet_config.key
+
     value = facet_value_for_facet_item(item)
 
     p = reset_search_params(source_params)
     p[:f] = (p[:f] || {}).dup # the command above is not deep in rails3, !@#$!@#$
-    p[:f][field] = (p[:f][field] || []).dup
+    p[:f][url_field] = (p[:f][url_field] || []).dup
 
-    if facet_config.single and not p[:f][field].empty?
-      p[:f][field] = []
+    if facet_config.single and not p[:f][url_field].empty?
+      p[:f][url_field] = []
     end
     
-    p[:f][field].push(value)
+    p[:f][url_field].push(value)
 
     if item and item.respond_to?(:fq) and item.fq
       item.fq.each do |f,v|
@@ -262,6 +264,10 @@ module Blacklight::UrlHelperBehavior
       field = item.field
     end
 
+    facet_config = facet_configuration_for_field(field)
+
+    url_field = facet_config.key
+
     value = facet_value_for_facet_item(item)
 
     p = reset_search_params(source_params)
@@ -269,9 +275,9 @@ module Blacklight::UrlHelperBehavior
     # if the values aren't dup'd, then the values
     # from the session will get remove in the show view...
     p[:f] = (p[:f] || {}).dup
-    p[:f][field] = (p[:f][field] || []).dup
-    p[:f][field] = p[:f][field] - [value]
-    p[:f].delete(field) if p[:f][field].size == 0
+    p[:f][url_field] = (p[:f][url_field] || []).dup
+    p[:f][url_field] = p[:f][url_field] - [value]
+    p[:f].delete(url_field) if p[:f][url_field].size == 0
     p.delete(:f) if p[:f].empty?
     p
   end

data/spec/helpers/url_helper_spec.rb

@@ -343,6 +343,16 @@ describe BlacklightUrlHelper do
       expect(result_params[:f]["facet_field"]).to eq ["facet_value"]
     end
 
+    it "should use 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"))
+
+      result_params = helper.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'
+    end
+
+
     it "should add a facet param to existing facet constraints" do
       allow(helper).to receive(:params).and_return(@params_existing_facets)
       
@@ -374,7 +384,7 @@ describe BlacklightUrlHelper do
     end    
 
     it "should replace facets for facets configured as single" do
-      allow(helper).to receive(:facet_configuration_for_field).with('single_value_facet_field').and_return(double(:single => true))
+      allow(helper).to receive(:facet_configuration_for_field).with('single_value_facet_field').and_return(double(single: true, key: "single_value_facet_field"))
       params = { :f => { 'single_value_facet_field' => 'other_value'}}
       allow(helper).to receive(:params).and_return params
 
@@ -441,6 +451,46 @@ describe BlacklightUrlHelper do
       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
   end
+
+  describe "#remove_facet_params" do
+    let(:query_params) { { f: facet_params }}
+    let(:facet_params) { { } }
+    it "should remove the facet / value tuple from the query parameters" do
+      facet_params['some_field'] = ['some_value', 'another_value']
+
+      params = helper.remove_facet_params('some_field', 'some_value', query_params)
+
+      expect(params[:f]['some_field']).not_to include 'some_value'
+      expect(params[:f]['some_field']).to include 'another_value'
+    end
+
+    it "should use the facet's key configuration" do
+      allow(helper).to receive(:facet_configuration_for_field).with('some_field').and_return(double(single: true, field: "a_solr_field", key: "some_key"))
+      facet_params['some_key'] = ['some_value', 'another_value']
+
+      params = helper.remove_facet_params('some_field', 'some_value', query_params)
+
+      expect(params[:f]['some_key']).not_to eq 'some_value'
+      expect(params[:f]['some_key']).to include 'another_value'
+    end
+
+    it "should remove the facet entirely when the last facet value is removed" do
+      facet_params['another_field'] = ['some_value']
+      facet_params['some_field'] = ['some_value']
+
+      params = helper.remove_facet_params('some_field', 'some_value', query_params)
+
+      expect(params[:f]).not_to include 'some_field'
+    end
+
+    it "should remove the 'f' parameter entirely when no facets remain" do
+      facet_params['some_field'] = ['some_value']
+
+      params = helper.remove_facet_params('some_field', 'some_value', query_params)
+
+      expect(params).not_to include :f
+    end
+  end
   
   describe "#bookmarks_export_url" do
     it "should be the bookmark url with an encrypted user token" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: blacklight
 version: !ruby/object:Gem::Version
-  version: 5.11.0
+  version: 5.11.1
 platform: ruby
 authors:
 - Jonathan Rochkind
@@ -17,7 +17,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-17 00:00:00.000000000 Z
+date: 2015-03-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails