RubyGems - blacklight - Versions diffs - 7.32.0 → 8.0.0.beta1 - Mend

blacklight 7.32.0 → 8.0.0.beta1

Files changed (390) hide show

data/app/controllers/concerns/blacklight/searchable.rb CHANGED Viewed

@@ -5,17 +5,28 @@
 # would be provided by Blacklight::Controller
 #  1. search_state
 #  2. blacklight_config
-#  3. search_service_class
 #
 # Additionally, the including class may override the search_service_context method to provide
 # further context to the SearchService. For example you could override this to provide the
 # currently signed in user.
 module Blacklight::Searchable
+  extend ActiveSupport::Concern
+  included do
+    # 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)
+    class_attribute :search_service_class
+    self.search_service_class = Blacklight::SearchService
+  end
   # @return [Blacklight::SearchService]
   def search_service
     search_service_class.new(config: blacklight_config, search_state: search_state, user_params: search_state.to_h, **search_service_context)
   end
+  # Override this method on the class that includes Blacklight::Searchable to provide more context to the search service if necessary.
+  # For example, if your search builder needs to be aware of the current user, override this method to return a hash including the current user.
+  # Then the search builder could use some property about the current user to construct a constraint on the search.
   # @return [Hash] a hash of context information to pass through to the search service
   def search_service_context
     {}

data/app/controllers/concerns/blacklight/token_based_user.rb CHANGED Viewed

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
 module Blacklight::TokenBasedUser
   extend ActiveSupport::Concern

data/app/controllers/search_history_controller.rb CHANGED Viewed

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

data/app/helpers/blacklight/blacklight_helper_behavior.rb CHANGED Viewed

@@ -1,10 +1,8 @@
 # frozen_string_literal: true
 # Methods added to this helper will be available to all templates in the hosting application
 module Blacklight::BlacklightHelperBehavior
-  extend Deprecation
   include Blacklight::UrlHelperBehavior
-  include Blacklight::HashAsHiddenFieldsHelperBehavior
   include Blacklight::LayoutHelperBehavior
   include Blacklight::IconHelperBehavior
@@ -51,15 +49,6 @@ module Blacklight::BlacklightHelperBehavior
     document_presenter(document).link_rel_alternates(options)
   end
-  ##
-  # Render OpenSearch headers for this search
-  # @deprecated
-  # @return [String]
-  def render_opensearch_response_metadata
-    render partial: 'catalog/opensearch_response_metadata', locals: { response: @response }
-  end
-  deprecation_deprecate render_opensearch_response_metadata: 'Use `render "catalog/opensearch_response_metadata"\' instead'
   ##
   # Render classes for the <body> element
   # @return [String]
@@ -72,198 +61,8 @@ module Blacklight::BlacklightHelperBehavior
   # @see render_body_class
   # @return [Array<String>]
   def extra_body_classes
-    @extra_body_classes ||= ['blacklight-' + controller.controller_name, 'blacklight-' + [controller.controller_name, controller.action_name].join('-')]
-  end
-  ##
-  # Render the search navbar
-  # @return [String]
-  def render_search_bar
-    if search_bar_presenter_class == Blacklight::SearchBarPresenter && partial_from_blacklight?(Blacklight::SearchBarPresenter.partial)
-      component_class = blacklight_config&.view_config(document_index_view_type)&.search_bar_component || Blacklight::SearchBarComponent
-      render component_class.new(
-        url: search_action_url,
-        advanced_search_url: search_action_url(action: 'advanced_search'),
-        params: search_state.params_for_search.except(:qt),
-        search_fields: Deprecation.silence(Blacklight::ConfigurationHelperBehavior) { search_fields },
-        autocomplete_path: search_action_path(action: :suggest)
-      )
-    else
-      search_bar_presenter.render
-    end
-  end
-  deprecation_deprecate render_search_bar: "Call `render Blacklight::SearchBarComponent.new' instead"
-  # @!group Presenter extension helpers
-  ##
-  # @return [Blacklight::SearchBarPresenter]
-  def search_bar_presenter
-    @search_bar ||= search_bar_presenter_class.new(controller, blacklight_config)
-  end
-  deprecation_deprecate :search_bar_presenter
-  # @!group Document helpers
-  ##
-  # Determine whether to render a given field in the index view.
-  #
-  # @deprecated
-  # @param [SolrDocument] document
-  # @param [Blacklight::Configuration::Field] field_config
-  # @return [Boolean]
-  def should_render_index_field? document, field_config
-    Deprecation.warn self, "should_render_index_field? is deprecated and will be removed in Blacklight 8. Use IndexPresenter#render_field? instead."
-    should_render_field?(field_config, document) && document_has_value?(document, field_config)
-  end
-  ##
-  # Determine whether to render a given field in the show view
-  #
-  # @deprecated
-  # @param [SolrDocument] document
-  # @param [Blacklight::Configuration::Field] field_config
-  # @return [Boolean]
-  def should_render_show_field? document, field_config
-    Deprecation.warn self, "should_render_show_field? is deprecated and will be removed in Blacklight 8. Use ShowPresenter#render_field? instead."
-    should_render_field?(field_config, document) && document_has_value?(document, field_config)
-  end
-  ##
-  # Check if a document has (or, might have, in the case of accessor methods) a value for
-  # the given solr
-  # @deprecated
-  # @param [SolrDocument] document
-  # @param [Blacklight::Configuration::Field] field_config
-  # @return [Boolean]
-  def document_has_value? document, field_config
-    Deprecation.warn self, "document_has_value? is deprecated and will be removed in Blacklight 8. Use DocumentPresenter#has_value? instead."
-    document.has?(field_config.field) ||
-      (document.has_highlight_field? field_config.field if field_config.highlight) ||
-      field_config.accessor
-  end
-  # @!group Search result helpers
-  ##
-  # Determine whether to display spellcheck suggestions
-  #
-  # @deprecated
-  # @param [Blacklight::Solr::Response] response
-  # @return [Boolean]
-  def should_show_spellcheck_suggestions? response
-    Deprecation.silence(Blacklight::ConfigurationHelperBehavior) do
-      # The spelling response field may be missing from non solr repositories.
-      response.total <= spell_check_max &&
-        (response.spelling&.words&.any? || response.spelling&.collation&.present? || false)
-    end
-  end
-  deprecation_deprecate should_show_spellcheck_suggestions?: 'moving into a private method of Blacklight::Response::SpellcheckComponent'
-  # @!group Document helpers
-  ##
-  # Render the index field label for a document
-  #
-  # Translations for index field labels should go under blacklight.search.fields
-  # They are picked up from there by a value "%{label}" in blacklight.search.index.label
-  #
-  # @deprecated
-  # @overload render_index_field_label(options)
-  #   Use the default, document-agnostic configuration
-  #   @param [Hash] opts
-  #   @option opts [String] :field
-  # @overload render_index_field_label(document, options)
-  #   Allow an extention point where information in the document
-  #   may drive the value of the field
-  #   @param [SolrDocument] doc
-  #   @param [Hash] opts
-  #   @option opts [String] :field
-  # @return [String]
-  def render_index_field_label *args
-    options = args.extract_options!
-    document = args.first
-    field = options[:field]
-    label = Deprecation.silence(Blacklight::ConfigurationHelperBehavior) do
-      options[:label] || index_field_label(document, field)
-    end
-    html_escape t(:"blacklight.search.index.#{document_index_view_type}.label", default: :'blacklight.search.index.label', label: label)
-  end
-  deprecation_deprecate render_index_field_label: 'Use Blacklight::MetadataFieldComponent instead'
-  ##
-  # Render the show field label for a document
-  #
-  # @deprecated
-  # @overload render_document_show_field_label(options)
-  #   Use the default, document-agnostic configuration
-  #   @param [Hash] opts
-  #   @option opts [String] :field
-  # @overload render_document_show_field_label(document, options)
-  #   Allow an extention point where information in the document
-  #   may drive the value of the field
-  #   @param [SolrDocument] doc
-  #   @param [Hash] opts
-  #   @option opts [String] :field
-  # @return [String]
-  def render_document_show_field_label *args
-    options = args.extract_options!
-    document = args.first
-    field = options[:field]
-    label = Deprecation.silence(Blacklight::ConfigurationHelperBehavior) do
-      options[:label] || document_show_field_label(document, field)
-    end
-    t(:'blacklight.search.show.label', label: label)
+    @extra_body_classes ||= ["blacklight-#{controller.controller_name}", "blacklight-#{[controller.controller_name, controller.action_name].join('-')}"]
   end
-  deprecation_deprecate render_document_show_field_label: 'Use Blacklight::MetadataFieldComponent instead'
-  ##
-  # Get the value of the document's "title" field, or a placeholder
-  # value (if empty)
-  #
-  # @deprecated
-  # @param [SolrDocument] document
-  # @return [String]
-  def document_heading document = nil
-    document ||= @document
-    document_presenter(document).heading
-  end
-  deprecation_deprecate document_heading: 'Use Blacklight::DocumentPresenter#heading instead'
-  ##
-  # Get the document's "title" to display in the <title> element.
-  # (by default, use the #document_heading)
-  #
-  # @deprecated
-  # @see #document_heading
-  # @param [SolrDocument] document
-  # @return [String]
-  def document_show_html_title document = nil
-    document ||= @document
-    document_presenter(document).html_title
-  end
-  deprecation_deprecate document_show_html_title: 'Use Blacklight::DocumentPresenter#html_title instead'
-  ##
-  # Render the document "heading" (title) in a content tag
-  # @deprecated
-  # @overload render_document_heading(document, options)
-  #   @param [SolrDocument] document
-  #   @param [Hash] options
-  #   @option options [Symbol] :tag
-  # @overload render_document_heading(options)
-  #   @param [Hash] options
-  #   @option options [Symbol] :tag
-  # @return [String]
-  def render_document_heading(*args)
-    options = args.extract_options!
-    document = args.first
-    tag = options.fetch(:tag, :h4)
-    document ||= @document
-    content_tag(tag, document_presenter(document).heading, itemprop: "name")
-  end
-  deprecation_deprecate render_document_heading: 'Removed without replacement'
   ##
   # Get the current "view type" (and ensure it is a valid type)
@@ -309,101 +108,24 @@ module Blacklight::BlacklightHelperBehavior
     response&.grouped?
   end
-  # @!group Presenter extension helpers
-  ##
-  # Returns a document presenter for the given document
-  # TODO: Move this to the controller. It can just pass a presenter or set of presenters.
-  # @deprecated
-  # @return [Blacklight::DocumentPresenter]
-  def presenter(document)
-    Deprecation.warn(Blacklight::BlacklightHelperBehavior, '#presenter is deprecated; use #document_presenter instead')
-    # As long as the presenter methods haven't been overridden, we can use the new behavior
-    if method(:show_presenter).owner == Blacklight::BlacklightHelperBehavior &&
-       method(:index_presenter).owner == Blacklight::BlacklightHelperBehavior
-      return document_presenter_class(document).new(document, self)
-    end
-    Deprecation.warn(Blacklight::BlacklightHelperBehavior, '#show_presenter and/or #index_presenter have been overridden; please override #document_presenter instead')
-    Deprecation.silence(Blacklight::BlacklightHelperBehavior) do
-      case action_name
-      when 'show', 'citation'
-        show_presenter(document)
-      else
-        index_presenter(document)
-      end
-    end
-  end
   ##
   # Returns a document presenter for the given document
   def document_presenter(document)
-    Deprecation.silence(Blacklight::BlacklightHelperBehavior) do
-      presenter(document)
-    end
-  end
-  # @deprecated
-  # @return [Blacklight::ShowPresenter]
-  def show_presenter(document)
-    Deprecation.warn(Blacklight::BlacklightHelperBehavior, '#show_presenter is deprecated; use #document_presenter instead')
-    if method(:show_presenter_class).owner != Blacklight::BlacklightHelperBehavior
-      Deprecation.warn(Blacklight::BlacklightHelperBehavior, '#show_presenter_class has been overridden; please override #document_presenter_class instead')
-    end
-    Deprecation.silence(Blacklight::BlacklightHelperBehavior) do
-      show_presenter_class(document).new(document, self)
-    end
-  end
-  # @deprecated
-  # @return [Blacklight::IndexPresenter]
-  def index_presenter(document)
-    Deprecation.warn(Blacklight::BlacklightHelperBehavior, '#index_presenter is deprecated; use #document_presenter instead')
-    if method(:index_presenter_class).owner != Blacklight::BlacklightHelperBehavior
-      Deprecation.warn(Blacklight::BlacklightHelperBehavior, '#index_presenter_class has been overridden; please override #document_presenter_class instead')
-    end
-    Deprecation.silence(Blacklight::BlacklightHelperBehavior) do
-      index_presenter_class(document).new(document, self)
-    end
+    document_presenter_class(document).new(document, self)
   end
   ##
   # Override this method if you want to use a differnet presenter for your documents
-  # @param [Blacklight::Document] document optional, here for extension + backwards compatibility only
-  def document_presenter_class(document = nil)
-    Deprecation.silence(Blacklight::BlacklightHelperBehavior) do
-      case action_name
-      when 'show', 'citation'
-        show_presenter_class(document)
-      else
-        index_presenter_class(document)
-      end
+  # @param [Blacklight::Document] _document optional, here for extension + backwards compatibility only
+  def document_presenter_class(_document = nil)
+    case action_name
+    when 'show', 'citation'
+      blacklight_config.view_config(:show, action_name: action_name).document_presenter_class
+    else
+      blacklight_config.view_config(document_index_view_type, action_name: action_name).document_presenter_class
     end
   end
-  ##
-  # Override this method if you want to use a different presenter class
-  # @deprecated
-  # @return [Class]
-  def show_presenter_class(_document)
-    Deprecation.warn(Blacklight::BlacklightHelperBehavior, '#show_presenter_class is deprecated; use #document_presenter_class instead')
-    blacklight_config.view_config(:show, action_name: action_name).document_presenter_class
-  end
-  # @deprecated
-  # @return [Class]
-  def index_presenter_class(_document)
-    Deprecation.warn(Blacklight::BlacklightHelperBehavior, '#index_presenter_class is deprecated; use #document_presenter_class instead')
-    blacklight_config.view_config(document_index_view_type, action_name: action_name).document_presenter_class
-  end
   # @return [Class]
   def search_bar_presenter_class
     blacklight_config.view_config(action_name: :index).search_bar_presenter_class
@@ -416,24 +138,4 @@ module Blacklight::BlacklightHelperBehavior
   def opensearch_description_tag title, href
     tag :link, href: href, title: title, type: "application/opensearchdescription+xml", rel: "search"
   end
-  # @private
-  def self.blacklight_path
-    @blacklight_path ||= Gem.loaded_specs["blacklight"].full_gem_path
-  end
-  def partial_from_blacklight?(partial)
-    path = if Rails::VERSION::MAJOR >= 6
-             name = partial.split('/').last
-             prefix = partial.split('/').first if partial.include?('/')
-             logger&.debug "Looking for document index partial #{partial}"
-             prefixes = lookup_context.prefixes + [prefix, ""].compact
-             lookup_context.find_all(name, prefixes, true).first&.identifier
-           else
-             lookup_context.find_all(partial, lookup_context.prefixes + [""], true).first&.identifier
-           end
-    path&.starts_with?(Blacklight::BlacklightHelperBehavior.blacklight_path)
-  end
 end

data/app/helpers/blacklight/catalog_helper_behavior.rb CHANGED Viewed

@@ -1,16 +1,11 @@
 # frozen_string_literal: true
 # Helper methods for catalog-like controllers
 module Blacklight::CatalogHelperBehavior
-  extend Deprecation
-  self.deprecation_horizon = 'blacklight 8.0'
   include Blacklight::ConfigurationHelperBehavior
   include Blacklight::ComponentHelperBehavior
   include Blacklight::FacetsHelperBehavior
-  include Blacklight::RenderConstraintsHelperBehavior
   include Blacklight::RenderPartialsHelperBehavior
-  include Blacklight::SearchHistoryConstraintsHelperBehavior
-  include Blacklight::SuggestHelperBehavior
   # @param [Hash] options
   # @option options :route_set the route scope to use when constructing the link
@@ -64,18 +59,18 @@ module Blacklight::CatalogHelperBehavior
               end
     case collection.total_count
-      when 0
-        t('blacklight.search.pagination_info.no_items_found', entry_name: entry_name).html_safe
-      when 1
-        t('blacklight.search.pagination_info.single_item_found', entry_name: entry_name).html_safe
-      else
-        t('blacklight.search.pagination_info.pages', entry_name: entry_name,
-                                                     current_page: collection.current_page,
-                                                     num_pages: collection.total_pages,
-                                                     start_num: number_with_delimiter(collection.offset_value + 1),
-                                                     end_num: number_with_delimiter(end_num),
-                                                     total_num: number_with_delimiter(collection.total_count),
-                                                     count: collection.total_pages).html_safe
+    when 0
+      t('blacklight.search.pagination_info.no_items_found', entry_name: entry_name).html_safe
+    when 1
+      t('blacklight.search.pagination_info.single_item_found', entry_name: entry_name).html_safe
+    else
+      t('blacklight.search.pagination_info.pages', entry_name: entry_name,
+                                                   current_page: collection.current_page,
+                                                   num_pages: collection.total_pages,
+                                                   start_num: number_with_delimiter(collection.offset_value + 1),
+                                                   end_num: number_with_delimiter(end_num),
+                                                   total_num: number_with_delimiter(collection.total_count),
+                                                   count: collection.total_pages).html_safe
     end
   end
@@ -94,20 +89,6 @@ module Blacklight::CatalogHelperBehavior
     end
   end
-  ##
-  # Like #page_entries_info above, but for an individual
-  # item show page. Displays "showing X of Y items" message.
-  #
-  # @deprecated
-  # @see #page_entries_info
-  # @return [String]
-  def item_page_entry_info
-    t('blacklight.search.entry_pagination_info.other', current: number_with_delimiter(search_session['counter']),
-                                                       total: number_with_delimiter(search_session['total']),
-                                                       count: search_session['total'].to_i).html_safe
-  end
-  deprecation_deprecate item_page_entry_info: 'Use Blacklight::SearchContextComponent methods instead'
   ##
   # Look up search field user-displayable label
   # based on params[:qt] and blacklight_configuration.
@@ -162,27 +143,10 @@ module Blacklight::CatalogHelperBehavior
   #
   # @param [SolrDocument] document
   # @return [String]
-  def render_document_sidebar_partial(document = nil)
-    unless document
-      Deprecation.warn(self, 'render_document_sidebar_partial expects one argument ' \
-        '(@document) and you passed none. This behavior will be removed in version 8')
-      document = @document
-    end
+  def render_document_sidebar_partial(document)
     render 'show_sidebar', document: document
   end
-  ##
-  # Render the main content partial for a document
-  # This is widely used as by downstream apps when they override their show view.
-  # See https://github.com/search?q=render_document_main_content_partial&type=Code
-  #
-  # @param [SolrDocument] _document
-  # @return [String]
-  def render_document_main_content_partial(_document = @document)
-    render partial: 'show_main_content'
-  end
-  deprecation_deprecate render_document_main_content_partial: "Use \"render 'show_main_content'\" instead"
   ##
   # Should we display the sort and per page widget?
   #
@@ -204,95 +168,16 @@ module Blacklight::CatalogHelperBehavior
   end
   ##
-  # If no search parameters have been given, we should
-  # auto-focus the user's cursor into the searchbox
-  #
-  # @deprecated
-  # @return [Boolean]
-  def should_autofocus_on_search_box?
-    controller.is_a?(Blacklight::Catalog) &&
-      action_name == "index" &&
-      !has_search_parameters?
-  end
-  deprecation_deprecate should_autofocus_on_search_box?: "use SearchBarPresenter#autofocus?"
-  ##
-  # Does the document have a thumbnail to render?
-  #
-  # @deprecated
-  # @param [SolrDocument] document
-  # @return [Boolean]
-  def has_thumbnail? document
-    document_presenter(document).thumbnail.exists?
-  end
-  deprecation_deprecate has_thumbnail?: "use IndexPresenter#thumbnail.exists?"
-  ##
-  # Render the thumbnail, if available, for a document and
-  # link it to the document record.
-  #
-  # @deprecated
-  # @param [SolrDocument] document
-  # @param [Hash] image_options to pass to the image tag
-  # @param [Hash] url_options to pass to #link_to_document
-  # @return [String]
-  def render_thumbnail_tag document, image_options = {}, url_options = {}
-    document_presenter(document).thumbnail.thumbnail_tag(image_options, url_options)
-  end
-  deprecation_deprecate render_thumbnail_tag: "Use IndexPresenter#thumbnail.thumbnail_tag"
-  ##
-  # Get the URL to a document's thumbnail image
-  #
-  # @deprecated
-  # @param [SolrDocument] document
-  # @return [String]
-  def thumbnail_url document
-    if document.has? blacklight_config.view_config(document_index_view_type).thumbnail_field
-      document.first(blacklight_config.view_config(document_index_view_type).thumbnail_field)
-    end
-  end
-  deprecation_deprecate thumbnail_url: "this method will be removed without replacement"
-  ##
-  # Render the view type icon for the results view picker
-  #
-  # @deprecated
-  # @param [String] view
-  # @return [String]
-  def render_view_type_group_icon view
-    blacklight_icon(view)
-  end
-  deprecation_deprecate render_view_type_group_icon: 'call blacklight_icon instead'
-  ##
-  # Get the default view type classes for a view in the results view picker
-  #
-  # @deprecated
-  # @param [String] view
-  # @return [String]
-  def default_view_type_group_icon_classes view
-    Deprecation.warn(Blacklight::CatalogHelperBehavior, 'This method has been deprecated, use blacklight_icons helper instead')
-    "glyphicon-#{view.to_s.parameterize} view-icon-#{view.to_s.parameterize}"
-  end
-  ##
-  # return the Bookmarks on a set of documents
-  # @param [Enumerable<Blacklight::Document>] documents_or_response
+  # return the Bookmarks on a set of documents (all bookmarks on the page)
+  # @private
   # @return [Enumerable<Bookmark>]
-  def current_bookmarks documents_or_response = nil
-    documents = if documents_or_response.respond_to? :documents
-                  Deprecation.warn(Blacklight::CatalogHelperBehavior, "Passing a response to #current_bookmarks is deprecated; pass response.documents instead")
-                  documents_or_response.documents
-                else
-                  documents_or_response
-                end
-    documents ||= [@document] if @document.present?
-    documents ||= @response.documents
-    @current_bookmarks ||= current_or_guest_user.bookmarks_for_documents(documents).to_a
+  def current_bookmarks
+    @current_bookmarks ||= begin
+      documents = @document.presence || @response.documents
+      current_or_guest_user.bookmarks_for_documents(Array(documents)).to_a
+    end
   end
+  private :current_bookmarks
   ##
   # Check if the document is in the user's bookmarks

data/app/helpers/blacklight/component_helper_behavior.rb CHANGED Viewed

@@ -1,31 +1,11 @@
 # frozen_string_literal: true
 module Blacklight
   module ComponentHelperBehavior
-    extend Deprecation
-    # @deprecated
-    def document_action_label action, opts
-      t("blacklight.tools.#{action}", default: opts.label || action.to_s.humanize)
-    end
-    deprecation_deprecate :document_action_label
-    # @deprecated
-    def document_action_path action_opts, url_opts = {}
-      if action_opts.path
-        send(action_opts.path, url_opts)
-      elsif url_opts[:id].class.respond_to?(:model_name)
-        url_for([action_opts.key.to_sym, url_opts[:id]])
-      else
-        send("#{action_opts.key}_#{controller_name}_path", url_opts)
-      end
-    end
-    deprecation_deprecate :document_action_path
     ##
     # Render "document actions" area for navigation header
     # (normally renders "Saved Searches", "History", "Bookmarks")
-    # These things are added by add_nav_action and the default config is
-    # provided by DefaultComponentConfiguration
+    # These things are added by add_nav_action
     #
     # @param [Hash] options
     # @return [String]
@@ -60,37 +40,6 @@ module Blacklight
       render(component.new(actions: actions, classes: wrapping_class))
     end
-    ##
-    # Render "document actions" for the item detail 'show' view.
-    # (this normally renders next to title)
-    #
-    # By default includes 'Bookmarks'
-    #
-    # @param [SolrDocument] document
-    # @param [Hash] options
-    # @return [String]
-    def render_show_doc_actions(document = @document, url_opts: {}, **options)
-      document = options[:document] if options.key? :document
-      actions = document_actions(document, options: options)
-      if block_given?
-        # TODO: Deprecate this behavior and replace it with a separate component?
-        # Deprecation.warn(Blacklight::ComponentHelperBehavior, 'Pass a block to #render_show_doc_actions is deprecated')
-        actions.each do |action|
-          yield action, render((action.component || Blacklight::Document::ActionComponent).new(action: action, document: document, options: options, url_opts: url_opts))
-        end
-        nil
-      else
-        render(Blacklight::Document::ActionsComponent.new(document: document, actions: actions, options: options, url_opts: url_opts))
-      end
-    end
-    def render_show_doc_actions_method_from_blacklight?
-      method(:render_show_doc_actions).owner == Blacklight::ComponentHelperBehavior
-    end
     def show_doc_actions?(document = @document, options = {})
       filter_partials(blacklight_config.view_config(:show).document_actions, { document: document }.merge(options)).any?
     end