From c068479f0a0f42af4525f13848387e157dde29aa Mon Sep 17 00:00:00 2001
From: tmaruyama <tmaruyama@e93f8b46-1217-0410-a6f0-8f06a7374b81>
Date: Mon, 2 Apr 2012 05:09:36 +0000
Subject: [PATCH] scm: fix revisions page "OK" button always returns 404 error
 (#10567)

git-svn-id: https://svn.redmine.org/redmine/trunk@9299 e93f8b46-1217-0410-a6f0-8f06a7374b81
---
 config/routes.rb                              | 10 ++++++----
 test/integration/routing/repositories_test.rb | 10 ++++++++++
 2 files changed, 16 insertions(+), 4 deletions(-)

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
-- 
GitLab