TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

remove param dup'ing logic

Browse Source 
since we are serializing parameters, we don't need to do all the dup
checks on each object.
This commit is contained in:
Aaron Patterson 2015-07-09 19:14:22 -07:00
parent c546a2b045
commit 79ab812663
1 changed files with 0 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
actionpack/lib/action_controller/test_case.rb
View File

@ -48,14 +48,6 @@ def assign_parameters(routes, controller_path, action, parameters = {})
non_path_parameters = {}.with_indifferent_access
parameters.each do |key, value|
if value.is_a?(Array) && (value.frozen? || value.any?(&:frozen?))
value = value.map{ |v| v.duplicable? ? v.dup : v }
elsif value.is_a?(Hash) && (value.frozen? || value.any?{ |k,v| v.frozen? })
value = Hash[value.map{ |k,v| [k, v.duplicable? ? v.dup : v] }]
elsif value.frozen? && value.duplicable?
value = value.dup
end
if extra_keys.include?(key) || key == :action || key == :controller
non_path_parameters[key] = value
else