Refactor the metadata.rb
from ActiveStorage::Analyzer::ImageAnalyzer::ImageMagick/Vips.new(blob).metadata
#254
Labels
metadata.rb
from ActiveStorage::Analyzer::ImageAnalyzer::ImageMagick/Vips.new(blob).metadata
#254
Reading Rails documentation to implement a feature for this gem, I came across the following ActiveStorage class: https://github.com/rails/rails/blob/55c4adeb36eff229972eecbb53723c1b80393091/activestorage/lib/active_storage/analyzer/image_analyzer.rb
It looks like it both supports
MiniMagick
andVips
since June 2021. Themetadata.rb
file however was built a few years before maybe explaining its presence?We could rely on
ActiveStorage::Analyzer::ImageAnalyzer
s to do this job, therefore decreasing the weight + associated unnecessary complexity of this gem?UPDATE : We can't rely on
ActiveStorage::Analyzer::ImageAnalyzer
s to do this job because it needs a saved blob, ie with its io uploaded. We only have not uploaded attachables at this point, but we can copy their structure to refactor metadata.rb (which could be a very nice addition)@igorkasyanchuk @gr8bit what do you think of it?
The text was updated successfully, but these errors were encountered: