Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorAkos Gyimesi <akos@gyim.hu>2018-01-13 14:06:03 +0300
committerNick Thomas <nick@gitlab.com>2018-12-06 20:20:08 +0300
commit5fd5dad492a2918c10633996bb29d656a4e9137a (patch)
tree95f07d6d53289920cd9c24771d6cb9e7318fce8d /app
parent2e3cefa6cdeabe9bfd169df9ed280895deb57be3 (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.rb4
-rw-r--r--app/models/merge_request.rb2
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