Add download link to spree admin

app/controllers/spree/admin/digitals_controller.rb

@@ -1,6 +1,8 @@
 module Spree
   module Admin
     class DigitalsController < ResourceController
+      include Downloadable
+
       belongs_to "spree/product", :find_by => :slug
 
       def create
@@ -16,6 +18,10 @@ def create
         end
       end
 
+      def show
+        send_file attachment.path, filename: attachment.original_filename, type: attachment.content_type, status: :ok and return
+      end
+
       protected
         def location_after_save
           spree.admin_product_digitals_path(@product)
@@ -28,6 +34,16 @@ def permitted_resource_params
       def permitted_digital_attributes
         [:variant_id, :attachment]
       end
+
+      private
+
+        def attachment
+          @attachment = Digital.find(params[:id]).attachment
+        end
+
+        def authorized?
+          true
+        end
     end
   end
-end
+end

app/controllers/spree/digitals_controller.rb

@@ -1,41 +1,23 @@
 module Spree
   class DigitalsController < Spree::StoreController
-    rescue_from ActiveRecord::RecordNotFound, with: :render_404
-
-    def show
-      if attachment.present?
-        # don't authorize the link unless the file exists
-        # the logger error will help track down customer issues easier
-        if attachment_is_file?
-          if digital_link.authorize!
-            if Paperclip::Attachment.default_options[:storage] == :s3
-              redirect_to attachment.expiring_url(Spree::DigitalConfiguration[:s3_expiration_seconds]) and return
-            else
-              send_file attachment.path, filename: attachment.original_filename, type: attachment.content_type, status: :ok and return
-            end
-          end
-        else
-          Rails.logger.error "Missing Digital Item: #{attachment.path}"
-        end
-      end
+    include Downloadable
 
-      render :unauthorized
-    end
+    rescue_from ActiveRecord::RecordNotFound, with: :render_404
 
     private
 
-      def attachment_is_file?
-        if Paperclip::Attachment.default_options[:storage] == :s3
-          attachment.exists?
-        else
-          File.file?(attachment.path)
-        end
-      end
-
       def digital_link
         @link ||= DigitalLink.find_by!(secret: params[:secret])
       end
 
+      def authorized?
+        digital_link.authorize!
+      end
+
+      def unauthorized_action
+        render :unauthorized
+      end
+
       def attachment
         @attachment ||= digital_link.digital.try(:attachment) if digital_link.present?
       end

app/controllers/spree/downloadable.rb

@@ -0,0 +1,36 @@
+module Spree
+  module Downloadable
+    def show
+      if attachment.present?
+        # don't authorize the link unless the file exists
+        # the logger error will help track down customer issues easier
+        if attachment_is_file?
+          if authorized?
+            if Paperclip::Attachment.default_options[:storage] == :s3
+              redirect_to attachment.expiring_url(Spree::DigitalConfiguration[:s3_expiration_seconds]) and return
+            else
+              send_file attachment.path, filename: attachment.original_filename, type: attachment.content_type, status: :ok and return
+            end
+          end
+        else
+          Rails.logger.error "Missing Digital Item: #{attachment.path}"
+        end
+      end
+
+      unauthorized_action
+    end
+
+    private
+
+      def unauthorized_action
+      end
+
+      def attachment_is_file?
+        if Paperclip::Attachment.default_options[:storage] == :s3
+          attachment.exists?
+        else
+          File.file?(attachment.path)
+        end
+      end
+  end
+end

app/views/spree/admin/digitals/_form.html.erb

@@ -14,7 +14,11 @@
           <tbody>
           <% variant.digitals.each do |digital| %>
             <tr>
-              <td><%= render digital %></td>
+              <td>
+		<%= link_to admin_product_digital_path(@product, digital) do %>
+		  <%= render digital %>
+		<% end %>
+	      </td>
               <td class="actions text-right">
                 <%#= link_to_with_icon 'delete', Spree.t(:delete_file, scope: 'digitals'), admin_product_digital_url(@product, digital), data: {confirm: Spree.t(:delete_file_confirmation, scope: 'digitals', filename: digital.attachment_file_name)}, method: :delete %>
                 <%= link_to_with_icon 'delete', Spree.t(:delete_file, scope: 'digitals'), admin_product_digital_url(@product, digital), data: {confirm: Spree.t(:delete_file_confirmation, scope: 'digitals', filename: digital.attachment_file_name)}, method: :delete, class: 'btn btn-danger btn-sm delete-resource' %>

spec/controllers/admin/digitals_controller_spec.rb

@@ -68,6 +68,19 @@
     end
   end
 
+  context '#show' do
+    let(:digital) { create(:digital) }
+    let!(:variant) { create(:variant, product: product, digitals: [digital]) }
+
+    it 'calls send_file' do
+      expect(controller).to receive(:send_file).with(digital.attachment.path,
+                                                 filename: digital.attachment.original_filename,
+                                                 type: digital.attachment.content_type, status: :ok){controller.head :ok,
+                                                                         content_type: digital.attachment.content_type, status: :ok }
+      spree_get :show, product_id: product.slug, id: digital.id
+    end
+  end
+
   context '#destroy' do
     let(:digital) { create(:digital) }
     let!(:variant) { create(:variant, product: product, digitals: [digital]) }