kt-paperclip 7.2.1 → 7.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 206e4e7dd6b6f2ad56c7a0395e9b845a8033c7efa20f19f4ef5c33d09edcbf39
-  data.tar.gz: 492bfb36e312bd31c4511a91f6b0b4d779d31b5d098f616cb75871c2a6a4e527
+  metadata.gz: 213cd04cf7f631853227e9c3fc6d6a38733f3c278907497e1f8e3c2e7a98bac8
+  data.tar.gz: 01aec044e7eff086cef50fab323730f617fd455e4ba70ffb99a58d099a4ffde5
 SHA512:
-  metadata.gz: 86a39461732fd83389253cc444168d30a9af957362d90694e04852d86619e3d4ac4a84ea71db0d904784de74d5c5e019bdcec86fc18484fce2e26568cf390d55
-  data.tar.gz: 9506c9fc401cb9ccee3dcfe9186ec8787df2abdc3472f947592d7a79a2e6c32e917cc3eae2de1bfc49f366129a67f793c436b69ecafe501aee0fb7c265662bdc
+  metadata.gz: a7e922d52efbde590030651a7cbdca2087aa3c163d6cc498273e9ca57ce5c2b371b44601ccb9e825ee196f1197689438cb91cfca240b6361f259650ea63e5d03
+  data.tar.gz: 412ef55d0817f23247e52859d1b39cb1358f55bfd8c8d67cb16df376b42cc2926611b938004d6cec95d7a211895702cb279721322c1ac16e235af954a6f1f40e

data/LICENSE

@@ -1,9 +1,9 @@
-
 LICENSE
 
 The MIT License
 
 Copyright (c) 2008-2016 Jon Yurek and thoughtbot, inc.
+Copyright (c) 2019-2025 Kreeti Technologies Pvt Ltd.
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/{NEWS → OLD_NEWS}

@@ -1,5 +1,14 @@
 [UNRELEASED]
+
+* Improvement: Support aws-sdk-s3 >= 1.196.1 and multipart file downloads
+
+7.2.1 (2023-09-09)
 * Improvement: Support file extension names both as symbols and strings for :content_type_mappings
+
+7.2.0 (2023-05-30)
+* Paperclip schema statements are consistent with ActiveRecord::Migration::Compatibility versioning. Old migrations containing Paperclip schema statements perform the same schema changes both before and after an ActiveRecord version upgrade.
+
+7.0.1 (2021-10-06)
 * Issue file delete only once per unique style when nullifying attachment or destroying an object. Avoids triggering a rate limit error on Google Cloud Storage.
 
 7.0.0 (2021-05-28)

data/lib/paperclip/attachment.rb

@@ -10,30 +10,31 @@ module Paperclip
   class Attachment
     def self.default_options
       @default_options ||= {
-        convert_options:                  {},
-        default_style:                    :original,
-        default_url:                      "/:attachment/:style/missing.png",
-        escape_url:                       true,
-        restricted_characters:            /[&$+,\/:;=?@<>\[\]\{\}\|\\\^~%# ]/,
-        filename_cleaner:                 nil,
-        hash_data:                        ":class/:attachment/:id/:style/:updated_at",
-        hash_digest:                      "SHA1",
-        interpolator:                     Paperclip::Interpolations,
-        only_process:                     [],
-        path:                             ":rails_root/public:url",
-        preserve_files:                   false,
-        processors:                       [:thumbnail],
-        source_file_options:              {},
-        storage:                          :filesystem,
-        styles:                           {},
-        url:                              "/system/:class/:attachment/:id_partition/:style/:filename",
-        url_generator:                    Paperclip::UrlGenerator,
-        use_default_time_zone:            true,
-        use_timestamp:                    true,
-        whiny:                            Paperclip.options[:whiny] || Paperclip.options[:whiny_thumbnails],
-        validate_media_type:              true,
-        adapter_options:                  { hash_digest: Digest::MD5 },
-        check_validity_before_processing: true
+        convert_options:                   {},
+        default_style:                     :original,
+        default_url:                       "/:attachment/:style/missing.png",
+        escape_url:                        true,
+        restricted_characters:             /[&$+,\/:;=?@<>\[\]\{\}\|\\\^~%# ]/,
+        filename_cleaner:                  nil,
+        hash_data:                         ":class/:attachment/:id/:style/:updated_at",
+        hash_digest:                       "SHA1",
+        interpolator:                      Paperclip::Interpolations,
+        only_process:                      [],
+        path:                              ":rails_root/public:url",
+        preserve_files:                    false,
+        processors:                        [:thumbnail],
+        source_file_options:               {},
+        storage:                           :filesystem,
+        styles:                            {},
+        url:                               "/system/:class/:attachment/:id_partition/:style/:filename",
+        url_generator:                     Paperclip::UrlGenerator,
+        use_default_time_zone:             true,
+        use_timestamp:                     true,
+        whiny:                             Paperclip.options[:whiny] || Paperclip.options[:whiny_thumbnails],
+        validate_media_type:               true,
+        adapter_options:                   { hash_digest: Digest::MD5 },
+        check_validity_before_processing:  true,
+        return_file_attributes_on_destroy: false
       }
     end
 
@@ -69,6 +70,8 @@ module Paperclip
     # +interpolator+ - the object used to interpolate filenames and URLs. Defaults to Paperclip::Interpolations
     # +url_generator+ - the object used to generate URLs, using the interpolator. Defaults to Paperclip::UrlGenerator
     # +escape_url+ - Perform URI escaping to URLs. Defaults to true
+    # +return_file_attributes_on_destroy+ - whether attachment-related attributes should be displayed when
+    #                                       destroying a record. Defaults to false.
     def initialize(name, instance, options = {})
       @name              = name.to_sym
       @name_string       = name.to_s
@@ -563,12 +566,10 @@ module Paperclip
           path(style) if exists?(style)
         end.compact
       end
-      instance_write(:file_name, nil)
-      instance_write(:content_type, nil)
-      instance_write(:file_size, nil)
-      instance_write(:fingerprint, nil)
-      instance_write(:created_at, nil) if has_enabled_but_unset_created_at?
-      instance_write(:updated_at, nil)
+
+      return if show_attrs_and_destroy_callback_triggered?
+
+      clear_all_attachment_attributes
     end
 
     def flush_errors #:nodoc:
@@ -613,5 +614,21 @@ module Paperclip
     def has_enabled_but_unset_created_at?
       able_to_store_created_at? && !instance_read(:created_at)
     end
+
+    # Checks whether the option to show attributes to be destroyed is enabled and whether a destroy callback has been
+    # invoked when deleting the entire record.
+    def show_attrs_and_destroy_callback_triggered?
+      @options[:return_file_attributes_on_destroy] && instance.instance_variable_get(:@_destroy_callback_already_called)
+    end
+
+    # Sets attachment-related attributes to `nil`.
+    def clear_all_attachment_attributes
+      instance_write(:file_name, nil)
+      instance_write(:content_type, nil)
+      instance_write(:file_size, nil)
+      instance_write(:fingerprint, nil)
+      instance_write(:created_at, nil) if has_enabled_but_unset_created_at?
+      instance_write(:updated_at, nil)
+    end
   end
 end

data/lib/paperclip/geometry.rb

@@ -72,7 +72,7 @@ module Paperclip
 
     # Returns the width and height in a format suitable to be passed to Geometry.parse
     def to_s
-      s = ""
+      s = String.new
       s << width.to_i.to_s if width > 0
       s << "x#{height.to_i}" if height > 0
       s << modifier.to_s

data/lib/paperclip/io_adapters/abstract_adapter.rb

@@ -17,7 +17,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

@@ -33,6 +33,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/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/schema.rb

@@ -11,7 +11,7 @@ module Paperclip
     def self.included(_base)
       ActiveRecord::ConnectionAdapters::Table.include TableDefinition
       ActiveRecord::ConnectionAdapters::TableDefinition.include TableDefinition
-      ActiveRecord::ConnectionAdapters::AbstractAdapter.include Statements
+      ActiveRecord::Migration.include Statements
       ActiveRecord::Migration::CommandRecorder.include CommandRecorder
     end
 

data/lib/paperclip/storage/fog.rb

@@ -214,9 +214,9 @@ module Paperclip
       def find_credentials(creds)
         case creds
         when File
-          YAML::safe_load(ERB.new(File.read(creds.path)).result)
+          load_credentials_from_file(creds.path)
         when String, Pathname
-          YAML::safe_load(ERB.new(File.read(creds)).result)
+          load_credentials_from_file(creds.path)
         when Hash
           creds
         else
@@ -228,6 +228,14 @@ module Paperclip
         end
       end
 
+      def load_credentials_from_file(path)
+        if Gem::Version.new(Psych::VERSION) >= Gem::Version.new("3.1.0")
+          YAML::safe_load(ERB.new(File.read(path)).result, aliases: true)
+        else
+          YAML::safe_load(ERB.new(File.read(path)).result, [], [], true)
+        end
+      end
+
       def connection
         @connection ||= ::Fog::Storage.new(fog_credentials)
       end

data/lib/paperclip/storage/s3.rb

@@ -1,3 +1,4 @@
+# coding: utf-8
 module Paperclip
   module Storage
     # Amazon's S3 file hosting service is a scalable, easy place to store files for
@@ -280,6 +281,16 @@ module Paperclip
         s3_bucket.object style_name_as_path(style_name)
       end
 
+      def s3_transfer_manager
+        @s3_transfer_manager ||= begin
+          if ::Aws::S3.const_defined?(:TransferManager, false)
+            ::Aws::S3::TransferManager.new(client: s3_interface.client)
+          else
+            nil
+          end
+        end
+      end
+
       def use_accelerate_endpoint?
         !!@use_accelerate_endpoint
       end
@@ -386,7 +397,12 @@ module Paperclip
             write_options[:metadata] = @s3_metadata unless @s3_metadata.empty?
             write_options.merge!(@s3_headers)
 
-            s3_object(style).upload_file(file.path, write_options)
+            if s3_transfer_manager
+              destination = style_name_as_path(style)
+              s3_transfer_manager.upload_file(file.path, **write_options, bucket: bucket_name, key: destination)
+            else
+              s3_object(style).upload_file(file.path, write_options)
+            end
           rescue ::Aws::S3::Errors::NoSuchBucket
             create_bucket
             retry
@@ -422,7 +438,13 @@ module Paperclip
 
       def copy_to_local_file(style, local_dest_path)
         log("copying #{path(style)} to local file #{local_dest_path}")
-        s3_object(style).download_file(local_dest_path)
+        
+        if s3_transfer_manager
+          source = style_name_as_path(style)
+          s3_transfer_manager.download_file(local_dest_path, bucket: bucket_name, key: source)
+        else
+          s3_object(style).download_file(local_dest_path)
+        end
       rescue Aws::Errors::ServiceError => e
         warn("#{e} - cannot copy #{path(style)} to local file #{local_dest_path}")
         false
@@ -433,7 +455,7 @@ module Paperclip
       def find_credentials(creds)
         case creds
         when File
-          load_credentials_from_file(creds.path)
+           load_credentials_from_file(creds.path)
         when String, Pathname
           load_credentials_from_file(creds)
         when Hash

data/lib/paperclip/version.rb

@@ -1,3 +1,3 @@
 module Paperclip
-  VERSION = "7.2.1" unless defined?(Paperclip::VERSION)
+  VERSION = "7.3.0" unless defined?(Paperclip::VERSION)
 end

data/paperclip.gemspec

@@ -26,7 +26,7 @@ Gem::Specification.new do |s|
   s.add_dependency("activesupport", ">= 4.2.0")
   s.add_dependency("mime-types")
   s.add_dependency("marcel", "~> 1.0.1")
-  s.add_dependency("terrapin", "~> 0.6.0")
+  s.add_dependency("terrapin", ">= 0.6.0", "< 2.0")
 
   s.add_development_dependency("activerecord", ">= 4.2.0")
   s.add_development_dependency("appraisal")

data/spec/paperclip/attachment_spec.rb

@@ -1267,7 +1267,7 @@ describe Paperclip::Attachment do
                 @existing_names.each { |f| assert_file_not_exists(f) }
               end
 
-              it "deletes the files when you call #delete" do
+              it "deletes the files when you call #destroy" do
                 expect(@attachment).to receive(:instance_write).with(:file_name, nil)
                 expect(@attachment).to receive(:instance_write).with(:content_type, nil)
                 expect(@attachment).to receive(:instance_write).with(:file_size, nil)
@@ -1277,6 +1277,32 @@ describe Paperclip::Attachment do
                 @existing_names.each { |f| assert_file_not_exists(f) }
               end
 
+              it "deletes the files when you destroy the model" do
+                expect(@attachment).to receive(:instance_write).with(:file_name, nil)
+                expect(@attachment).to receive(:instance_write).with(:content_type, nil)
+                expect(@attachment).to receive(:instance_write).with(:file_size, nil)
+                expect(@attachment).to receive(:instance_write).with(:fingerprint, nil)
+                expect(@attachment).to receive(:instance_write).with(:updated_at, nil)
+                @attachment.instance.destroy
+                @existing_names.each { |f| assert_file_not_exists(f) }
+              end
+
+              context "when 'return_file_attributes_on_destroy' option is set to true" do
+                before do
+                  @attachment.options[:return_file_attributes_on_destroy] = true
+                end
+
+                it "does not override attachment-related attributes and deletes the files" do
+                  expect(@attachment).not_to receive(:instance_write)
+                  expect(@attachment).not_to receive(:instance_write)
+                  expect(@attachment).not_to receive(:instance_write)
+                  expect(@attachment).not_to receive(:instance_write)
+                  expect(@attachment).not_to receive(:instance_write)
+                  @attachment.instance.destroy
+                  @existing_names.each { |f| assert_file_not_exists(f) }
+                end
+              end
+
               context "when keeping old files" do
                 before do
                   @attachment.options[:keep_old_files] = true
@@ -1304,7 +1330,7 @@ describe Paperclip::Attachment do
                   @existing_names.each { |f| assert_file_exists(f) }
                 end
 
-                it "keeps the files when you call #delete" do
+                it "keeps the files when you call #destroy" do
                   expect(@attachment).to receive(:instance_write).with(:file_name, nil)
                   expect(@attachment).to receive(:instance_write).with(:content_type, nil)
                   expect(@attachment).to receive(:instance_write).with(:file_size, nil)
@@ -1333,7 +1359,7 @@ describe Paperclip::Attachment do
 
   context "An attachment with only a avatar_file_name column" do
     before do
-      ActiveRecord::Base.connection.create_table :dummies, force: true do |table|
+      ActiveRecord::Migration.create_table :dummies, force: true do |table|
         table.column :avatar_file_name, :string
       end
       rebuild_class
@@ -1359,7 +1385,7 @@ describe Paperclip::Attachment do
 
     context "and avatar_created_at column" do
       before do
-        ActiveRecord::Base.connection.add_column :dummies, :avatar_created_at, :timestamp
+        ActiveRecord::Migration.add_column :dummies, :avatar_created_at, :timestamp
         rebuild_class
         @dummy = Dummy.new
       end
@@ -1396,7 +1422,7 @@ describe Paperclip::Attachment do
 
     context "and avatar_updated_at column" do
       before do
-        ActiveRecord::Base.connection.add_column :dummies, :avatar_updated_at, :timestamp
+        ActiveRecord::Migration.add_column :dummies, :avatar_updated_at, :timestamp
         rebuild_class
         @dummy = Dummy.new
       end
@@ -1426,7 +1452,7 @@ describe Paperclip::Attachment do
 
     context "and avatar_content_type column" do
       before do
-        ActiveRecord::Base.connection.add_column :dummies, :avatar_content_type, :string
+        ActiveRecord::Migration.add_column :dummies, :avatar_content_type, :string
         rebuild_class
         @dummy = Dummy.new
       end
@@ -1443,7 +1469,7 @@ describe Paperclip::Attachment do
 
     context "and avatar_file_size column" do
       before do
-        ActiveRecord::Base.connection.add_column :dummies, :avatar_file_size, :bigint
+        ActiveRecord::Migration.add_column :dummies, :avatar_file_size, :bigint
         rebuild_class
         @dummy = Dummy.new
       end
@@ -1467,7 +1493,7 @@ describe Paperclip::Attachment do
 
     context "and avatar_fingerprint column" do
       before do
-        ActiveRecord::Base.connection.add_column :dummies, :avatar_fingerprint, :string
+        ActiveRecord::Migration.add_column :dummies, :avatar_fingerprint, :string
         rebuild_class
         @dummy = Dummy.new
       end

data/spec/paperclip/io_adapters/attachment_adapter_spec.rb

@@ -6,6 +6,33 @@ describe Paperclip::AttachmentAdapter do
     @attachment = Dummy.new.avatar
   end
 
+  context "for an attachment that renames the source to the destination" do
+    before do
+      @file = File.new(fixture_file("5k.png"))
+      @file.binmode
+
+      @attachment.assign(@file)
+      @attachment.save
+
+      allow(@attachment).to receive(:copy_to_local_file) do |style, path|
+        FileUtils.cp(@file.path, "#{@file.path}.tmp")
+        FileUtils.mv("#{@file.path}.tmp", path)
+      end
+
+      @subject = Paperclip.io_adapters.for(@attachment,
+                                           hash_digest: Digest::MD5)
+    end
+
+    after do
+      @file.close
+      @subject.close
+    end
+
+    it "handles the case where the source is renamed to the destination" do
+      assert_equal @file.read, @subject.read
+    end
+  end
+
   context "for an attachment" do
     before do
       @file = File.new(fixture_file("5k.png"))

data/spec/paperclip/paperclip_missing_attachment_styles_spec.rb

@@ -49,7 +49,7 @@ describe "Missing Attachment Styles" do
     expected_hash = { Dummy: { avatar: [:export, :thumb] } }
     assert_equal expected_hash, Paperclip.missing_attachments_styles
 
-    ActiveRecord::Base.connection.create_table :books, force: true
+    ActiveRecord::Migration.create_table :books, force: true
     class ::Book < ActiveRecord::Base
       has_attached_file :cover, styles: { small: "x100", large: "1000x1000>" }
       has_attached_file :sample, styles: { thumb: "x100" }

data/spec/paperclip/schema_spec.rb

@@ -11,7 +11,7 @@ describe Paperclip::Schema do
 
   after do
     begin
-      Dummy.connection.drop_table :dummies
+      ActiveRecord::Migration.drop_table :dummies
     rescue StandardError
       nil
     end
@@ -23,7 +23,7 @@ describe Paperclip::Schema do
         ActiveSupport::Deprecation.silenced = false
       end
       it "creates attachment columns" do
-        Dummy.connection.create_table :dummies, force: true do |t|
+        ActiveRecord::Migration.create_table :dummies, force: true do |t|
           ActiveSupport::Deprecation.silence do
             t.has_attached_file :avatar
           end
@@ -38,7 +38,7 @@ describe Paperclip::Schema do
       end
 
       it "displays deprecation warning" do
-        Dummy.connection.create_table :dummies, force: true do |t|
+        ActiveRecord::Migration.create_table :dummies, force: true do |t|
           assert_deprecated do
             t.has_attached_file :avatar
           end
@@ -48,7 +48,7 @@ describe Paperclip::Schema do
 
     context "using #attachment" do
       before do
-        Dummy.connection.create_table :dummies, force: true do |t|
+        ActiveRecord::Migration.create_table :dummies, force: true do |t|
           t.attachment :avatar
         end
       end
@@ -65,7 +65,7 @@ describe Paperclip::Schema do
 
     context "using #attachment with options" do
       before do
-        Dummy.connection.create_table :dummies, force: true do |t|
+        ActiveRecord::Migration.create_table :dummies, force: true do |t|
           t.attachment :avatar, default: 1, file_name: { default: "default" }
         end
       end
@@ -83,13 +83,13 @@ describe Paperclip::Schema do
 
   context "within schema statement" do
     before do
-      Dummy.connection.create_table :dummies, force: true
+      ActiveRecord::Migration.create_table :dummies, force: true
     end
 
     context "migrating up" do
       context "with single attachment" do
         before do
-          Dummy.connection.add_attachment :dummies, :avatar
+          ActiveRecord::Migration.add_attachment :dummies, :avatar
         end
 
         it "creates attachment columns" do
@@ -104,7 +104,7 @@ describe Paperclip::Schema do
 
       context "with single attachment and options" do
         before do
-          Dummy.connection.add_attachment :dummies, :avatar, default: "1", file_name: { default: "default" }
+          ActiveRecord::Migration.add_attachment :dummies, :avatar, default: "1", file_name: { default: "default" }
         end
 
         it "sets defaults on columns" do
@@ -119,7 +119,7 @@ describe Paperclip::Schema do
 
       context "with multiple attachments" do
         before do
-          Dummy.connection.add_attachment :dummies, :avatar, :photo
+          ActiveRecord::Migration.add_attachment :dummies, :avatar, :photo
         end
 
         it "creates attachment columns" do
@@ -138,7 +138,7 @@ describe Paperclip::Schema do
 
       context "with multiple attachments and options" do
         before do
-          Dummy.connection.add_attachment :dummies, :avatar, :photo, default: "1", file_name: { default: "default" }
+          ActiveRecord::Migration.add_attachment :dummies, :avatar, :photo, default: "1", file_name: { default: "default" }
         end
 
         it "sets defaults on columns" do
@@ -157,7 +157,7 @@ describe Paperclip::Schema do
       context "with no attachment" do
         it "raises an error" do
           assert_raises ArgumentError do
-            Dummy.connection.add_attachment :dummies
+            ActiveRecord::Migration.add_attachment :dummies
           end
         end
       end
@@ -165,7 +165,7 @@ describe Paperclip::Schema do
 
     context "migrating down" do
       before do
-        Dummy.connection.change_table :dummies do |t|
+        ActiveRecord::Migration.change_table :dummies do |t|
           t.column :avatar_file_name, :string
           t.column :avatar_content_type, :string
           t.column :avatar_file_size, :bigint
@@ -179,7 +179,7 @@ describe Paperclip::Schema do
         end
         it "removes the attachment columns" do
           ActiveSupport::Deprecation.silence do
-            Dummy.connection.drop_attached_file :dummies, :avatar
+            ActiveRecord::Migration.drop_attached_file :dummies, :avatar
           end
 
           columns = Dummy.columns.map { |column| [column.name, column.sql_type] }
@@ -192,7 +192,7 @@ describe Paperclip::Schema do
 
         it "displays a deprecation warning" do
           assert_deprecated do
-            Dummy.connection.drop_attached_file :dummies, :avatar
+            ActiveRecord::Migration.drop_attached_file :dummies, :avatar
           end
         end
       end
@@ -200,7 +200,7 @@ describe Paperclip::Schema do
       context "using #remove_attachment" do
         context "with single attachment" do
           before do
-            Dummy.connection.remove_attachment :dummies, :avatar
+            ActiveRecord::Migration.remove_attachment :dummies, :avatar
           end
 
           it "removes the attachment columns" do
@@ -215,14 +215,14 @@ describe Paperclip::Schema do
 
         context "with multiple attachments" do
           before do
-            Dummy.connection.change_table :dummies do |t|
+            ActiveRecord::Migration.change_table :dummies do |t|
               t.column :photo_file_name, :string
               t.column :photo_content_type, :string
               t.column :photo_file_size, :bigint
               t.column :photo_updated_at, :datetime
             end
 
-            Dummy.connection.remove_attachment :dummies, :avatar, :photo
+            ActiveRecord::Migration.remove_attachment :dummies, :avatar, :photo
           end
 
           it "removes the attachment columns" do
@@ -242,7 +242,7 @@ describe Paperclip::Schema do
         context "with no attachment" do
           it "raises an error" do
             assert_raises ArgumentError do
-              Dummy.connection.remove_attachment :dummies
+              ActiveRecord::Migration.remove_attachment :dummies
             end
           end
         end

data/spec/paperclip/storage/s3_spec.rb

@@ -6,6 +6,10 @@ describe Paperclip::Storage::S3 do
     Aws.config[:stub_responses] = true
   end
 
+  def s3_uses_transfer_manager?
+    defined?(Aws::S3::TransferManager)
+  end
+
   def aws2_add_region
     { s3_region: "us-east-1" }
   end
@@ -389,27 +393,30 @@ describe Paperclip::Storage::S3 do
 
       @dummy = Dummy.new
       @dummy.avatar = @file
-
       object = double
-      allow(@dummy.avatar).to receive(:s3_object).with(:original).and_return(object)
-      allow(@dummy.avatar).to receive(:s3_object).with(:thumbnail).and_return(object)
 
+      if s3_uses_transfer_manager?
+        allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+      else
+        allow(@dummy.avatar).to receive(:s3_object).with(:original).and_return(object)
+        allow(@dummy.avatar).to receive(:s3_object).with(:thumbnail).and_return(object)
+      end
       expect(object).to receive(:upload_file).
         with(
           anything,
-          {
+          hash_including(
             content_type: "image/png",
             acl: :"public-read",
-          },
+          ),
         )
       expect(object).to receive(:upload_file).
         with(
           anything,
-          {
+          hash_including(
             content_type: "image/png",
             acl: :"public-read",
             cache_control: "max-age=31557600",
-          },
+          ),
         )
       @dummy.save
     end
@@ -417,7 +424,12 @@ describe Paperclip::Storage::S3 do
     after { @file.close }
 
     it "succeeds" do
-      assert_equal @dummy.counter, 7
+      if s3_uses_transfer_manager?
+        count = 9
+      else
+        count = 7
+      end
+      assert_equal @dummy.counter, count
     end
   end
 
@@ -445,22 +457,27 @@ describe Paperclip::Storage::S3 do
     context "reprocess" do
       before do
         @object = double
-        allow(@dummy.avatar).to receive(:s3_object).with(:original).and_return(@object)
-        allow(@dummy.avatar).to receive(:s3_object).with(:thumb).and_return(@object)
-        allow(@object).to receive(:get).and_yield(@file.read)
-        allow(@object).to receive(:exists?).and_return(true)
-        allow(@object).to receive(:download_file).with(anything)
+        if s3_uses_transfer_manager?
+          allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(@object)
+          allow(@object).to receive(:download_file).with(any_args)
+        else
+          allow(@dummy.avatar).to receive(:s3_object).with(:original).and_return(@object)
+          allow(@dummy.avatar).to receive(:s3_object).with(:thumb).and_return(@object)
+          allow(@object).to receive(:get).and_yield(@file.read)
+          allow(@object).to receive(:exists?).and_return(true)
+          allow(@object).to receive(:download_file).with(anything)
+        end
       end
 
       it "uploads original" do
         expect(@object).to receive(:upload_file).with(
           anything,
-          { content_type: "image/png" },
+          hash_including(content_type: "image/png"),
         ).and_return(true)
         @dummy.avatar.reprocess!
         expect(@object).to receive(:upload_file).with(
           anything,
-          { content_type: "image/png" },
+          hash_including(content_type: "image/png"),
         ).and_return(true)
         @dummy.avatar.reprocess!
       end
@@ -468,7 +485,7 @@ describe Paperclip::Storage::S3 do
       it "doesn't upload original" do
         expect(@object).to receive(:upload_file).with(
           anything,
-          { content_type: "image/png" },
+          hash_including(content_type: "image/png"),
         ).and_return(true)
         @dummy.avatar.reprocess!
       end
@@ -500,28 +517,33 @@ describe Paperclip::Storage::S3 do
     context "reprocess" do
       before do
         @object = double
-        allow(@dummy.avatar).to receive(:s3_object).with(:original).and_return(@object)
-        allow(@dummy.avatar).to receive(:s3_object).with(:thumb).and_return(@object)
-        allow(@object).to receive(:get).and_yield(@file.read)
-        allow(@object).to receive(:exists?).and_return(true)
-        allow(@object).to receive(:download_file).with(anything)
+        if s3_uses_transfer_manager?
+          allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(@object)
+          allow(@object).to receive(:download_file).with(any_args)
+        else
+          allow(@dummy.avatar).to receive(:s3_object).with(:original).and_return(@object)
+          allow(@dummy.avatar).to receive(:s3_object).with(:thumb).and_return(@object)
+          allow(@object).to receive(:get).and_yield(@file.read)
+          allow(@object).to receive(:exists?).and_return(true)
+          allow(@object).to receive(:download_file).with(anything)
+        end
       end
 
       it "uploads original" do
         expect(@object).to receive(:upload_file).with(
           anything,
-          {
+          hash_including(
             content_type: "image/png",
             acl: :"public-read",
-          },
+          ),
         ).and_return(true)
         @dummy.avatar.reprocess!
         expect(@object).to receive(:upload_file).with(
           anything,
-          {
+          hash_including(
             content_type: "image/png",
             acl: :"public-read",
-          },
+          ),
         ).and_return(true)
         @dummy.avatar.reprocess!
       end
@@ -529,10 +551,10 @@ describe Paperclip::Storage::S3 do
       it "doesn't upload original" do
         expect(@object).to receive(:upload_file).with(
           anything,
-          {
+          hash_including(
             content_type: "image/png",
             acl: :"public-read",
-          },
+          ),
         ).and_return(true)
         @dummy.avatar.reprocess!
       end
@@ -973,9 +995,12 @@ describe Paperclip::Storage::S3 do
 
       it "will retry to save again but back off on SlowDown" do
         allow(@dummy.avatar).to receive(:sleep)
-        allow_any_instance_of(Aws::S3::Object).to receive(:upload_file).
-          and_raise(Aws::S3::Errors::SlowDown.new(spy,
-                                                  spy(status: 503, body: "")))
+        err = Aws::S3::Errors::SlowDown.new(spy, spy(status: 503, body: ""))
+        if s3_uses_transfer_manager?
+          allow_any_instance_of(Aws::S3::TransferManager).to receive(:upload_file).and_raise(err)
+        else
+          allow_any_instance_of(Aws::S3::Object).to receive(:upload_file).and_raise(err)
+        end
         expect { @dummy.save }.to raise_error(Aws::S3::Errors::SlowDown)
         expect(@dummy.avatar).to have_received(:sleep).with(1)
         expect(@dummy.avatar).to have_received(:sleep).with(2)
@@ -987,9 +1012,13 @@ describe Paperclip::Storage::S3 do
       context "and saved" do
         before do
           object = double
-          allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+          if s3_uses_transfer_manager?
+            allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+          else
+            allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+          end
           expect(object).to receive(:upload_file).
-            with(anything, { content_type: "image/png", acl: :"public-read" })
+            with(anything, hash_including(content_type: "image/png", acl: :"public-read"))
           @dummy.save
         end
 
@@ -1146,16 +1175,19 @@ describe Paperclip::Storage::S3 do
       context "and saved" do
         before do
           object = double
-          allow(@dummy.avatar).to receive(:s3_object).and_return(object)
-
+          if s3_uses_transfer_manager?
+            allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+          else
+            allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+          end
           expect(object).to receive(:upload_file).
             with(
               anything,
-              {
+              hash_including(
                 content_type: "image/png",
                 acl: :"public-read",
                 cache_control: "max-age=31557600",
-              },
+              ),
             )
           @dummy.save
         end
@@ -1191,16 +1223,20 @@ describe Paperclip::Storage::S3 do
       context "and saved" do
         before do
           object = double
-          allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+          if s3_uses_transfer_manager?
+            allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+          else
+            allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+          end
 
           expect(object).to receive(:upload_file).
             with(
               anything,
-              {
+              hash_including(
                  content_type: "image/png",
                  acl: :"public-read",
                  metadata: { "color" => "red" },
-              },
+              ),
             )
           @dummy.save
         end
@@ -1236,16 +1272,20 @@ describe Paperclip::Storage::S3 do
       context "and saved" do
         before do
           object = double
-          allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+          if s3_uses_transfer_manager?
+            allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+          else
+            allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+          end
 
           expect(object).to receive(:upload_file).
             with(
               anything,
-              {
+              hash_including(
                  content_type: "image/png",
                  acl: :"public-read",
                  metadata: { "color" => "red" },
-              },
+              ),
             )
           @dummy.save
         end
@@ -1282,16 +1322,20 @@ describe Paperclip::Storage::S3 do
         context "and saved" do
           before do
             object = double
-            allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+            if s3_uses_transfer_manager?
+              allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+            else
+              allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+            end
 
             expect(object).to receive(:upload_file).
               with(
                 anything,
-                {
+                hash_including(
                    content_type: "image/png",
                    acl: :"public-read",
                    storage_class: "reduced_redundancy",
-                },
+                ),
               )
             @dummy.save
           end
@@ -1333,7 +1377,11 @@ describe Paperclip::Storage::S3 do
           before do
             object = double
             [:thumb, :original].each do |style|
-              allow(@dummy.avatar).to receive(:s3_object).with(style).and_return(object)
+              if s3_uses_transfer_manager?
+                allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+              else
+                allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+              end
 
               expected_options = {
                 content_type: "image/png",
@@ -1342,7 +1390,7 @@ describe Paperclip::Storage::S3 do
               expected_options.merge!(storage_class: :reduced_redundancy) if style == :thumb
 
               expect(object).to receive(:upload_file).
-                with(anything, expected_options)
+                with(anything, hash_including(expected_options))
             end
             @dummy.save
           end
@@ -1382,16 +1430,20 @@ describe Paperclip::Storage::S3 do
           before do
             object = double
             [:thumb, :original].each do |style|
-              allow(@dummy.avatar).to receive(:s3_object).with(style).and_return(object)
+              if s3_uses_transfer_manager?
+                allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+              else
+                allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+              end
 
               expect(object).to receive(:upload_file).
                 with(
                   anything,
-                  {
+                  hash_including(
                     content_type: "image/png",
                     acl: :"public-read",
                     storage_class: :reduced_redundancy,
-                  },
+                  ),
                 )
             end
             @dummy.save
@@ -1432,10 +1484,14 @@ describe Paperclip::Storage::S3 do
         context "and saved" do
           before do
             object = double
-            allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+            if s3_uses_transfer_manager?
+              allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+            else
+              allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+            end
 
             expect(object).to receive(:upload_file).
-              with(anything, { content_type: "image/png", acl: :"public-read" })
+              with(anything, hash_including(content_type: "image/png", acl: :"public-read"))
             @dummy.save
           end
 
@@ -1471,16 +1527,20 @@ describe Paperclip::Storage::S3 do
       context "and saved" do
         before do
           object = double
-          allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+          if s3_uses_transfer_manager?
+            allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+          else
+            allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+          end
 
           expect(object).to receive(:upload_file).
             with(
               anything,
-              {
+              hash_including(
                 content_type: "image/png",
                 acl: :"public-read",
                 server_side_encryption: "AES256",
-              },
+              ),
             )
           @dummy.save
         end
@@ -1516,16 +1576,20 @@ describe Paperclip::Storage::S3 do
       context "and saved" do
         before do
           object = double
-          allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+          if s3_uses_transfer_manager?
+            allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+          else
+            allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+          end
 
           expect(object).to receive(:upload_file).
             with(
               anything,
-              {
+              hash_including(
                 content_type: "image/png",
                 acl: :"public-read",
                 storage_class: :reduced_redundancy,
-              },
+              ),
             )
           @dummy.save
         end
@@ -1667,10 +1731,14 @@ describe Paperclip::Storage::S3 do
         context "and saved" do
           before do
             object = double
-            allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+            if s3_uses_transfer_manager?
+              allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+            else
+              allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+            end
 
             expect(object).to receive(:upload_file).
-              with(anything, { content_type: "image/png", acl: :"public-read" })
+              with(anything, hash_including(content_type: "image/png", acl: :"public-read"))
             @dummy.save
           end
 
@@ -1705,10 +1773,14 @@ describe Paperclip::Storage::S3 do
         context "and saved" do
           before do
             object = double
-            allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+            if s3_uses_transfer_manager?
+              allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object)
+            else
+              allow(@dummy.avatar).to receive(:s3_object).and_return(object)
+            end
 
             expect(object).to receive(:upload_file).
-              with(anything, { content_type: "image/png", acl: :private })
+              with(anything, hash_including(content_type: "image/png", acl: :private))
             @dummy.save
           end
 
@@ -1748,16 +1820,21 @@ describe Paperclip::Storage::S3 do
 
         context "and saved" do
           before do
-            [:thumb, :original].each do |style|
-              object = double
-              allow(@dummy.avatar).to receive(:s3_object).with(style).and_return(object)
+            object = double
+            allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object) if s3_uses_transfer_manager?
 
+            [:thumb, :original].each do |style|
+              expected_args = { content_type: "image/png", acl: style == :thumb ? :public_read : :private }
+              if s3_uses_transfer_manager?
+                expected_args = expected_args.merge({bucket: "testing", key: "avatars/#{style}/5k.png"})
+              else
+                allow(@dummy.avatar).to receive(:s3_object).with(style).and_return(object)
+              end
               expect(object).to receive(:upload_file).
-                with(anything,
-                     {
-                       content_type: "image/png",
-                       acl: style == :thumb ? :public_read : :private
-                     })
+                with(
+                  anything,
+                  hash_including(expected_args)
+                )
             end
             @dummy.save
           end
@@ -1821,18 +1898,24 @@ describe Paperclip::Storage::S3 do
 
       context "and saved" do
         before do
-          [:thumb, :original].each do |style|
-            object = double
-            allow(@dummy.avatar).to receive(:s3_object).with(style).and_return(object)
+          object = double
+          allow(@dummy.avatar).to receive(:s3_transfer_manager).and_return(object) if s3_uses_transfer_manager?
 
+          [:thumb, :original].each do |style|
+            expected_args = {
+              content_type: "image/png",
+              acl: :"public-read",
+              content_disposition: 'attachment; filename="Custom Avatar Name.png"'
+            }
+            if s3_uses_transfer_manager?
+              expected_args = expected_args.merge({bucket: "testing", key: "avatars/#{style}/5k.png"})
+            else
+              allow(@dummy.avatar).to receive(:s3_object).with(style).and_return(object)
+            end
             expect(object).to receive(:upload_file).
               with(
                 anything,
-                {
-                  content_type: "image/png",
-                  acl: :"public-read",
-                  content_disposition: 'attachment; filename="Custom Avatar Name.png"',
-                },
+                hash_including(expected_args)
               )
           end
           @dummy.save

data/spec/spec_helper.rb

@@ -21,6 +21,7 @@ FIXTURES_DIR = File.join(File.dirname(__FILE__), "fixtures")
 config = YAML::safe_load(IO.read(File.dirname(__FILE__) + "/database.yml"))
 ActiveRecord::Base.logger = Logger.new(File.dirname(__FILE__) + "/debug.log")
 ActiveRecord::Base.establish_connection(config["test"])
+ActiveRecord::Migration.verbose = false
 if ActiveRecord::VERSION::STRING >= "4.2" &&
    ActiveRecord::VERSION::STRING < "5.0"
   ActiveRecord::Base.raise_in_transactional_callbacks = true

data/spec/support/model_reconstruction.rb

@@ -28,15 +28,15 @@ module ModelReconstruction
 
   def reset_table(_table_name, &block)
     block ||= lambda { |_table| true }
-    ActiveRecord::Base.connection.create_table :dummies, **{ force: true }, &block
+    ActiveRecord::Migration.create_table :dummies, **{ force: true }, &block
   end
 
   def modify_table(&block)
-    ActiveRecord::Base.connection.change_table :dummies, &block
+    ActiveRecord::Migration.change_table :dummies, &block
   end
 
   def rebuild_model(options = {})
-    ActiveRecord::Base.connection.create_table :dummies, force: true do |table|
+    ActiveRecord::Migration.create_table :dummies, force: true do |table|
       table.column :title, :string
       table.column :other, :string
       table.column :avatar_file_name, :string

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kt-paperclip
 version: !ruby/object:Gem::Version
-  version: 7.2.1
+  version: 7.3.0
 platform: ruby
 authors:
 - Surendra Singhi
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-09-09 00:00:00.000000000 Z
+date: 2026-01-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -70,16 +70,22 @@ dependencies:
   name: terrapin
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.6.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.6.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
@@ -366,7 +372,7 @@ files:
 - CONTRIBUTING.md
 - Gemfile
 - LICENSE
-- NEWS
+- OLD_NEWS
 - README.md
 - RELEASING.md
 - Rakefile
@@ -428,6 +434,7 @@ files:
 - lib/paperclip/io_adapters/uploaded_file_adapter.rb
 - lib/paperclip/io_adapters/uri_adapter.rb
 - lib/paperclip/locales/en.yml
+- lib/paperclip/locales/fr.yml
 - lib/paperclip/locales/gd.yml
 - lib/paperclip/logger.rb
 - lib/paperclip/matchers.rb