Add support for combined MiniMagick transformations

This commit is contained in:
Robert Glaser 2017-12-22 16:44:34 +01:00
parent 58ff921c1d
commit 7a3a991f6d
2 changed files with 30 additions and 6 deletions

View File

@ -46,13 +46,15 @@ class ActiveStorage::Variation
# Accepts an open MiniMagick image instance, like what's returned by <tt>MiniMagick::Image.read(io)</tt>,
# and performs the +transformations+ against it. The transformed image instance is then returned.
def transform(image)
transformations.each do |method, argument|
image.mogrify do |command|
if eligible_argument?(argument)
command.public_send(method, argument)
else
command.public_send(method)
transformations.each do |(method, argument)|
if method.to_s == "combine_options"
image.combine_options do |combination|
argument.each do |(method, argument)|
pass_transform_argument(combination, method, argument)
end
end
else
pass_transform_argument(image, method, argument)
end
end
end
@ -66,4 +68,12 @@ class ActiveStorage::Variation
def eligible_argument?(argument)
argument.present? && argument != true
end
def pass_transform_argument(instance, method, argument)
if eligible_argument?(argument)
instance.public_send(method, argument)
else
instance.public_send(method)
end
end
end

View File

@ -25,6 +25,20 @@ class ActiveStorage::VariantTest < ActiveSupport::TestCase
assert_match(/Gray/, image.colorspace)
end
test "center-weighted crop of JPEG blob" do
blob = create_file_blob(filename: "racecar.jpg")
variant = blob.variant(combine_options: {
gravity: "center",
resize: "100x100^",
crop: "100x100+0+0",
}).processed
assert_match(/racecar\.jpg/, variant.service_url)
image = read_image(variant)
assert_equal 100, image.width
assert_equal 100, image.height
end
test "resized variation of PSD blob" do
blob = create_file_blob(filename: "icon.psd", content_type: "image/vnd.adobe.photoshop")
variant = blob.variant(resize: "20x20").processed