Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
张重言
rails
提交
90fb7798
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,发现更多精彩内容 >>
未验证
提交
90fb7798
编写于
2月 07, 2017
作者:
R
Rafael Mendonça França
浏览文件
操作
浏览文件
下载
差异文件
Merge pull request #25873 from schuetzm/warn_about_dirty_lock
Deprecate locking of dirty records
上级
9e98f3f7
578f2830
变更
3
隐藏空白更改
内联
并排
Showing
3 changed file
with
18 addition
and
2 deletion
+18
-2
activerecord/CHANGELOG.md
activerecord/CHANGELOG.md
+4
-0
activerecord/lib/active_record/locking/pessimistic.rb
activerecord/lib/active_record/locking/pessimistic.rb
+10
-1
activerecord/test/cases/locking_test.rb
activerecord/test/cases/locking_test.rb
+4
-1
未找到文件。
activerecord/CHANGELOG.md
浏览文件 @
90fb7798
*
Deprecate locking records with unpersisted changes.
*Marc Schütz*
*
Remove deprecated behavior that halts callbacks when the return is false.
*Rafael Mendonça França*
...
...
activerecord/lib/active_record/locking/pessimistic.rb
浏览文件 @
90fb7798
...
...
@@ -59,7 +59,16 @@ module Pessimistic
# or pass true for "FOR UPDATE" (the default, an exclusive row lock). Returns
# the locked record.
def
lock!
(
lock
=
true
)
reload
(
lock:
lock
)
if
persisted?
if
persisted?
if
changed?
ActiveSupport
::
Deprecation
.
warn
(
<<-
MSG
.
squish
)
Locking a record with unpersisted changes is deprecated and will raise an
exception in Rails 5.2. Use `save` to persist the changes, or `reload` to
discard them explicitly.
MSG
end
reload
(
lock:
lock
)
end
self
end
...
...
activerecord/test/cases/locking_test.rb
浏览文件 @
90fb7798
...
...
@@ -536,7 +536,10 @@ def test_sane_lock_method
Person
.
transaction
do
person
=
Person
.
find
1
old
,
person
.
first_name
=
person
.
first_name
,
"fooman"
person
.
lock!
# Locking a dirty record is deprecated
assert_deprecated
do
person
.
lock!
end
assert_equal
old
,
person
.
first_name
end
end
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录