refac(audit/task): early return if no related_object_id

Flagsmith · Jul 27, 2023 · 457d458 · 457d458
1 parent f4e4105
commit 457d458
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/api/audit/tasks.py b/api/audit/tasks.py
@@ -84,6 +84,12 @@ def create_audit_log_from_historical_record(
 
     environment, project = instance.get_environment_and_project()
 
+    related_object_id = instance.get_audit_log_related_object_id(history_instance)
+    related_object_type = instance.get_audit_log_related_object_type(history_instance)
+
+    if not related_object_id:
+        return
+
     log_message = {
         "+": instance.get_create_log_message,
         "-": instance.get_delete_log_message,
@@ -93,7 +99,7 @@ def create_audit_log_from_historical_record(
     related_object_id = instance.get_audit_log_related_object_id(history_instance)
     related_object_type = instance.get_audit_log_related_object_type(history_instance)
 
-    if not (log_message and related_object_id):
+    if not log_message:
         return
 
     AuditLog.objects.create(