summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2015-03-20 20:11:04 +0000
committerDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2015-03-20 20:11:04 +0000
commit65b392c3941885f01ff7375fdced8ece7f28d02b (patch)
tree54b6256b383549077066577cc332275b8112655b
parentd735321ebdee53b84ea45090227cf2a571a534e0 (diff)
parent415517918eca22dff524b5ce3ad48cb852887065 (diff)
downloadgitlab-ce-65b392c3941885f01ff7375fdced8ece7f28d02b.tar.gz
Merge branch 'merge-status-transition' into 'master'
Update merge_status state to allow more transitions Previously you could only transition from the unchecked state to one of the others. This meant that the mark_as_unmerged call in AutoMergeService would rarily be able to actually transition the state. As it would usually have already been set to can_be_merged before it hit that service. Spec and Spinach tests have been run locally and all pass. See merge request !405
-rw-r--r--app/models/merge_request.rb4
1 files changed, 2 insertions, 2 deletions
diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb
index 4cbdc612297..798306f6dcc 100644
--- a/app/models/merge_request.rb
+++ b/app/models/merge_request.rb
@@ -95,11 +95,11 @@ class MergeRequest < ActiveRecord::Base
end
event :mark_as_mergeable do
- transition unchecked: :can_be_merged
+ transition [:unchecked, :cannot_be_merged] => :can_be_merged
end
event :mark_as_unmergeable do
- transition unchecked: :cannot_be_merged
+ transition [:unchecked, :can_be_merged] => :cannot_be_merged
end
state :unchecked