diff --git a/config/routes.rb b/config/routes.rb index 25a7ca59b..5cba46c00 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -270,6 +270,9 @@ ActionController::Routing::Routes.draw do |map| repository_views.connect 'projects/:id/repository/:repository_id/show/*path', :requirements => { :path => /.+/ } + repository_views.connect 'projects/:id/repository/:repository_id/revision', + :action => 'revision' + repository_views.connect 'projects/:id/repository/revisions', :action => 'revisions' repository_views.connect 'projects/:id/repository/revisions.:format', @@ -296,13 +299,12 @@ ActionController::Routing::Routes.draw do |map| repository_views.connect 'projects/:id/repository/:action/*path', :requirements => { :action => /(browse|show|entry|changes|annotate|diff)/ } + repository_views.connect 'projects/:id/repository/revision', + :action => 'revision' + repository_views.connect 'projects/:id/repository/:repository_id', :action => 'show' end - - repositories.connect 'projects/:id/repository/revision', - :action => 'revision', - :conditions => {:method => [:get, :post]} end # additional routes for having the file name at the end of url diff --git a/test/integration/routing/repositories_test.rb b/test/integration/routing/repositories_test.rb index 14c835f37..60a1d408b 100644 --- a/test/integration/routing/repositories_test.rb +++ b/test/integration/routing/repositories_test.rb @@ -300,6 +300,11 @@ class RoutingRepositoriesTest < ActionController::IntegrationTest { :controller => 'repositories', :action => 'changes', :id => 'redmine', :path => @path_hash[:param] } ) + assert_routing( + { :method => 'get', + :path => "/projects/redmine/repository/revision" }, + { :controller => 'repositories', :action => 'revision', :id => 'redmine' } + ) end def test_repositories_non_revisions_path_with_repository_id @@ -339,6 +344,11 @@ class RoutingRepositoriesTest < ActionController::IntegrationTest { :controller => 'repositories', :action => 'changes', :id => 'redmine', :repository_id => 'foo', :path => @path_hash[:param] } ) + assert_routing( + { :method => 'get', + :path => "/projects/redmine/repository/foo/revision" }, + { :controller => 'repositories', :action => 'revision', :id => 'redmine', :repository_id => 'foo'} + ) end def test_repositories_related_issues