kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Add a specific method to force the transaction record state clear

Browse source 
We are using that code path in only one place so we should not add a
conditional to all the other cases. This will avoid performance
regressions on the old paths.
This commit is contained in:
Rafael Mendonça França 2014-05-16 15:03:40 -03:00
parent 9900703ebf
commit d89d859a41
1 changed files with 8 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
activerecord/lib/active_record/transactions.rb
View file

@ -295,7 +295,7 @@ module ActiveRecord
def committed! #:nodoc:
run_callbacks :commit if destroyed? || persisted?
ensure
clear_transaction_record_state(true)
force_clear_transaction_record_state
end
# Call the +after_rollback+ callbacks. The +force_restore_state+ argument indicates if the record
@ -353,11 +353,14 @@ module ActiveRecord
end
# Clear the new record state and id of a record.
def clear_transaction_record_state(force = false) #:nodoc:
def clear_transaction_record_state #:nodoc:
@_start_transaction_state[:level] = (@_start_transaction_state[:level] || 0) - 1
if force || @_start_transaction_state[:level] < 1
@_start_transaction_state.clear
end
@_start_transaction_state.clear @_start_transaction_state[:level] < 1
end
# Force to clear the teansaction record state.
def force_clear_transaction_record_state #:nodoc:
@_start_transaction_state.clear
end
# Restore the new record state and id of a record that was previously saved by a call to save_record_state.