Remove carrierwave dependency from Spotlight, relying on ActiveStorage

Author: mejackreed

app/models/spotlight/attachment.rb

@@ -4,11 +4,18 @@ module Spotlight
   ##
   # Sir-trevor image upload attachments
   class Attachment < ActiveRecord::Base
+    # Open to alternatives on how to do this, but this works
+    include Rails.application.routes.url_helpers
+
     belongs_to :exhibit
-    mount_uploader :file, Spotlight::AttachmentUploader
+    has_one_attached :file
 
-    def as_json(options = nil)
-      file.as_json(options).merge(name: name, uid: uid, attachment: to_global_id)
+    def as_json(_options = nil)
+      # as_json has problems with single has_one_attached content
+      # https://github.com/rails/rails/issues/33036
+      {
+        name: name, uid: uid, attachment: to_global_id, url: rails_blob_path(file, only_path: true)
+      }
     end
   end
 end

app/models/spotlight/bulk_update.rb

@@ -2,7 +2,7 @@
 
 module Spotlight
   class BulkUpdate < ActiveRecord::Base
-    mount_uploader :file, Spotlight::BulkUpdatesUploader
+    has_one_attached :file
     belongs_to :exhibit
   end
 end

app/models/spotlight/featured_image.rb

@@ -4,16 +4,16 @@ module Spotlight
   ##
   # Featured images for browse categories, feature pages, and exhibits
   class FeaturedImage < ActiveRecord::Base
-    mount_uploader :image, Spotlight::FeaturedImageUploader
+    has_one_attached :image
 
     before_validation do
       next unless upload_id.present? && source == 'remote'
 
       # copy the image from the temp upload
       temp_image = Spotlight::TemporaryImage.find(upload_id)
-      self.image = CarrierWave::SanitizedFile.new tempfile: StringIO.new(temp_image.image.read),
-                                                  filename: temp_image.image.filename || temp_image.image.identifier,
-                                                  content_type: temp_image.image.content_type
+      image.attach(io: StringIO.new(temp_image.image.read),
+                   filename: temp_image.image.filename || temp_image.image.identifier,
+                   content_type: temp_image.image.content_type)
 
       # Unset the incoming iiif_tilesource, which points at the temp image
       self.iiif_tilesource = nil
@@ -24,13 +24,6 @@ class FeaturedImage < ActiveRecord::Base
       Spotlight::TemporaryImage.find(upload_id).delete if upload_id.present?
     end
 
-    after_save do
-      if image.present?
-        image.cache! unless image.cached?
-        image.store!
-      end
-    end
-
     attr_accessor :upload_id
 
     def iiif_url
@@ -60,7 +53,7 @@ def document
     end
 
     def file_present?
-      image.file.present?
+      image.blob.present?
     end
 
     def iiif_tilesource

app/services/spotlight/carrierwave_file_resolver.rb

@@ -9,10 +9,10 @@ def initialize
     end
 
     def pattern(id)
-      uploaded_file = Spotlight::FeaturedImage.find(id).image.file
+      uploaded_file = Spotlight::FeaturedImage.find(id).image.blob
       raise Riiif::ImageNotFoundError, "unable to find file for #{id}" if uploaded_file.nil?
 
-      uploaded_file.file
+      ActiveStorage::Blob.service.path_for(uploaded_file.key)
     end
   end
 end

app/services/spotlight/exhibit_import_export_service.rb

@@ -186,9 +186,9 @@ def from_hash!(hash)
 
         # dedupe by something??
         ar = exhibit.attachments.build(attr)
-        ar.file = CarrierWave::SanitizedFile.new tempfile: StringIO.new(Base64.decode64(file[:content])),
-                                                 filename: file[:filename],
-                                                 content_type: file[:content_type]
+        ar.file.attach io: StringIO.new(Base64.decode64(file[:content])),
+                       filename: file[:filename],
+                       content_type: file[:content_type]
       end
 
       hash[:languages].each do |attr|
@@ -212,9 +212,9 @@ def deserialize_featured_image(obj, method, data)
       image = obj.public_send("build_#{method}")
       image.update(data)
       if file
-        image.image = CarrierWave::SanitizedFile.new tempfile: StringIO.new(Base64.decode64(file[:content])),
-                                                     filename: file[:filename],
-                                                     content_type: file[:content_type]
+        image.image.attach io: StringIO.new(Base64.decode64(file[:content])),
+                           filename: file[:filename],
+                           content_type: file[:content_type]
         # Unset the iiif_tilesource field as the new image should be different, because
         # the source has been reloaded
         image.iiif_tilesource = nil

app/uploaders/spotlight/attachment_uploader.rb

@@ -28,7 +28,6 @@ these collections.)
   s.add_dependency 'bootstrap_form', '~> 4.1'
   s.add_dependency 'breadcrumbs_on_rails', '>= 3.0', '< 5'
   s.add_dependency 'cancancan'
-  s.add_dependency 'carrierwave', '~> 2.2'
   s.add_dependency 'clipboard-rails', '~> 1.5'
   s.add_dependency 'devise', '~> 4.1'
   s.add_dependency 'devise_invitable'

app/uploaders/spotlight/bulk_updates_uploader.rb

@@ -29,6 +29,11 @@ def add_js
       run 'bundle exec rails webpacker:install'
     end
 
+    def setup_activestorage
+      run 'bundle exec rails active_storage:install'
+      run 'bundle exec rails db:migrate'
+    end
+
     def inject_spotlight_routes
       route "mount Spotlight::Engine, at: 'spotlight'"
       gsub_file 'config/routes.rb', /^\s*root.*/ do |match|