kt-paperclip 7.2.2 → 8.0.0

data/lib/paperclip/geometry.rb

@@ -1,35 +1,42 @@
+# frozen_string_literal: true
+
 module Paperclip
   # Defines the geometry of an image.
   class Geometry
-    attr_accessor :height, :width, :modifier
+    attr_accessor :height, :width, :modifier, :orientation
 
     EXIF_ROTATED_ORIENTATION_VALUES = [5, 6, 7, 8].freeze
 
     # Gives a Geometry representing the given height and width
-    def initialize(width = nil, height = nil, modifier = nil)
+    def initialize(width = nil, height = nil, modifier = nil, orientation = nil)
       if width.is_a?(Hash)
         options = width
         @height = options[:height].to_f
         @width = options[:width].to_f
-        @modifier = options[:modifier]
+        @modifier = options[:modifier].presence
         @orientation = options[:orientation].to_i
       else
         @height = height.to_f
         @width  = width.to_f
-        @modifier = modifier
+        @modifier = modifier.presence
+        @orientation = orientation.to_i
       end
     end
 
     # Extracts the Geometry from a file (or path to a file)
+    #
+    # @deprecated Will be removed in Paperclip 8.0.
     def self.from_file(file)
-      GeometryDetector.new(file).make
+      Commands::ImageMagick::GeometryParser.from_file(file)
     end
 
     # Extracts the Geometry from a "WxH,O" string
     # Where W is the width, H is the height,
     # and O is the EXIF orientation
+    #
+    # @deprecated Will be removed in Paperclip 8.0.
     def self.parse(string)
-      GeometryParser.new(string).make
+      Commands::ImageMagick::GeometryParser.parse(string)
     end
 
     # Swaps the height and width if necessary
@@ -72,13 +79,21 @@ module Paperclip
 
     # Returns the width and height in a format suitable to be passed to Geometry.parse
     def to_s
-      s = ""
+      s = +""
       s << width.to_i.to_s if width > 0
       s << "x#{height.to_i}" if height > 0
       s << modifier.to_s
       s
     end
 
+    def ==(other)
+      other.is_a?(Geometry) &&
+        width == other.width &&
+        height == other.height &&
+        modifier == other.modifier &&
+        orientation == other.orientation
+    end
+
     # Same as to_s
     def inspect
       to_s

data/lib/paperclip/geometry_detector_factory.rb

@@ -1,45 +1,25 @@
+# frozen_string_literal: true
+
 module Paperclip
+  # @deprecated Will be removed in Paperclip 8.0. Use +Paperclip::Commands::ImageMagick::GeometryParser+ instead.
   class GeometryDetector
+    # @deprecated Will be removed in Paperclip 8.0. Use +Paperclip::Commands::ImageMagick::GeometryParser+ instead.
     def initialize(file)
+      warn_deprecation
       @file = file
-      raise_if_blank_file
+      raise Errors::NotIdentifiedByImageMagickError.new("Cannot find the geometry of a file with a blank name") if @file.blank?
     end
 
+    # @deprecated Will be removed in Paperclip 8.0. Use +Paperclip::Commands::ImageMagick::GeometryParser+ instead.
     def make
-      geometry = GeometryParser.new(geometry_string.strip).make
-      geometry || raise(Errors::NotIdentifiedByImageMagickError.new("Could not identify image size"))
+      warn_deprecation
+      Paperclip::Commands::ImageMagick::GeometryParser.from_file(@file)
     end
 
     private
 
-    def geometry_string
-      orientation = Paperclip.options[:use_exif_orientation] ?
-        "%[exif:orientation]" : "1"
-      Paperclip.run(
-        Paperclip.options[:is_windows] ? "magick identify" : "identify",
-        "-format '%wx%h,#{orientation}' :file", {
-          file: "#{path}[0]"
-        },
-        swallow_stderr: true
-      )
-    rescue Terrapin::ExitStatusError
-      ""
-    rescue Terrapin::CommandNotFoundError => e
-      raise_because_imagemagick_missing
-    end
-
-    def path
-      @file.respond_to?(:path) ? @file.path : @file
-    end
-
-    def raise_if_blank_file
-      if path.blank?
-        raise Errors::NotIdentifiedByImageMagickError.new("Cannot find the geometry of a file with a blank name")
-      end
-    end
-
-    def raise_because_imagemagick_missing
-      raise Errors::CommandNotFoundError.new("Could not run the `identify` command. Please install ImageMagick.")
+    def warn_deprecation
+      Paperclip.deprecator.warn("Paperclip::GeometryDetector has been replaced by Paperclip::Commands::ImageMagick::GeometryParser")
     end
   end
 end

data/lib/paperclip/geometry_parser_factory.rb

@@ -1,31 +1,27 @@
+# frozen_string_literal: true
+
 module Paperclip
+  # @deprecated Will be removed in Paperclip 8.0. Use +Paperclip::Commands::ImageMagick::GeometryParser+ instead.
   class GeometryParser
-    FORMAT = /\b(\d*)x?(\d*)\b(?:,(\d?))?(\@\>|\>\@|[\>\<\#\@\%^!])?/i.freeze
+    # @deprecated Will be removed in Paperclip 8.0. Use +Paperclip::Commands::ImageMagick::GeometryParser::FORMAT+ instead.
+    FORMAT = Paperclip::Commands::ImageMagick::GeometryParser::FORMAT
+
+    # @deprecated Will be removed in Paperclip 8.0. Use +Paperclip::Commands::ImageMagick::GeometryParser+ instead.
     def initialize(string)
+      warn_deprecation
       @string = string
     end
 
+    # @deprecated Will be removed in Paperclip 8.0. Use +Paperclip::Commands::ImageMagick::GeometryParser+ instead.
     def make
-      if match
-        Geometry.new(
-          height: @height,
-          width: @width,
-          modifier: @modifier,
-          orientation: @orientation
-        )
-      end
+      warn_deprecation
+      Paperclip::Commands::ImageMagick::GeometryParser.parse(@string)
     end
 
     private
 
-    def match
-      if actual_match = @string && @string.match(FORMAT)
-        @width = actual_match[1]
-        @height = actual_match[2]
-        @orientation = actual_match[3]
-        @modifier = actual_match[4]
-      end
-      actual_match
+    def warn_deprecation
+      Paperclip.deprecator.warn("Paperclip::GeometryParser has been replaced by Paperclip::Commands::ImageMagick::GeometryParser")
     end
   end
 end

data/lib/paperclip/glue.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "paperclip/callbacks"
 require "paperclip/validators"
 require "paperclip/schema"

data/lib/paperclip/has_attached_file.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class HasAttachedFile
     def self.define_on(klass, name, options)

data/lib/paperclip/helpers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   module Helpers
     def configure
@@ -8,31 +10,9 @@ module Paperclip
       Paperclip::Interpolations[key] = block
     end
 
-    # The run method takes the name of a binary to run, the arguments
-    # to that binary, the values to interpolate and some local options.
-    #
-    #  :cmd -> The name of a binary to run.
-    #
-    #  :arguments -> The command line arguments to that binary.
-    #
-    #  :interpolation_values -> Values to be interpolated into the arguments.
-    #
-    #  :local_options -> The options to be used by Cocain::CommandLine.
-    #                    These could be: runner
-    #                                    logger
-    #                                    swallow_stderr
-    #                                    expected_outcodes
-    #                                    environment
-    #                                    runner_options
-    #
-    def run(cmd, arguments = "", interpolation_values = {}, local_options = {})
-      command_path = options[:command_path]
-      terrapin_path_array = Terrapin::CommandLine.path.try(:split, Terrapin::OS.path_separator)
-      Terrapin::CommandLine.path = [terrapin_path_array, command_path].flatten.compact.uniq
-      if logging? && (options[:log_command] || local_options[:log_command])
-        local_options = local_options.merge(logger: logger)
-      end
-      Terrapin::CommandLine.new(cmd, arguments, local_options).run(interpolation_values)
+    # @deprecated Will be removed in Paperclip 8.0. Use +Paperclip::Commands::Runner.run+ instead.
+    def run(command, arguments = nil, interpolation_values = {}, local_options = {})
+      Commands::Runner.run(command, nil, arguments, interpolation_values, local_options)
     end
 
     # Find all instances of the given Active Record model +klass+ with attachment +name+.

data/lib/paperclip/interpolations/plural_cache.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   module Interpolations
     class PluralCache

data/lib/paperclip/interpolations.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   # This module contains all the methods that are available for interpolation
   # in paths and urls. To add your own (or override an existing one), you

data/lib/paperclip/io_adapters/abstract_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_support/core_ext/module/delegation"
 
 module Paperclip
@@ -17,7 +19,7 @@ module Paperclip
       @fingerprint ||= begin
         digest = @options.fetch(:hash_digest).new
         File.open(path, "rb") do |f|
-          buf = ""
+          buf = String.new
           digest.update(buf) while f.read(16384, buf)
         end
         digest.hexdigest

data/lib/paperclip/io_adapters/attachment_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class AttachmentAdapter < AbstractAdapter
     def self.register
@@ -33,6 +35,15 @@ module Paperclip
       else
         begin
           source.copy_to_local_file(@style, destination.path)
+
+          # Some sources (like S3) writes to a temporary file first for multi-part downloads then moves
+          # it to the final destination path. This means our destination Tempfile reference is stale
+          # and reads will return an empty string. For compatibility, we need to copy the contents
+          # of the destination file to our current Tempfile instance.
+          if destination.size.zero?
+            IO.copy_stream(destination.path, destination)
+            destination.rewind
+          end
         rescue Errno::EACCES
           # clean up lingering tempfile if we cannot access source file
           destination.close(true)

data/lib/paperclip/io_adapters/data_uri_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class DataUriAdapter < StringioAdapter
     def self.register

data/lib/paperclip/io_adapters/empty_string_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class EmptyStringAdapter < AbstractAdapter
     def self.register

data/lib/paperclip/io_adapters/file_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class FileAdapter < AbstractAdapter
     def self.register

data/lib/paperclip/io_adapters/http_url_proxy_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class HttpUrlProxyAdapter < UriAdapter
     def self.register

data/lib/paperclip/io_adapters/identity_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class IdentityAdapter < AbstractAdapter
     def self.register

data/lib/paperclip/io_adapters/nil_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class NilAdapter < AbstractAdapter
     def self.register

data/lib/paperclip/io_adapters/registry.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class AdapterRegistry
     class NoHandlerError < Paperclip::Error; end

data/lib/paperclip/io_adapters/stringio_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class StringioAdapter < AbstractAdapter
     def self.register

data/lib/paperclip/io_adapters/uploaded_file_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class UploadedFileAdapter < AbstractAdapter
     def self.register

data/lib/paperclip/io_adapters/uri_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "open-uri"
 
 module Paperclip
@@ -50,20 +52,10 @@ module Paperclip
       "index.html"
     end
 
-    if RUBY_VERSION < '2.5'
-      def download_content
-        options = { read_timeout: Paperclip.options[:read_timeout] }.compact
-
-        # rubocop:disable Security/Open
-        open(@target, options)
-        # rubocop:enable Security/Open
-      end
-    else
-      def download_content
-        options = { read_timeout: Paperclip.options[:read_timeout] }.compact
+    def download_content
+      options = { read_timeout: Paperclip.options[:read_timeout] }.compact
 
-        URI.open(@target, options)
-      end
+      URI.open(@target, options)
     end
 
     def copy_to_tempfile(src)

data/lib/paperclip/locales/fr.yml

@@ -0,0 +1,18 @@
+fr:
+  errors:
+    messages:
+      in_between: "doit être entre %{min} et %{max}"
+      spoofed_media_type: "à du contenu qui ne correspond pas au type déclaré"
+
+  number:
+    human:
+      storage_units:
+        format: "%n %u"
+        units:
+          byte:
+            one:   "Octet"
+            other: "Octets"
+          kb: "ko"
+          mb: "Mo"
+          gb: "Go"
+          tb: "To"

data/lib/paperclip/locales/gd.yml

@@ -1,4 +1,4 @@
-gd:
+gd:
   errors:
     messages:
       in_between: "– feumaidh seo a bhith eadar %{min} ’s %{max}"

data/lib/paperclip/logger.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   module Logger
     # Log a paperclip-specific line. This will log to STDOUT

data/lib/paperclip/matchers/have_attached_file_matcher.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   module Shoulda
     module Matchers

data/lib/paperclip/matchers/validate_attachment_content_type_matcher.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   module Shoulda
     module Matchers
@@ -39,7 +41,7 @@ module Paperclip
         end
 
         def failure_message
-          "#{expected_attachment}\n".tap do |message|
+          (+"#{expected_attachment}\n").tap do |message|
             message << accepted_types_and_failures.to_s
             message << "\n\n" if @allowed_types.present? && @rejected_types.present?
             message << rejected_types_and_failures.to_s
@@ -54,7 +56,7 @@ module Paperclip
 
         def accepted_types_and_failures
           if @allowed_types.present?
-            "Accept content types: #{@allowed_types.join(', ')}\n".tap do |message|
+            (+"Accept content types: #{@allowed_types.join(', ')}\n").tap do |message|
               message << if @missing_allowed_types.present?
                            "  #{@missing_allowed_types.join(', ')} were rejected."
                          else
@@ -66,7 +68,7 @@ module Paperclip
 
         def rejected_types_and_failures
           if @rejected_types.present?
-            "Reject content types: #{@rejected_types.join(', ')}\n".tap do |message|
+            (+"Reject content types: #{@rejected_types.join(', ')}\n").tap do |message|
               message << if @missing_rejected_types.present?
                            "  #{@missing_rejected_types.join(', ')} were accepted."
                          else

data/lib/paperclip/matchers/validate_attachment_presence_matcher.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   module Shoulda
     module Matchers

data/lib/paperclip/matchers/validate_attachment_size_matcher.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   module Shoulda
     module Matchers

data/lib/paperclip/matchers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "paperclip/matchers/have_attached_file_matcher"
 require "paperclip/matchers/validate_attachment_presence_matcher"
 require "paperclip/matchers/validate_attachment_content_type_matcher"

data/lib/paperclip/media_type_spoof_detector.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class MediaTypeSpoofDetector
     def self.using(file, name, content_type)
@@ -65,22 +67,13 @@ module Paperclip
     end
 
     def calculated_content_type
-      @calculated_content_type ||= type_from_file_command.chomp
+      @calculated_content_type ||= ContentTypeDetector.new(@file.path).detect("")
     end
 
     def calculated_media_type
       @calculated_media_type ||= calculated_content_type.split("/").first
     end
 
-    def type_from_file_command
-      Paperclip.run("file", "-b --mime :file", file: @file.path).
-        split(/[:;\s]+/).first
-    rescue Terrapin::CommandLineError
-      Paperclip.log("Problem getting type from `file` command. Possible that `file` doesn't exist on this system. Content Type validations don't work without this.")
-
-      ""
-    end
-
     def mapped_content_type
       content_type_mappings = Paperclip.options[:content_type_mappings]
       content_type_mappings[filename_extension] || content_type_mappings[filename_extension.to_s]

data/lib/paperclip/missing_attachment_styles.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "paperclip/attachment_registry"
 require "set"
 

data/lib/paperclip/processor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   # Paperclip processors allow you to modify attached files when they are
   # attached in any way you are able. Paperclip itself uses command-line
@@ -15,9 +17,6 @@ module Paperclip
   #
   # #make must return an instance of File (Tempfile is acceptable) which
   # contains the results of the processing.
-  #
-  # See Paperclip.run for more information about using command-line
-  # utilities from within Processors.
   class Processor
     attr_accessor :file, :options, :attachment
 
@@ -33,24 +32,12 @@ module Paperclip
       new(file, options, attachment).make
     end
 
-    # The convert method runs the convert binary with the provided arguments.
-    # See Paperclip.run for the available options.
-    def convert(arguments = "", local_options = {})
-      Paperclip.run(
-        Paperclip.options[:is_windows] ? "magick convert" : "convert",
-        arguments,
-        local_options
-      )
+    def convert(arguments = nil, interpolation_values = {})
+      Paperclip::Commands::ImageMagick.convert(arguments, interpolation_values)
     end
 
-    # The identify method runs the identify binary with the provided arguments.
-    # See Paperclip.run for the available options.
-    def identify(arguments = "", local_options = {})
-      Paperclip.run(
-        Paperclip.options[:is_windows] ? "magick identify" : "identify",
-        arguments,
-        local_options
-      )
+    def identify(arguments = nil, interpolation_values = {})
+      Paperclip::Commands::ImageMagick.identify(arguments, interpolation_values)
     end
   end
 end

data/lib/paperclip/processor_helpers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   module ProcessorHelpers
     class NoSuchProcessor < StandardError; end

data/lib/paperclip/rails_environment.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   class RailsEnvironment
     def self.get

data/lib/paperclip/railtie.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "paperclip"
 require "paperclip/schema"
 

data/lib/paperclip/schema.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_support/deprecation"
 
 module Paperclip
@@ -25,7 +27,7 @@ module Paperclip
 
         attachment_names.each do |attachment_name|
           COLUMNS.each_pair do |column_name, column_type|
-            column_options = options.merge(options[column_name.to_sym] || {})
+            column_options = options.merge(options[column_name.to_sym] || {}).except(*COLUMNS.keys)
             add_column(table_name, "#{attachment_name}_#{column_name}", column_type, **column_options)
           end
         end
@@ -44,7 +46,7 @@ module Paperclip
       end
 
       def drop_attached_file(*args)
-        ActiveSupport::Deprecation.warn "Method `drop_attached_file` in the migration has been deprecated and will be replaced by `remove_attachment`."
+        Paperclip.deprecator.warn "Method `drop_attached_file` in the migration has been deprecated and will be replaced by `remove_attachment`."
         remove_attachment(*args)
       end
     end
@@ -54,14 +56,14 @@ module Paperclip
         options = attachment_names.extract_options!
         attachment_names.each do |attachment_name|
           COLUMNS.each_pair do |column_name, column_type|
-            column_options = options.merge(options[column_name.to_sym] || {})
+            column_options = options.merge(options[column_name.to_sym] || {}).except(*COLUMNS.keys)
             column("#{attachment_name}_#{column_name}", column_type, **column_options)
           end
         end
       end
 
       def has_attached_file(*attachment_names)
-        ActiveSupport::Deprecation.warn "Method `t.has_attached_file` in the migration has been deprecated and will be replaced by `t.attachment`."
+        Paperclip.deprecator.warn "Method `t.has_attached_file` in the migration has been deprecated and will be replaced by `t.attachment`."
         attachment(*attachment_names)
       end
     end

data/lib/paperclip/storage/filesystem.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Paperclip
   module Storage
     # The default place to store attachments is in the filesystem. Files on the local