当 commit=False 时，Django 表单 save() 方法中是否需要 save_m2m() ？

Question

文档似乎非常坚定地表明情况确实如此......

https://docs.djangoproject.com/en/dev/topics/forms/modelforms/#the-save-method

我特别参考了本节：

当您的模型与另一个模型具有多对多关系时，会出现使用 commit=False 的另一个副作用。如果您的模型具有多对多关系，并且您在保存表单时指定 commit=False，则 Django 无法立即保存多对多关系的表单数据。这是因为在实例存在于数据库中之前，无法保存该实例的多对多数据。

为了解决这个问题，每次使用 commit=False 保存表单时，Django 都会向 ModelForm 子类添加一个 save_m2m() 方法。手动保存表单生成的实例后，您可以调用 save_m2m() 来保存多对多表单数据。

我对 django 很陌生，昨天偶然发现了这个信息。

然而，我有一个观点，我不调用 save_m2m() 方法，但它实际上保存了 m2m 数据。

这是我的观点：

class SubscriberCreateView(AuthCreateView):
    model = Subscriber
    template_name = "forms/app.html"
    form_class = SubscriberForm
    success_url = "/app/subscribers/"

    def get_form_kwargs(self):
        kwargs = super(SubscriberCreateView, self).get_form_kwargs()
        kwargs.update({'user': self.request.user})
        return kwargs

    def form_valid(self, form):
        self.object = form.save(commit=False)
        self.object.user = self.request.user
        try:
            self.object.full_clean()
        except ValidationError:
            form._errors["email"] = ErrorList([u"This subscriber email is already in your account."])
            return super(SubscriberCreateView, self).form_invalid(form)
        return super(SubscriberCreateView, self).form_valid(form)

我的模型：

class Subscriber(models.Model):

    STATUS_CHOICES = (
        (1, ('Subscribed')),
        (2, ('Unsubscribed')),
        (3, ('Marked as Spam')),
        (4, ('Bounced')),
        (5, ('Blocked')),
        (6, ('Disabled')),
    )

    user = models.ForeignKey(User)
    status = models.IntegerField(('status'), choices=STATUS_CHOICES, default=1)
    email = models.EmailField()
    subscriber_list = models.ManyToManyField('SubscriberList')
    first_name = models.CharField(max_length=70, blank=True)
    last_name = models.CharField(max_length=70, blank=True)
    phone = models.CharField(max_length=20, blank=True)
    facebook_id = models.CharField(max_length=40, blank=True)
    twitter_id = models.CharField(max_length=40, blank=True)
    address1 = models.CharField(max_length=100, blank=True)
    address2 = models.CharField(max_length=100, blank=True)
    postcode = models.CharField(max_length=10, blank=True)
    city = models.CharField(max_length=30, blank=True)
    country = models.CharField(max_length=30, blank=True)
    date_joined = models.DateTimeField(auto_now_add=True)
    date_updated = models.DateTimeField(auto_now=True)

    class Meta:
        unique_together = (
            ('user', 'email',),
        )
    
    def __unicode__(self):
        return self.email

我的形式：

class SubscriberForm(ModelForm):
    def __init__(self, user, *args, **kwargs):
        super (SubscriberForm, self).__init__(*args, **kwargs)
        self.fields['subscriber_list'].queryset = SubscriberList.objects.filter(user=user)
    
    class Meta:
        model = Subscriber
        exclude = ('user', 'facebook_id', 'twitter_id')

那么为什么我的观点有效？ （这意味着，表单中某一字段的 m2m 关系实际上是在处理表单时保存的。）

Answer 1

父类之一正在执行模型对象及其 m2m 关系的完整保存。我不能确定，因为我没有 AuthCreateView 的声明，但命名约定表明它源于“CreateView”。如果是这样，你的视图的继承就像这样，SubscriberCreateView -> AuthCreateView->创建视图-> BaseCreateView-> ModelFormMixin。 ModelFormMixin 有一个 form_valid() 方法，您（可能）使用 super() 调用该方法。

这是 Django 1.4 的完整 方法 :

def form_valid(self, form):
    self.object = form.save()
    return super(ModelFormMixin, self).form_valid(form)

这样你就明白了。然而，让我指出一些潜在的困惑。 @Wogan 很聪明地指出你没有保存你的对象。按照您的代码的方式，您使用未保存的模型实例进行验证，然后它被丢弃，因为 ModelFormMixin 重新分配了 self.object。

1. 这意味着如果您稍后访问它，self.object 可能不是您所期望的。
2. 您会丢失 self.object.user 信息。 （因为 user 是模型上的必填字段，并且您在表单中将其排除，所以我预计 save() 会失败。因此父级 AuthCreateView< /code> 必须在做某事。当然，它可能正在处理整个 save() 并且根本不会命中 ModelFormMixin。）

为了避免这种混乱，只需不要这样做将您的实例分配给self.object。也许： validate_obj = form.save(commit=False)

Answer 2

我注意到您实际上并没有保存通过 form.save(commit=False) 调用获得的对象。所以在我看来，您的数据实际上保存在其他地方 - 可能在 AuthCreateView 的 form_valid 方法（或另一个祖先类）中。这可以解释为什么多对多对象被正确保存。

Answer 3

如果我们知道如何使用 save(commit=False)，这就是一个简单的答案。

带有 commit=False 的 save 方法不会更改您的数据库：

“如果您使用 commit=False 调用 save()，那么它将返回一个尚未被保存的对象。在这种情况下，您可以对生成的模型实例调用 save()，如果您想在保存对象之前对其进行自定义处理，或者您想使用其中之一，则这很有用。专门的模型保存选项。”

因此，在多对多关系的情况下，如果不将对象保存到数据库，就不可能保存 m2m 数据。通常的 save( commit=False ) 可以在保存部分数据（不是分配给 m2m 关系的数据）时更改对象。你确实不能仅将 M2M 关系存储在内存中。

如果您想在 save(commit=False) 后使用模型对象的 m2m 数据，则需要 save_m2m() 。您应该在使用 save(commit=False) 时执行此操作，否则您可能会在 save(commit=False) 之后得到一个与您的数据库（或数据库模型）不正确对应的中间对象。有时这可能是正常的（如果你不接触模型 m2m 部分处理中涉及的数据）。要恢复一致性，请在每次调用 save(commit=False) 时调用 save_m2m。

查看 save_m2m 实现：

def save_m2m():
    cleaned_data = form.cleaned_data
    for f in opts.many_to_many:
        if fields and f.name not in fields:
            continue
        if f.name in cleaned_data:
            f.save_form_data(instance, cleaned_data[f.name])