issuable.rb 11.3 KB
Newer Older
1
# == Issuable concern
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
2
#
3
# Contains common functionality shared between Issues and MergeRequests
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
4 5 6
#
# Used by Issue, MergeRequest
#
7
module Issuable
8
  extend ActiveSupport::Concern
9
  include CacheMarkdownField
10
  include Participable
11
  include Mentionable
12
  include Subscribable
13
  include StripAttribute
14
  include Awardable
15
  include Taskable
16
  include TimeTrackable
17

18
  # This object is used to gather issuable meta data for displaying
19
  # upvotes, downvotes, notes and closing merge requests count for issues and merge requests
20
  # lists avoiding n+1 queries and improving performance.
21
  IssuableMeta = Struct.new(:upvotes, :downvotes, :notes_count, :merge_requests_count)
22

23
  included do
24 25 26
    cache_markdown_field :title, pipeline: :single_line
    cache_markdown_field :description

27 28
    belongs_to :author, class_name: "User"
    belongs_to :assignee, class_name: "User"
29
    belongs_to :updated_by, class_name: "User"
30
    belongs_to :milestone
31
    has_many :notes, as: :noteable, inverse_of: :noteable, dependent: :destroy do
32
      def authors_loaded?
33
        # We check first if we're loaded to not load unnecessarily.
34 35
        loaded? && to_a.all? { |note| note.association(:author).loaded? }
      end
36 37 38 39 40

      def award_emojis_loaded?
        # We check first if we're loaded to not load unnecessarily.
        loaded? && to_a.all? { |note| note.association(:award_emoji).loaded? }
      end
41
    end
Timothy Andrew's avatar
Timothy Andrew committed
42

43 44
    has_many :label_links, as: :target, dependent: :destroy
    has_many :labels, through: :label_links
45
    has_many :todos, as: :target, dependent: :destroy
46

47 48
    has_one :metrics

Douwe Maan's avatar
Douwe Maan committed
49 50
    delegate :name,
             :email,
51
             :public_email,
Douwe Maan's avatar
Douwe Maan committed
52
             to: :author,
53
             allow_nil: true,
Douwe Maan's avatar
Douwe Maan committed
54 55 56 57
             prefix: true

    delegate :name,
             :email,
58
             :public_email,
Douwe Maan's avatar
Douwe Maan committed
59 60 61 62
             to: :assignee,
             allow_nil: true,
             prefix: true

Andrey Kumanyaev's avatar
Andrey Kumanyaev committed
63
    validates :author, presence: true
64
    validates :title, presence: true, length: { maximum: 255 }
65

66
    scope :authored, ->(user) { where(author_id: user) }
67
    scope :assigned_to, ->(u) { where(assignee_id: u.id)}
68
    scope :recent, -> { reorder(id: :desc) }
69
    scope :order_position_asc, -> { reorder(position: :asc) }
70 71
    scope :assigned, -> { where("assignee_id IS NOT NULL") }
    scope :unassigned, -> { where("assignee_id IS NULL") }
72
    scope :of_projects, ->(ids) { where(project_id: ids) }
73
    scope :of_milestones, ->(ids) { where(milestone_id: ids) }
74
    scope :with_milestone, ->(title) { left_joins_milestones.where(milestones: { title: title }) }
75
    scope :opened, -> { with_state(:opened, :reopened) }
76 77
    scope :only_opened, -> { with_state(:opened) }
    scope :only_reopened, -> { with_state(:reopened) }
78
    scope :closed, -> { with_state(:closed) }
79

80
    scope :left_joins_milestones,    -> { joins("LEFT OUTER JOIN milestones ON #{table_name}.milestone_id = milestones.id") }
81 82
    scope :order_milestone_due_desc, -> { left_joins_milestones.reorder('milestones.due_date IS NULL, milestones.id IS NULL, milestones.due_date DESC') }
    scope :order_milestone_due_asc,  -> { left_joins_milestones.reorder('milestones.due_date IS NULL, milestones.id IS NULL, milestones.due_date ASC') }
83

84
    scope :without_label, -> { joins("LEFT OUTER JOIN label_links ON label_links.target_type = '#{name}' AND label_links.target_id = #{table_name}.id").where(label_links: { id: nil }) }
85
    scope :join_project, -> { joins(:project) }
86
    scope :inc_notes_with_associations, -> { includes(notes: [:project, :author, :award_emoji]) }
87
    scope :references_project, -> { references(:project) }
88
    scope :non_archived, -> { join_project.where(projects: { archived: false }) }
89

90
    attr_mentionable :title, pipeline: :single_line
Yorick Peterse's avatar
Yorick Peterse committed
91 92 93 94 95 96
    attr_mentionable :description

    participant :author
    participant :assignee
    participant :notes_with_associations

97
    strip_attributes :title
98 99

    acts_as_paranoid
100 101

    after_save :update_assignee_cache_counts, if: :assignee_id_changed?
102
    after_save :record_metrics
103 104

    def update_assignee_cache_counts
105
      # make sure we flush the cache for both the old *and* new assignees(if they exist)
106
      previous_assignee = User.find_by_id(assignee_id_was) if assignee_id_was
107 108
      previous_assignee&.update_cache_counts
      assignee&.update_cache_counts
109
    end
110 111 112 113 114 115

    # We want to use optimistic lock for cases when only title or description are involved
    # http://api.rubyonrails.org/classes/ActiveRecord/Locking/Optimistic.html
    def locking_enabled?
      title_changed? || description_changed?
    end
116 117
  end

118
  module ClassMethods
119 120 121 122 123 124 125
    # Searches for records with a matching title.
    #
    # This method uses ILIKE on PostgreSQL and LIKE on MySQL.
    #
    # query - The search query as a String
    #
    # Returns an ActiveRecord::Relation.
126
    def search(query)
127
      where(arel_table[:title].matches("%#{query}%"))
128
    end
129

130 131 132 133 134 135 136
    # Searches for records with a matching title or description.
    #
    # This method uses ILIKE on PostgreSQL and LIKE on MySQL.
    #
    # query - The search query as a String
    #
    # Returns an ActiveRecord::Relation.
137
    def full_search(query)
138 139 140 141
      t = arel_table
      pattern = "%#{query}%"

      where(t[:title].matches(pattern).or(t[:description].matches(pattern)))
142 143
    end

144
    def sort(method, excluded_labels: [])
145 146 147 148 149
      sorted = case method.to_s
               when 'milestone_due_asc' then order_milestone_due_asc
               when 'milestone_due_desc' then order_milestone_due_desc
               when 'downvotes_desc' then order_downvotes_desc
               when 'upvotes_desc' then order_upvotes_desc
150 151
               when 'label_priority' then order_labels_priority(excluded_labels: excluded_labels)
               when 'priority' then order_due_date_and_labels_priority(excluded_labels: excluded_labels)
152
               when 'position_asc' then  order_position_asc
153 154 155 156 157 158
               else
                 order_by(method)
               end

      # Break ties with the ID column for pagination
      sorted.order(id: :desc)
159
    end
160

161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182
    def order_due_date_and_labels_priority(excluded_labels: [])
      # The order_ methods also modify the query in other ways:
      #
      # - For milestones, we add a JOIN.
      # - For label priority, we change the SELECT, and add a GROUP BY.#
      #
      # After doing those, we need to reorder to the order we want. The existing
      # ORDER BYs won't work because:
      #
      # 1. We need milestone due date first.
      # 2. We can't ORDER BY a column that isn't in the GROUP BY and doesn't
      #    have an aggregate function applied, so we do a useless MIN() instead.
      #
      milestones_due_date = 'MIN(milestones.due_date)'

      order_milestone_due_asc.
        order_labels_priority(excluded_labels: excluded_labels, extra_select_columns: [milestones_due_date]).
        reorder(Gitlab::Database.nulls_last_order(milestones_due_date, 'ASC'),
                Gitlab::Database.nulls_last_order('highest_priority', 'ASC'))
    end

    def order_labels_priority(excluded_labels: [], extra_select_columns: [])
183 184 185
      params = {
        target_type: name,
        target_column: "#{table_name}.id",
186
        project_column: "#{table_name}.#{project_foreign_key}",
187 188 189 190
        excluded_labels: excluded_labels
      }

      highest_priority = highest_label_priority(params).to_sql
Felipe Artur's avatar
Felipe Artur committed
191

192 193 194 195 196 197
      select_columns = [
        "#{table_name}.*",
        "(#{highest_priority}) AS highest_priority"
      ] + extra_select_columns

      select(select_columns.join(', ')).
198 199
        group(arel_table[:id]).
        reorder(Gitlab::Database.nulls_last_order('highest_priority', 'ASC'))
200 201
    end

202
    def with_label(title, sort = nil)
203
      if title.is_a?(Array) && title.size > 1
204
        joins(:labels).where(labels: { title: title }).group(*grouping_columns(sort)).having("COUNT(DISTINCT labels.title) = #{title.size}")
205 206 207 208
      else
        joins(:labels).where(labels: { title: title })
      end
    end
209 210 211 212 213

    # Includes table keys in group by clause when sorting
    # preventing errors in postgres
    #
    # Returns an array of arel columns
214 215
    def grouping_columns(sort)
      grouping_columns = [arel_table[:id]]
216

Douwe Maan's avatar
Douwe Maan committed
217
      if %w(milestone_due_desc milestone_due_asc).include?(sort)
218
        milestone_table = Milestone.arel_table
219 220
        grouping_columns << milestone_table[:id]
        grouping_columns << milestone_table[:due_date]
221 222
      end

223
      grouping_columns
224
    end
225 226 227 228

    def to_ability_name
      model_name.singular
    end
229 230 231 232 233 234 235 236 237
  end

  def today?
    Date.today == created_at.to_date
  end

  def new?
    today? && created_at == updated_at
  end
238 239 240 241 242

  def is_being_reassigned?
    assignee_id_changed?
  end

243 244 245 246
  def open?
    opened? || reopened?
  end

Z.J. van de Weg's avatar
Z.J. van de Weg committed
247
  def user_notes_count
248 249 250 251 252 253 254
    if notes.loaded?
      # Use the in-memory association to select and count to avoid hitting the db
      notes.to_a.count { |note| !note.system? }
    else
      # do the count query
      notes.user.count
    end
Z.J. van de Weg's avatar
Z.J. van de Weg committed
255 256
  end

257
  def subscribed_without_subscriptions?(user, project)
258 259 260
    participants(user).include?(user)
  end

Kirill Zaitsev's avatar
Kirill Zaitsev committed
261
  def to_hook_data(user)
262
    hook_data = {
263
      object_kind: self.class.name.underscore,
Kirill Zaitsev's avatar
Kirill Zaitsev committed
264
      user: user.hook_attrs,
265 266
      project: project.hook_attrs,
      object_attributes: hook_attrs,
267
      labels: labels.map(&:hook_attrs),
268 269
      # DEPRECATED
      repository: project.hook_attrs.slice(:name, :url, :description, :homepage)
270
    }
271
    hook_data[:assignee] = assignee.hook_attrs if assignee
272 273

    hook_data
274
  end
275

276 277 278 279
  def labels_array
    labels.to_a
  end

280 281 282 283
  def label_names
    labels.order('title ASC').pluck(:title)
  end

284 285 286 287 288 289 290
  # Convert this Issuable class name to a format usable by Ability definitions
  #
  # Examples:
  #
  #   issuable.class           # => MergeRequest
  #   issuable.to_ability_name # => "merge_request"
  def to_ability_name
291
    self.class.to_ability_name
292 293
  end

294 295 296 297 298 299 300 301 302 303 304
  # Convert this Issuable class name to a format usable by notifications.
  #
  # Examples:
  #
  #   issuable.class           # => MergeRequest
  #   issuable.human_class_name # => "merge request"

  def human_class_name
    @human_class_name ||= self.class.name.titleize.downcase
  end

305 306 307 308 309 310 311 312
  # Returns a Hash of attributes to be used for Twitter card metadata
  def card_attributes
    {
      'Author'   => author.try(:name),
      'Assignee' => assignee.try(:name)
    }
  end

313
  def notes_with_associations
314 315 316 317 318 319
    # If A has_many Bs, and B has_many Cs, and you do
    # `A.includes(b: :c).each { |a| a.b.includes(:c) }`, sadly ActiveRecord
    # will do the inclusion again. So, we check if all notes in the relation
    # already have their authors loaded (possibly because the scope
    # `inc_notes_with_associations` was used) and skip the inclusion if that's
    # the case.
320 321 322 323 324 325 326 327
    includes = []
    includes << :author unless notes.authors_loaded?
    includes << :award_emoji unless notes.award_emojis_loaded?
    if includes.any?
      notes.includes(includes)
    else
      notes
    end
328 329
  end

330 331 332
  def updated_tasks
    Taskable.get_updated_tasks(old_content: previous_changes['description'].first,
                               new_content: description)
333
  end
334 335 336 337 338 339 340 341 342

  ##
  # Method that checks if issuable can be moved to another project.
  #
  # Should be overridden if issuable can be moved.
  #
  def can_move?(*)
    false
  end
343

Yorick Peterse's avatar
Yorick Peterse committed
344 345 346 347 348
  def assignee_or_author?(user)
    # We're comparing IDs here so we don't need to load any associations.
    author_id == user.id || assignee_id == user.id
  end

349 350 351 352
  def record_metrics
    metrics = self.metrics || create_metrics
    metrics.record!
  end
353
end