blacklight 7.37.0 → 8.0.0.beta1

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

@@ -1,7 +1,7 @@
-<%= form_tag @url, method: @method, class: @classes.join(' '), role: 'search', aria: { label: scoped_t('submit') }, **@form_options do %>
-  <%= render_hash_as_hidden_fields(@params) %>
+<%= form_with url: @url, local: true, method: @method, class: @classes.join(' '), scope: @prefix, role: 'search', aria: { label: scoped_t('submit') }, **@form_options do |f| %>
+  <%= render Blacklight::HiddenSearchStateComponent.new(params: @params) %>
   <% if search_fields.length > 1 %>
-    <label for="search_field" class="sr-only visually-hidden"><%= scoped_t('search_field.label') %></label>
+    <%= f.label :search_field, scoped_t('search_field.label'), class: 'sr-only visually-hidden' %>
   <% end %>
   <% before_input_groups.each do |input_group| %>
     <%= input_group %>
@@ -10,22 +10,27 @@
     <%= prepend %>
 
     <% if search_fields.length > 1 %>
-        <%= select_tag(:search_field,
-                       options_for_select(search_fields, h(@search_field)),
-                       title: scoped_t('search_field.title'),
-                       id: "#{@prefix}search_field",
-                       class: "custom-select form-select search-field") %>
+      <%= f.select(:search_field,
+                   options_for_select(search_fields, h(@search_field)),
+                   {},
+                   title: scoped_t('search_field.title'),
+                   class: "custom-select form-select search-field") %>
     <% elsif search_fields.length == 1 %>
-      <%= hidden_field_tag :search_field, search_fields.first.last %>
+      <%= f.hidden_field :search_field, value: search_fields.first.last %>
     <% end %>
 
-    <label for="<%= @prefix %><%= @query_param %>" class="sr-only visually-hidden"><%= scoped_t('search.label') %></label>
-    <%= text_field_tag @query_param, @q, placeholder: scoped_t('search.placeholder'), class: "search-q q form-control rounded-#{search_fields.length > 1 ? '0' : 'left'}", id: "#{@prefix}q", autocomplete: autocomplete_path.present? ? "off" : "", autofocus: @autofocus, aria: { label: scoped_t('search.label') }, data: { autocomplete_enabled: autocomplete_path.present?, autocomplete_path: autocomplete_path }  %>
+    <%= f.label @query_param, scoped_t('search.label'), class: 'sr-only visually-hidden' %>
+    <% if autocomplete_path.present? %>
+      <auto-complete src="<%= autocomplete_path %>" for="autocomplete-popup" class="search-autocomplete-wrapper">
+        <%= f.search_field @query_param, value: @q, placeholder: scoped_t('search.placeholder'), class: "search-q q form-control rounded-#{search_fields.length > 1 ? '0' : 'left'}", autofocus: @autofocus, aria: { label: scoped_t('search.label'), autocomplete: 'list', controls: 'autocomplete-popup' }  %>
+        <ul id="autocomplete-popup" role="listbox" aria-label="<%= scoped_t('search.label') %>"></ul>
+      </auto-complete>
+    <% else %>
+      <%= f.search_field @query_param, value: @q, placeholder: scoped_t('search.placeholder'), class: "search-q q form-control rounded-#{search_fields.length > 1 ? '0' : 'left'}", autofocus: @autofocus, aria: { label: scoped_t('search.label') }  %>
+    <% end %>
 
-    <span class="input-group-append">
-      <%= append %>
-      <%= search_button || render(Blacklight::SearchButtonComponent.new(id: "#{@prefix}search", text: scoped_t('submit'))) %>
-    </span>
+    <%= append %>
+    <%= search_button || render(Blacklight::SearchButtonComponent.new(id: "#{@prefix}search", text: scoped_t('submit'))) %>
   </div>
 <% end %>
 

data/app/components/blacklight/search_bar_component.rb

@@ -2,8 +2,6 @@
 
 module Blacklight
   class SearchBarComponent < Blacklight::Component
-    include Blacklight::ContentAreasShim
-
     renders_one :append
     renders_one :prepend
     renders_one :search_button
@@ -12,10 +10,10 @@ module Blacklight
     # rubocop:disable Metrics/ParameterLists
     def initialize(
       url:, params:,
-      advanced_search_url: nil, presenter: nil,
+      advanced_search_url: nil,
       classes: ['search-query-form'], prefix: nil,
       method: 'GET', q: nil, query_param: :q,
-      search_field: nil, search_fields: nil, autocomplete_path: nil,
+      search_field: nil, autocomplete_path: nil,
       autofocus: nil, i18n: { scope: 'blacklight.search.form' },
       form_options: {}
     )
@@ -30,14 +28,8 @@ module Blacklight
       @method = method
       @autocomplete_path = autocomplete_path
       @autofocus = autofocus
-      @search_fields = search_fields
       @i18n = i18n
       @form_options = form_options
-
-      return if presenter.nil?
-
-      Deprecation.warn(self, 'SearchBarComponent no longer uses a SearchBarPresenter, the presenter: param will be removed in 8.0. ' \
-        'Set advanced_search.enabled, autocomplete_enabled, and enable_search_bar_autofocus on BlacklightConfiguration')
     end
     # rubocop:enable Metrics/ParameterLists
 
@@ -74,12 +66,6 @@ module Blacklight
       helpers.blacklight_config
     end
 
-    def render_hash_as_hidden_fields(*args)
-      Deprecation.silence(Blacklight::HashAsHiddenFieldsHelperBehavior) do
-        helpers.render_hash_as_hidden_fields(*args)
-      end
-    end
-
     def scoped_t(key, **args)
       t(key, default: t(key, scope: 'blacklight.search.form'), **@i18n, **args)
     end

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/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
-        with_button(classes: 'btn btn-outline-secondary dropdown-toggle', label: button_label) unless button
+        button(classes: 'btn btn-outline-secondary dropdown-toggle', label: button_label) unless button
 
         return if options.any?
 
-        with_options(@choices.map do |option|
+        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
-        with_message { @message } if @message
+        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

@@ -1,6 +1,6 @@
 <nav class="navbar navbar-expand-md navbar-dark bg-dark topbar" role="navigation">
   <div class="<%= container_classes %>">
-    <%= logo_link %>
+    <%= 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>

data/app/components/blacklight/top_navbar_component.rb

@@ -9,9 +9,5 @@ module Blacklight
     attr_reader :blacklight_config
 
     delegate :application_name, :container_classes, to: :helpers
-
-    def logo_link(title: application_name)
-      link_to title, blacklight_config.logo_link, class: 'mb-0 navbar-brand navbar-logo'
-    end
   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,15 +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",
-      ActiveSupport::Deprecation.new("8.0", "blacklight")
-    )
+    @response = search_service.search_results
 
     respond_to do |format|
       format.html {}
@@ -119,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,44 +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
 
-  Deprecation.silence(Blacklight::Base) do
-    include Blacklight::Base
-  end
-
-  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.',
-      ActiveSupport::Deprecation.new("8.0", "blacklight")
-    )
+    @response = search_service.search_results
 
     respond_to do |format|
       format.html { store_preferred_view }
@@ -56,12 +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.',
-      ActiveSupport::Deprecation.new("8.0", "blacklight")
-    )
+    @document = search_service.fetch(params[:id])
 
     respond_to do |format|
       format.html { @search_context = setup_next_and_previous_documents }
@@ -78,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
 
@@ -106,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
@@ -126,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
@@ -154,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
 
@@ -325,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,7 +22,6 @@ 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
     end
 
@@ -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
@@ -76,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
@@ -87,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
@@ -126,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)
@@ -159,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