Merge pull request #8 from 155pod/bw/artwork-action

Don't process images all at once

app/controllers/covers_controller.rb

@@ -3,6 +3,19 @@ class CoversController < ApplicationController
 
   before_action :password_required, only: [:index, :archive, :archive_all, :update_order]
 
+  def artwork
+    cover = Cover.find(params[:id])
+
+    redirect_to_admin unless cover.artwork.attached?
+
+    @artwork =
+      if cover.artwork.variable?
+        cover.artwork.variant(resize_to_limit: [1440,1440]).processed.url
+      else
+        rails_blob_path(cover.artwork, disposition: "inline")
+      end
+  end
+
   def new
     @cover = Cover.new
   end

app/views/covers/_list.html.erb

@@ -44,11 +44,9 @@
     <b>Artwork:</b>
     <%= link_to(
       cover.artwork.blob.filename.sanitized,
-      cover.artwork.variable? ?
-        cover.artwork.variant(resize_to_limit: [1440,1440]).processed.url :
-          rails_blob_path(cover.artwork, disposition: "inline"),
-      target: "_blank"
-    ) %><br />
+      artwork_cover_path(cover), target: "_blank"
+    ) %>
+    <br />
   <% end %>
 
   <% if cover.blurb.present? %>

app/views/covers/artwork.html.erb

@@ -0,0 +1 @@
+<%= image_tag @artwork, class: "d-block w-100" %>

config/routes.rb

@@ -13,6 +13,8 @@
       post :archive
       post :unarchive
       post :toggle_b_side
+
+      get :artwork
     end
     collection do
       post :archive_all