paper_trail 5.2.3 → 11.0.0

data/lib/paper_trail/events/create.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+require "paper_trail/events/base"
+
+module PaperTrail
+  module Events
+    # See docs in `Base`.
+    #
+    # @api private
+    class Create < Base
+      # Return attributes of nascent `Version` record.
+      #
+      # @api private
+      def data
+        data = {
+          item: @record,
+          event: @record.paper_trail_event || "create",
+          whodunnit: PaperTrail.request.whodunnit
+        }
+        if @record.respond_to?(:updated_at)
+          data[:created_at] = @record.updated_at
+        end
+        if record_object_changes? && changed_notably?
+          changes = notable_changes
+          data[:object_changes] = prepare_object_changes(changes)
+        end
+        merge_item_subtype_into(data)
+        merge_metadata_into(data)
+      end
+    end
+  end
+end

data/lib/paper_trail/events/destroy.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require "paper_trail/events/base"
+
+module PaperTrail
+  module Events
+    # See docs in `Base`.
+    #
+    # @api private
+    class Destroy < Base
+      # Return attributes of nascent `Version` record.
+      #
+      # @api private
+      def data
+        data = {
+          item_id: @record.id,
+          item_type: @record.class.base_class.name,
+          event: @record.paper_trail_event || "destroy",
+          whodunnit: PaperTrail.request.whodunnit
+        }
+        if record_object?
+          data[:object] = recordable_object(false)
+        end
+        if record_object_changes?
+          data[:object_changes] = prepare_object_changes(notable_changes)
+        end
+        merge_item_subtype_into(data)
+        merge_metadata_into(data)
+      end
+
+      private
+
+      # Rails' implementation (eg. `@record.saved_changes`) returns nothing on
+      # destroy, so we have to build the hash we want.
+      #
+      # @override
+      def changes_in_latest_version
+        @record.attributes.map { |attr, value| [attr, [value, nil]] }.to_h
+      end
+    end
+  end
+end

data/lib/paper_trail/events/update.rb

@@ -0,0 +1,60 @@
+# frozen_string_literal: true
+
+require "paper_trail/events/base"
+
+module PaperTrail
+  module Events
+    # See docs in `Base`.
+    #
+    # @api private
+    class Update < Base
+      # - is_touch - [boolean] - Used in the two situations that are touch-like:
+      #   - `after_touch` we call `RecordTrail#record_update`
+      # - force_changes - [Hash] - Only used by `RecordTrail#update_columns`,
+      #   because there dirty-tracking is off, so it has to track its own changes.
+      #
+      # @api private
+      def initialize(record, in_after_callback, is_touch, force_changes)
+        super(record, in_after_callback)
+        @is_touch = is_touch
+        @force_changes = force_changes
+      end
+
+      # Return attributes of nascent `Version` record.
+      #
+      # @api private
+      def data
+        data = {
+          item: @record,
+          event: @record.paper_trail_event || "update",
+          whodunnit: PaperTrail.request.whodunnit
+        }
+        if @record.respond_to?(:updated_at)
+          data[:created_at] = @record.updated_at
+        end
+        if record_object?
+          data[:object] = recordable_object(@is_touch)
+        end
+        if record_object_changes?
+          changes = @force_changes.nil? ? notable_changes : @force_changes
+          data[:object_changes] = prepare_object_changes(changes)
+        end
+        merge_item_subtype_into(data)
+        merge_metadata_into(data)
+      end
+
+      private
+
+      # `touch` cannot record `object_changes` because rails' `touch` does not
+      # perform dirty-tracking. Specifically, methods from `Dirty`, like
+      # `saved_changes`, return the same values before and after `touch`.
+      #
+      # See https://github.com/rails/rails/issues/33429
+      #
+      # @api private
+      def record_object_changes?
+        !@is_touch && super
+      end
+    end
+  end
+end

data/lib/paper_trail/frameworks/active_record.rb

@@ -1,4 +1,5 @@
+# frozen_string_literal: true
+
 # This file only needs to be loaded if the gem is being used outside of Rails,
 # since otherwise the model(s) will get loaded in via the `Rails::Engine`.
-require "paper_trail/frameworks/active_record/models/paper_trail/version_association"
 require "paper_trail/frameworks/active_record/models/paper_trail/version"

data/lib/paper_trail/frameworks/active_record/models/paper_trail/version.rb

@@ -1,10 +1,15 @@
+# frozen_string_literal: true
+
 require "paper_trail/version_concern"
 
 module PaperTrail
   # This is the default ActiveRecord model provided by PaperTrail. Most simple
-  # applications will only use this and its partner, `VersionAssociation`, but
-  # it is possible to sub-class, extend, or even do without this model entirely.
-  # See the readme for details.
+  # applications will use this model as-is, but it is possible to sub-class,
+  # extend, or even do without this model entirely. See documentation section
+  # 6.a. Custom Version Classes.
+  #
+  # The paper_trail-association_tracking gem provides a related model,
+  # `VersionAssociation`.
   class Version < ::ActiveRecord::Base
     include PaperTrail::VersionConcern
   end

data/lib/paper_trail/frameworks/cucumber.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 # before hook for Cucumber
 Before do
   PaperTrail.enabled = false
-  PaperTrail.enabled_for_controller = true
-  PaperTrail.whodunnit = nil
-  PaperTrail.controller_info = {} if defined? Rails
+  PaperTrail.request.enabled = true
+  PaperTrail.request.whodunnit = nil
+  PaperTrail.request.controller_info = {} if defined?(::Rails)
 end
 
 module PaperTrail

data/lib/paper_trail/frameworks/rails.rb

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 require "paper_trail/frameworks/rails/controller"
 require "paper_trail/frameworks/rails/engine"

data/lib/paper_trail/frameworks/rails/controller.rb

@@ -1,27 +1,17 @@
+# frozen_string_literal: true
+
 module PaperTrail
   module Rails
     # Extensions to rails controllers. Provides convenient ways to pass certain
     # information to the model layer, with `controller_info` and `whodunnit`.
-    # Also includes a convenient on/off switch, `enabled_for_controller`.
+    # Also includes a convenient on/off switch,
+    # `paper_trail_enabled_for_controller`.
     module Controller
-      def self.included(base)
-        before = [
+      def self.included(controller)
+        controller.before_action(
           :set_paper_trail_enabled_for_controller,
           :set_paper_trail_controller_info
-        ]
-        after = [
-          :warn_about_not_setting_whodunnit
-        ]
-
-        if base.respond_to? :before_action
-          # Rails 4+
-          before.map { |sym| base.before_action sym }
-          after.map  { |sym| base.after_action  sym }
-        else
-          # Rails 3.
-          before.map { |sym| base.before_filter sym }
-          after.map  { |sym| base.after_filter  sym }
-        end
+        )
       end
 
       protected
@@ -31,11 +21,11 @@ module PaperTrail
       #
       # Override this method in your controller to call a different
       # method, e.g. `current_person`, or anything you like.
+      #
+      # @api public
       def user_for_paper_trail
         return unless defined?(current_user)
-        ActiveSupport::VERSION::MAJOR >= 4 ? current_user.try!(:id) : current_user.try(:id)
-      rescue NoMethodError
-        current_user
+        current_user.try(:id) || current_user
       end
 
       # Returns any information about the controller or request that you
@@ -56,6 +46,8 @@ module PaperTrail
       # Use the `:meta` option to
       # `PaperTrail::Model::ClassMethods.has_paper_trail` to store any extra
       # model-level data you need.
+      #
+      # @api public
       def info_for_paper_trail
         {}
       end
@@ -65,6 +57,15 @@ module PaperTrail
       #
       # Override this method in your controller to specify when PaperTrail
       # should be off.
+      #
+      # ```
+      # def paper_trail_enabled_for_controller
+      #   # Don't omit `super` without a good reason.
+      #   super && request.user_agent != 'Disable User-Agent'
+      # end
+      # ```
+      #
+      # @api public
       def paper_trail_enabled_for_controller
         ::PaperTrail.enabled?
       end
@@ -73,39 +74,28 @@ module PaperTrail
 
       # Tells PaperTrail whether versions should be saved in the current
       # request.
+      #
+      # @api public
       def set_paper_trail_enabled_for_controller
-        ::PaperTrail.enabled_for_controller = paper_trail_enabled_for_controller
+        ::PaperTrail.request.enabled = paper_trail_enabled_for_controller
       end
 
       # Tells PaperTrail who is responsible for any changes that occur.
+      #
+      # @api public
       def set_paper_trail_whodunnit
-        @set_paper_trail_whodunnit_called = true
-        ::PaperTrail.whodunnit = user_for_paper_trail if ::PaperTrail.enabled_for_controller?
+        if ::PaperTrail.request.enabled?
+          ::PaperTrail.request.whodunnit = user_for_paper_trail
+        end
       end
 
       # Tells PaperTrail any information from the controller you want to store
       # alongside any changes that occur.
+      #
+      # @api public
       def set_paper_trail_controller_info
-        ::PaperTrail.controller_info = info_for_paper_trail if ::PaperTrail.enabled_for_controller?
-      end
-
-      def warn_about_not_setting_whodunnit
-        return unless ::PaperTrail.enabled_for_controller?
-
-        user_present = user_for_paper_trail.present?
-        whodunnit_blank = ::PaperTrail.whodunnit.blank?
-        if user_present && whodunnit_blank && !@set_paper_trail_whodunnit_called
-          source_file_location = self.class.instance_methods(false).map { |m|
-            self.class.instance_method(m).source_location.first
-          }.uniq.first
-          ::Kernel.warn <<-EOS.strip_heredoc
-            #{source_file_location}:
-            user_for_paper_trail is present, but whodunnit has not been set.
-            PaperTrail no longer adds the set_paper_trail_whodunnit callback for
-            you. To continue recording whodunnit, please add this before_action
-            callback to your ApplicationController . For more information,
-            please see https://git.io/vrTsk
-          EOS
+        if ::PaperTrail.request.enabled?
+          ::PaperTrail.request.controller_info = info_for_paper_trail
         end
       end
     end

data/lib/paper_trail/frameworks/rails/engine.rb

@@ -1,12 +1,45 @@
+# frozen_string_literal: true
+
 module PaperTrail
   module Rails
     # See http://guides.rubyonrails.org/engines.html
     class Engine < ::Rails::Engine
+      DPR_CONFIG_ENABLED = <<~EOS.squish.freeze
+        The rails configuration option config.paper_trail.enabled is deprecated.
+        Please use PaperTrail.enabled= instead. People were getting confused
+        that PT has both, specifically regarding *when* each was happening. If
+        you'd like to keep config.paper_trail, join the discussion at
+        https://github.com/paper-trail-gem/paper_trail/pull/1176
+      EOS
+      private_constant :DPR_CONFIG_ENABLED
+      DPR_RUDELY_ENABLING = <<~EOS.squish.freeze
+        At some point early in the rails boot process, you have set
+        PaperTrail.enabled = false. PT's rails engine is now overriding your
+        setting, and setting it to true. We're not sure why, but this is how PT
+        has worked since 5.0, when the config.paper_trail.enabled option was
+        introduced. This is now deprecated. In the future, PT will not override
+        your setting. See
+        https://github.com/paper-trail-gem/paper_trail/pull/1176 for discussion.
+      EOS
+      private_constant :DPR_RUDELY_ENABLING
+
       paths["app/models"] << "lib/paper_trail/frameworks/active_record/models"
+
+      # --- Begin deprecated section ---
       config.paper_trail = ActiveSupport::OrderedOptions.new
       initializer "paper_trail.initialisation" do |app|
-        PaperTrail.enabled = app.config.paper_trail.fetch(:enabled, true)
+        enable = app.config.paper_trail[:enabled]
+        if enable.nil?
+          unless PaperTrail.enabled?
+            ::ActiveSupport::Deprecation.warn(DPR_RUDELY_ENABLING)
+            PaperTrail.enabled = true
+          end
+        else
+          ::ActiveSupport::Deprecation.warn(DPR_CONFIG_ENABLED)
+          PaperTrail.enabled = enable
+        end
       end
+      # --- End deprecated section ---
     end
   end
 end

data/lib/paper_trail/frameworks/rspec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "rspec/core"
 require "rspec/matchers"
 require "paper_trail/frameworks/rspec/helpers"
@@ -8,9 +10,9 @@ RSpec.configure do |config|
 
   config.before(:each) do
     ::PaperTrail.enabled = false
-    ::PaperTrail.enabled_for_controller = true
-    ::PaperTrail.whodunnit = nil
-    ::PaperTrail.controller_info = {} if defined?(::Rails) && defined?(::RSpec::Rails)
+    ::PaperTrail.request.enabled = true
+    ::PaperTrail.request.whodunnit = nil
+    ::PaperTrail.request.controller_info = {} if defined?(::Rails) && defined?(::RSpec::Rails)
   end
 
   config.before(:each, versioning: true) do
@@ -25,5 +27,16 @@ end
 
 RSpec::Matchers.define :have_a_version_with do |attributes|
   # check if the model has a version with the specified attributes
-  match { |actual| actual.versions.where_object(attributes).any? }
+  match do |actual|
+    versions_association = actual.class.versions_association_name
+    actual.send(versions_association).where_object(attributes).any?
+  end
+end
+
+RSpec::Matchers.define :have_a_version_with_changes do |attributes|
+  # check if the model has a version changes with the specified attributes
+  match do |actual|
+    versions_association = actual.class.versions_association_name
+    actual.send(versions_association).where_object_changes(attributes).any?
+  end
 end

data/lib/paper_trail/frameworks/rspec/helpers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module PaperTrail
   module RSpec
     module Helpers

data/lib/paper_trail/has_paper_trail.rb

@@ -1,5 +1,5 @@
-require "active_support/core_ext/object" # provides the `try` method
-require "paper_trail/attribute_serializers/legacy_active_record_shim"
+# frozen_string_literal: true
+
 require "paper_trail/attribute_serializers/object_attribute"
 require "paper_trail/attribute_serializers/object_changes_attribute"
 require "paper_trail/model_config"
@@ -23,18 +23,18 @@ module PaperTrail
       # Options:
       #
       # - :on - The events to track (optional; defaults to all of them). Set
-      #   to an array of `:create`, `:update`, `:destroy` as desired.
-      # - :class_name - The name of a custom Version class.  This class should
-      #   inherit from `PaperTrail::Version`.
+      #   to an array of `:create`, `:update`, `:destroy` and `:touch` as desired.
+      # - :class_name (deprecated) - The name of a custom Version class that
+      #   includes `PaperTrail::VersionConcern`.
       # - :ignore - An array of attributes for which a new `Version` will not be
-      #   created if only they change. It can also aceept a Hash as an
+      #   created if only they change. It can also accept a Hash as an
       #   argument where the key is the attribute to ignore (a `String` or
       #   `Symbol`), which will only be ignored if the value is a `Proc` which
       #   returns truthily.
       # - :if, :unless - Procs that allow to specify conditions when to save
       #   versions for an object.
       # - :only - Inverse of `ignore`. A new `Version` will be created only
-      #   for these attributes if supplied it can also aceept a Hash as an
+      #   for these attributes if supplied it can also accept a Hash as an
       #   argument where the key is the attribute to track (a `String` or
       #   `Symbol`), which will only be counted if the value is a `Proc` which
       #   returns truthily.
@@ -47,328 +47,40 @@ module PaperTrail
       #   are called with `self`, i.e. the model with the paper trail).  See
       #   `PaperTrail::Controller.info_for_paper_trail` for how to store data
       #   from the controller.
-      # - :versions - The name to use for the versions association.  Default
-      #   is `:versions`.
+      # - :versions - Either,
+      #   - A String (deprecated) - The name to use for the versions
+      #     association.  Default is `:versions`.
+      #   - A Hash - options passed to `has_many`, plus `name:` and `scope:`.
       # - :version - The name to use for the method which returns the version
       #   the instance was reified from. Default is `:version`.
-      # - :save_changes - Whether or not to save changes to the object_changes
-      #   column if it exists. Default is true
-      # - :join_tables - If the model has a has_and_belongs_to_many relation
-      #   with an unpapertrailed model, passing the name of the association to
-      #   the join_tables option will paper trail the join table but not save
-      #   the other model, allowing reification of the association but with the
-      #   other models latest state (if the other model is paper trailed, this
-      #   option does nothing)
+      #
+      # Plugins like the experimental `paper_trail-association_tracking` gem
+      # may accept additional options.
+      #
+      # You can define a default set of options via the configurable
+      # `PaperTrail.config.has_paper_trail_defaults` hash in your applications
+      # initializer. The hash can contain any of the following options and will
+      # provide an overridable default for all models.
       #
       # @api public
       def has_paper_trail(options = {})
-        paper_trail.setup(options)
+        defaults = PaperTrail.config.has_paper_trail_defaults
+        paper_trail.setup(defaults.merge(options))
       end
 
       # @api public
       def paper_trail
         ::PaperTrail::ModelConfig.new(self)
       end
-
-      # @api private
-      def paper_trail_deprecate(new_method, old_method = nil)
-        old = old_method.nil? ? new_method : old_method
-        msg = format("Use paper_trail.%s instead of %s", new_method, old)
-        ::ActiveSupport::Deprecation.warn(msg, caller(2))
-      end
-
-      # @deprecated
-      def paper_trail_on_destroy(*args)
-        paper_trail_deprecate "on_destroy", "paper_trail_on_destroy"
-        paper_trail.on_destroy(*args)
-      end
-
-      # @deprecated
-      def paper_trail_on_update
-        paper_trail_deprecate "on_update", "paper_trail_on_update"
-        paper_trail.on_update
-      end
-
-      # @deprecated
-      def paper_trail_on_create
-        paper_trail_deprecate "on_create", "paper_trail_on_create"
-        paper_trail.on_create
-      end
-
-      # @deprecated
-      def paper_trail_off!
-        paper_trail_deprecate "disable", "paper_trail_off!"
-        paper_trail.disable
-      end
-
-      # @deprecated
-      def paper_trail_on!
-        paper_trail_deprecate "enable", "paper_trail_on!"
-        paper_trail.enable
-      end
-
-      # @deprecated
-      def paper_trail_enabled_for_model?
-        paper_trail_deprecate "enabled?", "paper_trail_enabled_for_model?"
-        paper_trail.enabled?
-      end
-
-      # @deprecated
-      def paper_trail_version_class
-        paper_trail_deprecate "version_class", "paper_trail_version_class"
-        paper_trail.version_class
-      end
     end
 
     # Wrap the following methods in a module so we can include them only in the
     # ActiveRecord models that declare `has_paper_trail`.
     module InstanceMethods
+      # @api public
       def paper_trail
         ::PaperTrail::RecordTrail.new(self)
       end
-
-      # @deprecated
-      def live?
-        self.class.paper_trail_deprecate "live?"
-        paper_trail.live?
-      end
-
-      # @deprecated
-      def paper_trail_originator
-        self.class.paper_trail_deprecate "originator", "paper_trail_originator"
-        paper_trail.originator
-      end
-
-      # @deprecated
-      def originator
-        self.class.paper_trail_deprecate "originator"
-        paper_trail.originator
-      end
-
-      # @deprecated
-      def clear_rolled_back_versions
-        self.class.paper_trail_deprecate "clear_rolled_back_versions"
-        paper_trail.clear_rolled_back_versions
-      end
-
-      # @deprecated
-      def source_version
-        self.class.paper_trail_deprecate "source_version"
-        paper_trail.source_version
-      end
-
-      # @deprecated
-      def version_at(*args)
-        self.class.paper_trail_deprecate "version_at"
-        paper_trail.version_at(*args)
-      end
-
-      # @deprecated
-      def versions_between(start_time, end_time, _reify_options = {})
-        self.class.paper_trail_deprecate "versions_between"
-        paper_trail.versions_between(start_time, end_time)
-      end
-
-      # @deprecated
-      def previous_version
-        self.class.paper_trail_deprecate "previous_version"
-        paper_trail.previous_version
-      end
-
-      # @deprecated
-      def next_version
-        self.class.paper_trail_deprecate "next_version"
-        paper_trail.next_version
-      end
-
-      # @deprecated
-      def paper_trail_enabled_for_model?
-        self.class.paper_trail_deprecate "enabled_for_model?", "paper_trail_enabled_for_model?"
-        paper_trail.enabled_for_model?
-      end
-
-      # @deprecated
-      def without_versioning(method = nil, &block)
-        self.class.paper_trail_deprecate "without_versioning"
-        paper_trail.without_versioning(method, &block)
-      end
-
-      # @deprecated
-      def appear_as_new_record(&block)
-        self.class.paper_trail_deprecate "appear_as_new_record"
-        paper_trail.appear_as_new_record(&block)
-      end
-
-      # @deprecated
-      def whodunnit(value, &block)
-        self.class.paper_trail_deprecate "whodunnit"
-        paper_trail.whodunnit(value, &block)
-      end
-
-      # @deprecated
-      def touch_with_version(name = nil)
-        self.class.paper_trail_deprecate "touch_with_version"
-        paper_trail.touch_with_version(name)
-      end
-
-      # `record_create` is deprecated in favor of `paper_trail.record_create`,
-      # but does not yet print a deprecation warning. When the `after_create`
-      # callback is registered (by ModelConfig#on_create) we still refer to this
-      # method by name, e.g.
-      #
-      #     @model_class.after_create :record_create, if: :save_version?
-      #
-      # instead of using the preferred method `paper_trail.record_create`, e.g.
-      #
-      #     @model_class.after_create { |r| r.paper_trail.record_create if r.save_version?}
-      #
-      # We still register the callback by name so that, if someone calls
-      # `has_paper_trail` twice, the callback will *not* be registered twice.
-      # Our own test suite calls `has_paper_trail` many times for the same
-      # class.
-      #
-      # In the future, perhaps we should require that users only set up
-      # PT once per class.
-      #
-      # @deprecated
-      def record_create
-        paper_trail.record_create
-      end
-
-      # See deprecation comment for `record_create`
-      # @deprecated
-      def record_update(force = nil)
-        paper_trail.record_update(force)
-      end
-
-      # @deprecated
-      def pt_record_object_changes?
-        self.class.paper_trail_deprecate "record_object_changes?", "pt_record_object_changes?"
-        paper_trail.record_object_changes?
-      end
-
-      # @deprecated
-      def pt_recordable_object
-        self.class.paper_trail_deprecate "recordable_object", "pt_recordable_object"
-        paper_trail.recordable_object
-      end
-
-      # @deprecated
-      def pt_recordable_object_changes
-        self.class.paper_trail_deprecate "recordable_object_changes", "pt_recordable_object_changes"
-        paper_trail.recordable_object_changes
-      end
-
-      # @deprecated
-      def changes_for_paper_trail
-        self.class.paper_trail_deprecate "changes", "changes_for_paper_trail"
-        paper_trail.changes
-      end
-
-      # See deprecation comment for `record_create`
-      # @deprecated
-      def clear_version_instance!
-        paper_trail.clear_version_instance
-      end
-
-      # See deprecation comment for `record_create`
-      # @deprecated
-      def reset_timestamp_attrs_for_update_if_needed!
-        paper_trail.reset_timestamp_attrs_for_update_if_needed
-      end
-
-      # See deprecation comment for `record_create`
-      # @deprecated
-      def record_destroy
-        paper_trail.record_destroy
-      end
-
-      # @deprecated
-      def save_associations(version)
-        self.class.paper_trail_deprecate "save_associations"
-        paper_trail.save_associations(version)
-      end
-
-      # @deprecated
-      def save_associations_belongs_to(version)
-        self.class.paper_trail_deprecate "save_associations_belongs_to"
-        paper_trail.save_associations_belongs_to(version)
-      end
-
-      # @deprecated
-      def save_associations_has_and_belongs_to_many(version)
-        self.class.paper_trail_deprecate(
-          "save_associations_habtm",
-          "save_associations_has_and_belongs_to_many"
-        )
-        paper_trail.save_associations_habtm(version)
-      end
-
-      # @deprecated
-      # @api private
-      def reset_transaction_id
-        ::ActiveSupport::Deprecation.warn(
-          "reset_transaction_id is deprecated, use PaperTrail.clear_transaction_id"
-        )
-        PaperTrail.clear_transaction_id
-      end
-
-      # @deprecated
-      # @api private
-      def merge_metadata(data)
-        self.class.paper_trail_deprecate "merge_metadata"
-        paper_trail.merge_metadata(data)
-      end
-
-      # @deprecated
-      def attributes_before_change
-        self.class.paper_trail_deprecate "attributes_before_change"
-        paper_trail.attributes_before_change
-      end
-
-      # @deprecated
-      def object_attrs_for_paper_trail
-        self.class.paper_trail_deprecate "object_attrs_for_paper_trail"
-        paper_trail.object_attrs_for_paper_trail
-      end
-
-      # @deprecated
-      def changed_notably?
-        self.class.paper_trail_deprecate "changed_notably?"
-        paper_trail.changed_notably?
-      end
-
-      # @deprecated
-      def ignored_attr_has_changed?
-        self.class.paper_trail_deprecate "ignored_attr_has_changed?"
-        paper_trail.ignored_attr_has_changed?
-      end
-
-      # @deprecated
-      def notably_changed
-        self.class.paper_trail_deprecate "notably_changed"
-        paper_trail.notably_changed
-      end
-
-      # @deprecated
-      def changed_and_not_ignored
-        self.class.paper_trail_deprecate "changed_and_not_ignored"
-        paper_trail.changed_and_not_ignored
-      end
-
-      # The new method is named "enabled?" for consistency.
-      # @deprecated
-      def paper_trail_switched_on?
-        self.class.paper_trail_deprecate "enabled?", "paper_trail_switched_on?"
-        paper_trail.enabled?
-      end
-
-      # @deprecated
-      # @api private
-      def save_version?
-        self.class.paper_trail_deprecate "save_version?"
-        paper_trail.save_version?
-      end
     end
   end
 end