stream publication files until we resolve that differently

app/admin/news_article.rb

@@ -66,7 +66,7 @@
       f.input :tpi_sector_ids, label: 'Sectors', as: :select,
                                collection: TPISector.order(:name), input_html: {multiple: true}
       f.input :keywords_string, label: 'Keywords', hint: t('hint.tag'), as: :tags, collection: Keyword.pluck(:name)
-      f.input :image, as: :file
+      f.input :image, as: :file, input_html: {accept: 'image/*'}
     end
 
     f.actions

app/admin/publication.rb

@@ -58,8 +58,13 @@
       f.input :tpi_sector_ids, label: 'Sectors', as: :select,
                                collection: TPISector.order(:name), input_html: {multiple: true}
       f.input :keywords_string, label: 'Keywords', hint: t('hint.tag'), as: :tags, collection: Keyword.pluck(:name)
-      f.input :file, as: :file
-      f.input :image, as: :file
+      f.input :file,
+              as: :file,
+              hint: preview_file_tag(f.object.file),
+              input_html: {
+                accept: 'application/pdf, application/vnd.ms-powerpoint, .ppt, .pptx, application/msword, .docx'
+              }
+      f.input :image, as: :file, hint: preview_file_tag(f.object.image), input_html: {accept: 'image/*'}
     end
 
     f.actions

app/controllers/tpi/publications_controller.rb

@@ -1,7 +1,10 @@
 module TPI
   class PublicationsController < TPIController
+    include ActionController::Live
+
     before_action :fetch_tags, only: [:index]
     before_action :fetch_sectors, only: [:index]
+    before_action :fetch_publication, only: [:show]
 
     def index
       @publications_and_articles = Queries::TPI::NewsPublicationsQuery
@@ -17,25 +20,41 @@ def partial
     end
 
     def show
-      @publication = params[:type].eql?('NewsArticle') ? NewsArticle.find(params[:id]) : Publication.find(params[:id])
       respond_to do |format|
-        format.html
-        format.pdf { redirect_to rails_blob_url(@publication.file, disposition: 'preview') }
+        format.html do
+          admin_panel_path = polymorphic_path([:admin, @publication])
+          fixed_navbar("#{@publication.class.name.underscore.humanize} #{@publication.title}", admin_panel_path)
+
+          redirect_to '' unless @publication
+        end
+        format.pdf { stream_publication_file }
       end
+    end
 
-      admin_panel_path = polymorphic_path([:admin, @publication])
+    private
 
-      fixed_navbar("#{@publication.class.name.underscore.humanize} #{@publication.title}", admin_panel_path)
+    def stream_publication_file
+      send_file_headers!(
+        type: @publication.file.content_type,
+        disposition: 'inline',
+        filename: @publication.file.filename.to_s
+      )
 
-      redirect_to '' unless @publication
+      @publication.file.download do |chunk|
+        response.stream.write(chunk)
+      end
+    ensure
+      response.stream.close
     end
 
-    private
-
     def filter_params
       params.permit(:tags, :sectors)
     end
 
+    def fetch_publication
+      @publication = params[:type].eql?('NewsArticle') ? NewsArticle.find(params[:id]) : Publication.find(params[:id])
+    end
+
     def fetch_tags
       @tags = Keyword
         .joins(:taggings)

app/views/admin/pages/_image_fields.html.erb

@@ -2,7 +2,7 @@
 
 <%= content_tag :div, class: 'nested-fields inline-fields', data: { controller: 'image', new_record: is_new_record } do %>
   <% if is_new_record || !form.object.logo.present? %>
-    <%= form.input :logo, as: :file %>
+    <%= form.input :logo, as: :file, input_html: {accept: 'image/*'} %>
   <% else %>
     <%= image_tag form.object.logo, style: 'height: 100px; width: 100px;' %>
   <% end %>