Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
张重言
rails
提交
1c4a7a0d
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,体验更适合开发者的 AI 搜索 >>
提交
1c4a7a0d
编写于
5月 12, 2020
作者:
E
Eugene Kenny
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Return enumerator from each_pair and each_value
This matches Hash's behaviour for those methods.
上级
25bc1c01
变更
3
隐藏空白更改
内联
并排
Showing
3 changed file
with
27 addition
and
0 deletion
+27
-0
actionpack/CHANGELOG.md
actionpack/CHANGELOG.md
+5
-0
actionpack/lib/action_controller/metal/strong_parameters.rb
actionpack/lib/action_controller/metal/strong_parameters.rb
+2
-0
actionpack/test/controller/parameters/accessors_test.rb
actionpack/test/controller/parameters/accessors_test.rb
+20
-0
未找到文件。
actionpack/CHANGELOG.md
浏览文件 @
1c4a7a0d
*
Calling
`each_pair`
or
`each_value`
on an
`ActionController::Parameters`
without passing a block now returns an enumerator.
*Eugene Kenny*
*
`fixture_file_upload`
now uses path relative to
`file_fixture_path`
Previously the path had to be relative to `fixture_path`.
...
...
actionpack/lib/action_controller/metal/strong_parameters.rb
浏览文件 @
1c4a7a0d
...
...
@@ -360,6 +360,7 @@ def to_unsafe_h
# Convert all hashes in values into parameters, then yield each pair in
# the same way as <tt>Hash#each_pair</tt>.
def
each_pair
(
&
block
)
return
to_enum
(
__callee__
)
unless
block_given?
@parameters
.
each_pair
do
|
key
,
value
|
yield
[
key
,
convert_hashes_to_parameters
(
key
,
value
)]
end
...
...
@@ -369,6 +370,7 @@ def each_pair(&block)
# Convert all hashes in values into parameters, then yield each value in
# the same way as <tt>Hash#each_value</tt>.
def
each_value
(
&
block
)
return
to_enum
(
:each_value
)
unless
block_given?
@parameters
.
each_pair
do
|
key
,
value
|
yield
convert_hashes_to_parameters
(
key
,
value
)
end
...
...
actionpack/test/controller/parameters/accessors_test.rb
浏览文件 @
1c4a7a0d
...
...
@@ -58,6 +58,11 @@ class ParametersAccessorsTest < ActiveSupport::TestCase
end
end
test
"each without a block returns an enumerator"
do
assert_kind_of
Enumerator
,
@params
.
each
assert_equal
@params
,
@params
.
each
.
to_h
end
test
"each_pair carries permitted status"
do
@params
.
permit!
@params
.
each_pair
{
|
key
,
value
|
assert
(
value
.
permitted?
)
if
key
==
"person"
}
...
...
@@ -75,6 +80,11 @@ class ParametersAccessorsTest < ActiveSupport::TestCase
end
end
test
"each_pair without a block returns an enumerator"
do
assert_kind_of
Enumerator
,
@params
.
each_pair
assert_equal
@params
,
@params
.
each_pair
.
to_h
end
test
"each_value carries permitted status"
do
@params
.
permit!
@params
.
each_value
do
|
value
|
...
...
@@ -88,6 +98,11 @@ class ParametersAccessorsTest < ActiveSupport::TestCase
end
end
test
"each_value without a block returns an enumerator"
do
assert_kind_of
Enumerator
,
@params
.
each_value
assert_equal
@params
.
values
,
@params
.
each_value
.
to_a
end
test
"each_key converts to hash for permitted"
do
@params
.
permit!
@params
.
each_key
{
|
key
|
assert_kind_of
(
String
,
key
)
if
key
==
"person"
}
...
...
@@ -97,6 +112,11 @@ class ParametersAccessorsTest < ActiveSupport::TestCase
@params
.
each_key
{
|
key
|
assert_kind_of
(
String
,
key
)
if
key
==
"person"
}
end
test
"each_key without a block returns an enumerator"
do
assert_kind_of
Enumerator
,
@params
.
each_key
assert_equal
@params
.
keys
,
@params
.
each_key
.
to_a
end
test
"empty? returns true when params contains no key/value pairs"
do
params
=
ActionController
::
Parameters
.
new
assert_empty
params
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录