bb883387f9
# Conflicts: # app/controllers/projects/merge_requests_controller.rb # app/models/note.rb # db/schema.rb # spec/models/note_spec.rb
27 lines
893 B
Ruby
27 lines
893 B
Ruby
class NotesFinder
|
|
FETCH_OVERLAP = 5.seconds
|
|
|
|
def execute(project, current_user, params)
|
|
target_type = params[:target_type]
|
|
target_id = params[:target_id]
|
|
# Default to 0 to remain compatible with old clients
|
|
last_fetched_at = Time.at(params.fetch(:last_fetched_at, 0).to_i)
|
|
|
|
notes =
|
|
case target_type
|
|
when "commit"
|
|
project.notes.for_commit_id(target_id).non_diff_notes
|
|
when "issue"
|
|
project.issues.find(target_id).notes.inc_author
|
|
when "merge_request"
|
|
project.merge_requests.find(target_id).mr_and_commit_notes.inc_author
|
|
when "snippet", "project_snippet"
|
|
project.snippets.find(target_id).notes
|
|
else
|
|
raise 'invalid target_type'
|
|
end
|
|
|
|
# Use overlapping intervals to avoid worrying about race conditions
|
|
notes.where('updated_at > ?', last_fetched_at - FETCH_OVERLAP).fresh
|
|
end
|
|
end
|