diff --git a/app/components/embed/footer_component.html.erb b/app/components/embed/footer_component.html.erb index 4d944f483..de0a84272 100644 --- a/app/components/embed/footer_component.html.erb +++ b/app/components/embed/footer_component.html.erb @@ -15,8 +15,8 @@ class='sul-embed-footer-tool sul-embed-btn sul-em bed-btn-toolbar sul-embed-btn-default sul-i-download-3' aria-expanded='false' aria-label='open download panel' data-sul-embed-toggle='sul-embed-download-panel'> - <% file_count = viewer.purl_object.downloadable_files.length if viewer.show_download_count? %> - <% if viewer.show_download_count? && file_count > 0 %> + <% file_count = viewer.purl_object.downloadable_files.length %> + <% if file_count > 0 %> <%= file_count %> <% end %> diff --git a/app/viewers/embed/viewer/common_viewer.rb b/app/viewers/embed/viewer/common_viewer.rb index 95693a233..1c2aa68f6 100644 --- a/app/viewers/embed/viewer/common_viewer.rb +++ b/app/viewers/embed/viewer/common_viewer.rb @@ -53,20 +53,10 @@ def show_download? self.class.show_download? && !@request.hide_download? end - # Should the download file count be shown (when download toolbar is shown)? - def show_download_count? - self.class.show_download_count? - end - def self.show_download? false end - # default is to show the download file count (when download toolbar is shown) - def self.show_download_count? - true - end - def tooltip_text(file) return unless file.stanford_only? diff --git a/spec/lib/embed/viewer/common_viewer_spec.rb b/spec/lib/embed/viewer/common_viewer_spec.rb index fe2764eb7..1f8ef6314 100644 --- a/spec/lib/embed/viewer/common_viewer_spec.rb +++ b/spec/lib/embed/viewer/common_viewer_spec.rb @@ -65,15 +65,6 @@ end end - describe '#show_download_count?' do - it 'true for non-image viewers' do - expect(file_viewer).to be_show_download_count - expect(geo_viewer).to be_show_download_count - expect(media_viewer).to be_show_download_count - expect(was_seed_viewer).to be_show_download_count - end - end - describe '#iframe_title' do it 'determines the title from the class name' do expect(file_viewer.iframe_title).to eq 'File viewer'