为什么 django 的 model.save() 不调用 full_clean()？

Question

我只是好奇是否有人知道 django 的 orm 是否有充分的理由不对模型调用“full_clean”，除非它被保存为模型表单的一部分。

请注意，当您调用模型的 save() 方法时，不会自动调用 full_clean()。当您想为自己手动创建的模型运行一步模型验证时，您需要手动调用它。 django's full clean doc

（注意：为 Django 1.6 更新了引用...以前的 django 文档也对 ModelForms 提出了警告。）

人们是否有充分的理由不想要这种行为？我认为如果您花时间向模型添加验证，您会希望每次保存模型时都运行验证。

我知道如何让一切正常工作，我只是在寻找解释。

Answer 1

如果您想始终确保模型验证，全局pre_save 信号可以很好地工作。但是，它会在当前版本（3.1.x）中遇到 Django 的身份验证问题，并可能导致您正在使用的其他应用程序的模型出现问题。

详细说明@Peter Shannon 的答案，此版本将仅验证您执行它的模块内的模型，跳过使用"raw" saves 的验证并将dispatch_uid 添加到avoid duplicate signals。

from django.db.models.signals import pre_save
import inspect
import sys

MODELS = [obj for name, obj in
    inspect.getmembers(sys.modules[__name__], inspect.isclass)]

def validate_model(sender, instance, **kwargs):
    if 'raw' in kwargs and not kwargs['raw']:
        if type(instance) in MODELS:
            instance.full_clean()

pre_save.connect(validate_model, dispatch_uid='validate_models')

Answer 2

调用full_clean 方法的最简单方法就是在你的模型中重写save 方法：

class YourModel(models.Model):
    ...  
    
    def save(self, *args, **kwargs):
        self.full_clean()
        return super(YourModel, self).save(*args, **kwargs)

Answer 3

评论@Alfred Huang 的回答和评论。可以通过在当前模块 (models.py) 中定义一个类列表并在 pre_save 钩子中检查它来将 pre_save 钩子锁定到应用程序：

CUSTOM_CLASSES = [obj for name, obj in
        inspect.getmembers(sys.modules[__name__])
        if inspect.isclass(obj)]

@receiver(pre_save)
def pre_save_handler(sender, instance, **kwargs):
    if type(instance) in CUSTOM_CLASSES:
        instance.full_clean()

Answer 4

我们可以将应用程序用作settings.py 中的INSTALLED_APPS 部分，而不是插入一段声明接收器的代码

INSTALLED_APPS = [
    # ...
    'django_fullclean',
    # your apps here,
]

在此之前，您可能需要使用 PyPI 安装 django-fullclean：

pip install django-fullclean

Answer 5

如果您有一个模型要确保至少有一个 FK 关系，并且您不想使用 null=False，因为这需要设置默认 FK（这将是垃圾数据），我最好的方法是想出的是添加自定义.clean() 和.save() 方法。 .clean() 引发验证错误，.save() 调用 clean。这样，表单和其他调用代码、命令行和测试都可以强制执行完整性。没有这个，（AFAICT）就无法编写测试来确保模型与特定选择（非默认）的其他模型具有 FK 关系。

class Payer(models.Model):

    name = models.CharField(blank=True, max_length=100)
    # Nullable, but will enforce FK in clean/save:
    payer_group = models.ForeignKey(PayerGroup, null=True, blank=True,)

    def clean(self):
        # Ensure every Payer is in a PayerGroup (but only via forms)
        if not self.payer_group:
            raise ValidationError(
                {'payer_group': 'Each Payer must belong to a PayerGroup.'})

    def save(self, *args, **kwargs):
        self.full_clean()
        return super().save(*args, **kwargs)

    def __str__(self):
        return self.name

Answer 6

出于兼容性考虑，django内核中没有启用保存时自动清理。

如果我们正在开始一个新项目，并且希望模型上的默认 save 方法可以自动清理，我们可以在保存每个模型之前使用以下信号进行清理。

from django.dispatch import receiver
from django.db.models.signals import pre_save, post_save

@receiver(pre_save)
def pre_save_handler(sender, instance, *args, **kwargs):
    instance.full_clean()

Answer 7

AFAIK，这是因为向后兼容。带有排除字段的 ModelForms、带有默认值的模型、pre_save() 信号等也存在问题。

您可能感兴趣的来源：

• http://code.djangoproject.com/ticket/13100
• http://groups.google.com/group/django-developers/browse_frm/thread/b888734b05878f87