Style/RedundantBegin

.rubocop_todo.yml

@@ -76,9 +76,3 @@ Security/YAMLLoad:
     - 'spec/paper_trail/events/destroy_spec.rb'
     - 'spec/paper_trail/model_spec.rb'
     - 'spec/paper_trail/serializer_spec.rb'
-
-# Offense count: 1
-# Cop supports --auto-correct.
-Style/RedundantBegin:
-  Exclude:
-    - 'lib/paper_trail/events/base.rb'

lib/paper_trail/events/base.rb

@@ -109,19 +109,11 @@ def changed_in_latest_version
         @changed_in_latest_version ||= changes_in_latest_version.keys
       end
 
-      # Rails 5.1 changed the API of `ActiveRecord::Dirty`. See
-      # https://github.com/paper-trail-gem/paper_trail/pull/899
+      # Memoized to reduce memory usage
       #
       # @api private
       def changes_in_latest_version
-        # Memoized to reduce memory usage
-        @changes_in_latest_version ||= begin
-          if @in_after_callback
-            @record.saved_changes
-          else
-            @record.changes
-          end
-        end
+        @changes_in_latest_version ||= load_changes_in_latest_version
       end
 
       # An attributed is "ignored" if it is listed in the `:ignore` option
@@ -134,6 +126,18 @@ def ignored_attr_has_changed?
         ignored.any? && (changed_in_latest_version & ignored).any?
       end
 
+      # Rails 5.1 changed the API of `ActiveRecord::Dirty`. See
+      # https://github.com/paper-trail-gem/paper_trail/pull/899
+      #
+      # @api private
+      def load_changes_in_latest_version
+        if @in_after_callback
+          @record.saved_changes
+        else
+          @record.changes
+        end
+      end
+
       # PT 10 has a new optional column, `item_subtype`
       #
       # @api private