blacklight 4.6.2 → 4.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2973df853ccd78aa817fb76e2b5f2aaec1273b55
-  data.tar.gz: 4e6e6d1ad0d678deb64588ce0a3b81628c8a10dc
+  metadata.gz: 5982a2428034dae378c31791d1d9d4a2cfcdd02d
+  data.tar.gz: dbe8685099072975e318962fd879410db0bec8f4
 SHA512:
-  metadata.gz: 0978b221b5f4308e8ee8011826ab39cad5fd69fc122399ae0b0d45ab1373e9db21ed099e6c4c69b159caa988633c02babb6988cd3eb2b6a52e684bf5df5eafe5
-  data.tar.gz: d0f6f2e7324af35f7c9d967de599ab24f0a9153942e2d15209d9b1aee19bd7c02b7b4cfe82690bbda17ec0e708aa6c9ca8deb5f41df9591bbb6d907fb70aee5e
+  metadata.gz: 49c5477a3ee2d0e5418a9cd4b1cad1683864398373be03b267a7896c4e8c6f850218163a98e6f0e4faf039bacdbef5ebf5782b23a06e29257780b83af685ce64
+  data.tar.gz: cee3f4ff93d8301b3e30e4fd8fc784486eaae15b47c209258b50c3d1b86adf53ccd7bd64ebae4233261f5c793312925164c43f8c9262c24fcecf81b1064b6592

data/LICENSE

@@ -1,5 +1,6 @@
 ##########################################################################
-# Copyright 2008 Rector and Visitors of the University of Virginia
+# Copyright 2008-2014 Rector and Visitors of the University of Virginia, The Board of Trustees of the Leland Stanford Junior University, Johns Hopkins Universities, and Data Curation Experts
+# Additional copyright may be held by others, as reflected in the commit log
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/VERSION

@@ -1 +1 @@
-4.6.2
+4.6.3

data/app/controllers/feedback_controller.rb

@@ -1,6 +1,9 @@
 # -*- encoding : utf-8 -*-
 class FeedbackController < ApplicationController
   
+  extend Deprecation
+  self.deprecation_horizon = 'Blacklight 5.x'
+
   # http://expressica.com/simple_captcha/
   # include SimpleCaptcha::ControllerHelpers
   
@@ -14,6 +17,7 @@ class FeedbackController < ApplicationController
       end
     end
   end
+  deprecation_deprecate :show
   
   protected
   

data/app/helpers/blacklight/blacklight_helper_behavior.rb

@@ -368,14 +368,18 @@ module Blacklight::BlacklightHelperBehavior
     ', '
   end
 
-  def document_index_view_type
-    if blacklight_config.document_index_view_types.include? params[:view]
-      params[:view]
+  def document_index_view_type query_params=params
+    if blacklight_config.document_index_view_types.include? query_params[:view]
+      query_params[:view]
     else
-      blacklight_config.document_index_view_types.first
+      default_document_index_view_type
     end
   end
 
+  def default_document_index_view_type
+    blacklight_config.document_index_view_types.first
+  end
+
   def render_document_index documents = nil, locals = {}
     documents ||= @document_list
     render_document_index_with_view(document_index_view_type, documents)
@@ -464,6 +468,17 @@ module Blacklight::BlacklightHelperBehavior
     link_to(query, link_url)
   end
 
+  ##
+  # Get the path to the search action with any parameters (e.g. view type)
+  # that should be persisted across search sessions.
+  def start_over_path query_params = params
+    h = { }
+    current_index_view_type = document_index_view_type(query_params)
+    h[:view] = current_index_view_type unless current_index_view_type == default_document_index_view_type
+
+    search_action_url(h)
+  end
+
   def render_document_index_label doc, opts
     label = nil
     label ||= doc.get(opts[:label], :sep => nil) if opts[:label].instance_of? Symbol

data/app/helpers/blacklight/catalog_helper_behavior.rb

@@ -19,6 +19,7 @@ module Blacklight::CatalogHelperBehavior
   # kaminari paginate, passed on through.
   # will output HTML pagination controls.
   def paginate_rsolr_response(response, options = {}, &block)
+    Deprecation.warn Blacklight::CatalogHelperBehavior, "#paginate_rsolr_response is deprecated; the original response object is Kaminari-compatible"
     paginate response, options, &block
   end
 

data/app/helpers/blacklight/facets_helper_behavior.rb

@@ -58,7 +58,23 @@ module Blacklight::FacetsHelperBehavior
   # @param [Blacklight::SolrResponse::Facets::FacetField] display_facet 
   def should_render_facet? display_facet
     # display when show is nil or true
-    display = facet_configuration_for_field(display_facet.name).show != false
+    facet_config = facet_configuration_for_field(display_facet.name)
+
+    display = case facet_config.show
+    when Symbol
+      arity = method(facet_config.show).arity
+
+      if arity == 0
+        send(facet_config.show)
+      else 
+        send(facet_config.show, display_facet)
+      end
+    when Proc
+      facet_config.show.call self, facet_config, display_facet
+    else
+      facet_config.show
+    end
+
     return display && display_facet.items.present?
   end
 

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

@@ -2,7 +2,7 @@
       <div id="appliedParams" class="constraints-container">
         <span class="constraints-label"><%= t('blacklight.search.filters.title') %></span>
   
-        <%=link_to t('blacklight.search.start_over'), url_for(:action=>'index'), :class => "catalog_startOverLink", :id=>"startOverLink" %>
+          <%=link_to t('blacklight.search.start_over'), start_over_path, :class => "catalog_startOverLink btn btn-sm btn-text", :id=>"startOverLink" %>
         <%= render_constraints(params) %>  
 
       </div>

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

@@ -10,7 +10,7 @@
 <div class="pull-right">
       <%= link_back_to_catalog %>
       |
-      <%=link_to "#{t('blacklight.search.start_over')}", catalog_index_path, :id=>"startOverLink" %>
+      <%=link_to "#{t('blacklight.search.start_over')}", start_over_path(current_search_session.try(:query_params) || {}), :id=>"startOverLink" %>
   </div>
 	</div>
 <% end %>

data/lib/blacklight/configuration.rb

@@ -25,7 +25,8 @@ module Blacklight
           :search_history_window => Blacklight::Catalog::SearchHistoryWindow,
           :document_index_view_types => ['list'],
           :add_facet_fields_to_solr_request => false,
-          :add_field_configuration_to_solr_request => false
+          :add_field_configuration_to_solr_request => false,
+          :http_method => :get
           }
         end
       end

data/lib/blacklight/configuration/facet_field.rb

@@ -7,6 +7,8 @@ module Blacklight
         self.tag = "#{self.field}_single"
         self.ex = "#{self.field}_single"
       end
+
+      self.show = true if self.show.nil?
       super
     end
   end

data/lib/blacklight/routes.rb

@@ -2,6 +2,26 @@
 module Blacklight
   class Routes
 
+    # adds as class and instance level accessors, default_route_sets
+    # returns an array of symbols for method names that define routes. 
+    # Order is important:.  (e.g. /catalog/email precedes /catalog/:id)
+    #
+    # Add-ons that want to add routes into default routing might
+    # monkey-patch Blacklight::Routes, say:
+    #
+    #     module MyWidget::Routes
+    #       extend ActiveSupport::Concern
+    #       included do |klass|
+    #         klass.default_route_sets += [:widget_routing]
+    #       end
+    #       def widget_routing(primary_resource)
+    #         get "#{primary_resource}/widget", "#{primary_resource}#widget"
+    #       end
+    #     end
+    #     Blacklight::Routes.send(:include, MyWidget::Routes)
+    class_attribute :default_route_sets
+    self.default_route_sets = [:bookmarks, :search_history, :saved_searches, :catalog, :solr_document, :feedback]
+
     def initialize(router, options)
       @router = router
       @options = options
@@ -23,10 +43,6 @@ module Blacklight
       (@options[:only] || default_route_sets) - (@options[:except] || [])
     end
 
-    def default_route_sets
-      [:bookmarks, :search_history, :saved_searches, :catalog, :solr_document, :feedback]
-    end
-
     module RouteSets
       def bookmarks
         add_routes do |options|

data/lib/blacklight/solr_helper.rb

@@ -415,7 +415,8 @@ module Blacklight::SolrHelper
     path = blacklight_config.solr_path
 
     # delete these parameters, otherwise rsolr will pass them through.
-    res = blacklight_solr.send_and_receive(path, :params=>solr_params)
+    key = blacklight_config.http_method == :post ? :data : :params
+    res = blacklight_solr.send_and_receive(path, {key=>solr_params, method:blacklight_config.http_method})
     
     solr_response = Blacklight::SolrResponse.new(force_to_utf8(res), solr_params)
 

data/spec/helpers/blacklight_helper_spec.rb

@@ -334,6 +334,26 @@ describe BlacklightHelper do
        params[:view] = 'not_in_list'
        document_index_view_type.should == 'list'
      end
+
+     it "should pluck values from supplied params" do
+       blacklight_config.stub(:document_index_view_types) { ['list', 'asdf'] }
+       params[:view] = 'asdf'
+       expect(document_index_view_type(:view => 'list')).to eq 'list'
+     end
+   end
+
+   describe "start_over_path" do
+    it 'should be the catalog path with the current view type' do
+      blacklight_config.stub(:document_index_view_types) { ['list', 'abc'] }
+      helper.stub(:blacklight_config => blacklight_config)
+      expect(helper.start_over_path(:view => 'abc')).to eq catalog_index_url(:view => 'abc')
+    end
+
+    it 'should not include the current view type if it is the default' do
+      blacklight_config.stub(:document_index_view_types) { ['list', 'abc'] }
+      helper.stub(:blacklight_config => blacklight_config)
+      expect(helper.start_over_path(:view => 'list')).to eq catalog_index_url
+    end
    end
 
    describe "render_document_index" do

data/spec/helpers/facets_helper_spec.rb

@@ -34,11 +34,16 @@ describe FacetsHelper do
     before do
       @config = Blacklight::Configuration.new do |config|
         config.add_facet_field 'basic_field'
-        config.add_facet_field 'no_show', :show=>false
+        config.add_facet_field 'no_show', :show => false
+        config.add_facet_field 'helper_show', :show => :my_helper
+        config.add_facet_field 'helper_with_an_arg_show', :show => :my_helper_with_an_arg
+        config.add_facet_field 'lambda_show', :show => lambda { |context, config, field| true }
+        config.add_facet_field 'lambda_no_show', :show => lambda { |context, config, field| false }
       end
 
       helper.stub(:blacklight_config => @config)
     end
+
     it "should render facets with items" do
       a = double(:items => [1,2], :name=>'basic_field')
       helper.should_render_facet?(a).should == true
@@ -52,6 +57,27 @@ describe FacetsHelper do
       a = double(:items => [1,2], :name=>'no_show')
       helper.should_render_facet?(a).should ==  false
     end
+
+    it "should call a helper to determine if it should render a field" do
+      helper.stub(:my_helper => true)
+      a = double(:items => [1,2], :name=>'helper_show')
+      expect(helper.should_render_facet?(a)).to be_true
+    end
+
+    it "should call a helper to determine if it should render a field" do
+      a = double(:items => [1,2], :name=>'helper_with_an_arg_show')
+      helper.should_receive(:my_helper_with_an_arg).with(a).and_return(true)
+      expect(helper.should_render_facet?(a)).to be_true
+    end
+
+
+    it "should evaluate a Proc to determine if it should render a field" do
+      a = double(:items => [1,2], :name=>'lambda_show')
+      expect(helper.should_render_facet?(a)).to be_true
+
+      a = double(:items => [1,2], :name=>'lambda_no_show')
+      expect(helper.should_render_facet?(a)).to be_false
+    end
   end
 
   describe "facet_by_field_name" do

data/spec/lib/solr_helper_spec.rb

@@ -1150,5 +1150,40 @@ describe 'Blacklight::SolrHelper' do
     end
   end
 
+  describe "http_method configuration" do
+    describe "using default" do
+      let (:blacklight_config) {Blacklight::Configuration.new}
+
+      it "defaults to get" do
+        expect(blacklight_config.http_method).to eq :get
+        Blacklight.solr.should_receive(:send_and_receive) do |path, params|
+          expect(path).to eq 'select'
+          expect(params[:method]).to eq :get
+          expect(params[:params]).to include(:q)
+        end.and_return({'response'=>{'docs'=>[]}})
+        get_search_results(:q => @all_docs_query)
+      end
+    end
+
+    describe "setting to post" do
+      let (:blacklight_config) {config = Blacklight::Configuration.new; config.http_method=:post; config}
+
+      it "keep value set to post" do
+        expect(blacklight_config.http_method).to eq :post
+        Blacklight.solr.should_receive(:send_and_receive) do |path, params|
+          expect(path).to eq 'select'
+          expect(params[:method]).to eq :post
+          expect(params[:data]).to include(:q)
+        end.and_return({'response'=>{'docs'=>[]}})
+        get_search_results(:q => @all_docs_query)
+      end
+
+      it "should send a post request to solr", :integration => true do
+        response, docs = get_search_results(:q => @all_docs_query)
+        expect(docs.length).to be >= 1
+     end
+    end
+  end
+
 end
 

data/spec/routing/routes_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+
+describe "Blacklight::Routes" do
+  describe "default_route_sets" do
+    around do |example|
+      @original = Blacklight::Routes.default_route_sets.dup.freeze
+
+      example.run
+
+      Blacklight::Routes.default_route_sets = @original
+    end
+
+    it "is settable" do
+      Blacklight::Routes.default_route_sets += [:foo]
+
+      # Order DOES matter. 
+      expect(Blacklight::Routes.default_route_sets).to eq(@original + [:foo])
+    end
+  end
+end

data/spec/views/catalog/_constraints.html.erb_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+describe "catalog/constraints" do
+  let :blacklight_config do
+    Blacklight::Configuration.new :document_index_view_types => ['list', 'xyz']
+  end
+
+  it "should render nothing if no constraints are set" do
+    view.stub(query_has_constraints?: false)
+    render partial: "catalog/constraints"
+    expect(rendered).to be_empty
+  end
+
+  it "should render a start over link" do
+    view.should_receive(:search_action_url).with({}).and_return('http://xyz')
+    view.stub(query_has_constraints?: true)
+    view.stub(:blacklight_config).and_return(blacklight_config)
+    render partial: "catalog/constraints"
+    expect(rendered).to have_selector("#startOverLink")
+    expect(rendered).to have_link("Start Over", :href => 'http://xyz')
+  end
+
+  it "should render a start over link with the current view type" do
+    view.should_receive(:search_action_url).with(view: 'xyz').and_return('http://xyz?view=xyz')
+    view.stub(query_has_constraints?: true)
+    params[:view] = 'xyz'
+    view.stub(:blacklight_config).and_return(blacklight_config)
+    render partial: "catalog/constraints"
+    expect(rendered).to have_selector("#startOverLink")
+    expect(rendered).to have_link("Start Over", :href => 'http://xyz?view=xyz')
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: blacklight
 version: !ruby/object:Gem::Version
-  version: 4.6.2
+  version: 4.6.3
 platform: ruby
 authors:
 - Jonathan Rochkind
@@ -17,7 +17,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-17 00:00:00.000000000 Z
+date: 2014-01-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -502,6 +502,7 @@ files:
 - spec/rcov.opts
 - spec/requests/alternate_controller_spec.rb
 - spec/routing/catalog_routing_spec.rb
+- spec/routing/routes_spec.rb
 - spec/spec.opts
 - spec/spec_helper.rb
 - spec/support/assert_difference.rb
@@ -511,6 +512,7 @@ files:
 - spec/test_app_templates/Gemfile.extra
 - spec/test_app_templates/lib/generators/test_app_generator.rb
 - spec/test_app_templates/lib/tasks/blacklight_test_app.rake
+- spec/views/catalog/_constraints.html.erb_spec.rb
 - spec/views/catalog/_constraints_element.html.erb_spec.rb
 - spec/views/catalog/_document.html.erb_spec.rb
 - spec/views/catalog/_document_list.html.erb_spec.rb
@@ -604,6 +606,7 @@ test_files:
 - spec/rcov.opts
 - spec/requests/alternate_controller_spec.rb
 - spec/routing/catalog_routing_spec.rb
+- spec/routing/routes_spec.rb
 - spec/spec.opts
 - spec/spec_helper.rb
 - spec/support/assert_difference.rb
@@ -613,6 +616,7 @@ test_files:
 - spec/test_app_templates/Gemfile.extra
 - spec/test_app_templates/lib/generators/test_app_generator.rb
 - spec/test_app_templates/lib/tasks/blacklight_test_app.rake
+- spec/views/catalog/_constraints.html.erb_spec.rb
 - spec/views/catalog/_constraints_element.html.erb_spec.rb
 - spec/views/catalog/_document.html.erb_spec.rb
 - spec/views/catalog/_document_list.html.erb_spec.rb