diff --git a/actionpack/CHANGELOG b/actionpack/CHANGELOG index 4f36209ccaf760258bc50029e5ed7b644a71f867..6f1bd4d3d7b10eccd883a4c30b30a6f5450d225b 100644 --- a/actionpack/CHANGELOG +++ b/actionpack/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Fixed that map.connect should convert arguments to strings #780 [Nicholas Seckar] + * Added UrlHelper#link_to_if/link_to_unless to enable other conditions that just link_to_unless_current #757 [mindel] * Fixed that single quote was not escaped in a UrlHelper#link_to javascript confirm #549 [Scott Barron] diff --git a/actionpack/lib/action_controller/routing.rb b/actionpack/lib/action_controller/routing.rb index 894b3476e18e4323a81cbaab44b3915b2e61b805..218697aca81ef10dceb89ef9adb2c44b42ee53ac 100644 --- a/actionpack/lib/action_controller/routing.rb +++ b/actionpack/lib/action_controller/routing.rb @@ -15,7 +15,7 @@ def initialize(path, hash={}) raise ArgumentError, "Regexp requirement on #{k}, but #{k} is not in this route's path!" unless @items.include? k @requirements[k] = v else - (@items.include?(k) ? @defaults : @requirements)[k] = v + (@items.include?(k) ? @defaults : @requirements)[k] = (v.nil? ? nil : v.to_s) end end diff --git a/actionpack/test/controller/routing_tests.rb b/actionpack/test/controller/routing_tests.rb index 360ca06cf8132c0a7d5ed93bb3eb57a0d7aa9332..d4f954e59059ea86a0171260cbd0f5b34b2b1c8f 100644 --- a/actionpack/test/controller/routing_tests.rb +++ b/actionpack/test/controller/routing_tests.rb @@ -490,6 +490,19 @@ def test_url_with_dots_in_controller @set.add_route(@rails_route) if @set.empty? assert_raises(ActionController::RoutingError) {@set.recognize!(@request)} end + + def test_generate_of_empty_url + @set.connect '', :controller => 'content', :action => 'view', :id => "1" + @set.add_route(@rails_route) + verify_generate('content/view/2', {:controller => 'content', :action => 'view', :id => 2}) + verify_generate('', {:controller => 'content', :action => 'view', :id => 1}) + end + def test_generate_of_empty_url_with_numeric_requirement + @set.connect '', :controller => 'content', :action => 'view', :id => 1 + @set.add_route(@rails_route) + verify_generate('content/view/2', {:controller => 'content', :action => 'view', :id => 2}) + verify_generate('', {:controller => 'content', :action => 'view', :id => 1}) + end end #require '../assertions/action_pack_assertions.rb'