nofusscomputing/centurion_erp
1
0
Fork 0
Code Issues Wiki Activity

fix(core): When creating the AuditHistory entry for a model, use the user from context

Browse Source 
ref: #772 #759
This commit is contained in:
Jon Lockwood
2025-05-25 10:29:11 +09:30
parent 197cf4eb73
commit 82e617afbe
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/core/signal/audit_history.py
View File

@ -6,7 +6,6 @@ from django.db.models.signals import (
) )
from django.dispatch import receiver from django.dispatch import receiver
from core.middleware.get_request import get_request
@receiver(post_delete, dispatch_uid="audit_history_delete") @receiver(post_delete, dispatch_uid="audit_history_delete")
@ -35,6 +34,6 @@ def audit_history(sender, instance, **kwargs):
model = instance._meta.model_name, model = instance._meta.model_name,
), ),
action = audit_action, action = audit_action,
user = get_request().user, user = instance.context['user'],
model = instance, model = instance,
) )

4
app/core/tests/unit/centurion_abstract/test_unit_centurion_abstract_model.py
View File

@ -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( model_object = model.objects.create(
**self.kwargs_create_item **self.kwargs_create_item