kotovalexarian-likes-gitlab/gitlab-org--gitlab-foss
1
0
Fork 0
Code Releases Activity

Explain why MR can't be reverted.

Browse source
This commit is contained in:
Rubén Dávila 2016-02-01 16:06:13 -05:00 committed by Robert Speicher
parent 5bb5019e04
commit ae46d6ea04
1 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
app/controllers/projects/merge_requests_controller.rb
View file

@ -10,7 +10,6 @@ class Projects::MergeRequestsController < Projects::ApplicationController
before_action :define_show_vars, only: [:show, :diffs, :commits, :builds]
before_action :define_widget_vars, only: [:merge, :cancel_merge_when_build_succeeds, :merge_check]
before_action :ensure_ref_fetched, only: [:show, :diffs, :commits, :builds]
before_action :check_target_branch_exists, only: [:revert]
# Allow read any merge_request
before_action :authorize_read_merge_request!
@ -194,8 +193,7 @@ class Projects::MergeRequestsController < Projects::ApplicationController
end
def revert
@repository.revert_merge(current_user, @merge_request)
target_branch_exists = @merge_request.target_branch_exists?
url_params = { merge_request: {
source_branch: @merge_request.revert_branch_name,
target_branch: @merge_request.target_branch,
@ -204,7 +202,13 @@ class Projects::MergeRequestsController < Projects::ApplicationController
description: "Reverts #{@merge_request.to_reference}"
}}
redirect_to new_namespace_project_merge_request_url(@project.namespace, @project, url_params)
if target_branch_exists
@repository.revert_merge(current_user, @merge_request)
redirect_to new_namespace_project_merge_request_url(@project.namespace, @project, url_params)
else
redirect_to namespace_project_merge_request_url(@project.namespace, @project, @merge_request),
alert: 'Merge Request cannot be reverted because target branch was deleted.'
end
end
def branch_from
@ -279,10 +283,6 @@ class Projects::MergeRequestsController < Projects::ApplicationController
return render_404 unless @project.merge_requests_enabled
end
def check_target_branch_exists
return render_404 unless @merge_request.target_branch_exists?
end
def validates_merge_request
# If source project was removed (Ex. mr from fork to origin)
return invalid_mr unless @merge_request.source_project