summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2013-09-26 22:19:01 +0300
committerDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2013-09-26 22:19:01 +0300
commit052a7cc9cdc0fb14a55e7a7d71a8e4696c909658 (patch)
tree4686d485901a31637fee8f8d51f306e905fc69dc
parent9bb1e15c830341f27aa976785bd1504ad98b5fb1 (diff)
parentfb8a3e319a59cedd19191a169cfae08b2923ecbe (diff)
downloadgitlab-ce-052a7cc9cdc0fb14a55e7a7d71a8e4696c909658.tar.gz
Merge branch 'patch-2' of https://github.com/zzet/gitlabhq into zzet-patch-2
Conflicts: app/controllers/projects/merge_requests_controller.rb
-rw-r--r--app/controllers/projects/merge_requests_controller.rb15
1 files changed, 9 insertions, 6 deletions
diff --git a/app/controllers/projects/merge_requests_controller.rb b/app/controllers/projects/merge_requests_controller.rb
index 3bc50b0418f..0cc09caf1d2 100644
--- a/app/controllers/projects/merge_requests_controller.rb
+++ b/app/controllers/projects/merge_requests_controller.rb
@@ -157,12 +157,15 @@ class Projects::MergeRequestsController < Projects::ApplicationController
end
def validates_merge_request
- # Show git not found page if target branch doesn't exist
- return invalid_mr unless @merge_request.target_project.repository.branch_names.include?(@merge_request.target_branch)
-
- # Show git not found page if source branch doesn't exist
- # and there is no saved commits between source & target branch
- return invalid_mr if !@merge_request.source_project.repository.branch_names.include?(@merge_request.source_branch) && @merge_request.commits.blank?
+ # Show git not found page
+ # if there is no saved commits between source & target branch
+ if @merge_request.commits.blank?
+ # and if source target doesn't exist
+ return invalid_mr unless @merge_request.target_project.repository.branch_names.include?(@merge_request.target_branch)
+
+ # or if source branch doesn't exist
+ return invalid_mr unless @merge_request.source_project.repository.branch_names.include?(@merge_request.source_branch)
+ end
end
def define_show_vars