1 files changed, 30 insertions, 17 deletions

diff --git a/app/services/issuable_base_service.rb b/app/services/issuable_base_service.rb
index 70ad97f8436..e24ae8f59f0 100644
--- a/app/services/issuable_base_service.rb
+++ b/app/services/issuable_base_service.rb
@@ -67,22 +67,14 @@ class IssuableBaseService < ::BaseProjectService
   end
 
   def filter_assignees(issuable)
-    filter_assignees_with_key(issuable, :assignee_ids, :assignees)
-    filter_assignees_with_key(issuable, :add_assignee_ids, :add_assignees)
-    filter_assignees_with_key(issuable, :remove_assignee_ids, :remove_assignees)
+    filter_assignees_using_checks(issuable, :assignee_ids)
+    filter_assignees_using_checks(issuable, :add_assignee_ids)
+    filter_assignees_using_checks(issuable, :remove_assignee_ids)
   end
 
-  def filter_assignees_with_key(issuable, id_key, key)
-    if params[key] && params[id_key].blank?
-      params[id_key] = params[key].map(&:id)
-    end
-
+  def filter_assignees_using_checks(issuable, id_key)
     return if params[id_key].blank?
 
-    filter_assignees_using_checks(issuable, id_key)
-  end
-
-  def filter_assignees_using_checks(issuable, id_key)
     unless issuable.allows_multiple_assignees?
       params[id_key] = params[id_key].first(1)
     end
@@ -154,10 +146,13 @@ class IssuableBaseService < ::BaseProjectService
   end
 
   def filter_escalation_status(issuable)
+    status_params = params.delete(:escalation_status) || {}
+    status_params.permit! if status_params.respond_to?(:permit!)
+
     result = ::IncidentManagement::IssuableEscalationStatuses::PrepareUpdateService.new(
       issuable,
       current_user,
-      params.delete(:escalation_status)
+      status_params
     ).execute
 
     return unless result.success? && result[:escalation_status].present?
@@ -266,11 +261,23 @@ class IssuableBaseService < ::BaseProjectService
     # To be overridden by subclasses
   end
 
-  def after_update(issuable)
+  def prepare_update_params(issuable)
     # To be overridden by subclasses
   end
 
+  def after_update(issuable, old_associations)
+    handle_description_updated(issuable)
+    handle_label_changes(issuable, old_associations[:labels])
+  end
+
+  def handle_description_updated(issuable)
+    return unless issuable.previous_changes.include?('description')
+
+    GraphqlTriggers.issuable_description_updated(issuable)
+  end
+
   def update(issuable)
+    prepare_update_params(issuable)
     handle_quick_actions(issuable)
     filter_params(issuable)
 
@@ -316,7 +323,7 @@ class IssuableBaseService < ::BaseProjectService
         affected_assignees = (old_associations[:assignees] + new_assignees) - (old_associations[:assignees] & new_assignees)
 
         invalidate_cache_counts(issuable, users: affected_assignees.compact)
-        after_update(issuable)
+        after_update(issuable, old_associations)
         issuable.create_new_cross_references!(current_user)
         execute_hooks(
           issuable,
@@ -356,7 +363,8 @@ class IssuableBaseService < ::BaseProjectService
 
         handle_task_changes(issuable)
         invalidate_cache_counts(issuable, users: issuable.assignees.to_a)
-        after_update(issuable)
+        # not passing old_associations here to keep `update_task` as fast as possible
+        after_update(issuable, {})
         execute_hooks(issuable, 'update', old_associations: nil)
 
         if issuable.is_a?(MergeRequest)
@@ -531,6 +539,8 @@ class IssuableBaseService < ::BaseProjectService
   end
 
   def has_label_changes?(issuable, old_labels)
+    return false if old_labels.nil?
+
     Set.new(issuable.labels) != Set.new(old_labels)
   end
 
@@ -542,12 +552,15 @@ class IssuableBaseService < ::BaseProjectService
 
   # override if needed
   def handle_label_changes(issuable, old_labels)
-    return unless has_label_changes?(issuable, old_labels)
+    return false unless has_label_changes?(issuable, old_labels)
 
     # reset to preserve the label sort order (title ASC)
     issuable.labels.reset
 
     GraphqlTriggers.issuable_labels_updated(issuable)
+
+    # return true here to avoid checking for label changes in sub classes
+    true
   end
 
   # override if needed