Merge branch 'refactor/projects_controllers' of /home/git/repositories/gitlab/gitlabhq into 6-0-dev

app/assets/javascripts/dispatcher.js.coffee

@@ -18,19 +18,19 @@ class Dispatcher
     path = page.split(':')
 
     switch page
-      when 'issues:index'
+      when 'projects:issues:index'
         Issues.init()
       when 'dashboard:show'
         new Dashboard()
-      when 'commit:show'
+      when 'projects:commit:show'
         new Commit()
-      when 'groups:show', 'teams:show', 'projects:show'
+      when 'groups:show', 'projects:show'
         Pager.init(20, true)
       when 'projects:new', 'projects:edit'
         new Project()
-      when 'walls:show'
+      when 'projects:walls:show'
         new Wall(project_id)
-      when 'teams:members:index'
+      when 'projects:teams:members:index'
         new TeamMembers()
       when 'groups:people'
         new GroupMembers()

app/controllers/project_resource_controller.rb

-class ProjectResourceController < ApplicationController
-  before_filter :project
-  before_filter :repository
-end

app/controllers/projects/application_controller.rb

 class Projects::ApplicationController < ApplicationController
   before_filter :project
   before_filter :repository
+  layout 'projects'
 end

app/controllers/blame_controller.rb → app/controllers/projects/blame_controller.rb

 # Controller for viewing a file's blame
-class BlameController < ProjectResourceController
+class Projects::BlameController < Projects::ApplicationController
   include ExtractsPath
 
   # Authorize

app/controllers/blob_controller.rb → app/controllers/projects/blob_controller.rb

 # Controller for viewing a file's blame
-class BlobController < ProjectResourceController
+class Projects::BlobController < Projects::ApplicationController
   include ExtractsPath
 
   # Authorize

app/controllers/commit_controller.rb → app/controllers/projects/commit_controller.rb

 # Controller for a specific Commit
 #
 # Not to be confused with CommitsController, plural.
-class CommitController < ProjectResourceController
+class Projects::CommitController < Projects::ApplicationController
   # Authorize
   before_filter :authorize_read_project!
   before_filter :authorize_code_access!

app/controllers/commits_controller.rb → app/controllers/projects/commits_controller.rb

 require "base64"
 
-class CommitsController < ProjectResourceController
+class Projects::CommitsController < Projects::ApplicationController
   include ExtractsPath
 
   # Authorize

app/controllers/compare_controller.rb → app/controllers/projects/compare_controller.rb

-class CompareController < ProjectResourceController
+class Projects::CompareController < Projects::ApplicationController
   # Authorize
   before_filter :authorize_read_project!
   before_filter :authorize_code_access!

app/controllers/deploy_keys_controller.rb → app/controllers/projects/deploy_keys_controller.rb

-class DeployKeysController < ProjectResourceController
+class Projects::DeployKeysController < Projects::ApplicationController
   respond_to :html
 
   # Authorize

app/controllers/edit_tree_controller.rb → app/controllers/projects/edit_tree_controller.rb

 # Controller for edit a repository's file
-class EditTreeController < ProjectResourceController
+class Projects::EditTreeController < Projects::ApplicationController
   include ExtractsPath
 
   # Authorize

app/controllers/graphs_controller.rb → app/controllers/projects/graphs_controller.rb

-class GraphsController < ProjectResourceController
+class Projects::GraphsController < Projects::ApplicationController
   # Authorize
   before_filter :authorize_read_project!
   before_filter :authorize_code_access!

app/controllers/hooks_controller.rb → app/controllers/projects/hooks_controller.rb

-class HooksController < ProjectResourceController
+class Projects::HooksController < Projects::ApplicationController
   # Authorize
   before_filter :authorize_read_project!
   before_filter :authorize_admin_project!, only: [:new, :create, :destroy]

app/controllers/issues_controller.rb → app/controllers/projects/issues_controller.rb

-class IssuesController < ProjectResourceController
+class Projects::IssuesController < Projects::ApplicationController
   before_filter :module_enabled
   before_filter :issue, only: [:edit, :update, :show]
 

app/controllers/labels_controller.rb → app/controllers/projects/labels_controller.rb

-class LabelsController < ProjectResourceController
+class Projects::LabelsController < Projects::ApplicationController
   before_filter :module_enabled
 
   # Allow read any issue

app/controllers/merge_requests_controller.rb → app/controllers/projects/merge_requests_controller.rb

 require 'gitlab/satellite/satellite'
 
-class MergeRequestsController < ProjectResourceController
+class Projects::MergeRequestsController < Projects::ApplicationController
   before_filter :module_enabled
   before_filter :merge_request, only: [:edit, :update, :show, :commits, :diffs, :automerge, :automerge_check, :ci_status]
   before_filter :validates_merge_request, only: [:show, :diffs]

app/controllers/milestones_controller.rb → app/controllers/projects/milestones_controller.rb

-class MilestonesController < ProjectResourceController
+class Projects::MilestonesController < Projects::ApplicationController
   before_filter :module_enabled
   before_filter :milestone, only: [:edit, :update, :destroy, :show]
 

app/controllers/network_controller.rb → app/controllers/projects/network_controller.rb

-class NetworkController < ProjectResourceController
+class Projects::NetworkController < Projects::ApplicationController
   include ExtractsPath
   include ApplicationHelper
 

app/controllers/notes_controller.rb → app/controllers/projects/notes_controller.rb

-class NotesController < ProjectResourceController
+class Projects::NotesController < Projects::ApplicationController
   # Authorize
   before_filter :authorize_read_note!
   before_filter :authorize_write_note!, only: [:create]

app/controllers/protected_branches_controller.rb → app/controllers/projects/protected_branches_controller.rb

-class ProtectedBranchesController < ProjectResourceController
+class Projects::ProtectedBranchesController < Projects::ApplicationController
   # Authorize
   before_filter :authorize_read_project!
   before_filter :require_non_empty_project

app/controllers/raw_controller.rb → app/controllers/projects/raw_controller.rb

 # Controller for viewing a file's raw
-class RawController < ProjectResourceController
+class Projects::RawController < Projects::ApplicationController
   include ExtractsPath
 
   # Authorize

app/controllers/refs_controller.rb → app/controllers/projects/refs_controller.rb

-class RefsController < ProjectResourceController
+class Projects::RefsController < Projects::ApplicationController
   include ExtractsPath
 
   # Authorize

app/controllers/repositories_controller.rb → app/controllers/projects/repositories_controller.rb

-class RepositoriesController < ProjectResourceController
+class Projects::RepositoriesController < Projects::ApplicationController
   # Authorize
   before_filter :authorize_read_project!
   before_filter :authorize_code_access!

app/controllers/services_controller.rb → app/controllers/projects/services_controller.rb

-class ServicesController < ProjectResourceController
+class Projects::ServicesController < Projects::ApplicationController
   # Authorize
   before_filter :authorize_admin_project!
   before_filter :service, only: [:edit, :update, :test]

app/controllers/projects/snippets_controller.rb

@@ -14,7 +14,7 @@ class Projects::SnippetsController < Projects::ApplicationController
   # Allow destroy snippet
   before_filter :authorize_admin_project_snippet!, only: [:destroy]
 
-  layout 'project_resource'
+  layout 'projects'
 
   respond_to :html
 

app/controllers/team_members_controller.rb → app/controllers/projects/team_members_controller.rb

-class TeamMembersController < ProjectResourceController
+class Projects::TeamMembersController < Projects::ApplicationController
   # Authorize
   before_filter :authorize_read_project!
   before_filter :authorize_admin_project!, except: [:index, :show]

app/controllers/tree_controller.rb → app/controllers/projects/tree_controller.rb

 # Controller for viewing a repository's file structure
-class TreeController < ProjectResourceController
+class Projects::TreeController < Projects::ApplicationController
   include ExtractsPath
 
   # Authorize

app/controllers/walls_controller.rb → app/controllers/projects/walls_controller.rb

-class WallsController < ProjectResourceController
+class Projects::WallsController < Projects::ApplicationController
   before_filter :module_enabled
 
   respond_to :js, :html

app/controllers/wikis_controller.rb → app/controllers/projects/wikis_controller.rb

-class WikisController < ProjectResourceController
+class Projects::WikisController < Projects::ApplicationController
   before_filter :authorize_read_wiki!
   before_filter :authorize_write_wiki!, only: [:edit, :create, :history]
   before_filter :authorize_admin_wiki!, only: :destroy

app/controllers/projects_controller.rb

-class ProjectsController < ProjectResourceController
+class ProjectsController < Projects::ApplicationController
   skip_before_filter :project, only: [:new, :create]
   skip_before_filter :repository, only: [:new, :create]
 

app/helpers/commits_helper.rb

@@ -109,7 +109,7 @@ module CommitsHelper
   end
 
   def commit_to_html commit
-    escape_javascript(render 'commits/commit', commit: commit)
+    escape_javascript(render 'projects/commits/commit', commit: commit)
   end
 
   def diff_line_content(line)

app/helpers/tree_helper.rb

@@ -10,13 +10,13 @@ module TreeHelper
     tree = ""
 
     # Render folders if we have any
-    tree += render partial: 'tree/tree_item', collection: folders, locals: {type: 'folder'} if folders.present?
+    tree += render partial: 'projects/tree/tree_item', collection: folders, locals: {type: 'folder'} if folders.present?
 
     # Render files if we have any
-    tree += render partial: 'tree/blob_item', collection: files, locals: {type: 'file'} if files.present?
+    tree += render partial: 'projects/tree/blob_item', collection: files, locals: {type: 'file'} if files.present?
 
     # Render submodules if we have any
-    tree += render partial: 'tree/submodule_item', collection: submodules if submodules.present?
+    tree += render partial: 'projects/tree/submodule_item', collection: submodules if submodules.present?
 
     tree.html_safe
   end

app/views/dashboard/issues.html.haml

@@ -22,7 +22,7 @@
 
           %ul.well-list.issues-list
             - group[1].each do |issue|
-              = render issue
+              = render 'projects/issues/issue', issue: issue
       %hr
       = paginate @issues, theme: "gitlab"
     - else

app/views/groups/issues.html.haml

@@ -16,7 +16,7 @@
             = link_to_project project
           %ul.well-list.issues-list
             - group[1].each do |issue|
-              = render issue
+              = render 'projects/issues/issue', issue: issue
       %hr
       = paginate @issues, theme: "gitlab"
     - else

app/views/merge_requests/diffs.js.haml

-:plain
-  merge_request.$(".diffs").html("#{escape_javascript(render(partial: "merge_requests/show/diffs"))}");

app/views/blame/show.html.haml → app/views/projects/blame/show.html.haml

@@ -17,7 +17,7 @@
       %span.file_name
         = @blob.name
         %small= number_to_human_size @blob.size
-      %span.options= render "blob/actions"
+      %span.options= render "projects/blob/actions"
     .file_content.blame
       %table
         - current_line = 1

app/views/commit/show.html.haml → app/views/projects/commit/show.html.haml

@@ -7,5 +7,5 @@
     and
     %span.cred #{@commit.stats.deletions} deletions
 
-= render "commits/diffs", diffs: @commit.diffs
-= render "notes/notes_with_form"
+= render "projects/commits/diffs", diffs: @commit.diffs
+= render "projects/notes/notes_with_form"

app/views/commits/_diffs.html.haml → app/views/projects/commits/_diffs.html.haml

@@ -10,7 +10,7 @@
 %p.cgray
   Showing #{pluralize(diffs.count, "changed file")}
 .file-stats
-  = render "commits/diff_head", diffs: diffs
+  = render "projects/commits/diff_head", diffs: diffs
 
 .files
   - unless @suppress_diff
@@ -41,9 +41,9 @@
           -# Skipp all non non-supported blobs
           - next unless file.respond_to?('text?')
           - if file.text?
-            = render "commits/text_file", diff: diff, index: i
+            = render "projects/commits/text_file", diff: diff, index: i
           - elsif file.image?
             - old_file = Gitlab::Git::Blob.new(@repository, @commit.parent_id, @ref, diff.old_path) if @commit.parent_id
-            = render "commits/image", diff: diff, old_file: old_file, file: file, index: i
+            = render "projects/commits/image", diff: diff, old_file: old_file, file: file, index: i
           - else
             %p.nothing_here_message No preview for this file type

app/views/commits/_text_file.html.haml → app/views/projects/commits/_text_file.html.haml

@@ -13,11 +13,11 @@
         %td.old_line
           = link_to raw(type == "new" ? " " : line_old), "##{line_code}", id: line_code
           - if @comments_allowed
-            = render "notes/diff_note_link", line_code: line_code
+            = render "projects/notes/diff_note_link", line_code: line_code
         %td.new_line= link_to raw(type == "old" ? " " : line_new) , "##{line_code}", id: line_code
         %td.line_content{class: "noteable_line #{type} #{line_code}", "line_code" => line_code}= raw diff_line_content(line)
 
     - if @reply_allowed
       - comments = @line_notes.select { |n| n.line_code == line_code }.sort_by(&:created_at)
       - unless comments.empty?
-        = render "notes/diff_notes_with_reply", notes: comments
+        = render "projects/notes/diff_notes_with_reply", notes: comments

app/views/compare/index.html.haml → app/views/projects/compare/index.html.haml

-= render "commits/head"
+= render "projects/commits/head"
 
 %h3.page_title
   Compare View

app/views/compare/show.html.haml → app/views/projects/compare/show.html.haml

-= render "commits/head"
+= render "projects/commits/head"
 
 %h3.page_title
   Compare View
@@ -20,4 +20,4 @@
 
   - unless @diffs.empty?
     %h4 Diff
-    = render "commits/diffs", diffs: @diffs
+    = render "projects/commits/diffs", diffs: @diffs

app/views/issues/index.html.haml → app/views/projects/issues/index.html.haml

-= render "issues/head"
+= render "head"
 .issues_content
   %h3.page_title
     Issues

app/views/issues/show.html.haml → app/views/projects/issues/show.html.haml

@@ -65,4 +65,4 @@
     - else
       = link_to 'Close Issue', project_issue_path(@project, @issue, issue: {state_event: :close }, status_only: true), method: :put, class: "btn grouped close_issue", title: "Close Issue"
 
-.voting_notes#notes= render "notes/notes_with_form"
+.voting_notes#notes= render "projects/notes/notes_with_form"

app/views/labels/index.html.haml → app/views/projects/labels/index.html.haml

-= render "issues/head"
+= render "projects/issues/head"
 
 %h3.page_title
   Labels

app/views/merge_requests/_show.html.haml → app/views/projects/merge_requests/_show.html.haml

 .merge-request
-  = render "merge_requests/show/mr_title"
-  = render "merge_requests/show/how_to_merge"
-  = render "merge_requests/show/mr_box"
-  = render "merge_requests/show/mr_accept"
+  = render "projects/merge_requests/show/mr_title"
+  = render "projects/merge_requests/show/how_to_merge"
+  = render "projects/merge_requests/show/mr_box"
+  = render "projects/merge_requests/show/mr_accept"
   - if @project.gitlab_ci?
-    = render "merge_requests/show/mr_ci"
-  = render "merge_requests/show/commits"
+    = render "projects/merge_requests/show/mr_ci"
+  = render "projects/merge_requests/show/commits"
 
   - if @commits.present?
     %ul.nav.nav-tabs
@@ -19,9 +19,9 @@
           Diff
 
   .notes.tab-content.voting_notes#notes{ class: (controller.action_name == 'show') ? "" : "hide" }
-    = render "notes/notes_with_form"
+    = render "projects/notes/notes_with_form"
   .diffs.tab-content
-    = render "merge_requests/show/diffs" if @diffs
+    = render "projects/merge_requests/show/diffs" if @diffs
   .status
 
 :javascript

app/views/projects/merge_requests/diffs.js.haml

+:plain
+  merge_request.$(".diffs").html("#{escape_javascript(render(partial: "projects/merge_requests/show/diffs"))}");

app/views/merge_requests/invalid.html.haml → app/views/projects/merge_requests/invalid.html.haml

 .merge-request
-  = render "merge_requests/show/mr_title"
-  = render "merge_requests/show/mr_box"
+  = render "projects/merge_requests/show/mr_title"
+  = render "projects/merge_requests/show/mr_box"
 
   .alert.alert-error
     %h5