[Analyzer] Do not require dependency if already defined

igorkasyanchuk · Dec 11, 2024 · 214e74e · 214e74e
1 parent 96033ed
commit 214e74e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/active_storage_validations/analyzer/image_analyzer/image_magick.rb b/lib/active_storage_validations/analyzer/image_analyzer/image_magick.rb
@@ -10,7 +10,7 @@ class Analyzer::ImageAnalyzer::ImageMagick < Analyzer::ImageAnalyzer
 
     def read_image
       begin
-        require "mini_magick"
+        require "mini_magick" unless defined?(MiniMagick)
       rescue LoadError
         logger.info "Skipping image analysis because the mini_magick gem isn't installed"
         return {}

diff --git a/lib/active_storage_validations/analyzer/image_analyzer/vips.rb b/lib/active_storage_validations/analyzer/image_analyzer/vips.rb
@@ -9,7 +9,7 @@ class Analyzer::ImageAnalyzer::Vips < Analyzer::ImageAnalyzer
 
     def read_image
       begin
-        require "ruby-vips"
+        require "vips" unless defined?(::Vips)
       rescue LoadError
         logger.info "Skipping image analysis because the ruby-vips gem isn't installed"
         return {}