From b91d6802572eeb152bd86dd68c54ef4151180848 Mon Sep 17 00:00:00 2001 From: Riyad Preukschas Date: Mon, 29 Oct 2012 15:50:30 +0100 Subject: [PATCH] Fix notes helper --- app/helpers/notes_helper.rb | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/app/helpers/notes_helper.rb b/app/helpers/notes_helper.rb index ffcc7acc..5cada379 100644 --- a/app/helpers/notes_helper.rb +++ b/app/helpers/notes_helper.rb @@ -9,16 +9,13 @@ module NotesHelper # Helps to distinguish e.g. commit notes in mr notes list def note_for_main_target?(note) - !@mixed_targets || @main_target_type == note.noteable_type + !@mixed_targets || (@main_target_type == note.noteable_type && !note.for_diff_line?) end def link_to_commit_diff_line_note(note) - commit = note.noteable - diff_index, diff_old_line, diff_new_line = note.line_code.split('_') + if note.for_commit_diff_line? + link_to "#{note.diff_file_name}:L#{note.diff_new_line}", project_commit_path(@project, note.noteable, anchor: note.line_code) + end - link_file = commit.diffs[diff_index.to_i].new_path - link_line = diff_new_line - - link_to "#{link_file}:L#{link_line}", project_commit_path(@project, commit, anchor: note.line_code) end end