blacklight 5.10.2 → 5.10.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1d53a3586635ad7513ad3cfbd39d178c248e7836
-  data.tar.gz: b3f18bb4c468758d1ed21e3a5faad5261513a824
+  metadata.gz: 0d8ff5b59fd2f2f4492d6a334429e34d65eecb73
+  data.tar.gz: 1c5518ceb4bbba537374113d4ba12830276dcb50
 SHA512:
-  metadata.gz: 9789e89bc7c855b96bf969f4b2691fa9c830cfcb5ca126004aeed2200a9d7fb659fa351da420b95454a5770a568de66f2ba5993765593dae98e93072e1e6cbeb
-  data.tar.gz: 6abdbabd10009fba2f694e7d7faca33b668dd3b7fe386cc32ba2608e04bd5367cf313485be7a1db2bf32a9111357c8a2bdc0feb60a8a8914997f95f7b1fc531e
+  metadata.gz: 56d549c7e95d7a3bc1aef93b961e8403a9697e07562c5f76e5f16da795d9545de919fec3455b31ee1453a323a16db89486f166774ce7568b84d703b5cce3ad03
+  data.tar.gz: 51b5768632c4336d9b1242d3b3872cd97de4e8e6ecf42b7d01d3e43d052d06d515e7dba26a1d60aba9d7c61d3dfd26851298b86d9254107d7b445c788503dffe

data/.travis.yml

@@ -15,6 +15,8 @@ matrix:
       env: "RAILS_VERSION=4.1.9"
     - rvm: 2.1.5
       env: "RAILS_VERSION=4.2.0"
+    - rvm: 2.1.5
+      env: "RAILS_VERSION=4.2.1"
     - rvm: jruby
       env: "RAILS_VERSION=4.2.0 JRUBY_OPTS=\"-J-Xms512m -J-Xmx1024m\""
     - rvm: 1.9.3

data/VERSION

@@ -1 +1 @@
-5.10.2
+5.10.3

data/app/helpers/blacklight/configuration_helper_behavior.rb

@@ -87,13 +87,11 @@ module Blacklight::ConfigurationHelperBehavior
   # Look up the label for the facet field
   def facet_field_label field
     field_config = blacklight_config.facet_fields[field]
+    defaults = [:"blacklight.search.fields.facet.#{field}", :"blacklight.search.fields.#{field}"]
+    defaults << field_config.label if field_config
+    defaults << field.to_s.humanize
 
-    field_label(
-      :"blacklight.search.fields.facet.#{field}",
-      :"blacklight.search.fields.#{field}",
-      (field_config.label if field_config),
-      field.to_s.humanize
-    )
+    field_label *defaults
   end
 
   ##
@@ -109,7 +107,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/models/bookmark.rb

@@ -5,7 +5,10 @@ class Bookmark < ActiveRecord::Base
   belongs_to :document, polymorphic: true
 
   validates_presence_of :user_id, :scope=>:document_id
-  attr_accessible :id, :document_id, :document_type, :title if Rails::VERSION::MAJOR < 4
+
+  if Blacklight::Utils.needs_attr_accessible?
+    attr_accessible :id, :document_id, :document_type, :title
+  end
 
   def document
     document_type.new document_type.unique_key => document_id

data/app/models/search.rb

@@ -5,10 +5,13 @@ class Search < ActiveRecord::Base
 
   serialize :query_params
 
-  if Rails::VERSION::MAJOR < 4
+  if Blacklight::Utils.needs_attr_accessible?
     attr_accessible :query_params 
-  
-    scope :none, where(:id => nil).where("id IS NOT ?", nil)
+  end
+
+  unless respond_to?(:none)
+    # polyfill
+    scope :none, where(id: nil).where("id IS NOT ?", nil)
   end
 
   # A Search instance is considered a saved search if it has a user_id.

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

@@ -1,9 +1,49 @@
-<h4><%= t('blacklight.welcome') %></h4>
-<p>&nbsp;</p>
-<p>To modify this text to your specifications, copy this file located in the blacklight plugin directory:<br/>
-&nbsp;&nbsp;&nbsp;<%= Blacklight.root %>/app/views/catalog/_home_text.html.erb</p>
-<p>to your top level rails app:<br/>
-&nbsp;&nbsp;&nbsp;<%= Rails.root %>/app/views/catalog/_home_text.html.erb</p>
-<p>Note that you will need to create the 'catalog' directory under views.</p>
-<% #Creates a bit space in the page -%>
+<div class="page-header row">
+  <h1 class="col-md-8"><%= t('blacklight.welcome') %></h1>
+  
+  <ul class="nav nav-pills col-md-4">
+    <li><a href="https://github.com/projectblacklight/blacklight/">Github</a></li>
+    <li><a href="https://github.com/projectblacklight/blacklight/wiki">Wiki</a></li>
+    <li><a href="https://github.com/projectblacklight/blacklight/releases/tag/v<%= Blacklight::VERSION %>">Release Notes</a></li>
+  </ul>
 
+
+</div>
+
+<div id="getting-started">
+  <h2>Here&rsquo;s how to get started:</h2>
+  
+  <ol>
+    <li>To modify this text, you need to <a href="http://guides.rubyonrails.org/engines.html#improving-engine-functionality">override the Blacklight-provided view</a>.
+You can copy this file, located in the blacklight gem: <br />
+<%= Blacklight.root %>/app/views/catalog/_home_text.html.erb <br />
+to your own application: <br />
+ <%= Rails.root %>/app/views/catalog/_home_text.html.erb 
+    </li>
+    <li><a href="https://github.com/projectblacklight/blacklight/wiki/Indexing-your-data-into-solr">Index your own data</a> into Solr</li>
+    <li><a href="https://github.com/projectblacklight/blacklight/wiki#blacklight-configuration">Configure Blacklight</a> to match your data and user-experience needs</li>
+    <li><a href="https://github.com/projectblacklight/blacklight/wiki#support">Get in touch</a> with your comments, questions, and ideas</li>
+  </ol>
+</div>
+
+<%# This is the same panel shown in the Rails welcome template %>
+<div id="about">
+  <h3><a href="/rails/info/properties">About your application&rsquo;s environment</a></h3>
+  <div id="about-content" class="well" style="display: none"></div>
+</div>
+
+<script>
+  $(function() {
+    $('#about a').on('click', function(e) {
+      e.preventDefault();
+
+      if ($('#about-content').html() == "") {
+        $('#about-content').load($(this).attr("href"), function() {
+          $('#about-content').show();
+        });
+      } else {
+        $('#about-content').toggle();
+      }
+    });
+  });
+</script>

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/config/jetty.yml

@@ -1,10 +1,13 @@
 development:
   startup_wait: 15
   jetty_port: 8983 
+  java_version: ">= 1.7"
 test:
   startup_wait: 60
   jetty_port: <%= ENV['TEST_JETTY_PORT'] || 8888 %> 
   <%= ENV['TEST_JETTY_PATH'] ? "jetty_home: " + ENV['TEST_JETTY_PATH'] : '' %>
+  java_version: ">= 1.7"
 production:
   startup_wait: 15
   jetty_port: 8983
+  java_version: ">= 1.7"

data/lib/blacklight.rb

@@ -1,6 +1,8 @@
 # -*- encoding : utf-8 -*-
 require 'kaminari'
 require 'deprecation'
+require 'blacklight/utils'
+
 module Blacklight
   autoload :Configurable, 'blacklight/configurable'
   autoload :Configuration, 'blacklight/configuration'
@@ -28,8 +30,7 @@ module Blacklight
   autoload :DocumentPresenter, 'blacklight/document_presenter'
 
   autoload :Routes, 'blacklight/routes'
-
-  autoload :OpenStructWithHashAccess, 'blacklight/utils'
+  
   autoload :SolrResponse, 'blacklight/solr_response'
   autoload :Facet, 'blacklight/facet'
 
@@ -144,7 +145,13 @@ module Blacklight
   end
 
   def self.logger
-    ::Rails.logger
+    @logger ||= begin
+      ::Rails.logger if defined? Rails and Rails.respond_to? :logger
+    end
+  end
+
+  def self.logger= logger
+    @logger = logger
   end
 
   #############  

data/lib/blacklight/abstract_repository.rb

@@ -23,7 +23,7 @@ module Blacklight
       end
 
       def logger
-        @logger ||= Rails.logger if defined? Rails
+        @logger ||= Blacklight.logger
       end
   end
 end

data/lib/blacklight/catalog/search_context.rb

@@ -102,7 +102,7 @@ module Blacklight::Catalog::SearchContext
       @next_document = documents.last
     end
   rescue Blacklight::Exceptions::InvalidRequest => e
-    Rails.logger.warn "Unable to setup next and previous documents: #{e}"
+    logger.warn "Unable to setup next and previous documents: #{e}"
   end
 
 end

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,23 @@ 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)
+    self[attr]
   end
 
   # Helper method to check if value/multi-values exist for a given key.
@@ -81,7 +93,7 @@ module Blacklight::Document
   end
 
   def key? k
-    @_source.key? k
+    _source.key? k
   end
 
   # helper
@@ -112,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
@@ -135,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
@@ -153,4 +171,4 @@ module Blacklight::Document
       self
     end  
   end
-end
+end

data/lib/blacklight/solr_repository.rb

@@ -45,8 +45,8 @@ module Blacklight
 
         solr_response = blacklight_config.response_model.new(res, solr_params, document_model: blacklight_config.document_model)
 
-        Rails.logger.debug("Solr query: #{solr_params.inspect}")
-        Rails.logger.debug("Solr response: #{solr_response.inspect}") if defined?(::BLACKLIGHT_VERBOSE_LOGGING) and ::BLACKLIGHT_VERBOSE_LOGGING
+        Blacklight.logger.debug("Solr query: #{solr_params.inspect}")
+        Blacklight.logger.debug("Solr response: #{solr_response.inspect}") if defined?(::BLACKLIGHT_VERBOSE_LOGGING) and ::BLACKLIGHT_VERBOSE_LOGGING
         solr_response
       end
     rescue Errno::ECONNREFUSED => e

data/lib/blacklight/utils.rb

@@ -1,5 +1,28 @@
 require 'ostruct'
 module Blacklight
+
+  module Utils
+    def self.needs_attr_accessible?
+      if rails_3?
+        !strong_parameters_enabled?
+      else
+        protected_attributes_enabled?
+      end
+    end
+
+    def self.rails_3?
+      Rails::VERSION::MAJOR == 3
+    end
+
+    def self.strong_parameters_enabled?
+      defined?(ActionController::StrongParameters)
+    end
+
+    def self.protected_attributes_enabled?
+      defined?(ActiveModel::MassAssignmentSecurity)
+    end
+  end
+
   ##
   # An OpenStruct that responds to common Hash methods
   class OpenStructWithHashAccess < OpenStruct

data/lib/generators/blacklight/templates/config/jetty.yml

@@ -1,10 +1,13 @@
 development:
   startup_wait: 15
   jetty_port: 8983
+  java_version: ">= 1.7"
 test:
   startup_wait: 60
   jetty_port: <%= ENV['TEST_JETTY_PORT'] || 8888 %>
   <%= ENV['TEST_JETTY_PATH'] ? "jetty_home: " + ENV['TEST_JETTY_PATH'] : '' %>
+  java_version: ">= 1.7"
 production:
   startup_wait: 15
   jetty_port: 8983
+  java_version: ">= 1.7"

data/lib/generators/blacklight/user_generator.rb

@@ -47,7 +47,9 @@ This generator makes the following changes to your application:
       file_path = "app/models/#{model_name.underscore}.rb"
       if File.exists?(file_path)
         inject_into_class file_path, model_name.classify do
-          "\n  attr_accessible :email, :password, :password_confirmation if Rails::VERSION::MAJOR < 4\n" +
+          "\n  if Blacklight::Utils.needs_attr_accessible?\n" +
+          "\n    attr_accessible :email, :password, :password_confirmation" +
+          "\n  end\n" +
             "# Connects this user object to Blacklights Bookmarks. " +
             "\n  include Blacklight::User\n"
         end

data/spec/helpers/configuration_helper_spec.rb

@@ -150,6 +150,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

@@ -449,4 +449,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.10.2
+  version: 5.10.3
 platform: ruby
 authors:
 - Jonathan Rochkind
@@ -17,7 +17,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-09 00:00:00.000000000 Z
+date: 2015-03-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails