From 005752e4c6485cf79b0a53583c03b5a69fe19f10 Mon Sep 17 00:00:00 2001
From: Marin Jankovski <marin@gitlab.com>
Date: Mon, 7 Jul 2014 15:46:57 +0200
Subject: Check if blob or tree and redirect to the correct one.

---
 app/controllers/projects/blob_controller.rb | 10 +++++++---
 app/controllers/projects/tree_controller.rb |  9 ++++++++-
 2 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/app/controllers/projects/blob_controller.rb b/app/controllers/projects/blob_controller.rb
index a1a8bed09f4..db3d173b98d 100644
--- a/app/controllers/projects/blob_controller.rb
+++ b/app/controllers/projects/blob_controller.rb
@@ -30,8 +30,12 @@ class Projects::BlobController < Projects::ApplicationController
   def blob
     @blob ||= @repository.blob_at(@commit.id, @path)
 
-    return not_found! unless @blob
-
-    @blob
+    if @blob
+      @blob
+    elsif tree.entries.any?
+      redirect_to project_tree_path(@project, File.join(@ref, @path)) and return
+    else
+      return not_found!
+    end
   end
 end
diff --git a/app/controllers/projects/tree_controller.rb b/app/controllers/projects/tree_controller.rb
index 30c94ec6da0..4d033b36848 100644
--- a/app/controllers/projects/tree_controller.rb
+++ b/app/controllers/projects/tree_controller.rb
@@ -1,7 +1,14 @@
 # Controller for viewing a repository's file structure
 class Projects::TreeController < Projects::BaseTreeController
   def show
-    return not_found! if tree.entries.empty?
+
+    if tree.entries.empty?
+      if @repository.blob_at(@commit.id, @path)
+        redirect_to project_blob_path(@project, File.join(@ref, @path)) and return
+      else
+        return not_found!
+      end
+    end
 
     respond_to do |format|
       format.html
-- 
cgit v1.2.1


From e87f2380779871a64742acdd9dd0358aac7eafce Mon Sep 17 00:00:00 2001
From: Marin Jankovski <marin@gitlab.com>
Date: Tue, 8 Jul 2014 20:25:25 +0200
Subject: Add tests for redirects.

---
 spec/controllers/blob_controller_spec.rb | 14 ++++++++++++++
 spec/controllers/tree_controller_spec.rb | 13 +++++++++++++
 2 files changed, 27 insertions(+)

diff --git a/spec/controllers/blob_controller_spec.rb b/spec/controllers/blob_controller_spec.rb
index cea6922e1c3..929f6d3b46d 100644
--- a/spec/controllers/blob_controller_spec.rb
+++ b/spec/controllers/blob_controller_spec.rb
@@ -34,4 +34,18 @@ describe Projects::BlobController do
       it { should respond_with(:not_found) }
     end
   end
+
+  describe 'GET show with tree path' do
+    render_views
+
+    before do
+      get :show, project_id: project.to_param, id: id
+      controller.instance_variable_set(:@blob, nil)
+    end
+
+    context 'redirect to tree' do
+      let(:id) { 'master/doc' }
+      it { should redirect_to("/#{project.path_with_namespace}/tree/master/doc") }
+    end
+  end
 end
diff --git a/spec/controllers/tree_controller_spec.rb b/spec/controllers/tree_controller_spec.rb
index 479118a3465..b169c2a678f 100644
--- a/spec/controllers/tree_controller_spec.rb
+++ b/spec/controllers/tree_controller_spec.rb
@@ -40,4 +40,17 @@ describe Projects::TreeController do
       it { should respond_with(:not_found) }
     end
   end
+
+  describe 'GET show with blob path' do
+    render_views
+
+    before do
+      get :show, project_id: project.to_param, id: id
+    end
+
+    context 'redirect to blob' do
+      let(:id) { 'master/README.md' }
+      it { should redirect_to("/#{project.path_with_namespace}/blob/master/README.md") }
+    end
+  end
 end
-- 
cgit v1.2.1