kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

require mini_magick at the level it is being used

Browse source
This commit is contained in:
Joel Cogen 2022-06-21 10:25:42 -03:00
parent b456656482
commit 95322be0ed
1 changed files with 9 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
activestorage/lib/active_storage/analyzer/image_analyzer/image_magick.rb
View file

@ -10,9 +10,14 @@ module ActiveStorage
private
def read_image
download_blob_to_tempfile do |file|
begin
require "mini_magick"
rescue LoadError
logger.info "Skipping image analysis because the mini_magick gem isn't installed"
return {}
end
download_blob_to_tempfile do |file|
image = instrument("mini_magick") do
MiniMagick::Image.new(file.path)
end
@ -23,13 +28,10 @@ module ActiveStorage
logger.info "Skipping image analysis because ImageMagick doesn't support the file"
{}
end
rescue MiniMagick::Error => error
logger.error "Skipping image analysis due to an ImageMagick error: #{error.message}"
{}
end
rescue LoadError
logger.info "Skipping image analysis because the mini_magick gem isn't installed"
{}
rescue MiniMagick::Error => error
logger.error "Skipping image analysis due to an ImageMagick error: #{error.message}"
{}
end
def rotated_image?(image)