riiif 1.3.0 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6b8dc5c1ffd5532b680b26ff6958a02261700d3e
-  data.tar.gz: 7719e41b21ee91d52eefe4cd5347fb87db25b47c
+  metadata.gz: 0aaf0ae50831c1b402852fee520e6a4a0c9b8760
+  data.tar.gz: 141b672ec9ce926ae80d139febb926a82d607cc2
 SHA512:
-  metadata.gz: 453bd022c4fb48291f987625db7c22d6709a7997bb26d9123da975b6ab7537776d847a7d618369788e3e756e08770a99aa29496126540a75a9ed1516b0666814
-  data.tar.gz: 1b96e14965119ee2454f3a44a144b572643af01a00616eff0e5b76fd1a604b1b58ed6f2185d02c33ba811cd8975baa9c8f68ab9c09c2a4139fe4d8e520810342
+  metadata.gz: 3729b7972bea7cfea2d4f1c21b75ad3c974b54c2307a1ab63e297a7b06cf5e5dc9efb9bc919a6c8b5c3e7d04dc9030866e10e2c0fc368dc8a2b1bc4518dd0b4f
+  data.tar.gz: ee0acb8006038a89caceb49cd4216ce08d3f65136ca38f5aff3ed2391bd0fb17ced7a48075e7f65565d3d0b600f199c004d2ee0e027b0ad80f3ff9ba48ecf338

data/app/services/riiif/command_runner.rb

@@ -16,7 +16,7 @@ module Riiif
         stdout.close
         err = stderr.read
         stderr.close
-        raise "Unable to execute command \"#{command}\"\n#{err}" unless wait_thr.value.success?
+        raise ConversionError, "Unable to execute command \"#{command}\"\n#{err}" unless wait_thr.value.success?
       end
       out
     end

data/app/services/riiif/imagemagick_command_factory.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Riiif
   # Builds a command to run a transformation using Imagemagick
   class ImagemagickCommandFactory
@@ -5,48 +7,84 @@ module Riiif
     # @param [String] path the location of the file
     # @param [Transformation] transformation
     # @param [Integer] compression (85) the compression level to use (set 0 for no compression)
+    # @param [String] sampling_factor ("4:2:0") the chroma sample factor (set 0 for no compression)
+    # @param [Boolean] strip_metadata (true) do we want to strip EXIF tags?
     # @return [String] a command for running imagemagick to produce the requested output
-    def self.build(path, transformation, compression: 85)
-      new(path, transformation, compression: compression).build
+    def self.build(path, transformation, compression: 85, sampling_factor: '4:2:0', strip_metadata: true)
+      new(path, transformation,
+          compression: compression,
+          sampling_factor: sampling_factor,
+          strip_metadata: strip_metadata).build
     end
 
     # A helper method to instantiate and invoke build
     # @param [String] path the location of the file
     # @param [Transformation] transformation
     # @param [Integer] compression the compression level to use (set 0 for no compression)
-    def initialize(path, transformation, compression:)
+    def initialize(path, transformation, compression:, sampling_factor:, strip_metadata:)
       @path = path
       @transformation = transformation
       @compression = compression
+      @sampling_factor = sampling_factor
+      @strip_metadata = strip_metadata
     end
 
-    attr_reader :path, :transformation, :compression
+    attr_reader :path, :transformation, :compression, :sampling_factor, :strip_metadata
 
     # @return [String] a command for running imagemagick to produce the requested output
     def build
-      command = 'convert'
-      command << " -crop #{transformation.crop}" if transformation.crop
-      command << " -resize #{transformation.size}" if transformation.size
-      if transformation.rotation
-        command << " -virtual-pixel white +distort srt #{transformation.rotation}"
-      end
-
-      case transformation.quality
-      when 'grey'
-        command << ' -colorspace Gray'
-      when 'bitonal'
-        command << ' -colorspace Gray'
-        command << ' -type Bilevel'
-      end
-      command << " -quality #{compression}" if use_compression?
-      command << " #{path} #{transformation.format}:-"
-      command
+      [command, crop, size, rotation, colorspace, quality, sampling, metadata, output].join
     end
 
     private
 
+      def command
+        'convert'
+      end
+
       def use_compression?
-        compression > 0 && transformation.format == 'jpg'
+        compression > 0 && jpeg?
+      end
+
+      def jpeg?
+        transformation.format == 'jpg'.freeze
+      end
+
+      def output
+        " #{path} #{transformation.format}:-"
+      end
+
+      def crop
+        " -crop #{transformation.crop}" if transformation.crop
+      end
+
+      def size
+        " -resize #{transformation.size}" if transformation.size
+      end
+
+      def rotation
+        " -virtual-pixel white +distort srt #{transformation.rotation}" if transformation.rotation
+      end
+
+      def quality
+        " -quality #{compression}" if use_compression?
+      end
+
+      def metadata
+        ' -strip' if strip_metadata
+      end
+
+      def sampling
+        " -sampling-factor #{sampling_factor}" if jpeg?
+      end
+
+      def colorspace
+        case transformation.quality
+        when 'grey'
+          ' -colorspace Gray'
+        when 'bitonal'
+          ' -colorspace Gray -type Bilevel'
+        end
       end
   end
 end

data/lib/riiif/version.rb

@@ -1,3 +1,3 @@
 module Riiif
-  VERSION = '1.3.0'.freeze
+  VERSION = '1.4.0'.freeze
 end

data/lib/riiif.rb

@@ -1,6 +1,5 @@
 require 'riiif/version'
 require 'riiif/engine'
-
 module Riiif
   extend ActiveSupport::Autoload
   autoload :Image
@@ -19,6 +18,8 @@ module Riiif
       @original_exception = orig
     end
   end
+  # This error is raised when Riiif can't convert an image
+  class ConversionError < Error; end
 
   Transformation = Struct.new(:crop, :size, :quality, :rotation, :format)
   ImageInformation = Struct.new(:width, :height)

data/spec/models/riiif/image_spec.rb

@@ -6,7 +6,9 @@ RSpec.describe Riiif::Image do
   subject { described_class.new('world') }
   describe 'happy path' do
     before do
-      expect(subject.image).to receive(:execute).with("convert -quality 85 #{filename} jpg:-").and_return('imagedata')
+      expect(subject.image).to receive(:execute)
+        .with("convert -quality 85 -sampling-factor 4:2:0 -strip #{filename} jpg:-")
+        .and_return('imagedata')
     end
     it 'renders' do
       expect(subject.render('size' => 'full', format: 'jpg')).to eq 'imagedata'
@@ -63,25 +65,25 @@ RSpec.describe Riiif::Image do
   describe 'mogrify' do
     describe 'region' do
       it 'returns the original when specifing full size' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -strip #{filename} png:-")
         subject.render(region: 'full', format: 'png')
       end
       it 'handles absolute geometry' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert -crop 60x75+80+15 #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -crop 60x75+80+15 -strip #{filename} png:-")
         subject.render(region: '80,15,60,75', format: 'png')
       end
 
       it 'handles percent geometry' do
         expect(Riiif::CommandRunner).to receive(:execute)
           .with("identify -format %hx%w #{filename}").and_return('131x175')
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert -crop 80%x70+18+13 #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -crop 80%x70+18+13 -strip #{filename} png:-")
         subject.render(region: 'pct:10,10,80,70', format: 'png')
       end
 
       it 'handles square geometry' do
         expect(Riiif::CommandRunner).to receive(:execute)
           .with("identify -format %hx%w #{filename}").and_return('131x175')
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert -crop 131x131+22+0 #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -crop 131x131+22+0 -strip #{filename} png:-")
         subject.render(region: 'square', format: 'png')
       end
       it 'raises an error for invalid geometry' do
@@ -91,31 +93,31 @@ RSpec.describe Riiif::Image do
 
     describe 'resize' do
       it 'returns the original when specifing full size' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -strip #{filename} png:-")
         subject.render(size: 'full', format: 'png')
       end
       it 'handles integer percent sizes' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize 50% #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize 50% -strip #{filename} png:-")
         subject.render(size: 'pct:50', format: 'png')
       end
       it 'handles float percent sizes' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize 12.5% #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize 12.5% -strip #{filename} png:-")
         subject.render(size: 'pct:12.5', format: 'png')
       end
       it 'handles w,' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize 50 #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize 50 -strip #{filename} png:-")
         subject.render(size: '50,', format: 'png')
       end
       it 'handles ,h' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize x50 #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize x50 -strip #{filename} png:-")
         subject.render(size: ',50', format: 'png')
       end
       it 'handles w,h' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize 150x75! #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize 150x75! -strip #{filename} png:-")
         subject.render(size: '150,75', format: 'png')
       end
       it 'handles bestfit (!w,h)' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize 150x75 #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -resize 150x75 -strip #{filename} png:-")
         subject.render(size: '!150,75', format: 'png')
       end
       it 'raises an error for invalid size' do
@@ -125,12 +127,12 @@ RSpec.describe Riiif::Image do
 
     describe 'rotate' do
       it 'returns the original when specifing full size' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -strip #{filename} png:-")
         subject.render(rotation: '0', format: 'png')
       end
       it 'handles floats' do
         expect(Riiif::CommandRunner).to receive(:execute)
-          .with("convert -virtual-pixel white +distort srt 22.5 #{filename} png:-")
+          .with("convert -virtual-pixel white +distort srt 22.5 -strip #{filename} png:-")
         subject.render(rotation: '22.5', format: 'png')
       end
       it 'raises an error for invalid angle' do
@@ -140,20 +142,20 @@ RSpec.describe Riiif::Image do
 
     describe 'quality' do
       it 'returns the original when specifing default' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -strip #{filename} png:-")
         subject.render(quality: 'default', format: 'png')
       end
       it 'returns the original when specifing color' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -strip #{filename} png:-")
         subject.render(quality: 'color', format: 'png')
       end
       it 'converts to grayscale' do
-        expect(Riiif::CommandRunner).to receive(:execute).with("convert -colorspace Gray #{filename} png:-")
+        expect(Riiif::CommandRunner).to receive(:execute).with("convert -colorspace Gray -strip #{filename} png:-")
         subject.render(quality: 'grey', format: 'png')
       end
       it 'converts to bitonal' do
         expect(Riiif::CommandRunner).to receive(:execute)
-          .with("convert -colorspace Gray -type Bilevel #{filename} png:-")
+          .with("convert -colorspace Gray -type Bilevel -strip #{filename} png:-")
         subject.render(quality: 'bitonal', format: 'png')
       end
       it 'raises an error for invalid angle' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: riiif
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.4.0
 platform: ruby
 authors:
 - Justin Coyne
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-10 00:00:00.000000000 Z
+date: 2017-04-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties