Handle update events when previous state was None
This commit is contained in:
@@ -17,9 +17,12 @@ class AuditableMixin(object):
|
|||||||
request_id = resource.auditable_request_id()
|
request_id = resource.auditable_request_id()
|
||||||
resource_type = resource.auditable_resource_type()
|
resource_type = resource.auditable_resource_type()
|
||||||
display_name = resource.auditable_displayname()
|
display_name = resource.auditable_displayname()
|
||||||
changed_state = resource.auditable_changed_state()
|
|
||||||
event_details = resource.auditable_event_details()
|
event_details = resource.auditable_event_details()
|
||||||
|
|
||||||
|
changed_state = (
|
||||||
|
resource.auditable_changed_state() if action == ACTION_UPDATE else None
|
||||||
|
)
|
||||||
|
|
||||||
audit_event = AuditEvent(
|
audit_event = AuditEvent(
|
||||||
user_id=user_id,
|
user_id=user_id,
|
||||||
workspace_id=workspace_id,
|
workspace_id=workspace_id,
|
||||||
@@ -69,7 +72,9 @@ class AuditableMixin(object):
|
|||||||
for attr in attrs:
|
for attr in attrs:
|
||||||
history = getattr(inspect(self).attrs, attr.key).history
|
history = getattr(inspect(self).attrs, attr.key).history
|
||||||
if history.has_changes():
|
if history.has_changes():
|
||||||
previous_state[attr.key] = [history.deleted.pop(), history.added.pop()]
|
deleted = history.deleted.pop() if history.deleted else None
|
||||||
|
added = history.added.pop() if history.added else None
|
||||||
|
previous_state[attr.key] = [deleted, added]
|
||||||
return previous_state
|
return previous_state
|
||||||
|
|
||||||
def auditable_changed_state(self):
|
def auditable_changed_state(self):
|
||||||
|
Reference in New Issue
Block a user