From 82e617afbe6f5e70abebf209b72fbabd113fb4a5 Mon Sep 17 00:00:00 2001 From: Jon Date: Sun, 25 May 2025 10:29:11 +0930 Subject: [PATCH] fix(core): When creating the AuditHistory entry for a model, use the user from context ref: #772 #759 --- app/core/signal/audit_history.py | 3 +-- .../centurion_abstract/test_unit_centurion_abstract_model.py | 4 +++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/app/core/signal/audit_history.py b/app/core/signal/audit_history.py index 9d2cf442..3f942337 100644 --- a/app/core/signal/audit_history.py +++ b/app/core/signal/audit_history.py @@ -6,7 +6,6 @@ from django.db.models.signals import ( ) from django.dispatch import receiver -from core.middleware.get_request import get_request @receiver(post_delete, dispatch_uid="audit_history_delete") @@ -35,6 +34,6 @@ def audit_history(sender, instance, **kwargs): model = instance._meta.model_name, ), action = audit_action, - user = get_request().user, + user = instance.context['user'], model = instance, ) diff --git a/app/core/tests/unit/centurion_abstract/test_unit_centurion_abstract_model.py b/app/core/tests/unit/centurion_abstract/test_unit_centurion_abstract_model.py index bb06b0f3..045617cd 100644 --- a/app/core/tests/unit/centurion_abstract/test_unit_centurion_abstract_model.py +++ b/app/core/tests/unit/centurion_abstract/test_unit_centurion_abstract_model.py @@ -225,7 +225,9 @@ class CenturionAbstractModelInheritedCases( - def test_model_creation(self, model): + def test_model_creation(self, model, user): + + model.context['user'] = user model_object = model.objects.create( **self.kwargs_create_item