active_storage_validations 0.9.6 → 0.9.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 52bbd6249bbadd7549ccd7daf17673e98ccfd757b15ec954b67c5d7ce3395c84
-  data.tar.gz: cf3664efb821cd7fe60b0536e0d394f900af6fb128161ab13d62d373b2018013
+  metadata.gz: 0f77bdb678731cdb6bb3aca1bddf3a3b1d1d4f34355a778c504931d067f6d0e2
+  data.tar.gz: cb723cb17a4cbfcfec2e7a9b8432ac0d2641afbef327a92e38baa376d15ba2fd
 SHA512:
-  metadata.gz: bece2608d56156a83e468ffaf4ed31dead1769882457c1ff8480586ce0335122b475084f24f914d4d8083c5b4f2b891bda7465b25b2d34272ba76631c313167f
-  data.tar.gz: 56a1cbb1bd5bddc4cce887405eecd6d5e27cc9bfcdca97e636ccf43cf98ce29d9077e1b3e7a602dcb229cd24d25125024c4d83beba6f82050edc773b32d21ca4
+  metadata.gz: d44aed46b1f790d3df1913009fc1c84ea5831be3c8809063c94a0c50de3d8b9f40613eab920c49fb8f81dd0808d9c489aebb1705e022defe1e745d1205a48203
+  data.tar.gz: 3d770860efb253f13ea1f18086c5b8a51253caf69875ba1d9d7921c17b9c5ce43f8001f4d8eeb1e095f8f37d1f0924dbddcdb35ca24b721d6fd5cf2a002d9a90

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
@@ -177,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:
@@ -301,10 +306,12 @@ 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
 ```
 
@@ -362,6 +369,12 @@ You are welcome to contribute.
 - 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
 
 ## 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
 
@@ -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]

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/metadata.rb

@@ -3,9 +3,22 @@ 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 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)
@@ -42,29 +55,53 @@ module ActiveStorageValidations
         tempfile.flush
         tempfile.rewind
 
-        image = MiniMagick::Image.new(tempfile.path)
+        image = if image_processor == :vips && Vips::get_suffixes.include?(File.extname(tempfile.path))
+                  Vips::Image.new_from_file(tempfile.path)
+                else
+                  MiniMagick::Image.new(tempfile.path)
+                end
       else
-        image = MiniMagick::Image.new(read_file_path)
+        image = if image_processor == :vips && Vips::get_suffixes.include?(File.extname(read_file_path))
+                  Vips::Image.new_from_file(read_file_path)
+                else
+                  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 Vips::Error => error
+      logger.error "Skipping image analysis due to a Vips error: #{error.message}"
+      {}
     ensure
       image = nil
     end
 
+    def valid_image?(image)
+      image_processor == :vips ? image.avg : image.valid?
+    rescue Vips::Error
+      false
+    end
+
     def rotated_image?(image)
-      %w[ RightTop LeftBottom ].include?(image["%[orientation]"])
+      if image_processor == :vips
+        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 Vips::Error # field "exif-ifd0-Orientation" not found
+      false
     end
 
     def read_file_path

data/lib/active_storage_validations/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_storage_validations
 version: !ruby/object:Gem::Version
-  version: 0.9.6
+  version: 0.9.7
 platform: ruby
 authors:
 - Igor Kasyanchuk
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-11-23 00:00:00.000000000 Z
+date: 2022-03-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activejob
@@ -94,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
@@ -136,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
@@ -180,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
@@ -195,8 +223,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: Validations for Active Storage
 test_files: []