diff options
| author | David Heinemeier Hansson <david@loudthinking.com> | 2005-02-24 01:38:10 +0000 |
|---|---|---|
| committer | David Heinemeier Hansson <david@loudthinking.com> | 2005-02-24 01:38:10 +0000 |
| commit | b7889524bfe861e38aa7f384ed116f2b15068fe1 (patch) | |
| tree | 02dd6e904a74daa135926fb397c1802376fc71f6 /actionpack/lib/action_controller | |
| parent | 74d5f6ec0c7e1c91b3620d6785689a765f505069 (diff) | |
| download | rails-b7889524bfe861e38aa7f384ed116f2b15068fe1.tar.gz rails-b7889524bfe861e38aa7f384ed116f2b15068fe1.tar.bz2 rails-b7889524bfe861e38aa7f384ed116f2b15068fe1.zip | |
Fix empty url_for with nested modules #707
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@782 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
Diffstat (limited to 'actionpack/lib/action_controller')
| -rw-r--r-- | actionpack/lib/action_controller/routing.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/actionpack/lib/action_controller/routing.rb b/actionpack/lib/action_controller/routing.rb index a522b34f73..d270defa4d 100644 --- a/actionpack/lib/action_controller/routing.rb +++ b/actionpack/lib/action_controller/routing.rb @@ -215,8 +215,9 @@ module ActionController options = options.symbolize_keys defaults = request.path_parameters.symbolize_keys - options = defaults if options.empty? # Get back the current url if no options was passed - expand_controller_path!(options, defaults) + if options.empty? then options = defaults.clone # Get back the current url if no options was passed + else expand_controller_path!(options, defaults) # Expand the supplied controller path. + end defaults.delete_if {|k, v| options.key?(k) && options[k].nil?} # Remove defaults that have been manually cleared using :name => nil failures = [] |