mirrors
/
rails
mirror of https://github.com/rails/rails
8
0
Fork 0
Code Issues Projects Releases Wiki Activity

Let's use the delegation to the transaction manager here 

`within_new_transaction` is delegated to the transaction manager. Let's remove
this explicit receiver to honor the delegation and for consistency with usage
of the also delegated `current_transaction`.
This commit is contained in:
Xavier Noria 2024-05-28 13:17:31 +02:00
parent 9ce4d4468e
commit 2aeb322806
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
activerecord/lib/active_record/connection_adapters/abstract/database_statements.rb
View File

@ -358,7 +358,7 @@ module ActiveRecord
end end
yield current_transaction yield current_transaction
else else
transaction_manager.within_new_transaction(isolation: isolation, joinable: joinable, &block) within_new_transaction(isolation: isolation, joinable: joinable, &block)
end end
rescue ActiveRecord::Rollback rescue ActiveRecord::Rollback
# rollbacks are silently swallowed # rollbacks are silently swallowed