Changeset 41146ff for asadb/groups

Timestamp:

Dec 28, 2011, 5:57:26 AM (14 years ago)

Author:

Alex Dehnert <adehnert@…>

Branches:

master, space-access, stable, stage

Children:

d6e6622

Parents:

beaf731

git-author:

Alex Dehnert <adehnert@…> (12/28/11 05:57:26)

git-committer:

Alex Dehnert <adehnert@…> (12/28/11 05:57:26)

Message:

Register the models with reversion directly too

Instead of just using VersionAdmin? to force models to be versioned,
register them directly in models.py. This has the advantage of meaning
that anywhere we use the models they'll be versioned.

IIRC, I'd tried this before and it didn't work, because I was registering
models more than once, but I can't remember the details of when it broke, and
I'm not seeing any breakage now. This may need to get reverted, though. (Or,
more likely, patched further to allow registering one model multiple times.)

File:

• asadb/groups/models.py (modified) (6 diffs)

asadb/groups/models.py

@@ -1 +1 @@
 from django.db import models
 from django.contrib.auth.models import User
+import reversion
 
 import datetime
@@ -99 +100 @@
             ('recognize_group', 'Recognize groups'),
         )
+reversion.register(Group)
 
 
@@ -121 +123 @@
     treasurer_name = models.CharField(max_length=50)
     treasurer_kerberos = models.CharField(max_length=8)
+reversion.register(GroupStartup)
 
 
@@ -152 +155 @@
             ('view_note_all',       'View all notes', ),
         )
+reversion.register(GroupNote)
 
 
@@ -184 +188 @@
     def retrieve(cls, slug, ):
         return cls.objects.get(slug=slug)
+reversion.register(OfficerRole)
 
 
@@ -216 +221 @@
     def __repr__(self, ):
         return str(self)
+reversion.register(OfficeHolder)