Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
张重言
rails
提交
92dadf6d
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,发现更多精彩内容 >>
提交
92dadf6d
编写于
3月 12, 2009
作者:
W
Will Bryant
提交者:
Pratik Naik
3月 12, 2009
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Fixed autosave checks on objects with hm:t in :include [#2213 state:resolved]
Signed-off-by:
N
Pratik Naik
<
pratiknaik@gmail.com
>
上级
be7b64b3
变更
4
隐藏空白更改
内联
并排
Showing
4 changed file
with
49 addition
and
1 deletion
+49
-1
activerecord/lib/active_record/autosave_association.rb
activerecord/lib/active_record/autosave_association.rb
+1
-1
activerecord/test/cases/associations/belongs_to_associations_test.rb
...d/test/cases/associations/belongs_to_associations_test.rb
+16
-0
activerecord/test/cases/associations/has_one_associations_test.rb
...cord/test/cases/associations/has_one_associations_test.rb
+16
-0
activerecord/test/cases/associations/has_one_through_associations_test.rb
...t/cases/associations/has_one_through_associations_test.rb
+16
-0
未找到文件。
activerecord/lib/active_record/autosave_association.rb
浏览文件 @
92dadf6d
...
@@ -310,7 +310,7 @@ def save_collection_association(reflection)
...
@@ -310,7 +310,7 @@ def save_collection_association(reflection)
# This all happens inside a transaction, _if_ the Transactions module is included into
# This all happens inside a transaction, _if_ the Transactions module is included into
# ActiveRecord::Base after the AutosaveAssociation module, which it does by default.
# ActiveRecord::Base after the AutosaveAssociation module, which it does by default.
def
save_has_one_association
(
reflection
)
def
save_has_one_association
(
reflection
)
if
association
=
association_instance_get
(
reflection
.
name
)
if
(
association
=
association_instance_get
(
reflection
.
name
))
&&
!
association
.
target
.
nil?
if
reflection
.
options
[
:autosave
]
&&
association
.
marked_for_destruction?
if
reflection
.
options
[
:autosave
]
&&
association
.
marked_for_destruction?
association
.
destroy
association
.
destroy
elsif
new_record?
||
association
.
new_record?
||
association
[
reflection
.
primary_key_name
]
!=
id
||
reflection
.
options
[
:autosave
]
elsif
new_record?
||
association
.
new_record?
||
association
[
reflection
.
primary_key_name
]
!=
id
||
reflection
.
options
[
:autosave
]
...
...
activerecord/test/cases/associations/belongs_to_associations_test.rb
浏览文件 @
92dadf6d
...
@@ -326,4 +326,20 @@ def test_belongs_to_proxy_should_respond_to_private_methods_via_send
...
@@ -326,4 +326,20 @@ def test_belongs_to_proxy_should_respond_to_private_methods_via_send
companies
(
:first_firm
).
send
(
:private_method
)
companies
(
:first_firm
).
send
(
:private_method
)
companies
(
:second_client
).
firm
.
send
(
:private_method
)
companies
(
:second_client
).
firm
.
send
(
:private_method
)
end
end
def
test_save_of_record_with_loaded_belongs_to
@account
=
companies
(
:first_firm
).
account
assert_nothing_raised
do
Account
.
find
(
@account
.
id
).
save!
Account
.
find
(
@account
.
id
,
:include
=>
:firm
).
save!
end
@account
.
firm
.
delete
assert_nothing_raised
do
Account
.
find
(
@account
.
id
).
save!
Account
.
find
(
@account
.
id
,
:include
=>
:firm
).
save!
end
end
end
end
activerecord/test/cases/associations/has_one_associations_test.rb
浏览文件 @
92dadf6d
...
@@ -290,4 +290,20 @@ def test_has_one_proxy_should_respond_to_private_methods_via_send
...
@@ -290,4 +290,20 @@ def test_has_one_proxy_should_respond_to_private_methods_via_send
companies
(
:first_firm
).
account
.
send
(
:private_method
)
companies
(
:first_firm
).
account
.
send
(
:private_method
)
end
end
def
test_save_of_record_with_loaded_has_one
@firm
=
companies
(
:first_firm
)
assert_not_nil
@firm
.
account
assert_nothing_raised
do
Firm
.
find
(
@firm
.
id
).
save!
Firm
.
find
(
@firm
.
id
,
:include
=>
:account
).
save!
end
@firm
.
account
.
destroy
assert_nothing_raised
do
Firm
.
find
(
@firm
.
id
).
save!
Firm
.
find
(
@firm
.
id
,
:include
=>
:account
).
save!
end
end
end
end
activerecord/test/cases/associations/has_one_through_associations_test.rb
浏览文件 @
92dadf6d
...
@@ -173,4 +173,20 @@ def test_preloading_has_one_through_on_belongs_to
...
@@ -173,4 +173,20 @@ def test_preloading_has_one_through_on_belongs_to
assert_not_nil
assert_no_queries
{
@new_detail
.
member_type
}
assert_not_nil
assert_no_queries
{
@new_detail
.
member_type
}
end
end
def
test_save_of_record_with_loaded_has_one_through
@club
=
@member
.
club
assert_not_nil
@club
.
sponsored_member
assert_nothing_raised
do
Club
.
find
(
@club
.
id
).
save!
Club
.
find
(
@club
.
id
,
:include
=>
:sponsored_member
).
save!
end
@club
.
sponsor
.
destroy
assert_nothing_raised
do
Club
.
find
(
@club
.
id
).
save!
Club
.
find
(
@club
.
id
,
:include
=>
:sponsored_member
).
save!
end
end
end
end
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录