Permalink
Browse files

Fix a bug in url generation for generic routes.

  • Loading branch information...
josevalim committed Apr 12, 2010
1 parent 25cb213 commit 86defed5ad6c1ec95b2f07a93982a514e2347749
Showing with 21 additions and 2 deletions.
  1. +9 −2 actionpack/lib/action_dispatch/routing/mapper.rb
  2. +12 −0 actionpack/test/dispatch/routing_test.rb
@@ -129,7 +129,10 @@ def default_controller_and_action
{}
end
- defaults[:controller] ||= @options[:controller] || default_controller
+ defaults[:controller] ||= default_controller
+
+ defaults.delete(:controller) if defaults[:controller].blank?
+ defaults.delete(:action) if defaults[:action].blank?
if defaults[:controller].blank? && segment_keys.exclude?("controller")
raise ArgumentError, "missing :controller"
@@ -177,7 +180,11 @@ def to
end
def default_controller
- @scope[:controller].to_s if @scope[:controller]
+ if @options[:controller]
+ @options[:controller].to_s
+ elsif @scope[:controller]
+ @scope[:controller].to_s
+ end
end
end
@@ -197,6 +197,8 @@ def self.matches?(request)
resource :me
match '/' => 'mes#index'
end
+
+ match "whatever/:controller(/:action(/:id))"
end
end
@@ -979,6 +981,16 @@ def test_path_scope
end
end
+ def test_url_generator_for_generic_route
+ with_test_routes do
+ get 'whatever/foo/bar'
+ assert_equal 'foo#bar', @response.body
+
+ assert_equal 'http://www.example.com/whatever/foo/bar/1',
+ url_for(:controller => "foo", :action => "bar", :id => 1)
+ end
+ end
+
private
def with_test_routes
yield

0 comments on commit 86defed

Please sign in to comment.