From 9ce12723c09e10c92c2520a45085b922e758dabb Mon Sep 17 00:00:00 2001 From: Krzysztof Zalewski Date: Sun, 3 Jun 2012 02:38:30 +0200 Subject: [PATCH 1/2] use regexp to find multi-params keys --- lib/action_controller/parameters.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/action_controller/parameters.rb b/lib/action_controller/parameters.rb index 1acda2d..f34e72e 100644 --- a/lib/action_controller/parameters.rb +++ b/lib/action_controller/parameters.rb @@ -103,9 +103,9 @@ def each_element(object) end def multi_param_keys_for(filter) - keys.select { |key| - key.index("#{filter}(") == 0 && key.index(")") == key.length - 1 - } + keys.select do |key| + key.match /#{filter}\(\d+i\)/ + end end end From 8f12759c55e969d7d0cef9579d221c24469bcd5b Mon Sep 17 00:00:00 2001 From: Krzysztof Zalewski Date: Mon, 4 Jun 2012 14:39:27 +0200 Subject: [PATCH 2/2] use String#start_with? and String#end_with? instead of regexp --- lib/action_controller/parameters.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/action_controller/parameters.rb b/lib/action_controller/parameters.rb index f34e72e..cba4f8d 100644 --- a/lib/action_controller/parameters.rb +++ b/lib/action_controller/parameters.rb @@ -104,7 +104,7 @@ def each_element(object) def multi_param_keys_for(filter) keys.select do |key| - key.match /#{filter}\(\d+i\)/ + key.start_with? "#{filter}(" and key.end_with? ')' end end end