Permalink
Browse files

We actually don't need a reverse_merge! here. dup removed was giving …

…error with nil class.
  • Loading branch information...
arunagw committed Aug 12, 2011
1 parent d1f1b04 commit 8196c842b6f32271c23cba6b97918d32747018d8
Showing with 1 addition and 1 deletion.
  1. +1 −1 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.dup || {}).reverse_merge!(url_options).symbolize_keys)
+ _routes.url_for((options || {}).reverse_merge(url_options).symbolize_keys)
else
polymorphic_url(options)
end

0 comments on commit 8196c84

Please sign in to comment.