no empty sidebar, codesnippets only when dataset

sul-dlss · Sep 4, 2024 · 0df3b3c · 0df3b3c
1 parent b87598b
commit 0df3b3c
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 10 deletions.
diff --git a/app/components/document/sidebar_component.html.erb b/app/components/document/sidebar_component.html.erb
@@ -1,12 +1,14 @@
-<div class="bg-fog-light component-container rounded">
-  <% filtered_components = components.map{|component | render component }.uniq.compact_blank %>
-  <%= filtered_components.join('<hr style="border-top: dotted 1px;" />').html_safe if filtered_components.any? %>
-  <% tools = render_show_tools %>
-  <% if filtered_components.any? && tools.present? %>
-    <hr style="border-top: dotted 1px;" />
-  <% end %>
-  <%= tools %>
-</div>
+<% filtered_components = components.map{|component | render component }.uniq.compact_blank %>
+<% tools = render_show_tools %>
+<% if filtered_components.any? || tools.present?  %>
+  <div class="bg-fog-light component-container rounded">
+    <%= filtered_components.join('<hr style="border-top: dotted 1px;" />').html_safe if filtered_components.any? %>
+    <% if filtered_components.any? && tools.present? %>
+      <hr style="border-top: dotted 1px;" />
+    <% end %>
+    <%= tools %>
+  </div>
+<% end %>
 
 <%= render AlsoAvailableComponent.new(document:) %>
 

diff --git a/app/controllers/catalog_controller.rb b/app/controllers/catalog_controller.rb
@@ -333,7 +333,7 @@ class CatalogController < ApplicationController
                                                  }
     config.add_show_tools_partial :code_snippet_link, component: CodeSnippetLinkComponent,
                                                       if: proc { |_context, _config, options|
-                                                        options[:document] &&
+                                                        options[:document]&.dataset? &&
                                                           !options[:document].restricted?
                                                       }
     config.show.document_actions.delete(:sms)

diff --git a/app/models/solr_document.rb b/app/models/solr_document.rb
@@ -12,6 +12,10 @@ def institution
     fetch(Settings.FIELDS.PROVIDER, '')
   end
 
+  def dataset?
+    fetch(Settings.FIELDS.RESOURCE_CLASS, []).include?('Datasets')
+  end
+
   def contact_email
     Settings.INSTITUTIONS.send(institution)&.email
   end