From 24bcef23c637ed16db8481bd21b0078736016ceb Mon Sep 17 00:00:00 2001
From: Eric <eric@commitchange.com>
Date: Fri, 29 May 2020 16:54:53 -0500
Subject: [PATCH] Migrate from imagemagick to vips for image transformation

---
 Gemfile                                            | 1 -
 Gemfile.lock                                       | 1 -
 app/models/concerns/image/attachment_extensions.rb | 2 +-
 config/application.rb                              | 2 ++
 4 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/Gemfile b/Gemfile
index 612c0cc6..08e371ae 100755
--- a/Gemfile
+++ b/Gemfile
@@ -52,7 +52,6 @@ gem 'param_validation', path: 'gems/ruby-param-validation'
 gem 'qx', path: 'gems/ruby-qx'
 
 # Images
-gem 'mini_magick', '~> 4.10.1'
 gem 'image_processing', '~> 1.10.3'
 
 # User authentication
diff --git a/Gemfile.lock b/Gemfile.lock
index 1a850739..a51e8511 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -486,7 +486,6 @@ DEPENDENCIES
   listen
   lograge (~> 0.11.2)
   mail_view (~> 2.0)
-  mini_magick (~> 4.10.1)
   nearest_time_zone (~> 0.0.4)
   parallel (~> 1.17)
   parallel_tests (~> 2.32)
diff --git a/app/models/concerns/image/attachment_extensions.rb b/app/models/concerns/image/attachment_extensions.rb
index 4d10c0d4..296c29c0 100644
--- a/app/models/concerns/image/attachment_extensions.rb
+++ b/app/models/concerns/image/attachment_extensions.rb
@@ -27,7 +27,7 @@ module Image::AttachmentExtensions
                     #{sizes.map do |k,v| 
                         <<-INNER
                         when :#{k.to_sym}
-                            return #{attribute}.variant(resize: "#{v[0]}x#{v[1]}")
+                            return #{attribute}.variant(resize_to_limit: [#{v[0]}, #{v[1]}])
                         INNER
                     end.join("\n")}
                      else
diff --git a/config/application.rb b/config/application.rb
index ee4c5499..c8af71bd 100755
--- a/config/application.rb
+++ b/config/application.rb
@@ -108,5 +108,7 @@ module Commitchange
 
     # we don't require belongs_to associations to be required for historical reasons.
     config.active_record.belongs_to_required_by_default = false
+
+    config.active_storage.variant_processor = :vips
   end
 end