3288e1a874
This style change enforces `return if ...` instead of `return nil if ...` to save maintainers a few minor review points
63 lines
1.6 KiB
Ruby
63 lines
1.6 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
module Gitlab
|
|
module CycleAnalytics
|
|
class PlanEventFetcher < BaseEventFetcher
|
|
def initialize(*args)
|
|
@projections = [mr_diff_table[:id],
|
|
issue_metrics_table[:first_mentioned_in_commit_at]]
|
|
|
|
super(*args)
|
|
end
|
|
|
|
def events_query
|
|
base_query
|
|
.join(mr_diff_table)
|
|
.on(mr_diff_table[:merge_request_id].eq(mr_table[:id]))
|
|
|
|
super
|
|
end
|
|
|
|
private
|
|
|
|
def allowed_ids
|
|
nil
|
|
end
|
|
|
|
def merge_request_diff_commits
|
|
@merge_request_diff_commits ||=
|
|
MergeRequestDiffCommit
|
|
.where(merge_request_diff_id: event_result.map { |event| event['id'] })
|
|
.group_by(&:merge_request_diff_id)
|
|
end
|
|
|
|
def serialize(event)
|
|
commit = first_time_reference_commit(event)
|
|
|
|
return unless commit
|
|
|
|
serialize_commit(event, commit, query)
|
|
end
|
|
|
|
def first_time_reference_commit(event)
|
|
return unless event && merge_request_diff_commits
|
|
|
|
commits = merge_request_diff_commits[event['id'].to_i]
|
|
|
|
return if commits.blank?
|
|
|
|
commits.find do |commit|
|
|
next unless commit[:committed_date] && event['first_mentioned_in_commit_at']
|
|
|
|
commit[:committed_date].to_i == DateTime.parse(event['first_mentioned_in_commit_at'].to_s).to_i
|
|
end
|
|
end
|
|
|
|
def serialize_commit(event, commit, query)
|
|
commit = Commit.from_hash(commit.to_hash, @project)
|
|
|
|
AnalyticsCommitSerializer.new(project: @project, total_time: event['total_time']).represent(commit)
|
|
end
|
|
end
|
|
end
|
|
end
|