Comment #7 on issue 25 by carl.j.meyer: importing notification broken http://code.google.com/p/django-notification/issues/detail?id=25
I'm running into this same problem. I understand the rationale for using .get_app(), but it is not safe to use at module level in models.py. Importing it from any other models.py will then always cause a circular import, because models.get_app() requires the app cache to be fully populated, which requires loading the other models.py file that originally tried to load django-notification's models... The proper solution is to check directly in settings.INSTALLED_APPS. This is how this same bug was fixed in django-email-confirmation recently: http://code.google.com/p/django-email-confirmation/source/detail?r=40 Patch attached. Attachments: issue25.diff 599 bytes -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "pinax-updates" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/pinax-updates?hl=en -~----------~----~----~----~------~----~------~--~---
