Make MergeRequest respond to assignee_ids & assignee_ids=

To make it simpler to assign users to an Issuable, make MergeRequest
support the attribute `assignee_ids`.

1 files changed, 12 insertions, 17 deletions

diff --git a/app/services/quick_actions/interpret_service.rb b/app/services/quick_actions/interpret_service.rb
index 4ceabaf021e..df13976fb3b 100644
--- a/app/services/quick_actions/interpret_service.rb
+++ b/app/services/quick_actions/interpret_service.rb
@@ -107,13 +107,12 @@ module QuickActions
     command :assign do |users|
       next if users.empty?
 
-      if issuable.allows_multiple_assignees?
-        @updates[:assignee_ids] = issuable.assignees.pluck(:id) + users.map(&:id)
-      elsif issuable.supports_multiple_assignees?
-        @updates[:assignee_ids] = [users.last.id]
-      else
-        @updates[:assignee_id] = users.last.id
-      end
+      @updates[:assignee_ids] =
+        if issuable.allows_multiple_assignees?
+          issuable.assignees.pluck(:id) + users.map(&:id)
+        else
+          [users.last.id]
+        end
     end
 
     desc do
@@ -138,16 +137,12 @@ module QuickActions
       # When multiple users are assigned, all will be unassigned if multiple assignees are no longer allowed
       users = extract_users(unassign_param) if issuable.allows_multiple_assignees?
 
-      if issuable.supports_multiple_assignees?
-        @updates[:assignee_ids] =
-          if users&.any?
-            issuable.assignees.pluck(:id) - users.map(&:id)
-          else
-            []
-          end
-      else
-        @updates[:assignee_id] = nil
-      end
+      @updates[:assignee_ids] =
+        if users&.any?
+          issuable.assignees.pluck(:id) - users.map(&:id)
+        else
+          []
+        end
     end
 
     desc do