nofusscomputing/centurion_erp
1
0
Fork 0
Code Issues Wiki Activity

fix(core): Correct before lookup for current models audit history

Browse Source 
ref: #772 #759 #767
This commit is contained in:
Jon Lockwood
2025-05-25 08:26:56 +09:30
parent 64e53f6980
commit 105d89ee61
2 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/core/models/centurion.py
View File

@ -84,7 +84,7 @@ class CenturionModel(
self._after = {}
self._before = self.objects.get( id = self.id ).get_audit_values()
self._before = type(self).objects.get( id = self.id ).get_audit_values()
super().delete(using = using, keep_parents = keep_parents)
@ -213,7 +213,7 @@ class CenturionModel(
if self.id:
self._before = self.objects.get( id = self.id ).get_audit_values()
self._before = type(self).objects.get( id = self.id ).get_audit_values()
super().save(force_insert=force_insert, force_update=force_update, using=using, update_fields=update_fields)

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

@ -149,7 +149,7 @@ class CenturionAbstractModelTestCases(
def get(*args, **kwargs):
return model_instance
model_instance.objects = MockManager()
mocker.patch('access.models.tenancy_abstract.TenancyAbstractModel.objects', new_callable=MockManager)
super_delete = mocker.patch('django.db.models.base.Model.delete', return_value = None)
@ -315,7 +315,7 @@ class CenturionAbstractModelPyTest(
def get(*args, **kwargs):
return model_instance
model_instance.objects = MockManager()
mocker.patch('access.models.tenancy_abstract.TenancyAbstractModel.objects', new_callable=MockManager)
model_instance._audit_enabled = True
@ -360,7 +360,7 @@ class CenturionAbstractModelPyTest(
def get(*args, **kwargs):
return model_instance
model_instance.objects = MockManager()
mocker.patch('access.models.tenancy_abstract.TenancyAbstractModel.objects', new_callable=MockManager)
model_instance._is_submodel = True
@ -387,7 +387,7 @@ class CenturionAbstractModelPyTest(
def get(*args, **kwargs):
return model_instance
model_instance.objects = MockManager()
mocker.patch('access.models.tenancy_abstract.TenancyAbstractModel.objects', new_callable=MockManager)
model_instance._is_submodel = False
@ -417,7 +417,7 @@ class CenturionAbstractModelPyTest(
def get(*args, **kwargs):
return model_instance
model_instance.objects = MockManager()
mocker.patch('access.models.tenancy_abstract.TenancyAbstractModel.objects', new_callable=MockManager)
super_delete = mocker.patch('django.db.models.base.Model.delete', return_value = None)
@ -836,7 +836,7 @@ class CenturionAbstractModelPyTest(
def get(self, *args, **kwargs):
return self.MockObj()
model_instance.objects = MockManager()
mocker.patch('access.models.tenancy_abstract.TenancyAbstractModel.objects', new_callable=MockManager)
model_instance.id = 1
@ -879,7 +879,7 @@ class CenturionAbstractModelPyTest(
def get(self, *args, **kwargs):
return self.MockObj()
model_instance.objects = MockManager()
mocker.patch('access.models.tenancy_abstract.TenancyAbstractModel.objects', new_callable=MockManager)
model_instance.id = 1