active_storage_validations 0.9.5 → 0.9.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: efa69aeeea1b171286c6ca464e38146dedef79d22853c35ec6ea0ce0ca089812
-  data.tar.gz: e186561942afb6b295839565323d0bbed704de9e413b79ecc1fb194ada643ca5
+  metadata.gz: c9b366848f85ac520038a9efe0bee0514338f4d28b8f292ef87ad06d1ee0da28
+  data.tar.gz: 651195be6d5d23bf20c4600656e799005e3e1fc118ac21cc5fe023b24f43ae73
 SHA512:
-  metadata.gz: c31ec657e1121427f19d7617d5efbeab639c6ebd87c0333c36510064e7b4d8125ffd8e4f5afa86121e626dd608d2ad1b859a3515c8bc7683e0793291a30c7fd2
-  data.tar.gz: 3141abc08b6b33d90f9874a74e93f6aaddb0ebbd52bdd36ee292a1e0ff7d37cf65a8ced082d316f48cc8e0132320c4f83304d9823971906526096264129c0145
+  metadata.gz: f6231454bce1c4b09bf4dba3a83238f09b3fa0593c0c04146342ac6dd035048f890edcb594d24747f9d5e55d5ef86a35eba7fa93b109f101906b0f5b4f373c06
+  data.tar.gz: 6a5a9a0abd3f7abbddd92d8f1e5601859bbcc06e17f049046e5c9e2744760aa4fd7fa77462c22fb8d076cc4aef801e9b814a6b8663edb08e9c32a497ec041908

data/README.md

@@ -26,16 +26,17 @@ For example you have a model like this and you want to add validation.
 class User < ApplicationRecord
   has_one_attached :avatar
   has_many_attached :photos
+  has_one_attached :image
 
   validates :name, presence: true
 
   validates :avatar, attached: true, content_type: 'image/png',
                                      dimension: { width: 200, height: 200 }
-  validates :photos, attached: true, content_type: ['image/png', 'image/jpg', 'image/jpeg'],
+  validates :photos, attached: true, content_type: ['image/png', 'image/jpeg'],
                                      dimension: { width: { min: 800, max: 2400 },
                                                   height: { min: 600, max: 1800 }, message: 'is not given between dimension' }
   validates :image, attached: true,
-                    content_type: ['image/png', 'image/jpg'],
+                    content_type: ['image/png', 'image/jpeg'],
                     aspect_ratio: :landscape
 end
 ```
@@ -44,13 +45,15 @@ or
 
 ```ruby
 class Project < ApplicationRecord
+  has_one_attached :logo
   has_one_attached :preview
   has_one_attached :attachment
   has_many_attached :documents
 
   validates :title, presence: true
 
-  validates :preview, attached: true, size: { less_than: 100.megabytes , message: 'is not given between size' }
+  validates :logo, attached: true, size: { less_than: 100.megabytes , message: 'is too large' }
+  validates :preview, attached: true, size: { between: 1.kilobyte..100.megabytes , message: 'is not given between size' }
   validates :attachment, attached: true, content_type: { in: 'application/pdf', message: 'is not a PDF' }
   validates :documents, limit: { min: 1, max: 3 }
 end
@@ -58,14 +61,15 @@ end
 
 ### More examples
 
-- Content type validation using symbols. In order to infer the correct mime type from the symbol, the types must be registered with `MimeMagic::EXTENSIONS`.
+- Content type validation using symbols. In order to infer the correct mime type from the symbol, the types must be registered with `Marcel::EXTENSIONS` (`MimeMagic::EXTENSIONS` for Rails <= 6.1.3).
 
 ```ruby
 class User < ApplicationRecord
   has_one_attached :avatar
   has_many_attached :photos
 
-  validates :avatar, attached: true, content_type: :png # MimeMagic.by_extension(:png).to_s => 'image/png'
+  validates :avatar, attached: true, content_type: :png # Marcel::Magic.by_extension(:png).to_s => 'image/png'
+                                                        # Rails <= 6.1.3; MimeMagic.by_extension(:png).to_s => 'image/png'
   # or
   validates :photos, attached: true, content_type: [:png, :jpg, :jpeg]
   # or
@@ -176,6 +180,8 @@ gem 'active_storage_validations'
 
 # Optional, to use :dimension validator or :aspect_ratio validator
 gem 'mini_magick', '>= 4.9.5'
+# Or
+gem 'ruby-vips', '>= 2.1.0'
 ```
 
 And then execute:
@@ -292,6 +298,22 @@ To run tests in root folder of gem:
 * `BUNDLE_GEMFILE=gemfiles/rails_5_2.gemfile bundle exec rake test` to run for Rails 5.2
 * `BUNDLE_GEMFILE=gemfiles/rails_6_0.gemfile bundle exec rake test` to run for Rails 6.0
 * `BUNDLE_GEMFILE=gemfiles/rails_6_1.gemfile bundle exec rake test` to run for Rails 6.1
+* `BUNDLE_GEMFILE=gemfiles/rails_next.gemfile bundle exec rake test` to run for Rails main branch
+
+Snippet to run in console:
+
+```
+BUNDLE_GEMFILE=gemfiles/rails_5_2.gemfile bundle
+BUNDLE_GEMFILE=gemfiles/rails_6_0.gemfile bundle
+BUNDLE_GEMFILE=gemfiles/rails_6_1.gemfile bundle
+BUNDLE_GEMFILE=gemfiles/rails_7_0.gemfile bundle
+BUNDLE_GEMFILE=gemfiles/rails_next.gemfile bundle
+BUNDLE_GEMFILE=gemfiles/rails_5_2.gemfile bundle exec rake test
+BUNDLE_GEMFILE=gemfiles/rails_6_0.gemfile bundle exec rake test
+BUNDLE_GEMFILE=gemfiles/rails_6_1.gemfile bundle exec rake test
+BUNDLE_GEMFILE=gemfiles/rails_7_0.gemfile bundle exec rake test
+BUNDLE_GEMFILE=gemfiles/rails_next.gemfile bundle exec rake test
+```
 
 ## Known issues
 
@@ -344,7 +366,18 @@ You are welcome to contribute.
 - https://github.com/jayshepherd
 - https://github.com/ohbarye
 - https://github.com/randsina
-- https://github.com/...
+- https://github.com/vietqhoang
+- https://github.com/kemenaran
+- https://github.com/jrmhaig
+- https://github.com/tagliala
+- https://github.com/evedovelli
+- https://github.com/JuanVqz
+- https://github.com/luiseugenio
+- https://github.com/equivalent
+- https://github.com/NARKOZ
+- https://github.com/stephensolis
+- https://github.com/kwent
+= https://github.com/Animesh-Ghosh
 
 ## License
 

data/config/locales/pt-BR.yml

@@ -2,21 +2,21 @@ pt-BR:
   errors:
     messages:
       content_type_invalid: "tem um tipo de arquivo inválido"
-      file_size_out_of_range: "tamanho %{file_size} está fora da faixa de tamanho válida"
-      limit_out_of_range: "número total está fora do limite"
+      file_size_out_of_range: "tem tamanho %{file_size} e está fora da faixa de tamanho válida"
+      limit_out_of_range: "o número total está fora do limite"
       image_metadata_missing: "não é uma imagem válida"
-      dimension_min_inclusion: "deve ser maior ou igual a %{width} x %{height} pixel"
-      dimension_max_inclusion: "deve ser menor ou igual a %{width} x %{height} pixel"
-      dimension_width_inclusion: "largura não está entre %{min} e %{max} pixel"
-      dimension_height_inclusion: "altura não está entre %{min} e %{max} pixel"
-      dimension_width_greater_than_or_equal_to: "largura deve ser maior ou igual a %{length} pixel"
-      dimension_height_greater_than_or_equal_to: "altura deve ser maior ou igual a %{length} pixel"
-      dimension_width_less_than_or_equal_to: "largura deve ser menor ou igual a %{length} pixel"
-      dimension_height_less_than_or_equal_to: "altura deve ser menor ou igual a %{length} pixel"
-      dimension_width_equal_to: "largura deve ser igual a %{length} pixel"
-      dimension_height_equal_to: "altura deve ser igual a %{length} pixel"
+      dimension_min_inclusion: "deve ser maior ou igual a %{width} x %{height} pixels"
+      dimension_max_inclusion: "deve ser menor ou igual a %{width} x %{height} pixels"
+      dimension_width_inclusion: "deve ter largura entre %{min} e %{max} pixels"
+      dimension_height_inclusion: "deve ter altura entre %{min} e %{max} pixels"
+      dimension_width_greater_than_or_equal_to: "deve ter largura maior ou igual a %{length} pixels"
+      dimension_height_greater_than_or_equal_to: "deve ter altura maior ou igual a %{length} pixels"
+      dimension_width_less_than_or_equal_to: "deve ter largura menor ou igual a %{length} pixels"
+      dimension_height_less_than_or_equal_to: "deve ter altura menor ou igual a %{length} pixels"
+      dimension_width_equal_to: "deve ter largura igual a %{length} pixels"
+      dimension_height_equal_to: "deve ter altura igual a %{length} pixels"
       aspect_ratio_not_square: "não é uma imagem quadrada"
-      aspect_ratio_not_portrait: "não contém uma imagem no formato retrato"
-      aspect_ratio_not_landscape: "não contém uma imagem no formato paisagem"
+      aspect_ratio_not_portrait: "não está no formato retrato"
+      aspect_ratio_not_landscape: "não está no formato paisagem"
       aspect_ratio_is_not: "não contém uma proporção de %{aspect_ratio}"
       aspect_ratio_unknown: "não tem uma proporção definida"

data/lib/active_storage_validations/aspect_ratio_validator.rb

@@ -8,7 +8,6 @@ module ActiveStorageValidations
     PRECISION = 3
 
     def initialize(options)
-      require 'mini_magick' unless defined?(MiniMagick)
       super(options)
     end
 
@@ -18,7 +17,7 @@ module ActiveStorageValidations
       raise ArgumentError, 'You must pass "aspect_ratio: :OPTION" option to the validator'
     end
 
-    if Rails::VERSION::MAJOR >= 6
+    if Rails.gem_version >= Gem::Version.new('6.0.0')
       def validate_each(record, attribute, _value)
         return true unless record.send(attribute).attached?
 
@@ -37,14 +36,14 @@ module ActiveStorageValidations
       # Rails 5
       def validate_each(record, attribute, _value)
         return true unless record.send(attribute).attached?
-  
+
         files = Array.wrap(record.send(attribute))
-  
+
         files.each do |file|
           # Analyze file first if not analyzed to get all required metadata.
           file.analyze; file.reload unless file.analyzed?
           metadata = file.metadata
-  
+
           next if is_valid?(record, attribute, metadata)
           break
         end

data/lib/active_storage_validations/dimension_validator.rb

@@ -7,8 +7,6 @@ module ActiveStorageValidations
     AVAILABLE_CHECKS = %i[width height min max].freeze
 
     def initialize(options)
-      require 'mini_magick' unless defined?(MiniMagick)
-
       [:width, :height].each do |length|
         if options[length] and options[length].is_a?(Hash)
           if range = options[length][:in]
@@ -34,7 +32,7 @@ module ActiveStorageValidations
     end
 
 
-    if Rails::VERSION::MAJOR >= 6
+    if Rails.gem_version >= Gem::Version.new('6.0.0')
       def validate_each(record, attribute, _value)
         return true unless record.send(attribute).attached?
 
@@ -91,29 +89,32 @@ module ActiveStorageValidations
 
       # Validation based on checks :width and :height.
       else
+        width_or_height_invalid = false
         [:width, :height].each do |length|
           next unless options[length]
           if options[length].is_a?(Hash)
             if options[length][:in] && (file_metadata[length] < options[length][:min] || file_metadata[length] > options[length][:max])
               add_error(record, attribute, options[:message].presence || :"dimension_#{length}_inclusion", min: options[length][:min], max: options[length][:max])
-              return false
+              width_or_height_invalid = true
             else
               if options[length][:min] && file_metadata[length] < options[length][:min]
                 add_error(record, attribute, options[:message].presence || :"dimension_#{length}_greater_than_or_equal_to", length: options[length][:min])
-                return false
+                width_or_height_invalid = true
               end
               if options[length][:max] && file_metadata[length] > options[length][:max]
                 add_error(record, attribute, options[:message].presence || :"dimension_#{length}_less_than_or_equal_to", length: options[length][:max])
-                return false
+                width_or_height_invalid = true
               end
             end
           else
             if file_metadata[length] != options[length]
               add_error(record, attribute, options[:message].presence || :"dimension_#{length}_equal_to", length: options[length])
-              return false
+              width_or_height_invalid = true
             end
           end
         end
+
+        return false if width_or_height_invalid
       end
 
       true # valid file

data/lib/active_storage_validations/matchers/content_type_validator_matcher.rb

@@ -35,7 +35,7 @@ module ActiveStorageValidations
       def failure_message
         <<~MESSAGE
           Expected #{@attribute_name}
-            
+
             Accept content types: #{allowed_types.join(", ")}
               #{accepted_types_and_failures}
 
@@ -57,7 +57,7 @@ module ActiveStorageValidations
       end
 
       def rejected_types
-        @rejected_types || (Marcel::TYPES.keys - allowed_types)
+        @rejected_types || (content_type_keys - allowed_types)
       end
 
       def allowed_types_allowed?
@@ -96,6 +96,16 @@ module ActiveStorageValidations
         suffix = type.to_s.split('/').last
         { io: Tempfile.new('.'), filename: "test.#{suffix}", content_type: type }
       end
+
+      private
+
+      def content_type_keys
+        if Rails.gem_version < Gem::Version.new('6.1.0')
+          Mime::LOOKUP.keys
+        else
+          Marcel::TYPES.keys
+        end
+      end
     end
   end
 end

data/lib/active_storage_validations/matchers.rb

@@ -22,7 +22,7 @@ module ActiveStorageValidations
     end
 
     def self.mock_metadata(attachment, width, height)
-      if Rails::VERSION::MAJOR >= 6
+      if Rails.gem_version >= Gem::Version.new('6.0.0')
         # Mock the Metadata class for rails 6
         mock = OpenStruct.new(metadata: { width: width, height: height })
         stub_method(ActiveStorageValidations::Metadata, :new, mock) do

data/lib/active_storage_validations/metadata.rb

@@ -3,9 +3,30 @@ module ActiveStorageValidations
     attr_reader :file
 
     def initialize(file)
+      require_image_processor
       @file = file
     end
 
+    def image_processor
+      Rails.application.config.active_storage.variant_processor
+    end
+    
+    def exception_class
+      if image_processor == :vips && defined?(Vips)
+        Vips::Error
+      elsif defined?(MiniMagick)
+        MiniMagick::Error
+      end
+    end
+
+    def require_image_processor
+      if image_processor == :vips
+        require 'vips' unless defined?(Vips)
+      else
+        require 'mini_magick' unless defined?(MiniMagick)
+      end
+    end
+
     def metadata
       read_image do |image|
         if rotated_image?(image)
@@ -21,17 +42,16 @@ module ActiveStorageValidations
     def read_image
       is_string = file.is_a?(String)
       if is_string || file.is_a?(ActiveStorage::Blob)
-        if is_string
-          # If Rails 5.2 or 6.0, use `find_signed`
-          if Rails::VERSION::MAJOR < 6 || (Rails::VERSION::MAJOR == 6 && Rails::VERSION::MINOR == 0)
-            blob = ActiveStorage::Blob.find_signed(file)
-          # If Rails 6.1 or higher, use `find_signed!`
-          elsif Rails::VERSION::MAJOR > 6 || (Rails::VERSION::MAJOR == 6 && Rails::VERSION::MINOR >= 1)
-            blob = ActiveStorage::Blob.find_signed!(file)
+        blob =
+          if is_string
+            if Rails.gem_version < Gem::Version.new('6.1.0')
+              ActiveStorage::Blob.find_signed(file)
+            else
+              ActiveStorage::Blob.find_signed!(file)
+            end
+          else
+            file
           end
-        else
-          blob = file
-        end
 
         tempfile = Tempfile.new(["ActiveStorage-#{blob.id}-", blob.filename.extension_with_delimiter])
         tempfile.binmode
@@ -43,29 +63,50 @@ module ActiveStorageValidations
         tempfile.flush
         tempfile.rewind
 
-        image = MiniMagick::Image.new(tempfile.path)
+        image = if image_processor == :vips && defined?(Vips) && Vips::get_suffixes.include?(File.extname(tempfile.path).downcase)
+                  Vips::Image.new_from_file(tempfile.path)
+                elsif defined?(MiniMagick)
+                  MiniMagick::Image.new(tempfile.path)
+                end
       else
-        image = MiniMagick::Image.new(read_file_path)
+        image = if image_processor == :vips && defined?(Vips) && Vips::get_suffixes.include?(File.extname(read_file_path).downcase)
+                  Vips::Image.new_from_file(read_file_path)
+                elsif defined?(MiniMagick)
+                  MiniMagick::Image.new(read_file_path)
+                end
       end
 
-      if image.valid?
+      if image && valid_image?(image)
         yield image
       else
-        logger.info "Skipping image analysis because ImageMagick doesn't support the file"
+        logger.info "Skipping image analysis because ImageMagick or Vips doesn't support the file"
         {}
       end
-    rescue LoadError
-      logger.info "Skipping image analysis because the mini_magick gem isn't installed"
+    rescue LoadError, NameError
+      logger.info "Skipping image analysis because the mini_magick or ruby-vips gem isn't installed"
       {}
-    rescue MiniMagick::Error => error
-      logger.error "Skipping image analysis due to an ImageMagick error: #{error.message}"
+    rescue exception_class => error
+      logger.error "Skipping image analysis due to an #{exception_class.name.split('::').map(&:downcase).join(' ').capitalize} error: #{error.message}"
       {}
     ensure
       image = nil
     end
 
+    def valid_image?(image)
+      image_processor == :vips && image.is_a?(Vips::Image) ? image.avg : image.valid?
+    rescue exception_class
+      false
+    end
+
     def rotated_image?(image)
-      %w[ RightTop LeftBottom ].include?(image["%[orientation]"])
+      if image_processor == :vips && image.is_a?(Vips::Image)
+        image.get('exif-ifd0-Orientation').include?('Right-top') ||
+          image.get('exif-ifd0-Orientation').include?('Left-bottom')
+      else
+        %w[ RightTop LeftBottom ].include?(image["%[orientation]"])
+      end
+    rescue exception_class # field "exif-ifd0-Orientation" not found
+      false
     end
 
     def read_file_path

data/lib/active_storage_validations/size_validator.rb

@@ -25,6 +25,9 @@ module ActiveStorageValidations
         next if content_size_valid?(file.blob.byte_size)
 
         errors_options[:file_size] = number_to_human_size(file.blob.byte_size)
+        errors_options[:min_size] = number_to_human_size(min_size)
+        errors_options[:max_size] = number_to_human_size(max_size)
+
         record.errors.add(attribute, :file_size_out_of_range, **errors_options)
         break
       end
@@ -43,5 +46,13 @@ module ActiveStorageValidations
         file_size >= options[:greater_than_or_equal_to]
       end
     end
+
+    def min_size
+      options[:between]&.min || options[:greater_than] || options[:greater_than_or_equal_to]
+    end
+
+    def max_size
+      options[:between]&.max || options[:less_than] || options[:less_than_or_equal_to]
+    end
   end
 end

data/lib/active_storage_validations/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveStorageValidations
-  VERSION = '0.9.5'
+  VERSION = '0.9.8'
 end

metadata

@@ -1,17 +1,59 @@
 --- !ruby/object:Gem::Specification
 name: active_storage_validations
 version: !ruby/object:Gem::Version
-  version: 0.9.5
+  version: 0.9.8
 platform: ruby
 authors:
 - Igor Kasyanchuk
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-26 00:00:00.000000000 Z
+date: 2022-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rails
+  name: activejob
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.2.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.2.0
+- !ruby/object:Gem::Dependency
+  name: activemodel
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.2.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.2.0
+- !ruby/object:Gem::Dependency
+  name: activestorage
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.2.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.2.0
+- !ruby/object:Gem::Dependency
+  name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -52,6 +94,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 4.9.5
+- !ruby/object:Gem::Dependency
+  name: ruby-vips
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.1.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.1.0
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
@@ -94,6 +150,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: marcel
+  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'
 description: Validations for Active Storage (presence)
 email:
 - igorkasyanchuk@gmail.com
@@ -138,7 +208,7 @@ homepage: https://github.com/igorkasyanchuk/active_storage_validations
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -153,8 +223,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: Validations for Active Storage
 test_files: []