summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2013-02-12 01:50:03 -0800
committerDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2013-02-12 01:50:03 -0800
commit9f2041ad0edd2919d9d1a41e0426b4fa4294bfe5 (patch)
tree0e3f91c0ae48d4d203152b8c47ce9b5e04fb024e
parentbf908989d0867fdb6e2f81dd688ca323e8cca47f (diff)
parent2d0c3e4c6dff291aa85e8bda18b8ac85e92c1994 (diff)
downloadgitlab-ce-9f2041ad0edd2919d9d1a41e0426b4fa4294bfe5.tar.gz
Merge pull request #2965 from yurifury/fix-blob-route
Gitlab incorrectly routes blobs to the compare controller depending on filename.
-rw-r--r--config/routes.rb2
-rw-r--r--spec/routing/project_routing_spec.rb1
2 files changed, 2 insertions, 1 deletions
diff --git a/config/routes.rb b/config/routes.rb
index 47c8a4122f5..88667db130e 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -166,12 +166,12 @@ Gitlab::Application.routes.draw do
get "files"
end
+ resources :blob, only: [:show], constraints: {id: /.+/}
resources :tree, only: [:show, :edit, :update], constraints: {id: /.+/}
resources :commit, only: [:show], constraints: {id: /[[:alnum:]]{6,40}/}
resources :commits, only: [:show], constraints: {id: /.+/}
resources :compare, only: [:index, :create]
resources :blame, only: [:show], constraints: {id: /.+/}
- resources :blob, only: [:show], constraints: {id: /.+/}
resources :graph, only: [:show], constraints: {id: /.+/}
match "/compare/:from...:to" => "compare#show", as: "compare",
:via => [:get, :post], constraints: {from: /.+/, to: /.+/}
diff --git a/spec/routing/project_routing_spec.rb b/spec/routing/project_routing_spec.rb
index f94bedc79a1..9cf5d91349f 100644
--- a/spec/routing/project_routing_spec.rb
+++ b/spec/routing/project_routing_spec.rb
@@ -392,6 +392,7 @@ end
describe BlobController, "routing" do
it "to #show" do
get("/gitlabhq/blob/master/app/models/project.rb").should route_to('blob#show', project_id: 'gitlabhq', id: 'master/app/models/project.rb')
+ get("/gitlabhq/blob/master/app/models/compare.rb").should route_to('blob#show', project_id: 'gitlabhq', id: 'master/app/models/compare.rb')
end
end