Commit 77926ea0 authored by Lin Jen-Shin's avatar Lin Jen-Shin

Merge branch 'projects-controller-private' into 'master'

Separate private methods under the keyword on projects controller

See merge request gitlab-org/gitlab-ce!31223
parents e2ce2326 8476516e
...@@ -284,6 +284,18 @@ class ProjectsController < Projects::ApplicationController ...@@ -284,6 +284,18 @@ class ProjectsController < Projects::ApplicationController
end end
# rubocop: enable CodeReuse/ActiveRecord # rubocop: enable CodeReuse/ActiveRecord
def resolve
@project = Project.find(params[:id])
if can?(current_user, :read_project, @project)
redirect_to @project
else
render_404
end
end
private
# Render project landing depending of which features are available # Render project landing depending of which features are available
# So if page is not available in the list it renders the next page # So if page is not available in the list it renders the next page
# #
...@@ -453,14 +465,4 @@ class ProjectsController < Projects::ApplicationController ...@@ -453,14 +465,4 @@ class ProjectsController < Projects::ApplicationController
def present_project def present_project
@project = @project.present(current_user: current_user) @project = @project.present(current_user: current_user)
end end
def resolve
@project = Project.find(params[:id])
if can?(current_user, :read_project, @project)
redirect_to @project
else
render_404
end
end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment