Merge pull request #24 from Fingertips/manfred-23-support-no-autorotate

Don't autorotate PNG and GIF images

Author: Manfred

Diff for: shard.yml

@@ -1,5 +1,5 @@
 name: greitspitz
-version: 0.2.0
+version: 0.2.1
 license: MIT
 dependencies:
   awscr-s3:

Diff for: spec/cases/transformer_spec.cr

@@ -162,4 +162,12 @@ describe Greitspitz::Transformer do
     image = Vips::Image.new_from_buffer(output)
     image.get_fields.includes?("exif-ifd0-Model").should be_false
   end
+
+  it "transforms a PNG" do
+    image = Spec.transform_image("fit:40x40", extension: ".png")
+    image.width.should eq(40)
+    image.height.should eq(40)
+    content_type = `file --mime-type #{image.filename}`.split(":").last.strip
+    content_type.should eq("image/jpeg")
+  end
 end

Diff for: spec/files/small.png

@@ -75,9 +75,9 @@ module Spec
     end
   end
 
-  def self.transform_image(instructions : String)
+  def self.transform_image(instructions : String, extension = ".jpg")
     temporary_filename = Spec.generate_tmp_filename
-    File.open(Spec.root.join("spec/files/small.jpg"), "rb") do |input|
+    File.open(Spec.root.join("spec/files/small#{extension}"), "rb") do |input|
       File.open(temporary_filename, "wb") do |output|
         Greitspitz::Transformer.new(input, Greitspitz::Instructions.new(instructions)).write(output)
       end

Diff for: spec/spec_helper.cr

@@ -21,7 +21,8 @@ module Greitspitz
     end
 
     def write(output : IO)
-      image = Vips::Image.new_from_buffer(@input, autorotate: true)
+      image = Vips::Image.new_from_buffer(@input)
+      image, _angle, _flip = image.autorot
       image = image.colourspace(Vips::Enums::Interpretation::Srgb)
       @instructions.transformations.each do |name, value|
         case name