note.rb 9.68 KB
Newer Older
1 2 3 4 5 6 7 8
# == Schema Information
#
# Table name: notes
#
#  id            :integer          not null, primary key
#  note          :text
#  noteable_type :string(255)
#  author_id     :integer
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
9 10
#  created_at    :datetime
#  updated_at    :datetime
11 12 13
#  project_id    :integer
#  attachment    :string(255)
#  line_code     :string(255)
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
14 15
#  commit_id     :string(255)
#  noteable_id   :integer
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
16
#  system        :boolean          default(FALSE), not null
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
17
#  st_diff       :text
Stan Hu's avatar
Stan Hu committed
18
#  updated_by_id :integer
Stan Hu's avatar
Stan Hu committed
19
#  is_award      :boolean          default(FALSE), not null
20 21
#

gitlabhq's avatar
gitlabhq committed
22 23 24 25
require 'carrierwave/orm/activerecord'
require 'file_size_validator'

class Note < ActiveRecord::Base
26
  include Gitlab::CurrentSettings
27
  include Participable
28
  include Mentionable
Valery Sizov's avatar
Valery Sizov committed
29
  include Elastic::NotesSearch
30

31 32
  default_value_for :system, false

33
  attr_mentionable :note, cache: true, pipeline: :note
34
  participant :author
35

gitlabhq's avatar
gitlabhq committed
36
  belongs_to :project
37
  belongs_to :noteable, polymorphic: true, touch: true
Nihad Abbasov's avatar
Nihad Abbasov committed
38
  belongs_to :author, class_name: "User"
39
  belongs_to :updated_by, class_name: "User"
gitlabhq's avatar
gitlabhq committed
40

41
  has_many :todos, dependent: :destroy
42

43
  delegate :gfm_reference, :local_reference, to: :noteable
Nihad Abbasov's avatar
Nihad Abbasov committed
44 45
  delegate :name, to: :project, prefix: true
  delegate :name, :email, to: :author, prefix: true
46

47 48
  before_validation :set_award!

49
  validates :note, :project, presence: true
Valery Sizov's avatar
Valery Sizov committed
50
  validates :note, uniqueness: { scope: [:author, :noteable_type, :noteable_id] }, if: ->(n) { n.is_award }
51
  validates :note, inclusion: { in: Emoji.emojis_names }, if: ->(n) { n.is_award }
52
  validates :line_code, line_code: true, allow_blank: true
53 54
  # Attachments are deprecated and are handled by Markdown uploader
  validates :attachment, file_size: { maximum: :max_attachment_size }
gitlabhq's avatar
gitlabhq committed
55

56 57
  validates :noteable_id, presence: true, if: ->(n) { n.noteable_type.present? && n.noteable_type != 'Commit' }
  validates :commit_id, presence: true, if: ->(n) { n.noteable_type == 'Commit' }
Valery Sizov's avatar
Valery Sizov committed
58
  validates :author, presence: true
59

60
  mount_uploader :attachment, AttachmentUploader
Andrey Kumanyaev's avatar
Andrey Kumanyaev committed
61 62

  # Scopes
Valery Sizov's avatar
Valery Sizov committed
63 64
  scope :awards, ->{ where(is_award: true) }
  scope :nonawards, ->{ where(is_award: false) }
65
  scope :searchable, ->{ where("is_award IS FALSE AND system IS FALSE") }
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
66
  scope :for_commit_id, ->(commit_id) { where(noteable_type: "Commit", commit_id: commit_id) }
67 68
  scope :inline, ->{ where("line_code IS NOT NULL") }
  scope :not_inline, ->{ where(line_code: [nil, '']) }
69
  scope :system, ->{ where(system: true) }
70
  scope :user, ->{ where(system: false) }
71
  scope :common, ->{ where(noteable_type: ["", nil]) }
72
  scope :fresh, ->{ order(created_at: :asc, id: :asc) }
73 74
  scope :inc_author_project, ->{ includes(:project, :author) }
  scope :inc_author, ->{ includes(:author) }
gitlabhq's avatar
gitlabhq committed
75

76
  scope :with_associations, -> do
77
    includes(:author, :noteable, :updated_by,
78
             project: [:project_members, { group: [:group_members] }])
79
  end
gitlabhq's avatar
gitlabhq committed
80

81
  serialize :st_diff
82
  before_create :set_diff, if: ->(n) { n.line_code.present? }
83

84 85 86 87 88 89 90 91 92
  class << self
    def discussions_from_notes(notes)
      discussion_ids = []
      discussions = []

      notes.each do |note|
        next if discussion_ids.include?(note.discussion_id)

        # don't group notes for the main target
93
        if !note.for_diff_line? && note.for_merge_request?
94 95 96 97 98 99 100 101 102 103 104
          discussions << [note]
        else
          discussions << notes.select do |other_note|
            note.discussion_id == other_note.discussion_id
          end
          discussion_ids << note.discussion_id
        end
      end

      discussions
    end
105

106 107 108 109
    def build_discussion_id(type, id, line_code)
      [:discussion, type.try(:underscore), id, line_code].join("-").to_sym
    end

110
    def search(query)
111
      where("LOWER(note) like :query", query: "%#{query.downcase}%")
112
    end
Valery Sizov's avatar
Valery Sizov committed
113 114

    def grouped_awards
115 116
      notes = {}

Valery Sizov's avatar
Valery Sizov committed
117
      awards.select(:note).distinct.map do |note|
118
        notes[note.note] = where(note: note.note)
Valery Sizov's avatar
Valery Sizov committed
119
      end
120 121 122 123 124

      notes["thumbsup"] ||= Note.none
      notes["thumbsdown"] ||= Note.none

      notes
Valery Sizov's avatar
Valery Sizov committed
125
    end
126
  end
127

128 129 130 131
  def searchable?
    !is_award && !system
  end

132
  def cross_reference?
133
    system && SystemNoteService.cross_reference?(note)
134 135
  end

136 137 138 139
  def max_attachment_size
    current_application_settings.max_attachment_size.megabytes.to_i
  end

140
  def find_diff
141 142
    return nil unless noteable
    return @diff if defined?(@diff)
143

144 145
    # Don't use ||= because nil is a valid value for @diff
    @diff = noteable.diffs(Commit.max_diff_options).find do |d|
146
      Digest::SHA1.hexdigest(d.new_path) == diff_file_index if d.new_path
147
    end
148 149
  end

150 151 152 153
  def hook_attrs
    attributes
  end

154 155 156
  def set_diff
    # First lets find notes with same diff
    # before iterating over all mr diffs
157
    diff = diff_for_line_code unless for_merge_request?
158 159 160 161 162 163 164 165 166
    diff ||= find_diff

    self.st_diff = diff.to_hash if diff
  end

  def diff
    @diff ||= Gitlab::Git::Diff.new(st_diff) if st_diff.respond_to?(:map)
  end

167 168 169 170
  def diff_for_line_code
    Note.where(noteable_id: noteable_id, noteable_type: noteable_type, line_code: line_code).last.try(:diff)
  end

Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
171 172 173
  # Check if such line of code exists in merge request diff
  # If exists - its active discussion
  # If not - its outdated diff
174
  def active?
175
    return true unless self.diff
176
    return false unless noteable
177
    return @active if defined?(@active)
178

179 180
    diffs = noteable.diffs(Commit.max_diff_options)
    notable_diff = diffs.find { |d| d.new_path == self.diff.new_path }
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
181

182
    return @active = false if notable_diff.nil?
183

184 185 186
    parsed_lines = Gitlab::Diff::Parser.new.parse(notable_diff.diff.each_line)
    # We cannot use ||= because @active may be false
    @active = parsed_lines.any? { |line_obj| line_obj.text == diff_line }
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
187 188 189 190
  end

  def outdated?
    !active?
191 192
  end

193
  def diff_file_index
194
    line_code.split('_')[0] if line_code
195 196 197
  end

  def diff_file_name
198
    diff.new_path if diff
199 200
  end

201 202 203 204 205 206 207 208
  def file_path
    if diff.new_path.present?
      diff.new_path
    elsif diff.old_path.present?
      diff.old_path
    end
  end

209
  def diff_old_line
210
    line_code.split('_')[1].to_i if line_code
211 212 213
  end

  def diff_new_line
214
    line_code.split('_')[2].to_i if line_code
215 216
  end

217 218 219 220
  def generate_line_code(line)
    Gitlab::Diff::LineCode.generate(file_path, line.new_pos, line.old_pos)
  end

221
  def diff_line
222 223
    return @diff_line if @diff_line

224
    if diff
225
      diff_lines.each do |line|
226 227 228
        if generate_line_code(line) == self.line_code
          @diff_line = line.text
        end
229
      end
230
    end
231 232

    @diff_line
233 234
  end

235 236 237 238 239 240 241 242 243 244 245 246 247 248
  def diff_line_type
    return @diff_line_type if @diff_line_type

    if diff
      diff_lines.each do |line|
        if generate_line_code(line) == self.line_code
          @diff_line_type = line.type
        end
      end
    end

    @diff_line_type
  end

249 250 251 252 253
  def truncated_diff_lines
    max_number_of_lines = 16
    prev_match_line = nil
    prev_lines = []

254
    highlighted_diff_lines.each do |line|
255 256 257
      if line.type == "match"
        prev_lines.clear
        prev_match_line = line
258 259
      else
        prev_lines << line
260

261 262 263
        break if generate_line_code(line) == self.line_code

        prev_lines.shift if prev_lines.length >= max_number_of_lines
264 265
      end
    end
266 267

    prev_lines
268 269 270
  end

  def diff_lines
271
    @diff_lines ||= Gitlab::Diff::Parser.new.parse(diff.diff.each_line)
272 273 274 275
  end

  def highlighted_diff_lines
    Gitlab::Diff::Highlight.new(diff_lines).highlight
276 277
  end

278
  def discussion_id
279
    @discussion_id ||= Note.build_discussion_id(noteable_type, noteable_id || commit_id, line_code)
280 281 282 283 284 285 286 287 288 289 290 291 292 293
  end

  def for_commit?
    noteable_type == "Commit"
  end

  def for_commit_diff_line?
    for_commit? && for_diff_line?
  end

  def for_diff_line?
    line_code.present?
  end

Riyad Preukschas's avatar
Riyad Preukschas committed
294 295 296 297
  def for_issue?
    noteable_type == "Issue"
  end

298 299 300 301 302 303
  def for_merge_request?
    noteable_type == "MergeRequest"
  end

  def for_merge_request_diff_line?
    for_merge_request? && for_diff_line?
Cedric Gatay's avatar
Cedric Gatay committed
304
  end
305

306 307 308 309
  def for_project_snippet?
    noteable_type == "Snippet"
  end

Riyad Preukschas's avatar
Riyad Preukschas committed
310 311 312
  # override to return commits, which are not active record
  def noteable
    if for_commit?
313
      project.commit(commit_id)
314
    else
Riyad Preukschas's avatar
Riyad Preukschas committed
315
      super
316
    end
317 318
  # Temp fix to prevent app crash
  # if note commit id doesn't exist
319
  rescue
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
320
    nil
321
  end
322

Andrew8xx8's avatar
Andrew8xx8 committed
323
  # FIXME: Hack for polymorphic associations with STI
Steven Burgart's avatar
Steven Burgart committed
324
  #        For more information visit http://api.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html#label-Polymorphic+Associations
325 326
  def noteable_type=(noteable_type)
    super(noteable_type.to_s.classify.constantize.base_class.to_s)
Andrew8xx8's avatar
Andrew8xx8 committed
327
  end
Drew Blessing's avatar
Drew Blessing committed
328 329 330 331 332 333 334 335 336 337 338

  # Reset notes events cache
  #
  # Since we do cache @event we need to reset cache in special cases:
  # * when a note is updated
  # * when a note is removed
  # Events cache stored like  events/23-20130109142513.
  # The cache key includes updated_at timestamp.
  # Thus it will automatically generate a new fragment
  # when the event is updated because the key changes.
  def reset_events_cache
339
    Event.reset_event_cache_for(self)
Drew Blessing's avatar
Drew Blessing committed
340
  end
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
341

342
  def downvote?
343
    is_award && note == "thumbsdown"
344 345 346
  end

  def upvote?
347
    is_award && note == "thumbsup"
348 349
  end

350
  def editable?
351
    !system? && !is_award
352
  end
353

354 355 356 357
  def cross_reference_not_visible_for?(user)
    cross_reference? && referenced_mentionables(user).empty?
  end

358
  # Checks if note is an award added as a comment
359
  #
360 361
  # If note is an award, this method sets is_award to true
  #   and changes content of the note to award name.
362 363 364 365
  #
  # Method is executed as a before_validation callback.
  #
  def set_award!
366
    return unless awards_supported? && contains_emoji_only?
367

368 369 370 371
    self.is_award = true
    self.note = award_emoji_name
  end

372 373
  private

374
  def awards_supported?
375
    (for_issue? || for_merge_request?) && !for_diff_line?
376 377
  end

378
  def contains_emoji_only?
379
    note =~ /\A#{Banzai::Filter::EmojiFilter.emoji_pattern}\s?\Z/
380 381 382
  end

  def award_emoji_name
383
    original_name = note.match(Banzai::Filter::EmojiFilter.emoji_pattern)[1]
Valery Sizov's avatar
Valery Sizov committed
384
    AwardEmoji.normilize_emoji_name(original_name)
385
  end
gitlabhq's avatar
gitlabhq committed
386
end