blacklight 7.33.0 → 8.0.0.beta2

data/app/components/blacklight/search_button_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Blacklight
-  class SearchButtonComponent < ::ViewComponent::Base
+  class SearchButtonComponent < Blacklight::Component
     def initialize(text:, id:)
       @text = text
       @id = id
@@ -9,8 +9,8 @@ module Blacklight
 
     def call
       tag.button(class: 'btn btn-primary search-btn', type: 'submit', id: @id) do
-        tag.span(@text, class: "submit-search-text") +
-          blacklight_icon(:search, aria_hidden: true)
+        tag.span(@text, class: "visually-hidden-sm me-sm-1 submit-search-text") +
+          render(Blacklight::Icons::SearchComponent.new)
       end
     end
   end

data/app/components/blacklight/search_context_component.rb

@@ -13,22 +13,56 @@ module Blacklight
       @search_context.present? && (@search_context[:prev] || @search_context[:next])
     end
 
+    ##
+    # Displays "showing X of Y items" message.
+    #
+    # @return [String]
     def item_page_entry_info
-      Deprecation.silence(Blacklight::CatalogHelperBehavior) do
-        helpers.item_page_entry_info
-      end
+      t('blacklight.search.entry_pagination_info.other', current: number_with_delimiter(count),
+                                                         total: number_with_delimiter(total),
+                                                         count: total).html_safe
     end
 
-    def link_to_previous_document(document = nil, *args, **kwargs)
-      Deprecation.silence(Blacklight::UrlHelperBehavior) do
-        helpers.link_to_previous_document(document || @search_context[:prev], *args, **kwargs)
+    def link_to_previous_document(previous_document = nil, classes: 'previous', **link_opts)
+      previous_document ||= @search_context[:prev]
+      link_opts = session_tracking_params(previous_document, count - 1, per_page: per_page, search_id: search_id).merge(class: classes, rel: 'prev').merge(link_opts)
+      link_to_unless previous_document.nil?, raw(t('views.pagination.previous')), url_for_document(previous_document), link_opts do
+        tag.span raw(t('views.pagination.previous')), class: 'previous'
       end
     end
 
-    def link_to_next_document(document = nil, *args, **kwargs)
-      Deprecation.silence(Blacklight::UrlHelperBehavior) do
-        helpers.link_to_next_document(document || @search_context[:next], *args, **kwargs)
+    def link_to_next_document(next_document = nil, classes: 'next', **link_opts)
+      next_document ||= @search_context[:next]
+      link_opts = session_tracking_params(next_document, count + 1, per_page: per_page, search_id: search_id).merge(class: classes, rel: 'next').merge(link_opts)
+      link_to_unless next_document.nil?, raw(t('views.pagination.next')), url_for_document(next_document), link_opts do
+        tag.span raw(t('views.pagination.next')), class: 'next'
       end
     end
+
+    private
+
+    def count
+      @search_session['counter'].to_i
+    end
+
+    def per_page
+      @search_session['per_page']
+    end
+
+    def total
+      @search_session['total'].to_i
+    end
+
+    def search_id
+      controller.current_search_session&.id
+    end
+
+    def session_tracking_params(...)
+      helpers.session_tracking_params(...)
+    end
+
+    def url_for_document(...)
+      helpers.search_state.url_for_document(...)
+    end
   end
 end

data/app/components/blacklight/search_header_component.html.erb

@@ -0,0 +1,2 @@
+<%= render 'did_you_mean' %>
+<%= render 'sort_and_per_page' %>

data/app/components/blacklight/search_header_component.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+module Blacklight
+  class SearchHeaderComponent < Blacklight::Component
+  end
+end

data/app/components/blacklight/search_navbar_component.html.erb

@@ -0,0 +1,5 @@
+<%= tag.div class: 'navbar-search navbar navbar-light bg-light', role: 'navigation', aria: { label: t('blacklight.search.header') } do %>
+  <div class="<%= container_classes %>">
+    <%= search_bar %>
+  </div>
+<% end %>

data/app/components/blacklight/search_navbar_component.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+module Blacklight
+  class SearchNavbarComponent < Blacklight::Component
+    def initialize(blacklight_config:)
+      @blacklight_config = blacklight_config
+    end
+
+    attr_reader :blacklight_config
+
+    delegate :container_classes, to: :helpers
+
+    def search_bar
+      render search_bar_component
+    end
+
+    def search_bar_component
+      search_bar_component_class.new(
+        url: helpers.search_action_url,
+        advanced_search_url: helpers.search_action_url(action: 'advanced_search'),
+        params: helpers.search_state.params_for_search.except(:qt),
+        autocomplete_path: suggest_index_catalog_path
+      )
+    end
+
+    def search_bar_component_class
+      view_config&.search_bar_component || Blacklight::SearchBarComponent
+    end
+
+    def view_config
+      blacklight_config&.view_config(helpers.document_index_view_type)
+    end
+  end
+end

data/app/components/blacklight/start_over_button_component.rb

@@ -12,9 +12,11 @@ module Blacklight
     # 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
-      Deprecation.silence(Blacklight::UrlHelperBehavior) do
-        helpers.start_over_path(query_params)
-      end
+      h = {}
+      current_index_view_type = helpers.document_index_view_type(query_params)
+      h[:view] = current_index_view_type unless current_index_view_type == helpers.default_document_index_view_type
+
+      helpers.search_action_path(h)
     end
   end
 end

data/app/components/blacklight/system/dropdown_component.rb

@@ -29,11 +29,11 @@ module Blacklight
       end
 
       def before_render
-        button(classes: 'btn btn-outline-secondary dropdown-toggle', label: button_label) unless button
+        with_button(classes: 'btn btn-outline-secondary dropdown-toggle', label: button_label) unless button
 
         return if options.any?
 
-        options(@choices.map do |option|
+        with_options(@choices.map do |option|
           text, value = option_text_and_value(option)
           { text: text, url: helpers.url_for(@search_state.params_for_search(@param => value)), selected: @selected == value }
         end)

data/app/components/blacklight/system/flash_message_component.html.erb

@@ -1,4 +1,6 @@
 <div class="alert <%= @classes %>">
   <%= message %>
-  <a class="btn-close close" data-dismiss="alert" data-bs-dismiss="alert" href="#">&times;</a>
+  <%= tag.button_tag button_contents, type: "button", class: "btn-close",
+                     data: { dismiss: "alert", bs_dismiss: "alert" },
+                     aria: { label: "Close" } %>
 </div>

data/app/components/blacklight/system/flash_message_component.rb

@@ -7,13 +7,22 @@ module Blacklight
 
       with_collection_parameter :message
 
-      def initialize(message: nil, type:)
+      def initialize(type:, message: nil)
         @message = message
         @classes = alert_class(type)
       end
 
       def before_render
-        message { @message } if @message
+        with_message { @message } if @message
+      end
+
+      # Bootstrap 4 requires the span, but Bootstrap 5 should not have it.
+      # See https://getbootstrap.com/docs/4.6/components/alerts/#dismissing
+      #     https://getbootstrap.com/docs/5.1/components/alerts/#dismissing
+      def button_contents
+        return if helpers.controller.blacklight_config.bootstrap_version == 5
+
+        tag.span '×'.html_safe, aria: { hidden: true }
       end
 
       def alert_class(type)

data/app/components/blacklight/system/modal_component.html.erb

@@ -6,7 +6,7 @@
       <h1 class="modal-title"><%= title %></h1>
     <% end) %>
 
-    <button type="button" class="blacklight-modal-close btn-close close" data-bs-dismiss="modal" data-dismiss="modal" aria-label="<%= t('blacklight.modal.close') %>">
+    <button type="button" class="blacklight-modal-close btn-close close" data-bl-dismiss="modal" aria-label="<%= t('blacklight.modal.close') %>">
       <span aria-hidden="true" class="visually-hidden">&times;</span>
     </button>
   </div>

data/app/components/blacklight/system/modal_component.rb

@@ -3,8 +3,6 @@
 module Blacklight
   module System
     class ModalComponent < ViewComponent::Base
-      include Blacklight::ContentAreasShim
-
       renders_one :prefix
       renders_one :header
       renders_one :title

data/app/components/blacklight/top_navbar_component.html.erb

@@ -0,0 +1,12 @@
+<nav class="navbar navbar-expand-md navbar-dark bg-dark topbar" role="navigation">
+  <div class="<%= container_classes %>">
+    <%= link_to application_name, blacklight_config.logo_link, class: 'mb-0 navbar-brand navbar-logo' %>
+    <button class="navbar-toggler navbar-toggler-right" type="button" data-toggle="collapse" data-bs-toggle="collapse" data-target="#user-util-collapse" data-bs-target="#user-util-collapse" aria-controls="user-util-collapse" aria-expanded="false" aria-label="Toggle navigation">
+      <span class="navbar-toggler-icon"></span>
+    </button>
+
+    <div class="collapse navbar-collapse justify-content-md-end" id="user-util-collapse">
+      <%= render 'shared/user_util_links' %>
+    </div>
+  </div>
+</nav>

data/app/components/blacklight/top_navbar_component.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module Blacklight
+  class TopNavbarComponent < Blacklight::Component
+    def initialize(blacklight_config:)
+      @blacklight_config = blacklight_config
+    end
+
+    attr_reader :blacklight_config
+
+    delegate :application_name, :container_classes, to: :helpers
+  end
+end

data/app/controllers/bookmarks_controller.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 class BookmarksController < CatalogController
   include Blacklight::Bookmarks
 end

data/app/controllers/catalog_controller.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 class CatalogController < ApplicationController
   include Blacklight::Catalog
 end

data/app/controllers/concerns/blacklight/bookmarks.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
-# note that while this is mostly restful routing, the #update and #destroy actions
+
+# NOTE: that while this is mostly restful routing, the #update and #destroy actions
 # take the Solr document ID as the :id, NOT the id of the actual Bookmark action.
 module Blacklight::Bookmarks
   extend ActiveSupport::Concern
@@ -20,6 +21,7 @@ module Blacklight::Bookmarks
 
     blacklight_config.show.document_actions[:bookmark].if = false if blacklight_config.show.document_actions[:bookmark]
     blacklight_config.show.document_actions[:sms].if = false if blacklight_config.show.document_actions[:sms]
+    blacklight_config.search_builder_class = Blacklight::BookmarksSearchBuilder
   end
 
   def action_documents
@@ -38,11 +40,14 @@ module Blacklight::Bookmarks
     search_catalog_url(*args)
   end
 
+  # @return [Hash] a hash of context information to pass through to the search service
+  def search_service_context
+    { bookmarks: @bookmarks }
+  end
+
   def index
     @bookmarks = token_or_current_or_guest_user.bookmarks
-    bookmark_ids = @bookmarks.collect { |b| b.document_id.to_s }
-    @response, deprecated_document_list = search_service.fetch(bookmark_ids)
-    @document_list = ActiveSupport::Deprecation::DeprecatedObjectProxy.new(deprecated_document_list, "The @document_list instance variable is now deprecated and will be removed in Blacklight 8.0")
+    @response = search_service.search_results
 
     respond_to do |format|
       format.html {}
@@ -115,7 +120,7 @@ module Blacklight::Bookmarks
         redirect_back fallback_location: bookmarks_path, notice: I18n.t('blacklight.bookmarks.remove.success')
       end
     elsif request.xhr?
-      head 500 # ajaxy request needs no redirect and should not have flash set
+      head :internal_server_error # ajaxy request needs no redirect and should not have flash set
     else
       redirect_back fallback_location: bookmarks_path, flash: { error: I18n.t('blacklight.bookmarks.remove.failure') }
     end

data/app/controllers/concerns/blacklight/catalog.rb

@@ -1,37 +1,29 @@
 # frozen_string_literal: true
+
 module Blacklight::Catalog
   extend ActiveSupport::Concern
 
   # MimeResponds is part of ActionController::Base, but not ActionController::API
   include ActionController::MimeResponds
 
-  include Blacklight::Base
-  include Blacklight::Facet
+  include Blacklight::Configurable
+  include Blacklight::SearchContext
   include Blacklight::Searchable
 
-  extend Deprecation
-
   # The following code is executed when someone includes blacklight::catalog in their
   # own controller.
   included do
     if respond_to? :helper_method
-      helper_method :sms_mappings, :has_search_parameters?, :facet_limit_for
+      helper_method :sms_mappings, :has_search_parameters?
+      helper_method :search_facet_path
     end
 
-    helper Blacklight::Facet if respond_to? :helper
-
-    # The index action will more than likely throw this one.
-    # Example: when the standard query parser is used, and a user submits a "bad" query.
-    rescue_from Blacklight::Exceptions::InvalidRequest, with: :handle_request_error
-
     record_search_parameters
   end
 
   # get search results from the solr index
   def index
-    (@response, deprecated_document_list) = search_service.search_results
-
-    @document_list = ActiveSupport::Deprecation::DeprecatedObjectProxy.new(deprecated_document_list, 'The @document_list instance variable is deprecated; use @response.documents instead.')
+    @response = search_service.search_results
 
     respond_to do |format|
       format.html { store_preferred_view }
@@ -49,8 +41,7 @@ module Blacklight::Catalog
   # get a single document from the index
   # to add responses for formats other than html or json see _Blacklight::Document::Export_
   def show
-    deprecated_response, @document = search_service.fetch(params[:id])
-    @response = ActiveSupport::Deprecation::DeprecatedObjectProxy.new(deprecated_response, 'The @response instance variable is deprecated; use @document.response instead.')
+    @document = search_service.fetch(params[:id])
 
     respond_to do |format|
       format.html { @search_context = setup_next_and_previous_documents }
@@ -67,7 +58,7 @@ module Blacklight::Catalog
   def raw
     raise(ActionController::RoutingError, 'Not Found') unless blacklight_config.raw_endpoint.enabled
 
-    _, @document = search_service.fetch(params[:id])
+    @document = search_service.fetch(params[:id])
     render json: @document
   end
 
@@ -95,7 +86,7 @@ module Blacklight::Catalog
     @response = search_service.facet_field_response(@facet.key)
     @display_facet = @response.aggregations[@facet.field]
 
-    @presenter = (@facet.presenter || Blacklight::FacetFieldPresenter).new(@facet, @display_facet, view_context)
+    @presenter = @facet.presenter.new(@facet, @display_facet, view_context)
     @pagination = @presenter.paginator
     respond_to do |format|
       format.html do
@@ -115,21 +106,19 @@ module Blacklight::Catalog
     end
   end
 
+  # Returns the dropdown list for autocomplete
   def suggest
-    respond_to do |format|
-      format.json do
-        render json: suggestions_service.suggestions
-      end
-    end
+    @suggestions = suggestions_service.suggestions
+    render 'suggest', layout: false
   end
 
   # @return [Array] first value is a Blacklight::Solr::Response and the second
   #                 is a list of documents
   def action_documents
-    deprecated_response, @documents = search_service.fetch(Array(params[:id]))
+    @documents = search_service.fetch(Array(params[:id]))
     raise Blacklight::Exceptions::RecordNotFound if @documents.blank?
 
-    [deprecated_response, @documents]
+    @documents
   end
 
   def action_success_redirect_path
@@ -143,34 +132,14 @@ module Blacklight::Catalog
     params[:search_field].present? || search_state.has_constraints?
   end
 
-  # TODO: deprecate this constant with #facet_limit_for
-  DEFAULT_FACET_LIMIT = 10
-
-  # Look up facet limit for given facet_field. Will look at config, and
-  # if config is 'true' will look up from Solr @response if available. If
-  # no limit is available, returns nil. Used from #add_facetting_to_solr
-  # to supply f.fieldname.facet.limit values in solr request (no @response
-  # available), and used in display (with @response available) to create
-  # a facet paginator with the right limit.
-  def facet_limit_for(facet_field)
-    facet = blacklight_config.facet_fields[facet_field]
-    return if facet.blank?
-
-    if facet.limit && @response && @response.aggregations[facet.field]
-      limit = @response.aggregations[facet.field].limit
-
-      if limit.nil? # we didn't get or a set a limit, so infer one.
-        facet.limit if facet.limit != true
-      elsif limit == -1 # limit -1 is solr-speak for unlimited
-        nil
-      else
-        limit.to_i - 1 # we added 1 to find out if we needed to paginate
-      end
-    elsif facet.limit
-      facet.limit == true ? DEFAULT_FACET_LIMIT : facet.limit
-    end
+  def search_facet_path(options = {})
+    opts = search_state
+           .to_h
+           .merge(action: "facet", only_path: true)
+           .merge(options)
+           .except(:page)
+    url_for opts
   end
-  deprecation_deprecate facet_limit_for: 'moving to private logic in Blacklight::FacetFieldPresenter'
 
   private
 
@@ -314,26 +283,6 @@ module Blacklight::Catalog
     action_name == 'show' ? 'catalog_result' : super
   end
 
-  # when a method throws a Blacklight::Exceptions::InvalidRequest, this method is executed.
-  def handle_request_error(exception)
-    # Rails own code will catch and give usual Rails error page with stack trace
-    raise exception if Rails.env.development? || Rails.env.test?
-
-    flash_notice = I18n.t('blacklight.search.errors.request_error')
-
-    # If there are errors coming from the index page, we want to trap those sensibly
-
-    if flash[:notice] == flash_notice
-      logger&.error "Cowardly aborting rsolr_request_error exception handling, because we redirected to a page that raises another exception"
-      raise exception
-    end
-
-    logger&.error exception
-
-    flash[:notice] = flash_notice
-    redirect_to search_action_url
-  end
-
   def blacklight_advanced_search_form_search_service
     form_search_state = search_state_class.new(blacklight_advanced_search_form_params, blacklight_config, self)
 

data/app/controllers/concerns/blacklight/controller.rb

@@ -1,14 +1,11 @@
 # frozen_string_literal: true
+
 # Filters added to this controller apply to all controllers in the hosting application
 # as this module is mixed-in to the application controller in the hosting app on installation.
 module Blacklight::Controller
   extend ActiveSupport::Concern
-  extend Deprecation
 
   included do
-    include Blacklight::SearchFields
-    helper Blacklight::SearchFields if respond_to? :helper
-
     include ActiveSupport::Callbacks
 
     # now in application.rb file under config.filter_parameters
@@ -16,6 +13,7 @@ module Blacklight::Controller
 
     # handle basic authorization exception with #access_denied
     rescue_from Blacklight::Exceptions::AccessDenied, with: :access_denied
+    helper BlacklightHelper
 
     if respond_to? :helper_method
       helper_method :current_user_session, :current_user, :current_or_guest_user
@@ -24,8 +22,7 @@ module Blacklight::Controller
       helper_method :has_user_authentication_provider?
       helper_method :blacklight_config, :blacklight_configuration_context # move to Catalog
       helper_method :search_action_url, :search_action_path
-      helper_method :search_facet_path # move to catalog? deprecate?
-      helper_method :search_state # move to catalog?
+      helper_method :search_state
     end
 
     # Which class to use for the search state. You can subclass SearchState if you
@@ -33,12 +30,6 @@ module Blacklight::Controller
     # TODO: move to Searchable
     class_attribute :search_state_class
     self.search_state_class = Blacklight::SearchState
-
-    # Which class to use for the search service. You can subclass SearchService if you
-    # want to override any of the methods (e.g. SearchService#fetch)
-    # TODO: move to Searchable
-    class_attribute :search_service_class
-    self.search_service_class = Blacklight::SearchService
   end
 
   # @private
@@ -64,8 +55,9 @@ module Blacklight::Controller
     has_user_authentication_provider? && current_or_guest_user.present?
   end
 
+  # This must be on every controller that uses the layout, because it is used in
+  # the header to draw Blacklight::SearchNavbarComponent (via the shared/header_navbar template)
   # @return [Blacklight::SearchState] a memoized instance of the parameter state.
-  # TODO: move to catalog?
   def search_state
     @search_state ||= search_state_class.new(params, blacklight_config, self)
   end
@@ -75,7 +67,7 @@ module Blacklight::Controller
   # which action the search form should use
   def search_action_url options = {}
     # Rails 4.2 deprecated url helpers accepting string keys for 'controller' or 'action'
-    search_catalog_url(options.except(:controller, :action))
+    search_catalog_url(options.to_h.except(:controller, :action))
   end
 
   def search_action_path *args
@@ -86,22 +78,6 @@ module Blacklight::Controller
     search_action_url(*args)
   end
 
-  # TODO: move to catalog? deprecate?
-  def search_facet_path(options = {})
-    opts = search_state
-           .to_h
-           .merge(action: "facet", only_path: true)
-           .merge(options)
-           .except(:page)
-    url_for opts
-  end
-
-  # Returns a list of Searches from the ids in the user's history.
-  # TODO: move to catalog? SearchHistory?
-  def searches_from_history
-    session[:history].blank? ? ::Search.none : ::Search.where(id: session[:history]).order("updated_at desc")
-  end
-
   # Should be provided by authentication provider
   # def current_user
   # end
@@ -125,21 +101,9 @@ module Blacklight::Controller
     respond_to? :current_user
   end
 
-  def require_user_authentication_provider
-    raise ActionController::RoutingError, 'Not Found' unless has_user_authentication_provider?
-  end
-  deprecation_deprecate require_user_authentication_provider: 'removed without replacement'
-
   ##
   # When a user logs in, transfer any saved searches or bookmarks to the current_user
   def transfer_guest_to_user
-    Deprecation.silence(Blacklight::Controller) do
-      transfer_guest_user_actions_to_current_user
-    end
-  end
-
-  # @deprecated use canonical `transfer_guest_to_user` method instead
-  def transfer_guest_user_actions_to_current_user
     return unless respond_to?(:current_user) && respond_to?(:guest_user) && current_user && guest_user
 
     current_user_searches = current_user.searches.pluck(:query_params)
@@ -158,7 +122,6 @@ module Blacklight::Controller
     # let guest_user know we've moved some bookmarks from under it
     guest_user.reload if guest_user.persisted?
   end
-  deprecation_deprecate :transfer_guest_user_actions_to_current_user
 
   ##
   # To handle failed authorization attempts, redirect the user to the

data/app/controllers/concerns/blacklight/search_context.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Blacklight::SearchContext
   extend ActiveSupport::Concern
 
@@ -17,6 +18,16 @@ module Blacklight::SearchContext
     end
   end
 
+  # The current search session
+  def current_search_session
+    @current_search_session ||= find_search_session
+  end
+
+  # Returns a list of Searches from the ids in the user's history.
+  def searches_from_history
+    session[:history].blank? ? ::Search.none : ::Search.where(id: session[:history]).order("updated_at desc")
+  end
+
   private
 
   # sets up the session[:search] hash if it doesn't already exist
@@ -27,11 +38,6 @@ module Blacklight::SearchContext
     session[:search]
   end
 
-  # The current search session
-  def current_search_session
-    @current_search_session ||= find_search_session
-  end
-
   # Persist the current search session id to the user's session
   def set_current_search_session
     search_session['id'] = current_search_session.id if current_search_session
@@ -79,7 +85,7 @@ module Blacklight::SearchContext
   end
 
   def find_or_initialize_search_session_from_params params
-    params_copy = params.reject { |k, v| blacklisted_search_session_params.include?(k.to_sym) || v.blank? }
+    params_copy = params.reject { |k, v| nonpersisted_search_session_params.include?(k.to_sym) || v.blank? }
 
     return if params_copy.reject { |k, _v| [:action, :controller].include? k.to_sym }.blank?
 
@@ -102,10 +108,22 @@ module Blacklight::SearchContext
   end
 
   # A list of query parameters that should not be persisted for a search
-  def blacklisted_search_session_params
+  def nonpersisted_search_session_params
+    unless method(:blacklisted_search_session_params).source_location.first.end_with?('deprecation/method_wrappers.rb')
+      # The blacklisted_search_session_params was overridden, so call it.
+      Blacklight.deprecation.warn(self, "blacklisted_search_session_params was overriden in your app, " \
+                                        "but that method should be renamed to `nonpersisted_search_session_params'. " \
+                                        "The original behavior will be removed in the next major release.")
+      return blacklisted_search_session_params
+    end
     [:commit, :counter, :total, :search_id, :page, :per_page]
   end
 
+  def blacklisted_search_session_params
+    nonpersisted_search_session_params
+  end
+  Blacklight.deprecation.deprecate_methods(self, blacklisted_search_session_params: 'use nonpersisted_search_session_params instead')
+
   # calls setup_previous_document then setup_next_document.
   # used in the show action for single view pagination.
   def setup_next_and_previous_documents