Merge pull request #1091 from ZmagoD/zd_SCI_2254

adds correct preview images to files [fixes SCI-2254]

app/helpers/file_icons_helper.rb

@@ -9,10 +9,14 @@ module FileIconsHelper
     file_ext = asset.file_file_name.split('.').last
     if %w(doc docm docx dot dotm dotx odt rtf).include?(file_ext)
       fa_class = 'fa-file-word'
-    elsif %w(csv ods xls xlsb xlsm xlsx).include?(file_ext)
+    elsif %w(ods xls xlsb xlsm xlsx).include?(file_ext)
       fa_class = 'fa-file-excel'
     elsif %w(odp pot potm potx pps ppsm ppsx ppt pptm pptx).include?(file_ext)
       fa_class = 'fa-file-powerpoint'
+    elsif %w(pdf).include?(file_ext)
+      fa_class = 'fa-file-pdf'
+    elsif %w(txt csv tab tex).include?(file_ext)
+      fa_class = 'fa-file-alt'
     end
 
     # Now check for custom mappings or possible overrides

app/views/my_modules/archive.html.erb

@@ -6,14 +6,13 @@
   <div class="col-xs-3">
     <% i = 0 %>
     <% @archived_results.each do |result| %>
-
       <%= render partial: "my_modules/archive/result.html.erb", locals: { result: result } %>
-
       <% i = i + 1 %>
     <% end %>
-
     <% if i == 0 %>
       <em><%=t "my_modules.module_archive.no_archived_results" %></em>
     <% end %>
   </div>
 </div>
+
+<script>window.initPreviewModal()</script>

app/views/my_modules/archive/_result.html.erb

@@ -9,7 +9,10 @@
       <li>
         <% option_text = t("my_modules.module_archive.option_download") %>
         <% if result.is_asset %>
-          <%= link_to option_text, download_asset_path(result.asset), data: {no_turbolink: true}  %>
+          <%= link_to t('protocols.index.archive_results.preview'), download_asset_path(result.asset),
+              class: 'file-preview-link',
+              id: "modal_link#{result.asset.id}",
+              data: { no_turbolink: true, id: true, status: 'asset-present', 'preview-url': asset_file_preview_path(result.asset) } %>
         <% elsif result.is_text %>
           <%= link_to option_text, result_text_download_path(result.result_text_id), data: {no_turbolink: true} %>
         <% elsif result.is_table %>

config/locales/en.yml

@@ -1655,6 +1655,7 @@ en:
         row_success: "Archived"
         row_renamed: "Archived & renamed"
         row_failed: "Failed"
+        preview: "View"
       restore_results:
         title: "Restore results"
         message_failed: "Failed to restore %{nr} protocols."