Django 2.0中post_save信号添加多对多关系未持久化问题

我来帮你分析一下这个问题：你已经在post_save信号里正确获取了要添加的Member，并且内存中能看到关联，但数据库里没有，核心原因可能有这几个：

1. 多余的instance.save()调用是不必要的（甚至可能干扰） #

对于Django的ManyToMany字段，调用add()方法本身就会直接向中间表插入关联数据，不需要再调用instance.save()——save()方法只负责保存模型本身的字段（比如Event的name、is_opt_out这些），和ManyToMany的关联关系无关。你额外调用save()不仅没用，还可能在某些场景下触发不必要的信号循环，或者覆盖一些未预期的状态。

先把最后一行的instance.save()删掉试试。

2. 事务提交时机的问题 #

如果你的Event创建操作是在事务原子块（比如用了@transaction.atomic装饰器）中执行的，那么post_save信号会在事务提交之前触发。这时候你在信号里添加的ManyToMany关联，会和Event的创建处于同一个事务中——如果事务还没提交（比如视图还没执行完），你直接去数据库查询是看不到这些关联的。

解决这个问题的方法是，把添加关联的逻辑放到事务提交后执行，用Django的transaction.on_commit()方法：

修改你的信号函数：

from django.db import transaction

@receiver(signal=post_save, sender='events.Event')
def auto_opt_in(sender, instance, created, **kwargs):
    if created and instance.is_opt_out:
        from database.models import Member
        
        def add_participants():
            members = Member.objects.none()
            for status in instance.opt_out_member_statuses:
                members = members.union(Member.objects.filter(member_status=status))
            instance.participants.add(*members)
        
        # 确保在事务提交后执行关联添加
        transaction.on_commit(add_participants)

这样就能保证只有当Event的创建事务成功提交后，才会去添加关联，这时候数据库里就能看到数据了。

3. 简化查询集的写法（优化建议） #

虽然你说打印能看到正确结果，但可以把循环union的逻辑简化成更高效的__in查询，避免多次union可能带来的潜在问题：

# 替代循环union的写法，更简洁高效
members = Member.objects.filter(member_status__in=instance.opt_out_member_statuses)

总结一下，优先去掉多余的instance.save()，然后用transaction.on_commit()确保操作在事务提交后执行，同时简化查询集的写法，应该就能解决你的问题了。

内容的提问来源于stack exchange，提问作者Mathis