diff options
| author | Rémy Coutable <remy@rymai.me> | 2016-11-03 17:15:03 +0300 |
|---|---|---|
| committer | Rémy Coutable <remy@rymai.me> | 2016-11-03 20:29:00 +0300 |
| commit | 46689162a3902ab294341b5860f16947ed030170 (patch) | |
| tree | aa7f74a8630ae5b5e7f6962fcdb5c0fcabbdb07e /app/models/merge_request.rb | |
| parent | 651bdd7c34c9ebf686b8357d8fc3ca4d0a58792e (diff) | |
Ensure merge request's "remove branch" accessors return booleans
Signed-off-by: Rémy Coutable <remy@rymai.me>
Diffstat (limited to 'app/models/merge_request.rb')
| -rw-r--r-- | app/models/merge_request.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb index 0397c57f935..6b8ac3fb48b 100644 --- a/app/models/merge_request.rb +++ b/app/models/merge_request.rb @@ -441,11 +441,11 @@ class MergeRequest < ActiveRecord::Base end def should_remove_source_branch? - merge_params['should_remove_source_branch'].present? + Gitlab::Utils.to_boolean(merge_params['should_remove_source_branch']) end def force_remove_source_branch? - merge_params['force_remove_source_branch'].present? + Gitlab::Utils.to_boolean(merge_params['force_remove_source_branch']) end def remove_source_branch? |