diff options
author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2013-09-26 22:19:01 +0300 |
---|---|---|
committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2013-09-26 22:19:01 +0300 |
commit | 052a7cc9cdc0fb14a55e7a7d71a8e4696c909658 (patch) | |
tree | 4686d485901a31637fee8f8d51f306e905fc69dc | |
parent | 9bb1e15c830341f27aa976785bd1504ad98b5fb1 (diff) | |
parent | fb8a3e319a59cedd19191a169cfae08b2923ecbe (diff) | |
download | gitlab-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.rb | 15 |
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 |