Enable routes to recognize uppercase controller names. Closes #1635

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@1765 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
This commit is contained in:
Nicholas Seckar 2005-07-07 19:51:20 +00:00
parent 5727dc2f42
commit f4bfacb24f
2 changed files with 13 additions and 1 deletions

View File

@ -210,7 +210,7 @@ module ActionController
mod_name = controller_name = segment = nil mod_name = controller_name = segment = nil
while index < length while index < length
return nil unless /^[a-z][a-z\d_]*$/ =~ (segment = segments[index]) return nil unless /^[A-Za-z][A-Za-z\d_]*$/ =~ (segment = segments[index])
index += 1 index += 1
mod_name = segment.camelize mod_name = segment.camelize

View File

@ -109,6 +109,9 @@ module Object::Controllers
class UserController class UserController
end end
class NewsFeedController
end
end end
end end
@ -718,6 +721,15 @@ class RouteSetTests < Test::Unit::TestCase
def test_action_expiry def test_action_expiry
assert_equal ['/content', {}], rs.generate({:controller => 'content'}, {:controller => 'content', :action => 'show'}) assert_equal ['/content', {}], rs.generate({:controller => 'content'}, {:controller => 'content', :action => 'show'})
end end
def test_recognition_with_uppercase_controller_name
assert_equal({'controller' => ::Controllers::ContentController, 'action' => 'index'}, rs.recognize_path(%w(Content)))
assert_equal({'controller' => ::Controllers::ContentController, 'action' => 'list'}, rs.recognize_path(%w(Content list)))
assert_equal({'controller' => ::Controllers::ContentController, 'action' => 'show', 'id' => '10'}, rs.recognize_path(%w(Content show 10)))
assert_equal({'controller' => ::Controllers::Admin::NewsFeedController, 'action' => 'index'}, rs.recognize_path(%w(Admin NewsFeed)))
assert_equal({'controller' => ::Controllers::Admin::NewsFeedController, 'action' => 'index'}, rs.recognize_path(%w(Admin News_Feed)))
end
end end
end end