diff options
author | Akos Gyimesi <akos@gyim.hu> | 2018-01-13 14:06:03 +0300 |
---|---|---|
committer | Nick Thomas <nick@gitlab.com> | 2018-12-06 20:20:08 +0300 |
commit | 5fd5dad492a2918c10633996bb29d656a4e9137a (patch) | |
tree | 95f07d6d53289920cd9c24771d6cb9e7318fce8d /app | |
parent | 2e3cefa6cdeabe9bfd169df9ed280895deb57be3 (diff) |
Improve error message when attempting to reopen MR and there's a new open MR for the same branch
Diffstat (limited to 'app')
-rw-r--r-- | app/controllers/projects/merge_requests_controller.rb | 4 | ||||
-rw-r--r-- | app/models/merge_request.rb | 2 |
2 files changed, 5 insertions, 1 deletions
diff --git a/app/controllers/projects/merge_requests_controller.rb b/app/controllers/projects/merge_requests_controller.rb index d521db79f85..9781d159540 100644 --- a/app/controllers/projects/merge_requests_controller.rb +++ b/app/controllers/projects/merge_requests_controller.rb @@ -122,6 +122,10 @@ class Projects::MergeRequestsController < Projects::MergeRequests::ApplicationCo respond_to do |format| format.html do + if merge_request_params[:state_event] && @merge_request.errors.any? + flash[:alert] = @merge_request.errors.values.flatten.to_sentence + end + if @merge_request.valid? redirect_to([@merge_request.target_project.namespace.becomes(Namespace), @merge_request.target_project, @merge_request]) else diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb index d0811a715bc..b50cfa639e0 100644 --- a/app/models/merge_request.rb +++ b/app/models/merge_request.rb @@ -547,7 +547,7 @@ class MergeRequest < ActiveRecord::Base similar_mrs = similar_mrs.where('id not in (?)', self.id) if self.id if similar_mrs.any? errors.add :validate_branches, - "Cannot Create: This merge request already exists: #{similar_mrs.pluck(:title)}" + "Cannot open this Merge Request because another open Merge Request already exists for this source branch: #{similar_mrs.first.title}" end end end |