Skip to content

Commit

Permalink
Merge pull request #2709 from projectblacklight/deprecate_helper
Browse files Browse the repository at this point in the history
Remove render_document_main_content_partial helper
  • Loading branch information
barmintor committed May 7, 2022
2 parents ed767db + ffc52e2 commit 4d061f0
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 12 deletions.
11 changes: 0 additions & 11 deletions app/helpers/blacklight/catalog_helper_behavior.rb
Original file line number Diff line number Diff line change
Expand Up @@ -147,17 +147,6 @@ 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

##
# Should we display the sort and per page widget?
#
Expand Down
2 changes: 1 addition & 1 deletion app/views/catalog/show.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
</div>
<% end %>
<%= render_document_main_content_partial %>
<%= render 'show_main_content' %>
<% content_for(:sidebar) do %>
<%= render_document_sidebar_partial @document %>
Expand Down

0 comments on commit 4d061f0

Please sign in to comment.