diff --git a/app/helpers/notes_helper.rb b/app/helpers/notes_helper.rb index 226f0423132fe91c668935a2722fe2aac7f3e81d..57d32165e702b3e0484e3a1f17eb01a0957aee64 100644 --- a/app/helpers/notes_helper.rb +++ b/app/helpers/notes_helper.rb @@ -57,7 +57,7 @@ module NotesHelper data.merge!( note_type: LegacyDiffNote.name, - discussion_id: discussion_id + discussion_id: Digest::SHA1.hexdigest(discussion_id) ) else discussion_id = DiffNote.build_discussion_id( @@ -69,7 +69,7 @@ module NotesHelper data.merge!( position: position.to_json, note_type: DiffNote.name, - discussion_id: discussion_id + discussion_id: Digest::SHA1.hexdigest(discussion_id) ) end diff --git a/app/models/diff_note.rb b/app/models/diff_note.rb index 3edbdf9f9663ecaaa8f27e8f7aedde33ee97d9b9..8c99be7df8a582e7300b39c8921db052f3692e0c 100644 --- a/app/models/diff_note.rb +++ b/app/models/diff_note.rb @@ -34,11 +34,11 @@ class DiffNote < Note end def discussion_id - @discussion_id ||= self.class.build_discussion_id(noteable_type, noteable_id || commit_id, position) + @discussion_id ||= Digest::SHA1.hexdigest(self.class.build_discussion_id(noteable_type, noteable_id || commit_id, position)) end def original_discussion_id - @original_discussion_id ||= self.class.build_discussion_id(noteable_type, noteable_id || commit_id, original_position) + @original_discussion_id ||= Digest::SHA1.hexdigest(self.class.build_discussion_id(noteable_type, noteable_id || commit_id, original_position)) end def position=(new_position) diff --git a/app/models/legacy_diff_note.rb b/app/models/legacy_diff_note.rb index 04a651d50abd4e99c5b6326f2bab10e4e23c219b..7bff9f4f052a3fca9befd4d4f97c3daeaf3c8528 100644 --- a/app/models/legacy_diff_note.rb +++ b/app/models/legacy_diff_note.rb @@ -22,7 +22,7 @@ class LegacyDiffNote < Note end def discussion_id - @discussion_id ||= self.class.build_discussion_id(noteable_type, noteable_id || commit_id, line_code) + @discussion_id ||= Digest::SHA1.hexdigest(self.class.build_discussion_id(noteable_type, noteable_id || commit_id, line_code)) end def diff_file_hash diff --git a/app/models/note.rb b/app/models/note.rb index 7668f3378e9b863ee480feb6666ffe4922a6c8ec..2c23e04e5459030669a57554861c0fcf7874e5d6 100644 --- a/app/models/note.rb +++ b/app/models/note.rb @@ -145,9 +145,9 @@ class Note < ActiveRecord::Base def discussion_id @discussion_id ||= if for_merge_request? - [:discussion, :note, id].join("-") + Digest::SHA1.hexdigest([:discussion, :note, id].join("-")) else - self.class.build_discussion_id(noteable_type, noteable_id || commit_id) + Digest::SHA1.hexdigest(self.class.build_discussion_id(noteable_type, noteable_id || commit_id)) end end