From 519b5ab77c02ea3709b4196d45b12d44bcff6eb1 Mon Sep 17 00:00:00 2001 From: Michael DeHaan Date: Sat, 23 Mar 2013 17:07:24 -0400 Subject: [PATCH] Add part of list to show audit trail by organization, still need to hook create to auto-establish the audit trail records. --- lib/main/models/__init__.py | 2 +- lib/main/tests.py | 5 +++-- lib/main/views.py | 2 +- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/lib/main/models/__init__.py b/lib/main/models/__init__.py index c767c0047d..bdc02ece21 100644 --- a/lib/main/models/__init__.py +++ b/lib/main/models/__init__.py @@ -90,7 +90,7 @@ class Tag(models.Model): return True -class AuditTrail(CommonModel): +class AuditTrail(models.Model): ''' changing any object records the change ''' diff --git a/lib/main/tests.py b/lib/main/tests.py index afa1768db3..85d99fc7ba 100644 --- a/lib/main/tests.py +++ b/lib/main/tests.py @@ -269,8 +269,9 @@ class OrganizationsTest(BaseTest): org1_tags = self.get(org1_tags_url, expect=403, auth=self.get_other_credentials()) def test_get_item_subobjects_audit_trail(self): - # FIXME - pass + url = '/api/v1/organizations/2/audit_trail/' + self.get(url, expect=200, auth=self.get_normal_credentials()) + # FIXME: verify that some audit trail records are auto-created on save AND post def test_post_item(self): diff --git a/lib/main/views.py b/lib/main/views.py index 9948d966e7..936c2ad5aa 100644 --- a/lib/main/views.py +++ b/lib/main/views.py @@ -57,7 +57,7 @@ class OrganizationsAuditTrailList(BaseSubList): if not (self.request.user.is_superuser or self.request.user in organization.admins.all()): # FIXME: use: organization.can_user_administrate(self.request.user) raise PermissionDenied() - return AuditTrail.objects.filter(audit_trail_by_tag__in = [ organization ]) + return AuditTrail.objects.filter(organization_by_audit_trail__in = [ organization ]) class OrganizationsUsersList(BaseSubList):