Style/RedundantBegin
This commit is contained in:
parent
860004f11d
commit
43ee89fe0d
|
@ -76,9 +76,3 @@ Security/YAMLLoad:
|
||||||
- 'spec/paper_trail/events/destroy_spec.rb'
|
- 'spec/paper_trail/events/destroy_spec.rb'
|
||||||
- 'spec/paper_trail/model_spec.rb'
|
- 'spec/paper_trail/model_spec.rb'
|
||||||
- 'spec/paper_trail/serializer_spec.rb'
|
- 'spec/paper_trail/serializer_spec.rb'
|
||||||
|
|
||||||
# Offense count: 1
|
|
||||||
# Cop supports --auto-correct.
|
|
||||||
Style/RedundantBegin:
|
|
||||||
Exclude:
|
|
||||||
- 'lib/paper_trail/events/base.rb'
|
|
||||||
|
|
|
@ -109,19 +109,11 @@ module PaperTrail
|
||||||
@changed_in_latest_version ||= changes_in_latest_version.keys
|
@changed_in_latest_version ||= changes_in_latest_version.keys
|
||||||
end
|
end
|
||||||
|
|
||||||
# Rails 5.1 changed the API of `ActiveRecord::Dirty`. See
|
# Memoized to reduce memory usage
|
||||||
# https://github.com/paper-trail-gem/paper_trail/pull/899
|
|
||||||
#
|
#
|
||||||
# @api private
|
# @api private
|
||||||
def changes_in_latest_version
|
def changes_in_latest_version
|
||||||
# Memoized to reduce memory usage
|
@changes_in_latest_version ||= load_changes_in_latest_version
|
||||||
@changes_in_latest_version ||= begin
|
|
||||||
if @in_after_callback
|
|
||||||
@record.saved_changes
|
|
||||||
else
|
|
||||||
@record.changes
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|
||||||
# An attributed is "ignored" if it is listed in the `:ignore` option
|
# An attributed is "ignored" if it is listed in the `:ignore` option
|
||||||
|
@ -134,6 +126,18 @@ module PaperTrail
|
||||||
ignored.any? && (changed_in_latest_version & ignored).any?
|
ignored.any? && (changed_in_latest_version & ignored).any?
|
||||||
end
|
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`
|
# PT 10 has a new optional column, `item_subtype`
|
||||||
#
|
#
|
||||||
# @api private
|
# @api private
|
||||||
|
|
Loading…
Reference in New Issue