fix: reorder deletion of TriggerTask to ensure triggers are deployed correctly

Author: liuruibin

apps/tools/serializers/tool.py

@@ -545,15 +545,16 @@ def delete(self):
             QuerySet(Tool).filter(id=self.data.get('id')).delete()
             ResourceMapping.objects.filter(target_id=self.data.get('id')).delete()
             QuerySet(ToolRecord).filter(tool_id=self.data.get('id')).delete()
-            trigger_ids = QuerySet(TriggerTask).filter(
-                source_type="TOOL", source_id=self.data.get('id')
-            ).values('trigger_id')
+            trigger_ids = list(
+                QuerySet(TriggerTask).filter(
+                    source_type="TOOL", source_id=self.data.get('id')
+                ).values('trigger_id').distinct()
+            )
+            QuerySet(TriggerTask).filter(source_type="TOOL", source_id=self.data.get('id')).delete()
             for trigger_id in trigger_ids:
                 trigger = Trigger.objects.filter(id=trigger_id['trigger_id']).first()
                 if trigger and trigger.is_active:
                     deploy(TriggerModelSerializer(trigger).data, **{})
-            QuerySet(TriggerTask).filter(source_type="TOOL", source_id=self.data.get('id')).delete()
-
 
         def one(self):
             self.is_one_valid(raise_exception=True)