Skip to content

Commit 6484ac7

Browse files
izimobilarneb
authored andcommitted
Import Site model only if necessary (#133)
When not using the Django sites framework, you get a RuntimeError because it is not installed, even if you set DJANGO_MESSAGES_NOTIFY to False. This can be avoided by importing the site model only when necessary.
1 parent ddc037f commit 6484ac7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Diff for: django_messages/utils.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
import django
33
from django.utils.text import wrap
44
from django.utils.translation import ugettext, ugettext_lazy as _
5-
from django.contrib.sites.models import Site
65
from django.template.loader import render_to_string
76
from django.conf import settings
87

@@ -73,6 +72,7 @@ def new_message_email(sender, instance, signal,
7372

7473
if 'created' in kwargs and kwargs['created']:
7574
try:
75+
from django.contrib.sites.models import Site
7676
current_domain = Site.objects.get_current().domain
7777
subject = subject_prefix % {'subject': instance.subject}
7878
message = render_to_string(template_name, {

0 commit comments

Comments
 (0)