Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
张重言
rails
提交
f7114754
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,发现更多精彩内容 >>
提交
f7114754
编写于
6月 14, 2013
作者:
R
Rafael Mendonça França
浏览文件
操作
浏览文件
下载
差异文件
Merge pull request #10898 from dmitry/find_first_refactor_duplication
Refactored ActiveRecord `first` method to get rid of duplication.
上级
067e1505
e720b50f
变更
1
隐藏空白更改
内联
并排
Showing
1 changed file
with
10 addition
and
11 deletion
+10
-11
activerecord/lib/active_record/relation/finder_methods.rb
activerecord/lib/active_record/relation/finder_methods.rb
+10
-11
未找到文件。
activerecord/lib/active_record/relation/finder_methods.rb
浏览文件 @
f7114754
...
...
@@ -124,11 +124,7 @@ def take!
#
def
first
(
limit
=
nil
)
if
limit
if
order_values
.
empty?
&&
primary_key
order
(
arel_table
[
primary_key
].
asc
).
limit
(
limit
).
to_a
else
limit
(
limit
).
to_a
end
find_first_with_limit
(
order_values
,
limit
)
else
find_first
end
...
...
@@ -358,12 +354,15 @@ def find_first
if
loaded?
@records
.
first
else
@first
||=
if
with_default_scope
.
order_values
.
empty?
&&
primary_key
order
(
arel_table
[
primary_key
].
asc
).
limit
(
1
).
to_a
.
first
else
limit
(
1
).
to_a
.
first
end
@first
||=
find_first_with_limit
(
with_default_scope
.
order_values
,
1
).
first
end
end
def
find_first_with_limit
(
order_values
,
limit
)
if
order_values
.
empty?
&&
primary_key
order
(
arel_table
[
primary_key
].
asc
).
limit
(
limit
).
to_a
else
limit
(
limit
).
to_a
end
end
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录