Browse files

when calling url_for with a hash, additional (likely unwanted) values…

… (such as :host) would be returned in the hash... calling #dup on the hash prevents this
  • Loading branch information...
1 parent e6fdfd0 commit 61579b76616d06ccb8268411421c23fb612e5113 @akaspick akaspick committed Aug 11, 2011
Showing with 13 additions and 1 deletion.
  1. +1 −1 actionpack/lib/action_dispatch/routing/url_for.rb
  2. +12 −0 actionpack/test/dispatch/routing_test.rb
View
2 actionpack/lib/action_dispatch/routing/url_for.rb
@@ -140,7 +140,7 @@ def url_for(options = nil)
when String
options
when nil, Hash
- _routes.url_for((options || {}).reverse_merge!(url_options).symbolize_keys)
+ _routes.url_for((options.dup || {}).reverse_merge!(url_options).symbolize_keys)
else
polymorphic_url(options)
end
View
12 actionpack/test/dispatch/routing_test.rb
@@ -851,6 +851,18 @@ def test_local
end
end
+ # tests the use of dup in url_for
+ def test_url_for_with_no_side_effects
+ # without dup, additional (and possibly unwanted) values will be present in the options (eg. :host)
+ original_options = {:controller => 'projects', :action => 'status'}
+ options = original_options.dup
+
+ url_for options
+
+ # verify that the options passed in have not changed from the original ones
+ assert_equal original_options, options
+ end
+
def test_projects_status
with_test_routes do
assert_equal '/projects/status', url_for(:controller => 'projects', :action => 'status', :only_path => true)

0 comments on commit 61579b7

Please sign in to comment.