Remove conditional assignment

c7ae5917 · Kerri Miller · 49c44b65 · c7ae5917
Commit c7ae5917 authored Mar 24, 2020 by Kerri Miller
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

app/controllers/projects/merge_requests/diffs_controller.rb app/controllers/projects/merge_requests/diffs_controller.rb +2 -2

No files found.
--- a/app/controllers/projects/merge_requests/diffs_controller.rb
+++ b/app/controllers/projects/merge_requests/diffs_controller.rb
@@ -92,8 +92,8 @@ class Projects::MergeRequests::DiffsController < Projects::MergeRequests::Applic
  # Deprecated: https://gitlab.com/gitlab-org/gitlab/issues/37735
  def find_merge_request_diff_compare
    @merge_request_diff =
-      if diff_id = params[:diff_id].presence
-        @merge_request.merge_request_diffs.viewable.find_by(id: diff_id)
+      if params[:diff_id].present?
+        @merge_request.merge_request_diffs.viewable.find_by(id: params[:diff_id])
      else
        @merge_request.merge_request_diff
      end