Added in_after_callback argument to save_with_version

CHANGELOG.md

@@ -5,6 +5,10 @@ recommendations of [keepachangelog.com](http://keepachangelog.com/).
 
 ## Unreleased
 
+- Added `in_after_callback` argument to `PaperTrail::RecordTrail#save_with_version`,
+  to allow the caller to indicate if this method is being called during an `after`
+  callback. Defaults to `false`.
+
 ### Breaking Changes
 
 - None

lib/paper_trail/record_trail.rb

@@ -194,15 +194,17 @@ def source_version
     # Save, and create a version record regardless of options such as `:on`,
     # `:if`, or `:unless`.
     #
-    # Arguments are passed to `save`.
+    # `in_after_callback`: Indicates if this method is being called within an
+    #                      `after` callback. Defaults to `false`.
+    # `options`: Optional arguments passed to `save`.
     #
     # This is an "update" event. That is, we record the same data we would in
     # the case of a normal AR `update`.
-    def save_with_version(**options)
+    def save_with_version(in_after_callback: false, **options)
       ::PaperTrail.request(enabled: false) do
         @record.save(**options)
       end
-      record_update(force: true, in_after_callback: false, is_touch: false)
+      record_update(force: true, in_after_callback: in_after_callback, is_touch: false)
     end
 
     # Like the `update_column` method from `ActiveRecord::Persistence`, but also

spec/dummy_app/db/migrate/20110208155312_set_up_test_tables.rb

@@ -144,6 +144,7 @@ def up
     end
 
     create_table :not_on_updates, force: true do |t|
+      t.string :name
       t.timestamps null: true, limit: 6
     end
 

spec/models/not_on_update_spec.rb

@@ -11,5 +11,12 @@
         PaperTrail::Version.count
       }.by(+1)
     end
+
+    it "captures changes when in_after_callback is true" do
+      record.name = "test"
+      record.paper_trail.save_with_version(in_after_callback: true)
+      changeset = record.versions.last.changeset
+      expect(changeset[:name]).to eq([nil, "test"])
+    end
   end
 end