dynamic_image 2.0.20 → 2.0.25

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f0a95aa4e97d054b1bd7d757c14f0a98790061806a930b6a2c8bc9294d85d00a
-  data.tar.gz: dbe583ffcfe362d0406c0b272bc1f7a53d625be553d0e804baa5bf6753100cae
+  metadata.gz: 20b1cfa9c326762d6b3d1bca63b4d8d65516cc266abd03f55328b4cca8086dc3
+  data.tar.gz: 4a0496d6507534664dc8e39bc59358a2607df6aaaa966ee0eda3990f72cae8bf
 SHA512:
-  metadata.gz: 681e4621406b2e6c4eae033dcba0ca1ea085c3429307a29e897eed62cbf22f2810183107142aae0432c351e2a8a47f5a25aa7ca901afe4e908109cf6126d54d3
-  data.tar.gz: a7e5975010f4eb0f3dfaa3bf9185ea5a8e4d134f6f963d353da7d7dca0c4894094aee2a8557083bf1870ef9770fc26e3a5e58346fe55742f0af07f73e691f612
+  metadata.gz: c04933c776c41870142473869f983fd0d8d1a0b98602abf41b3fb5392f6d6ef8c4cb2f77c875044f32cbc7903d1dd9de06c644fcfc73c913bd414583ee27dfae
+  data.tar.gz: 79ad9d9b59398d56765878253170eae968d2d8320c31b992fd9758ff2dd00b8131db1fae2746f6af359eeb8b9934dfe4020b30cd91602f1b6872302c95b2d1d6

data/README.md

@@ -1,5 +1,5 @@
 [![Version](https://img.shields.io/gem/v/dynamic_image.svg?style=flat)](https://rubygems.org/gems/dynamic_image)
-[![Build Status](https://travis-ci.org/elektronaut/dynamic_image.svg?branch=master)](https://travis-ci.org/elektronaut/dynamic_image)
+![Build](https://github.com/elektronaut/dynamic_image/workflows/Build/badge.svg)
 [![Code Climate](https://codeclimate.com/github/elektronaut/dynamic_image/badges/gpa.svg)](https://codeclimate.com/github/elektronaut/dynamic_image)
 [![Code Climate](https://codeclimate.com/github/elektronaut/dynamic_image/badges/coverage.svg)](https://codeclimate.com/github/elektronaut/dynamic_image)
 [![Inline docs](http://inch-ci.org/github/elektronaut/dynamic_image.svg)](http://inch-ci.org/github/elektronaut/dynamic_image)

data/lib/dynamic_image.rb

@@ -12,11 +12,12 @@ require "dynamic_image/errors"
 require "dynamic_image/helper"
 require "dynamic_image/image_reader"
 require "dynamic_image/image_sizing"
+require "dynamic_image/jobs"
 require "dynamic_image/metadata"
 require "dynamic_image/model"
 require "dynamic_image/processed_image"
 require "dynamic_image/routing"
 
 module DynamicImage
-  cattr_accessor :digest_verifier
+  cattr_accessor :digest_verifier, :process_later_limit
 end

data/lib/dynamic_image/belongs_to.rb

@@ -8,7 +8,7 @@ module DynamicImage
 
     module ClassMethods
       def belongs_to_image(name, scope = nil, **options)
-        belongs_to(name, scope, options)
+        belongs_to(name, scope, **options)
 
         define_method "#{name}=" do |new_image|
           if new_image.present? && !new_image.is_a?(DynamicImage::Model)

data/lib/dynamic_image/controller.rb

@@ -44,13 +44,21 @@ module DynamicImage
     private
 
     def cache_expiration_header
-      expires_in 30.days, public: true
+      expires_in 30.days, public: true if response.status == 200
     end
 
     def find_record
       @record = model.find(params[:id])
     end
 
+    def process_later?(processed_image, size)
+      return false unless DynamicImage.process_later_limit
+
+      image_size = processed_image.record.size.x * processed_image.record.size.y
+      image_size > DynamicImage.process_later_limit &&
+        !processed_image.find_variant(size)
+    end
+
     def render_image(options)
       return unless stale?(@record)
 
@@ -59,8 +67,8 @@ module DynamicImage
           render(file: File.join(File.dirname(__FILE__), "templates/show"),
                  layout: false, locals: { options: options })
         end
-        format.any(:gif, :jpeg, :png, :tiff, :webp) do
-          send_image(DynamicImage::ProcessedImage.new(@record, options))
+        format.any(:gif, :jpeg, :jpg, :png, :tiff, :webp) do
+          send_image(@record, options)
         end
       end
     end
@@ -69,7 +77,7 @@ module DynamicImage
       return unless stale?(@record)
 
       respond_to do |format|
-        format.any(:gif, :jpeg, :png, :tiff, :webp) do
+        format.any(:gif, :jpeg, :jpg, :png, :tiff, :webp) do
           send_data(@record.data,
                     filename: filename,
                     content_type: @record.content_type,
@@ -82,12 +90,17 @@ module DynamicImage
       params[:format]
     end
 
-    def send_image(processed_image)
-      send_data(
-        processed_image.cropped_and_resized(requested_size),
-        content_type: processed_image.content_type,
-        disposition: "inline"
-      )
+    def send_image(image, options)
+      processed_image = DynamicImage::ProcessedImage.new(image, options)
+      if process_later?(processed_image, requested_size)
+        DynamicImage::Jobs::CreateVariant
+          .perform_later(image, options, requested_size.to_s)
+        head 503, retry_after: 10
+      else
+        send_data(processed_image.cropped_and_resized(requested_size),
+                  content_type: processed_image.content_type,
+                  disposition: "inline")
+      end
     end
 
     def verify_signed_params

data/lib/dynamic_image/engine.rb

@@ -17,11 +17,21 @@ module DynamicImage
       end
     end
 
+    initializer "dynamic_image.mime_types" do
+      Mime::Type.register "image/bmp", :bmp
+      Mime::Type.register "image/gif", :gif
+      Mime::Type.register "image/jpeg", :jpg
+      Mime::Type.register "image/jpeg", :jpeg
+      Mime::Type.register "image/png", :png
+      Mime::Type.register "image/tiff", :tiff
+      Mime::Type.register "image/webp", :webp
+    end
+
     initializer "dynamic_image.extensions", before: :load_active_support do
       ActiveSupport.on_load(:active_record) do
         send :include, DynamicImage::BelongsTo
       end
-      ActionDispatch::Routing::Mapper.send :include, DynamicImage::Routing
+      ActionDispatch::Routing::Mapper.include DynamicImage::Routing
 
       ActionDispatch::ExceptionWrapper.rescue_responses.merge!(
         "DynamicImage::Errors::InvalidSignature" => :unauthorized

data/lib/dynamic_image/image_sizing.rb

@@ -149,8 +149,8 @@ module DynamicImage
       @uncropped
     end
 
-    def vector(x, y)
-      Vector2d.new(x, y)
+    def vector(width, height)
+      Vector2d.new(width, height)
     end
   end
 end

data/lib/dynamic_image/jobs.rb

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+require "dynamic_image/jobs/create_variant"

data/lib/dynamic_image/jobs/create_variant.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module DynamicImage
+  module Jobs
+    # = Create variant
+    #
+    # Creates an image variant.
+    class CreateVariant < ActiveJob::Base
+      queue_as :dis
+
+      def perform(record, options, size)
+        size_v = Vector2d.parse(size)
+        DynamicImage::ProcessedImage.new(record, options)
+                                    .find_or_create_variant(size_v)
+      end
+    end
+  end
+end

data/lib/dynamic_image/model/dimensions.rb

@@ -82,8 +82,8 @@ module DynamicImage
 
       private
 
-      def vector(x, y)
-        Vector2d.new(x, y)
+      def vector(width, height)
+        Vector2d.new(width, height)
       end
     end
   end

data/lib/dynamic_image/processed_image.rb

@@ -6,6 +6,8 @@ module DynamicImage
   # Handles all processing of images. Takes an instance of
   # +DynamicImage::Model+ as argument.
   class ProcessedImage
+    attr_reader :record
+
     def initialize(record, options = {})
       @record    = record
       @uncropped = options[:uncropped] ? true : false
@@ -40,10 +42,24 @@ module DynamicImage
       find_or_create_variant(size).data
     end
 
+    # Find or create a variant with the given size.
+    def find_or_create_variant(size)
+      find_variant(size) || create_variant(size)
+    rescue ActiveRecord::RecordNotUnique
+      find_variant(size)
+    end
+
+    # Find a variant with the given size.
+    def find_variant(size)
+      return nil unless record.persisted?
+
+      record.variants.find_by(variant_params(size))
+    end
+
     # Normalizes the image.
     #
     # * Applies EXIF rotation
-    # * CMYK images are converted to sRGB
+    # * Converts to sRGB
     # * Strips metadata
     # * Optimizes GIFs
     # * Performs format conversion if the requested format is different
@@ -59,7 +75,7 @@ module DynamicImage
       process_data do |image|
         image.combine_options do |combined|
           combined.auto_orient
-          combined.colorspace("sRGB") if needs_colorspace_conversion?
+          convert_to_srgb(image, combined)
           yield(combined) if block_given?
           optimize(combined)
         end
@@ -70,27 +86,29 @@ module DynamicImage
     private
 
     def coalesced(image)
-      return image unless gif?
+      gif? ? DynamicImage::ImageReader.new(image.coalesce.to_blob).read : image
+    end
+
+    def convert_to_srgb(image, combined)
+      combined.profile(srgb_profile) if image.data["profiles"].present?
+      combined.colorspace("sRGB") if record.cmyk?
+    end
 
-      DynamicImage::ImageReader.new(image.coalesce.to_blob).read
+    def create_variant(size)
+      record.variants.create(
+        variant_params(size).merge(filename: record.filename,
+                                   content_type: content_type,
+                                   data: crop_and_resize(size))
+      )
     end
 
     def crop_and_resize(size)
       normalized do |image|
-        if record.cropped? || size != record.size
-          image.crop(image_sizing.crop_geometry_string(size))
-          image.resize(size)
-        end
-      end
-    end
+        next unless record.cropped? || size != record.size
 
-    def find_or_create_variant(size)
-      record.variants.find_by(variant_params(size)) ||
-        record.variants.create(
-          variant_params(size).merge(filename: record.filename,
-                                     content_type: content_type,
-                                     data: crop_and_resize(size))
-        )
+        image.crop(image_sizing.crop_geometry_string(size))
+        image.resize(size)
+      end
     end
 
     def format
@@ -102,7 +120,7 @@ module DynamicImage
     end
 
     def jpeg?
-      content_type == "image/jpeg" || content_type == "image/jpeg"
+      content_type == "image/jpeg"
     end
 
     def image_sizing
@@ -110,10 +128,6 @@ module DynamicImage
         DynamicImage::ImageSizing.new(record, uncropped: @uncropped)
     end
 
-    def needs_colorspace_conversion?
-      record.cmyk?
-    end
-
     def needs_format_conversion?
       format != record_format
     end
@@ -133,15 +147,9 @@ module DynamicImage
       result
     end
 
-    attr_reader :record
-
     def record_format
-      { "image/bmp" => "BMP",
-        "image/png" => "PNG",
-        "image/gif" => "GIF",
-        "image/jpeg" => "JPEG",
-        "image/pjpeg" => "JPEG",
-        "image/tiff" => "TIFF",
+      { "image/bmp" => "BMP", "image/png" => "PNG", "image/gif" => "GIF",
+        "image/jpeg" => "JPEG", "image/pjpeg" => "JPEG", "image/tiff" => "TIFF",
         "image/webp" => "WEBP" }[record.content_type]
     end
 
@@ -149,15 +157,16 @@ module DynamicImage
       raise DynamicImage::Errors::InvalidImage unless record.valid?
     end
 
+    def srgb_profile
+      File.join(File.dirname(__FILE__), "profiles/sRGB_ICC_v4_Appearance.icc")
+    end
+
     def variant_params(size)
       crop_size, crop_start = image_sizing.crop_geometry(size)
 
-      { width: size.x.round,
-        height: size.y.round,
-        crop_width: crop_size.x,
-        crop_height: crop_size.y,
-        crop_start_x: crop_start.x,
-        crop_start_y: crop_start.y,
+      { width: size.x.round, height: size.y.round,
+        crop_width: crop_size.x, crop_height: crop_size.y,
+        crop_start_x: crop_start.x, crop_start_y: crop_start.y,
         format: format }
     end
   end

data/lib/dynamic_image/routing.rb

@@ -13,8 +13,9 @@ module DynamicImage
       options = {
         path: "#{resource_name}/:digest(/:size)",
         constraints: { size: /\d+x\d+/ },
-        only: [:show]
+        only: %i[show]
       }.merge(options)
+
       resources resource_name, options do
         get :uncropped, on: :member
         get :original, on: :member

data/lib/dynamic_image/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DynamicImage
-  VERSION = "2.0.20"
+  VERSION = "2.0.25"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dynamic_image
 version: !ruby/object:Gem::Version
-  version: 2.0.20
+  version: 2.0.25
 platform: ruby
 authors:
 - Inge Jørgensen
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-03-13 00:00:00.000000000 Z
+date: 2020-11-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dis
@@ -78,6 +78,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 2.2.1
+- !ruby/object:Gem::Dependency
+  name: rails-controller-testing
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
@@ -92,6 +106,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 3.7.0
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.17.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.17.1
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -127,6 +155,8 @@ files:
 - lib/dynamic_image/helper.rb
 - lib/dynamic_image/image_reader.rb
 - lib/dynamic_image/image_sizing.rb
+- lib/dynamic_image/jobs.rb
+- lib/dynamic_image/jobs/create_variant.rb
 - lib/dynamic_image/metadata.rb
 - lib/dynamic_image/model.rb
 - lib/dynamic_image/model/dimensions.rb
@@ -134,6 +164,7 @@ files:
 - lib/dynamic_image/model/validations.rb
 - lib/dynamic_image/model/variants.rb
 - lib/dynamic_image/processed_image.rb
+- lib/dynamic_image/profiles/sRGB_ICC_v4_Appearance.icc
 - lib/dynamic_image/routing.rb
 - lib/dynamic_image/templates/show.html.erb
 - lib/dynamic_image/version.rb
@@ -142,7 +173,7 @@ homepage: https://github.com/elektronaut/dynamic_image
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -158,7 +189,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: Rails plugin that simplifies image uploading and processing
 test_files: []