Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
张重言
rails
提交
93e10f99
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,发现更多精彩内容 >>
提交
93e10f99
编写于
7月 23, 2008
作者:
J
Jan De Poorter
提交者:
Pratik Naik
7月 23, 2008
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Ensure NamedScope#any? uses COUNT query wherever possible. [#680 state:resolved]
Signed-off-by:
N
Pratik Naik
<
pratiknaik@gmail.com
>
上级
26816854
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
31 addition
and
1 deletion
+31
-1
activerecord/lib/active_record/named_scope.rb
activerecord/lib/active_record/named_scope.rb
+9
-1
activerecord/test/cases/named_scope_test.rb
activerecord/test/cases/named_scope_test.rb
+22
-0
未找到文件。
activerecord/lib/active_record/named_scope.rb
浏览文件 @
93e10f99
...
...
@@ -103,7 +103,7 @@ class Scope
attr_reader
:proxy_scope
,
:proxy_options
[].
methods
.
each
do
|
m
|
unless
m
=~
/(^__|^nil\?|^send|^object_id$|class|extend|find|count|sum|average|maximum|minimum|paginate|first|last|empty?)/
unless
m
=~
/(^__|^nil\?|^send|^object_id$|class|extend|find|count|sum|average|maximum|minimum|paginate|first|last|empty?
|any?
)/
delegate
m
,
:to
=>
:proxy_found
end
end
...
...
@@ -140,6 +140,14 @@ def empty?
@found
?
@found
.
empty?
:
count
.
zero?
end
def
any?
if
block_given?
proxy_found
.
any?
{
|*
block_args
|
yield
(
*
block_args
)
}
else
!
empty?
end
end
protected
def
proxy_found
@found
||
load_found
...
...
activerecord/test/cases/named_scope_test.rb
浏览文件 @
93e10f99
...
...
@@ -184,6 +184,28 @@ def test_empty_should_not_load_results
end
end
def
test_any_should_not_load_results
topics
=
Topic
.
base
assert_queries
(
1
)
do
topics
.
expects
(
:empty?
).
returns
(
true
)
assert
!
topics
.
any?
end
end
def
test_any_should_call_proxy_found_if_using_a_block
topics
=
Topic
.
base
assert_queries
(
1
)
do
topics
.
expects
(
:empty?
).
never
topics
.
any?
{
true
}
end
end
def
test_any_should_not_fire_query_if_named_scope_loaded
topics
=
Topic
.
base
topics
.
collect
# force load
assert_no_queries
{
assert
topics
.
any?
}
end
def
test_should_build_with_proxy_options
topic
=
Topic
.
approved
.
build
({})
assert
topic
.
approved
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录