ChatGPT解决这个技术问题 Extra ChatGPT

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

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

请注意,当您调用模型的 save() 方法时,不会自动调用 full_clean()。当您想为自己手动创建的模型运行一步模型验证时,您需要手动调用它。 django 的完整文档

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

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

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

非常感谢您提出这个问题,它阻止了我更多时间用头撞墙。我创建了一个可以帮助其他人的 mixin。查看要点:gist.github.com/glarrain/5448253
最后我使用信号来捕获 pre_save 钩子并在所有捕获的模型上执行 full_clean

l
lqc

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

您可能感兴趣的来源:

http://code.djangoproject.com/ticket/13100

http://groups.google.com/group/django-developers/browse_frm/thread/b888734b05878f87


第二个参考资料中最有用的摘录(恕我直言):“开发一个“自动”验证选项,它既简单到实际有用,又足够强大以处理所有边缘情况——如果可能的话——远远超过可以在 1.2 的时间范围内完成。因此,目前,Django 没有任何这样的东西,并且不会在 1.2 中拥有它。如果你认为你可以让它在 1.3 上工作,你最好的选择是建立一个提案,至少包括一些示例代码,以及如何使其保持简单和健壮的说明。”
这现在已经很过时了,你知道现在还是这样吗?
A
Alfred Huang

由于考虑到兼容性,在 django 内核中未启用保存时自动清理。

如果我们正在开始一个新项目并希望 Model 上的默认 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()

为什么这比覆盖某些 BaseModel(所有其他人将继承自)上的 save 方法先调用 full_clean,然后调用 super() 更好(或更差)?
我发现这种方法有两个问题 1)如果 ModelForm 的 full_clean() 会被调用两次:通过表单和信号 2)如果表单排除了某些字段,它们仍然会被信号验证。
@mehmet 所以也许你可以在 pre_save_handler 中添加这些 if send == somemodel, then exclude some fields
对于那些正在使用或考虑使用这种方法的人:请记住,这种方法不受 Django 官方支持,并且在可预见的将来也不会得到支持(请参阅 Django 错误跟踪器中的此评论:code.djangoproject.com/ticket/29655#comment:3),所以你是如果您为所有模型启用验证,可能会偶然发现一些缺陷,例如身份验证停止工作 (code.djangoproject.com/ticket/29655)。您将不得不自己处理这些问题。但是,没有更好的方法 atm。
从 Django 2.2.3 开始,这会导致基本身份验证系统出现问题。您将获得一个 ValidationError: Session with this Session key already exists。为避免这种情况,您需要为 sender in list_of_model_classes 添加一个 if 语句,以防止信号覆盖 Django 的默认身份验证模型。根据您的选择定义 list_of_model_classes
M
M.Void

调用 full_clean 方法的最简单方法就是覆盖模型中的 save 方法:

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

为什么这比使用信号更好(或更差)?
我发现这种方法有两个问题 1)如果 ModelForm 的 full_clean() 会被调用两次:通过表单和保存 2)如果表单排除了某些字段,它们仍然会被保存验证。
可能更好地定义一个新的 clean_save() 方法,该方法调用 full_clean() 然后 save() 并在手动保存自定义模型时显式使用它?
F
Flimm

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

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

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

pip install django-fullclean

为什么您要pip install 一些包含 4 行代码的应用程序(检查 source code)而不是自己编写这些代码行?
我自己没有尝试过的另一个库:github.com/danielgatis/django-smart-save
P
Peter Shannon

评论@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()

s
shacker

如果您有一个要确保至少有一个 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

M
Matt Sanders

如果您希望始终确保模型验证,全局 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')