Django 多对多关系不保存

Update:

对于任何好奇的人，我弄清楚了它是什么、为什么以及如何解决它。 在我看来，我有：fields = ['html', 'tags', 'title', 'text', 'taken_date', 'image']我在我的模板中使用 {{ form.as_p }} 。显然，一旦从表单中发布，它真的非常不希望任何其他内容触及表单中尚未存在的表单字段。 所以我从我的视图中取出了“标签”字段，它起作用了。

感谢所有回复的人。

原始问题:

使用 Django 2.0.1 和 PostgreSQL 9.2.18

我正在编写一个简单的照片库应用程序。其中我有一个照片对象和 PhotoTag 对象。照片可以有很多标签，标签可以与很多照片相关联，因此它需要是一个ManyToManyField。

保存提交的照片后，post_save 接收器调用函数来制作缩略图（工作正常）和更新标签的函数。

照片保存得很好，update_tags 被调用很好，标签从照片中读取很好，标签保存到 PhotoTag 中很好。但是将两者绑定在一起的多对多表不会插入新行。除非代码在 update_tags 函数或 post_save 接收函数期间异常退出，否则在 update_tags 后调用thumbs。

我什至尝试使用 connection.cursor 直接写入 m2m 表，它具有相同的行为。

如果我尝试再次对 Photo 对象调用 save() ，由于 post_save 信号，我就会进入无限循环。

我对发生的事情感到困惑。有什么线索吗？

# models.py

def update_tags(instance):
    tags = get_tags(instance.image)

    # Set initial values
    pt = []
    tagid = ''
    photoid = instance.id

    # Loop through tag list and insert into PhotoTag and m2m relation
    for x in range(0, len(tags)):
        # Make sure this tag doesn't already exist
        if PhotoTag.objects.filter(tag_text=tags[x]).count() == 0:
            pt = PhotoTag.objects.create(tag_text=tags[x])
            tagid = PhotoTag.objects.latest('id').id
            instance.tags.add(pt)
        else:
            # Only working with new tags right now
            pass

    return


class Photo(models.Model):
    author = models.ForeignKey(settings.AUTH_USER_MODEL,
                               on_delete=models.CASCADE)
    title = models.CharField(max_length=200, null=True, blank=True)
    text = models.TextField(null=True, blank=True)
    html = models.BooleanField(default=False)
    filename = models.CharField(default='', max_length=100, blank=True,
                                null=True)
    image = models.ImageField(upload_to=upload_path)
    location = models.CharField(max_length=100, blank=True, null=True)
    entry_date = models.DateTimeField(default=timezone.now)
    taken_date = models.DateTimeField(blank=True, null=True)

    tags = models.ManyToManyField(PhotoTag, blank=True)


@receiver(post_save, sender=Photo)
def thumbs(sender, instance, **kwargs):
    """
    Upon photo save, create thumbnails and then
    update PhotoTag and m2m with any Exif/XMP tags
    in the photo.
    """

    mk_thumb(instance.image, 'mid')
    mk_thumb(instance.image, 'th')
    mk_thumb(instance.image, 'sm')

    update_tags(instance)

    return

-------------
From views.py
-------------

class PhotoCreate(LoginRequiredMixin, CreateView):
    model = Photo
    template_name = 'photogallery/photo_edit.html'
    fields = ['html', 'tags', 'title', 'text', 'taken_date', 'image']

    def get_initial(self):
        self.initial = {'entry_date': timezone.now()}
        return self.initial

    def form_valid(self, form):
        form.instance.author = self.request.user

        return super(PhotoCreate, self).form_valid(form)

Update:

def save(self, mkthumb='', *args, **kwargs):
      super(Photo, self).save(*args, **kwargs)
      if mkthumb != "thumbs":
          self.mk_thumb(self.image, 'mid')
          self.mk_thumb(self.image, 'th')
          self.mk_thumb(self.image, 'sm')

          self.update_tags()

          mkthumb = "thumbs"

      return

I had a 类似问题 https://stackoverflow.com/a/53608043/2996101保存用户实例时我试图添加一个组。

为什么会发生这种情况的答案在于docs https://docs.djangoproject.com/en/dev/topics/forms/modelforms/#the-save-method更明确地（使用代码）位于这张票 https://code.djangoproject.com/ticket/30022#comment:4.

保存时ModelForm()（在管理中点击“保存”），首先保存对象的实例，然后触发其所有信号等。第三步是使用保存所有 m2m 关系ModelForm().cleaned_data. If ModelForm().cleaned_data['tags'] is None，根据您的信号创建的所有关系都将被删除。

• 一个黑客解决方案是使用post_save信号与transaction.on_commit() https://docs.djangoproject.com/en/dev/topics/db/transactions/#django.db.transaction.on_commit它将在现有事务（包括保存所有M2M关系的过程）提交到数据库后执行相关代码。

  def on_transaction_commit(func):
      ''' Create the decorator '''
      def inner(*args, **kwargs):
          transaction.on_commit(lambda: func(*args, **kwargs))
  
      return inner
  
  
  @receiver(post_save, sender=Photo)
  @on_transaction_commit
  def tags(instance, raw, **kwargs):
      """
      Create the relevant tags after the transaction 
      of instance is committed, unless the database is 
      populated with fixtures. 
      """
      if not raw:
          update_tags(instance)
  

• 如果您的多对多关系还没有，更健全的解决方案blank=True是使用一个m2m_changed() https://docs.djangoproject.com/en/dev/ref/signals/#m2m-changed信号，如所解释的在这篇文章中 https://stackoverflow.com/questions/23795811/django-accessing-manytomany-fields-from-post-save-signal或前面提到的ticket https://code.djangoproject.com/ticket/30022#comment:4.

• 最棒的是，放弃信号 https://lincolnloop.com/blog/django-anti-patterns-signals/并覆盖ModelForm().clean()方法用于以下情况：ModelForm()被使用，并且还覆盖Model().save()方法如果直接保存模型。

  A ModelForm().instance.my_flag将会很有用，因此您可以检查现有的Model().my_flag in Model().save()以避免访问两次数据库。