Fix AuthorityAnalyzer

1988e81a · Jacob Vosmaer · d7fa60c6 · 1988e81a
Commit 1988e81a authored Mar 10, 2016 by Jacob Vosmaer
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

lib/gitlab/authority_analyzer.rb lib/gitlab/authority_analyzer.rb +4 -1

No files found.
--- a/lib/gitlab/authority_analyzer.rb
+++ b/lib/gitlab/authority_analyzer.rb
@@ -28,7 +28,10 @@ module Gitlab
    end

    def list_of_involved_files
-      compare_diffs = @merge_request.compare_diffs || @merge_request.diffs
+      diffable = [@merge_request.compare, @merge_request.merge_request_diff].compact
+      return [] if diffable.empty?
+
+      compare_diffs = diffable.first.diffs

      return [] unless compare_diffs.present?