Fixes #3164 - Collection Thumbnails don't show in search results.

samvera · Sep 24, 2018 · c6c3f26 · c6c3f26
1 parent 6e1970b
commit c6c3f26
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/app/views/catalog/_thumbnail_list_collection.html.erb b/app/views/catalog/_thumbnail_list_collection.html.erb
@@ -1,3 +1,9 @@
 <div class="col-md-2">
-  <span class="<%= Hyrax::ModelIcon.css_class_for(Collection) %> collection-icon-search" aria-hidden="true"></span>
+  <% collection_presenter = Hyrax::CollectionPresenter.new(document, current_ability) %>
+  <% if (collection_presenter.thumbnail_path == nil) %>
+    <span class="<%= Hyrax::ModelIcon.css_class_for(Collection) %> collection-icon-search" aria-hidden="true"></span>
+  <% else %>
+    <%= image_tag(collection_presenter.thumbnail_path) %>
+  <% end %>
 </div>
+
diff --git a/spec/features/search_spec.rb b/spec/features/search_spec.rb
@@ -53,7 +53,7 @@
       expect(page).to have_content('Search Results')
       expect(page).to have_content "Toothbrush"
       expect(page).to have_content('collection title abc')
-      expect(page).to have_css("span.collection-icon-search")
+      expect(page).to have_selector("//img")
 
       expect(page.body).to include "<span itemprop=\"keywords\"><a href=\"/catalog?f%5Bkeyword_sim%5D%5B%5D=taco&amp;locale=en\">taco</a></span>"
       expect(page.body).to include "<span itemprop=\"keywords\"><a href=\"/catalog?f%5Bkeyword_sim%5D%5B%5D=mustache&amp;locale=en\">mustache</a></span>"