在 Django 项目中保存 signals.py 文件的正确位置

根据我读过的 Django 的文档，似乎应用程序文件夹中的 signals.py是一个很好的起点，但我面临的问题是，当我为 pre_save创建信号并试图从模型中导入类时，它与我的模型中的 import冲突。

# models.py


from django.contrib.auth.models import User
from django.db import models
from django.utils.translation import gettext as _
from signals import *


class Comm_Queue(CommunicatorAbstract):
queue_statuses = (
('P', _('Pending')),
('S', _('Sent')),
('E', _('Error')),
('R', _('Rejected')),
)
status          = models.CharField(max_length=10, db_index=True, default='P')
is_html         = models.BooleanField(default=False)
language        = models.CharField(max_length=6, choices=settings.LANGUAGES)
sender_email    = models.EmailField()
recipient_email = models.EmailField()
subject         = models.CharField(max_length=100)
content         = models.TextField()

# signals.py


from django.conf import settings
from django.db.models.signals import pre_save
from django.dispatch import receiver
from models import Comm_Queue


@receiver(pre_save, sender=Comm_Queue)
def get_sender_email_from_settings(sender, **kwargs):
obj=kwargs['instance']
if not obj.sender_email:
obj.sender_email='%s' % settings.ADMINS[0][1]

这段代码不会运行，因为我在 signals.py中导入了 Comm_Queue，并且在 models.py中也导入了信号。

有没有人能给我一些建议，告诉我怎样才能克服这个问题？

问候

37669 次浏览
小开

I'm guessing that you're doing that so your signals are registered, so that they're found somewhere. I just put my signals right in a models.py file normally.

小开

I also put signals in signals.py file and also have this code snippet that loads all signals:

# import this in url.py file !


import logging


from importlib import import_module


from django.conf import settings


logger = logging.getLogger(__name__)


signal_modules = {}


for app in settings.INSTALLED_APPS:
signals_module = '%s.signals' % app
try:
logger.debug('loading "%s" ..' % signals_module)
signal_modules[app] = import_module(signals_module)
except ImportError as e:
logger.warning(
'failed to import "%s", reason: %s' % (signals_module, str(e)))

This is for project, I'm not sure if it works at app level.

小开

An alternative is to import the callback functions from signals.py and connect them in models.py:

signals.py

def pre_save_callback_function(sender, instance, **kwargs):
# Do stuff here

model.py

# Your imports here
from django.db.models.signals import pre_save
from yourapp.signals import pre_save_callback_function


class YourModel:
# Model stuff here
pre_save.connect(pre_save_callback_function, sender=YourModel)

Ps: Importing YourModel in signals.py will create a recursion; use sender, instead.

Ps2: Saving the instance again in the callback function will create a recursion. You can make a control argument in .save method to control it.

小开

To solve your problem you just have to import signals.py after your model definition. That's all.

小开
最佳答案

Original answer, for Django < 1.7:

You can register the signals by importing signals.py in the app's __init__.py file:

# __init__.py
import signals

This will allow to import models.py from signals.py without circular import errors.

One problem with this approach is that it messes up the coverage results if you're using coverage.py.

Related discussion

Edit: For Django >= 1.7:

Since AppConfig was introduced, the recommended way of importing signals is in its init() function. See Eric Marcos' answer for more details.

小开

If you're using Django<=1.6 I'd recommend Kamagatos solution: just import your signals at the end of your models module.

For future versions of Django (>=1.7), the recommended way is to import your signals module in your app's config ready() function:

my_app/apps.py

from django.apps import AppConfig


class MyAppConfig(AppConfig):
name = 'my_app'


def ready(self):
import my_app.signals

my_app/__init__.py

default_app_config = 'my_app.apps.MyAppConfig'
小开

In old Django versions would be fine to put the signals on the __init__.py or maybe in the models.py(although at the end models will be way to large for my taste).

With Django 1.9, it is better I think, to place the signals on a signals.py file and import them with the apps.py, where they are going to be loaded after loading the model.

apps.py:

from django.apps import AppConfig




class PollsConfig(AppConfig):
name = 'polls'


def ready(self):
from . import signals  # NOQA

You can also divide your signals on signals.py and handlers.py in another folder within your model named signals as well, but for me that is just over engineering. Take a look at Placing Signals

小开

This only applies if you have your signals in a separate signals.py file

In completely agree with the answer of @EricMarcos but it should be stated that the django docs explicitly advice not to use the default_app_config variable (although it is not wrong). For current versions, correct way would be:

my_app/apps.py

from django.apps import AppConfig


class MyAppConfig(AppConfig):
name = 'my_app'


def ready(self):
import my_app.signals

settings.py

(Make sure you don't just have your app name in installed apps but instead the relative path to your AppConfig)

INSTALLED_APPS = [
'my_app.apps.MyAppConfig',
# ...
]

提交答案