2015-09-11 14:41:01 +05:30
|
|
|
# SystemNoteService
|
|
|
|
#
|
|
|
|
# Used for creating system notes (e.g., when a user references a merge request
|
|
|
|
# from an issue, an issue's assignee changes, an issue is closed, etc.)
|
|
|
|
class SystemNoteService
|
|
|
|
# Called when commits are added to a Merge Request
|
|
|
|
#
|
|
|
|
# noteable - Noteable object
|
|
|
|
# project - Project owning noteable
|
|
|
|
# author - User performing the change
|
|
|
|
# new_commits - Array of Commits added since last push
|
|
|
|
# existing_commits - Array of Commits added in a previous push
|
|
|
|
# oldrev - Optional String SHA of a previous Commit
|
|
|
|
#
|
|
|
|
# See new_commit_summary and existing_commit_summary.
|
|
|
|
#
|
|
|
|
# Returns the created Note object
|
|
|
|
def self.add_commits(noteable, project, author, new_commits, existing_commits = [], oldrev = nil)
|
|
|
|
total_count = new_commits.length + existing_commits.length
|
|
|
|
commits_text = "#{total_count} commit".pluralize(total_count)
|
|
|
|
|
|
|
|
body = "Added #{commits_text}:\n\n"
|
|
|
|
body << existing_commit_summary(noteable, existing_commits, oldrev)
|
|
|
|
body << new_commit_summary(new_commits).join("\n")
|
|
|
|
|
|
|
|
create_note(noteable: noteable, project: project, author: author, note: body)
|
|
|
|
end
|
|
|
|
|
|
|
|
# Called when the assignee of a Noteable is changed or removed
|
|
|
|
#
|
|
|
|
# noteable - Noteable object
|
|
|
|
# project - Project owning noteable
|
|
|
|
# author - User performing the change
|
|
|
|
# assignee - User being assigned, or nil
|
|
|
|
#
|
|
|
|
# Example Note text:
|
|
|
|
#
|
|
|
|
# "Assignee removed"
|
|
|
|
#
|
|
|
|
# "Reassigned to @rspeicher"
|
|
|
|
#
|
|
|
|
# Returns the created Note object
|
|
|
|
def self.change_assignee(noteable, project, author, assignee)
|
|
|
|
body = assignee.nil? ? 'Assignee removed' : "Reassigned to @#{assignee.username}"
|
|
|
|
|
|
|
|
create_note(noteable: noteable, project: project, author: author, note: body)
|
|
|
|
end
|
|
|
|
|
|
|
|
# Called when one or more labels on a Noteable are added and/or removed
|
|
|
|
#
|
|
|
|
# noteable - Noteable object
|
|
|
|
# project - Project owning noteable
|
|
|
|
# author - User performing the change
|
|
|
|
# added_labels - Array of Labels added
|
|
|
|
# removed_labels - Array of Labels removed
|
|
|
|
#
|
|
|
|
# Example Note text:
|
|
|
|
#
|
|
|
|
# "Added ~1 and removed ~2 ~3 labels"
|
|
|
|
#
|
|
|
|
# "Added ~4 label"
|
|
|
|
#
|
|
|
|
# "Removed ~5 label"
|
|
|
|
#
|
|
|
|
# Returns the created Note object
|
|
|
|
def self.change_label(noteable, project, author, added_labels, removed_labels)
|
|
|
|
labels_count = added_labels.count + removed_labels.count
|
|
|
|
|
|
|
|
references = ->(label) { "~#{label.id}" }
|
|
|
|
added_labels = added_labels.map(&references).join(' ')
|
|
|
|
removed_labels = removed_labels.map(&references).join(' ')
|
|
|
|
|
|
|
|
body = ''
|
|
|
|
|
|
|
|
if added_labels.present?
|
|
|
|
body << "added #{added_labels}"
|
|
|
|
body << ' and ' if removed_labels.present?
|
|
|
|
end
|
|
|
|
|
|
|
|
if removed_labels.present?
|
|
|
|
body << "removed #{removed_labels}"
|
|
|
|
end
|
|
|
|
|
|
|
|
body << ' ' << 'label'.pluralize(labels_count)
|
|
|
|
body = "#{body.capitalize}"
|
|
|
|
|
|
|
|
create_note(noteable: noteable, project: project, author: author, note: body)
|
|
|
|
end
|
|
|
|
|
|
|
|
# Called when the milestone of a Noteable is changed
|
|
|
|
#
|
|
|
|
# noteable - Noteable object
|
|
|
|
# project - Project owning noteable
|
|
|
|
# author - User performing the change
|
|
|
|
# milestone - Milestone being assigned, or nil
|
|
|
|
#
|
|
|
|
# Example Note text:
|
|
|
|
#
|
|
|
|
# "Milestone removed"
|
|
|
|
#
|
|
|
|
# "Miletone changed to 7.11"
|
|
|
|
#
|
|
|
|
# Returns the created Note object
|
|
|
|
def self.change_milestone(noteable, project, author, milestone)
|
|
|
|
body = 'Milestone '
|
|
|
|
body += milestone.nil? ? 'removed' : "changed to #{milestone.title}"
|
|
|
|
|
|
|
|
create_note(noteable: noteable, project: project, author: author, note: body)
|
|
|
|
end
|
|
|
|
|
|
|
|
# Called when the status of a Noteable is changed
|
|
|
|
#
|
|
|
|
# noteable - Noteable object
|
|
|
|
# project - Project owning noteable
|
|
|
|
# author - User performing the change
|
|
|
|
# status - String status
|
|
|
|
# source - Mentionable performing the change, or nil
|
|
|
|
#
|
|
|
|
# Example Note text:
|
|
|
|
#
|
|
|
|
# "Status changed to merged"
|
|
|
|
#
|
|
|
|
# "Status changed to closed by bc17db76"
|
|
|
|
#
|
|
|
|
# Returns the created Note object
|
|
|
|
def self.change_status(noteable, project, author, status, source)
|
|
|
|
body = "Status changed to #{status}"
|
|
|
|
body += " by #{source.gfm_reference}" if source
|
|
|
|
|
|
|
|
create_note(noteable: noteable, project: project, author: author, note: body)
|
|
|
|
end
|
|
|
|
|
|
|
|
# Called when the title of a Noteable is changed
|
|
|
|
#
|
|
|
|
# noteable - Noteable object that responds to `title`
|
|
|
|
# project - Project owning noteable
|
|
|
|
# author - User performing the change
|
|
|
|
# old_title - Previous String title
|
|
|
|
#
|
|
|
|
# Example Note text:
|
|
|
|
#
|
|
|
|
# "Title changed from **Old** to **New**"
|
|
|
|
#
|
|
|
|
# Returns the created Note object
|
|
|
|
def self.change_title(noteable, project, author, old_title)
|
|
|
|
return unless noteable.respond_to?(:title)
|
|
|
|
|
|
|
|
body = "Title changed from **#{old_title}** to **#{noteable.title}**"
|
|
|
|
create_note(noteable: noteable, project: project, author: author, note: body)
|
|
|
|
end
|
|
|
|
|
|
|
|
# Called when a branch in Noteable is changed
|
|
|
|
#
|
|
|
|
# noteable - Noteable object
|
|
|
|
# project - Project owning noteable
|
|
|
|
# author - User performing the change
|
|
|
|
# branch_type - 'source' or 'target'
|
|
|
|
# old_branch - old branch name
|
|
|
|
# new_branch - new branch nmae
|
|
|
|
#
|
|
|
|
# Example Note text:
|
|
|
|
#
|
|
|
|
# "Target branch changed from `Old` to `New`"
|
|
|
|
#
|
|
|
|
# Returns the created Note object
|
|
|
|
def self.change_branch(noteable, project, author, branch_type, old_branch, new_branch)
|
|
|
|
body = "#{branch_type} branch changed from `#{old_branch}` to `#{new_branch}`".capitalize
|
|
|
|
create_note(noteable: noteable, project: project, author: author, note: body)
|
|
|
|
end
|
|
|
|
|
2015-10-24 18:46:33 +05:30
|
|
|
# Called when a branch in Noteable is added or deleted
|
|
|
|
#
|
|
|
|
# noteable - Noteable object
|
|
|
|
# project - Project owning noteable
|
|
|
|
# author - User performing the change
|
|
|
|
# branch_type - :source or :target
|
|
|
|
# branch - branch name
|
|
|
|
# presence - :add or :delete
|
|
|
|
#
|
|
|
|
# Example Note text:
|
|
|
|
#
|
|
|
|
# "Restored target branch `feature`"
|
|
|
|
#
|
|
|
|
# Returns the created Note object
|
|
|
|
def self.change_branch_presence(noteable, project, author, branch_type, branch, presence)
|
|
|
|
verb =
|
|
|
|
if presence == :add
|
|
|
|
'restored'
|
|
|
|
else
|
|
|
|
'deleted'
|
|
|
|
end
|
|
|
|
body = "#{verb} #{branch_type.to_s} branch `#{branch}`".capitalize
|
|
|
|
create_note(noteable: noteable, project: project, author: author, note: body)
|
|
|
|
end
|
|
|
|
|
2015-09-11 14:41:01 +05:30
|
|
|
# Called when a Mentionable references a Noteable
|
|
|
|
#
|
|
|
|
# noteable - Noteable object being referenced
|
|
|
|
# mentioner - Mentionable object
|
|
|
|
# author - User performing the reference
|
|
|
|
#
|
|
|
|
# Example Note text:
|
|
|
|
#
|
|
|
|
# "mentioned in #1"
|
|
|
|
#
|
|
|
|
# "mentioned in !2"
|
|
|
|
#
|
|
|
|
# "mentioned in 54f7727c"
|
|
|
|
#
|
|
|
|
# See cross_reference_note_content.
|
|
|
|
#
|
|
|
|
# Returns the created Note object
|
|
|
|
def self.cross_reference(noteable, mentioner, author)
|
|
|
|
return if cross_reference_disallowed?(noteable, mentioner)
|
|
|
|
|
|
|
|
gfm_reference = mentioner.gfm_reference(noteable.project)
|
|
|
|
|
|
|
|
note_options = {
|
|
|
|
project: noteable.project,
|
|
|
|
author: author,
|
|
|
|
note: cross_reference_note_content(gfm_reference)
|
|
|
|
}
|
|
|
|
|
|
|
|
if noteable.kind_of?(Commit)
|
|
|
|
note_options.merge!(noteable_type: 'Commit', commit_id: noteable.id)
|
|
|
|
else
|
|
|
|
note_options.merge!(noteable: noteable)
|
|
|
|
end
|
|
|
|
|
|
|
|
create_note(note_options)
|
|
|
|
end
|
|
|
|
|
|
|
|
def self.cross_reference?(note_text)
|
|
|
|
note_text.start_with?(cross_reference_note_prefix)
|
|
|
|
end
|
|
|
|
|
|
|
|
# Check if a cross-reference is disallowed
|
|
|
|
#
|
|
|
|
# This method prevents adding a "mentioned in !1" note on every single commit
|
|
|
|
# in a merge request. Additionally, it prevents the creation of references to
|
|
|
|
# external issues (which would fail).
|
|
|
|
#
|
|
|
|
# noteable - Noteable object being referenced
|
|
|
|
# mentioner - Mentionable object
|
|
|
|
#
|
|
|
|
# Returns Boolean
|
|
|
|
def self.cross_reference_disallowed?(noteable, mentioner)
|
|
|
|
return true if noteable.is_a?(ExternalIssue)
|
|
|
|
return false unless mentioner.is_a?(MergeRequest)
|
|
|
|
return false unless noteable.is_a?(Commit)
|
|
|
|
|
|
|
|
mentioner.commits.include?(noteable)
|
|
|
|
end
|
|
|
|
|
|
|
|
# Check if a cross reference to a noteable from a mentioner already exists
|
|
|
|
#
|
|
|
|
# This method is used to prevent multiple notes being created for a mention
|
|
|
|
# when a issue is updated, for example.
|
|
|
|
#
|
|
|
|
# noteable - Noteable object being referenced
|
|
|
|
# mentioner - Mentionable object
|
|
|
|
#
|
|
|
|
# Returns Boolean
|
|
|
|
def self.cross_reference_exists?(noteable, mentioner)
|
|
|
|
# Initial scope should be system notes of this noteable type
|
|
|
|
notes = Note.system.where(noteable_type: noteable.class)
|
|
|
|
|
|
|
|
if noteable.is_a?(Commit)
|
|
|
|
# Commits have non-integer IDs, so they're stored in `commit_id`
|
|
|
|
notes = notes.where(commit_id: noteable.id)
|
|
|
|
else
|
|
|
|
notes = notes.where(noteable_id: noteable.id)
|
|
|
|
end
|
|
|
|
|
|
|
|
gfm_reference = mentioner.gfm_reference(noteable.project)
|
|
|
|
notes = notes.where(note: cross_reference_note_content(gfm_reference))
|
|
|
|
|
|
|
|
notes.count > 0
|
|
|
|
end
|
|
|
|
|
|
|
|
private
|
|
|
|
|
|
|
|
def self.create_note(args = {})
|
|
|
|
Note.create(args.merge(system: true))
|
|
|
|
end
|
|
|
|
|
|
|
|
def self.cross_reference_note_prefix
|
|
|
|
'mentioned in '
|
|
|
|
end
|
|
|
|
|
|
|
|
def self.cross_reference_note_content(gfm_reference)
|
|
|
|
"#{cross_reference_note_prefix}#{gfm_reference}"
|
|
|
|
end
|
|
|
|
|
|
|
|
# Build an Array of lines detailing each commit added in a merge request
|
|
|
|
#
|
|
|
|
# new_commits - Array of new Commit objects
|
|
|
|
#
|
|
|
|
# Returns an Array of Strings
|
|
|
|
def self.new_commit_summary(new_commits)
|
|
|
|
new_commits.collect do |commit|
|
|
|
|
"* #{commit.short_id} - #{commit.title}"
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
# Build a single line summarizing existing commits being added in a merge
|
|
|
|
# request
|
|
|
|
#
|
|
|
|
# noteable - MergeRequest object
|
|
|
|
# existing_commits - Array of existing Commit objects
|
|
|
|
# oldrev - Optional String SHA of a previous Commit
|
|
|
|
#
|
|
|
|
# Examples:
|
|
|
|
#
|
|
|
|
# "* ea0f8418...2f4426b7 - 24 commits from branch `master`"
|
|
|
|
#
|
|
|
|
# "* ea0f8418..4188f0ea - 15 commits from branch `fork:master`"
|
|
|
|
#
|
|
|
|
# "* ea0f8418 - 1 commit from branch `feature`"
|
|
|
|
#
|
|
|
|
# Returns a newline-terminated String
|
|
|
|
def self.existing_commit_summary(noteable, existing_commits, oldrev = nil)
|
|
|
|
return '' if existing_commits.empty?
|
|
|
|
|
|
|
|
count = existing_commits.size
|
|
|
|
|
|
|
|
commit_ids = if count == 1
|
|
|
|
existing_commits.first.short_id
|
|
|
|
else
|
2015-10-24 18:46:33 +05:30
|
|
|
if oldrev && !Gitlab::Git.blank_ref?(oldrev)
|
2015-09-11 14:41:01 +05:30
|
|
|
"#{Commit.truncate_sha(oldrev)}...#{existing_commits.last.short_id}"
|
|
|
|
else
|
|
|
|
"#{existing_commits.first.short_id}..#{existing_commits.last.short_id}"
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
commits_text = "#{count} commit".pluralize(count)
|
|
|
|
|
|
|
|
branch = noteable.target_branch
|
|
|
|
branch = "#{noteable.target_project_namespace}:#{branch}" if noteable.for_fork?
|
|
|
|
|
|
|
|
"* #{commit_ids} - #{commits_text} from branch `#{branch}`\n"
|
|
|
|
end
|
|
|
|
end
|