提交 58201f7d 编写于 作者: X Xavier Noria

revises style in recent code (of mine)

This alternative case expressions read better for my taste, and look more uniform
in a file where other similar case expressions are used (without dynamic clauses).
上级 ee7556cb
...@@ -807,11 +807,12 @@ def hash_filter(params, filter) ...@@ -807,11 +807,12 @@ def hash_filter(params, filter)
def permit_any_in_parameters(params) def permit_any_in_parameters(params)
self.class.new.tap do |sanitized| self.class.new.tap do |sanitized|
params.each do |key, value| params.each do |key, value|
if permitted_scalar?(value) case value
when ->(v) { permitted_scalar?(v) }
sanitized[key] = value sanitized[key] = value
elsif value.is_a?(Array) when Array
sanitized[key] = permit_any_in_array(value) sanitized[key] = permit_any_in_array(value)
elsif value.is_a?(Parameters) when Parameters
sanitized[key] = permit_any_in_parameters(value) sanitized[key] = permit_any_in_parameters(value)
else else
# Filter this one out. # Filter this one out.
...@@ -824,9 +825,10 @@ def permit_any_in_parameters(params) ...@@ -824,9 +825,10 @@ def permit_any_in_parameters(params)
def permit_any_in_array(array) def permit_any_in_array(array)
[].tap do |sanitized| [].tap do |sanitized|
array.each do |element| array.each do |element|
if permitted_scalar?(element) case element
when ->(e) { permitted_scalar?(e) }
sanitized << element sanitized << element
elsif element.is_a?(Parameters) when Parameters
sanitized << permit_any_in_parameters(element) sanitized << permit_any_in_parameters(element)
else else
# Filter this one out. # Filter this one out.
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册