diff options
author | Valery Sizov <valery@gitlab.com> | 2017-05-05 18:22:54 +0300 |
---|---|---|
committer | Valery Sizov <valery@gitlab.com> | 2017-05-05 18:22:54 +0300 |
commit | 9c0f2485b5d7fed5b538313e5bf867110273c004 (patch) | |
tree | efe6b72d97557bff4c1d0b1e6bc065517fd714db /app | |
parent | 40f51c8e8172933ec511dd828a589abb52d83c45 (diff) |
Multiple issue assignee: resolve conflicts after merging upstream
Diffstat (limited to 'app')
-rw-r--r-- | app/services/slash_commands/interpret_service.rb | 32 |
1 files changed, 13 insertions, 19 deletions
diff --git a/app/services/slash_commands/interpret_service.rb b/app/services/slash_commands/interpret_service.rb index 78a6e072f26..a7e13648b54 100644 --- a/app/services/slash_commands/interpret_service.rb +++ b/app/services/slash_commands/interpret_service.rb @@ -91,41 +91,35 @@ module SlashCommands end desc 'Assign' - explanation do |user| - "Assigns #{user.to_reference}." if user + explanation do |users| + "Assigns #{users.map(&:to_reference).to_sentence}." if users.any? end params '@user' condition do current_user.can?(:"admin_#{issuable.to_ability_name}", project) end -<<<<<<< HEAD - command :assign do |assignee_param| - user_ids = extract_references(assignee_param, :user).map(&:id) + parse_params do |assignee_param| + users = extract_references(assignee_param, :user) - if user_ids.empty? - user_ids = User.where(username: assignee_param.split(' ').map(&:strip)).pluck(:id) + if users.empty? + users = User.where(username: assignee_param.split(' ').map(&:strip)) end - next if user_ids.empty? + users + end + command :assign do |users| + next if users.empty? if issuable.is_a?(Issue) - @updates[:assignee_ids] = user_ids + @updates[:assignee_ids] = users.map(&:id) else - @updates[:assignee_id] = user_ids.last + @updates[:assignee_id] = users.last.id end -======= - parse_params do |assignee_param| - extract_references(assignee_param, :user).first || - User.find_by(username: assignee_param) - end - command :assign do |user| - @updates[:assignee_id] = user.id if user ->>>>>>> 10c1bf2d77fd0ab21309d0b136cbc0ac11f56c77 end desc 'Remove assignee' explanation do - "Removes assignee #{issuable.assignee.to_reference}." + "Removes assignee #{issuable.assignees.first.to_reference}." end condition do issuable.persisted? && |