Merge pull request #11 from grahamu/fix8

Moving post_save handler to receivers.py

pinax/documents/apps.py

@@ -1,3 +1,5 @@
+import importlib
+
 from django.apps import AppConfig as BaseAppConfig
 from django.utils.translation import ugettext_lazy as _
 
@@ -6,3 +8,6 @@ class AppConfig(BaseAppConfig):
 
     name = "pinax.documents"
     verbose_name = _("Pinax Documents")
+
+    def ready(self):
+        importlib.import_module("pinax.documents.receivers")

pinax/documents/models.py

@@ -5,8 +5,6 @@
 
 from django.core.urlresolvers import reverse
 from django.db import models
-from django.db.models.signals import post_save
-from django.dispatch import receiver
 from django.utils import timezone
 
 from django.contrib.auth import get_user_model
@@ -314,10 +312,3 @@ def color(self):
         if p >= 90 and p <= 100:
             return "danger"
         raise ValueError("percentage out of range")
-
-
-@receiver(post_save, sender=settings.AUTH_USER_MODEL)
-def ensure_userstorage(sender, **kwargs):
-    if kwargs["created"]:
-        user = kwargs["instance"]
-        UserStorage.objects.create(user=user, bytes_total=(1024 * 1024 * 50))

pinax/documents/receivers.py

@@ -0,0 +1,12 @@
+from django.db.models.signals import post_save
+from django.dispatch import receiver
+
+from .conf import settings
+from .models import UserStorage
+
+
+@receiver(post_save, sender=settings.AUTH_USER_MODEL)
+def ensure_userstorage(sender, **kwargs):
+    if kwargs["created"]:
+        user = kwargs["instance"]
+        UserStorage.objects.create(user=user, bytes_total=(1024 * 1024 * 50))