blacklight 5.11.2 → 5.11.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fbed788b9563305a5fdc48a4d8c24b8a96387e6b
-  data.tar.gz: 0bd82d604396b69091d6f50fba853f2af67f06aa
+  metadata.gz: d66f63cc8f730572a615a21e6e02c6f2bec32a04
+  data.tar.gz: 0e0a72eaec2d8ac932f37c96cff9c3482e1c884d
 SHA512:
-  metadata.gz: ef476787e4c81ac178e2ed8f2c86bcac92709d023e1a241d0581b30e39196b0d687df4979aa749077c61c9a65d97f0b2fb391e9feb8dee087a9a6fd4ef9af6e2
-  data.tar.gz: c5e5be39ab6a4e978d550ae344a6aa543d633f65f1ca74f04310c91984b321db9f4e7a9a433ce7e8bb7325919b80106d2d6c99474212d00411ecd2670492f4ea
+  metadata.gz: 2cfd5cd4a3c90a7f88aace8b30f706adc7a932ae8f1b49f30464051fddaf35cc48de2259c307d6501d4e555c934c4196ed5a841cac232d97cca1322ea1c146fb
+  data.tar.gz: 87e32807796642701f8891a2fb8c2ae6027bc8fc1b24d9e2dc1e3fe66387c6c4710af5ae306e0ef2bf3258b3f055ceb8d915729c4b9b1d35c53c35979cb103ca

data/VERSION

@@ -1 +1 @@
-5.11.2
+5.11.3

data/app/helpers/blacklight/configuration_helper_behavior.rb

@@ -118,7 +118,7 @@ module Blacklight::ConfigurationHelperBehavior
   #   @param [Symbol] any number of additional keys
   #   @param [Symbol] ...
   def field_label *i18n_keys
-    first, *rest = i18n_keys
+    first, *rest = i18n_keys.compact
 
     t(first, default: rest)
   end

data/app/helpers/blacklight/url_helper_behavior.rb

@@ -82,10 +82,15 @@ module Blacklight::UrlHelperBehavior
       return {}
     end
   
-    { :data => {:'context-href' => track_solr_document_path(document, per_page: params.fetch(:per_page, search_session['per_page']), counter: counter, search_id: current_search_session.try(:id))}}
+    { :data => {:'context-href' => session_tracking_path(document, per_page: params.fetch(:per_page, search_session['per_page']), counter: counter, search_id: current_search_session.try(:id))}}
   end
   protected :session_tracking_params
   
+  ##
+  # Get the URL for tracking search sessions across pages using polymorphic routing
+  def session_tracking_path document, params = {}
+    polymorphic_path([:track, document], params)
+  end
 
   #
   # link based helpers ->

data/app/views/catalog/_sms_form.html.erb

@@ -19,7 +19,7 @@
 
     </div>
     <% @documents.each do |doc| %>
-       <%=hidden_field_tag "id[]", doc.get(:id)%>
+       <%=hidden_field_tag "id[]", doc.id %>
     <% end %>
 </div>
 <div class="modal-footer">

data/lib/blacklight/configuration/facet_field.rb

@@ -5,7 +5,7 @@ module Blacklight
 
       self.collapse = true if self.collapse.nil?
       self.show = true if self.show.nil?
-      self.if ||= self.show
+      self.if = self.show if self.if.nil?
 
       super
       

data/lib/blacklight/configuration/search_field.rb

@@ -1,7 +1,7 @@
 module Blacklight
   class Configuration::SearchField < Blacklight::Configuration::Field
     def normalize! blacklight_config = nil
-      self.if ||= self.include_in_simple_select
+      self.if = self.include_in_simple_select if self.if.nil?
 
       super
       self.qt ||= blacklight_config.default_solr_params[:qt] if blacklight_config && blacklight_config.default_solr_params

data/lib/blacklight/document.rb

@@ -31,7 +31,7 @@ module Blacklight::Document
     include Blacklight::Document::Extensions
   end    
 
-  attr_reader :response
+  attr_reader :response, :_source
   alias_method :solr_response, :response
 
   def initialize(source_doc={}, response=nil)
@@ -52,11 +52,19 @@ module Blacklight::Document
   # If a method is missing, it gets sent to @_source
   # with all of the original params and block
   def method_missing(m, *args, &b)
-    @_source.send(m, *args, &b)
+    if _source and _source.respond_to? m
+      _source.send(m, *args, &b)
+    else
+      super
+    end
+  end
+
+  def respond_to_missing? *args
+    (_source && _source.respond_to?(*args)) || super
   end
 
   def [] *args
-    @_source.send :[], *args
+    _source.send :[], *args
   end
 
   def _read_attribute(attr)
@@ -85,7 +93,7 @@ module Blacklight::Document
   end
 
   def key? k
-    @_source.key? k
+    _source.key? k
   end
 
   # helper
@@ -116,7 +124,7 @@ module Blacklight::Document
   end
 
   def as_json(options = nil)
-    @_source.as_json(options)
+    _source.as_json(options)
   end
 
   def to_partial_path
@@ -139,6 +147,12 @@ module Blacklight::Document
     nil
   end
 
+  ##
+  # Implementations that support More-Like-This should override this method
+  # to return an array of documents that are like this one.
+  def more_like_this
+    []
+  end
 
   # Certain class-level methods needed for the document-specific
   # extendability architecture

data/spec/helpers/configuration_helper_spec.rb

@@ -159,6 +159,13 @@ describe BlacklightConfigurationHelper do
 
       label = helper.field_label :key_a, :key_b, "default text"
     end
+
+    it "should compact nil keys (fixes rails/rails#19419)" do
+      allow(helper).to receive(:t).with(:key_a, default: [:key_b])
+
+      label = helper.field_label :key_a, nil, :key_b
+
+    end
   end
   
   describe "#default_per_page" do

data/spec/helpers/url_helper_spec.rb

@@ -499,4 +499,15 @@ describe BlacklightUrlHelper do
       expect(url).to eq helper.bookmarks_url(format: :html, encrypted_user_id: 'xyz')
     end
   end
+
+  describe "#session_tracking_path" do
+    let(:document) { SolrDocument.new(id: 1) }
+    it "should determine the correct route for the document class" do
+      expect(helper.session_tracking_path(document)).to eq helper.track_solr_document_path(document)
+    end
+
+    it "should pass through tracking parameters" do
+      expect(helper.session_tracking_path(document, x: 1)).to eq helper.track_solr_document_path(document, x: 1)
+    end
+  end
 end

data/spec/lib/blacklight/configuration_spec.rb

@@ -213,11 +213,25 @@ describe "Blacklight::Configuration" do
         "another_field_facet" => {},
         "a_facet_field" => {},
         })
-      expect { |b| @config.add_index_field "*_facet", &b }.to yield_control.twice
-
-      expect(@config.index_fields.keys).to eq ["some_field_facet", "another_field_facet"]
+      expect { |b| @config.add_facet_field "*_facet", &b }.to yield_control.twice
+      expect(@config.facet_fields.keys).to eq ["some_field_facet", "another_field_facet"]
     end
 
+    describe "if/unless conditions with legacy show parameter" do
+      it "should be hidden if the if condition is false" do
+        expect(@config.add_facet_field("hidden", if: false).if).to eq false
+        expect(@config.add_facet_field("hidden_with_legacy", if: false, show: true).if).to eq false
+      end
+
+      it "should be true if the if condition is true" do
+        expect(@config.add_facet_field("hidden", if: true).if).to eq true
+        expect(@config.add_facet_field("hidden_with_legacy", if: true, show: false).if).to eq true
+      end
+
+      it "should be true if the if condition is missing" do
+        expect(@config.add_facet_field("hidden", show: true).if).to eq true
+      end
+    end
   end
   
   describe "add_index_field" do
@@ -261,7 +275,6 @@ describe "Blacklight::Configuration" do
 
       expect(@config.index_fields.keys).to eq ["some_field_display", "another_field_display"]
     end
-
   end
   
   describe "add_show_field" do
@@ -370,8 +383,22 @@ describe "Blacklight::Configuration" do
       
       expect(@config.search_fields["author_name"].label).to eq "Author Name"
     end
-                
-        
+
+    describe "if/unless conditions with legacy include_in_simple_search" do
+      it "should be hidden if the if condition is false" do
+        expect(@config.add_search_field("hidden", if: false).if).to eq false
+        expect(@config.add_search_field("hidden_with_legacy", if: false, include_in_simple_search: true).if).to eq false
+      end
+
+      it "should be true if the if condition is true" do
+        expect(@config.add_search_field("hidden", if: true).if).to eq true
+        expect(@config.add_search_field("hidden_with_legacy", if: true, include_in_simple_search: false).if).to eq true
+      end
+
+      it "should be true if the if condition is missing" do
+        expect(@config.add_search_field("hidden", include_in_simple_search: true).if).to eq true
+      end
+    end
   end
   
   describe "add_sort_field" do

metadata

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