Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
张重言
rails
提交
f1a6307a
R
rails
项目概览
张重言
/
rails
通知
1
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
R
rails
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
提交
f1a6307a
编写于
2月 04, 2014
作者:
R
Rafael Mendonça França
浏览文件
操作
浏览文件
下载
差异文件
Merge pull request #13935 from arthurnn/fix_12566
Make sure transaction state resets after commit
上级
75510a62
9b66d6d4
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
16 addition
and
1 deletion
+16
-1
activerecord/lib/active_record/transactions.rb
activerecord/lib/active_record/transactions.rb
+1
-1
activerecord/test/cases/transaction_callbacks_test.rb
activerecord/test/cases/transaction_callbacks_test.rb
+15
-0
未找到文件。
activerecord/lib/active_record/transactions.rb
浏览文件 @
f1a6307a
...
...
@@ -295,7 +295,7 @@ def rollback_active_record_state!
def
committed!
#:nodoc:
run_callbacks
:commit
if
destroyed?
||
persisted?
ensure
clear_transaction_record_state
@_start_transaction_state
.
clear
end
# Call the +after_rollback+ callbacks. The +force_restore_state+ argument indicates if the record
...
...
activerecord/test/cases/transaction_callbacks_test.rb
浏览文件 @
f1a6307a
...
...
@@ -16,6 +16,11 @@ class ReplyWithCallbacks < ActiveRecord::Base
after_commit
:do_after_commit
,
on: :create
attr_accessor
:save_on_after_create
after_create
do
self
.
save!
if
save_on_after_create
end
def
history
@history
||=
[]
end
...
...
@@ -107,6 +112,16 @@ def test_only_call_after_commit_on_create_after_transaction_commits_for_new_reco
assert_equal
[],
reply
.
history
end
def
test_only_call_after_commit_on_create_and_doesnt_leaky
r
=
ReplyWithCallbacks
.
new
(
content:
'foo'
)
r
.
save_on_after_create
=
true
r
.
save!
r
.
content
=
'bar'
r
.
save!
r
.
save!
assert_equal
[
:commit_on_create
],
r
.
history
end
def
test_only_call_after_commit_on_update_after_transaction_commits_for_existing_record_on_touch
add_transaction_execution_blocks
@first
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录