From a794b161cd996f3328c0b9a60d0e3cbdaee2a914 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mica=C3=ABl=20Bergeron?= Date: Mon, 18 Dec 2017 12:02:45 -0500 Subject: rework the merge_request_version_params to use Object#tap --- app/models/diff_discussion.rb | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/app/models/diff_discussion.rb b/app/models/diff_discussion.rb index 758a26d3c70..d67b16584a4 100644 --- a/app/models/diff_discussion.rb +++ b/app/models/diff_discussion.rb @@ -23,9 +23,8 @@ class DiffDiscussion < Discussion def merge_request_version_params return unless for_merge_request? - version_params do |params| + version_params.tap do |params| params[:commit_id] = commit_id if on_merge_request_commit? - params end end @@ -39,11 +38,8 @@ class DiffDiscussion < Discussion private def version_params - params = if active? - {} - else - noteable.version_params_for(position.diff_refs) - end - yield params + return {} if active? + + noteable.version_params_for(position.diff_refs) end end -- cgit v1.2.1