Remove Commits#compare, add CompareController

4 files changed, 45 insertions, 12 deletions

diff --git a/app/controllers/commits_controller.rb b/app/controllers/commits_controller.rb
index 712b842c66d..b508d2877a4 100644
--- a/app/controllers/commits_controller.rb
+++ b/app/controllers/commits_controller.rb
@@ -45,18 +45,6 @@ class CommitsController < ApplicationController
   #   end
   # end
 
-  def compare
-    result = Commit.compare(project, params[:from], params[:to])
-
-    @commits = result[:commits]
-    @commit  = result[:commit]
-    @diffs   = result[:diffs]
-    @refs_are_same = result[:same]
-    @line_notes = []
-
-    @commits = CommitDecorator.decorate(@commits)
-  end
-
   def patch
     @commit = project.commit(params[:id])
 
diff --git a/app/controllers/compare_controller.rb b/app/controllers/compare_controller.rb
new file mode 100644
index 00000000000..1124fd0caa3
--- /dev/null
+++ b/app/controllers/compare_controller.rb
@@ -0,0 +1,22 @@
+class CompareController < ApplicationController
+  before_filter :project
+  layout "project"
+
+  # Authorize
+  before_filter :add_project_abilities
+  before_filter :authorize_read_project!
+  before_filter :authorize_code_access!
+  before_filter :require_non_empty_project
+
+  def show
+    result = Commit.compare(project, params[:from], params[:to])
+
+    @commits       = result[:commits]
+    @commit        = result[:commit]
+    @diffs         = result[:diffs]
+    @refs_are_same = result[:same]
+    @line_notes    = []
+
+    @commits = CommitDecorator.decorate(@commits)
+  end
+end
diff --git a/app/views/compare/_head.html.haml b/app/views/compare/_head.html.haml
new file mode 100644
index 00000000000..a8111a72ed5
--- /dev/null
+++ b/app/views/compare/_head.html.haml
@@ -0,0 +1,23 @@
+%ul.nav.nav-tabs
+  %li= render partial: 'shared/ref_switcher', locals: {destination: 'commits'}
+  %li{class: "#{'active' if current_page?(project_commits_path(@project)) }"}
+    = link_to project_commits_path(@project) do
+      Commits
+  %li{class: "#{'active' if current_page?(compare_project_commits_path(@project)) }"}
+    = link_to compare_project_commits_path(@project) do
+      Compare
+  %li{class: "#{branches_tab_class}"}
+    = link_to project_repository_path(@project) do
+      Branches
+      %span.badge= @project.repo.branch_count
+
+  %li{class: "#{'active' if current_page?(tags_project_repository_path(@project)) }"}
+    = link_to tags_project_repository_path(@project) do
+      Tags
+      %span.badge= @project.repo.tag_count
+
+  - if current_page?(project_commits_path(@project)) && current_user.private_token
+    %li.right
+      %span.rss-icon
+        = link_to project_commits_path(@project, :atom, { private_token: current_user.private_token, ref: @ref }), title: "Feed" do
+          = image_tag "rss_ui.png", title: "feed"
diff --git a/app/views/commits/compare.html.haml b/app/views/compare/show.html.haml
index db15ba53823..db15ba53823 100644
--- a/app/views/commits/compare.html.haml
+++ b/app/views/compare/show.html.haml