从 model.save() 中删除关联的模型实例不起作用

Question

我遇到了一个难以解释的问题，请多多包涵。首先，以下是相关版本，以防万一这很重要：Django 2.0.3、Python 3.6.4、PostgreSQL 10.3。

基本上，我的一些 Django 模型具有这种结构：

class Capability(models.Model):
    relationships = models.ManyToManyField('self',
                                           symmetrical=False,
                                           through='CapabilityRelationship',
                                           blank=True)
    name = models.CharField(max_length=255)


class CapabilityRelationship(models.Model):
    from_capability = models.ForeignKey(Capability,
                                        on_delete=models.CASCADE,
                                        related_name='from_set',
                                        blank=True,
                                        null=True)
    to_capability = models.ForeignKey(Capability,
                                      on_delete=models.CASCADE,
                                      related_name='to_set')
    number = models.PositiveSmallIntegerField(validators=[MinValueValidator(1)])

    def _get_complete_numbers(self):
        # generate the complete numbers using depth-first search

    def save(self, *args, **kwargs):
        # Get the complete numbers before saving. If we're not able to generate the complete numbers, something is
        # wrong, an error will be raised, and we don't want to save.
        complete_numbers = self._get_complete_numbers()

        super().save(*args, **kwargs)  # Call the "real" save() method.

        # Delete all old complete numbers.
        self.capabilityrelationshipcompletenumber_set.all().delete()

        # Save the new complete numbers.
        for complete_number in complete_numbers:
            CapabilityRelationshipCompleteNumber.objects.create(capability_relationship=self,
                                                                complete_number=complete_number)


class CapabilityRelationshipCompleteNumber(models.Model):
    capability_relationship = models.ForeignKey(CapabilityRelationship, on_delete=models.CASCADE)
    complete_number = models.CharField(max_length=255, unique=True)

为了用文字描述这些模型，我有一个Capability 模型，它与自身具有多对多关系（在CapabilityRelationship 中捕获）。在实践中，这将是一个“树”，其中每个节点可以有多个子节点和多个父节点（即，它是一个有向无环图）。最后，每个关系实例可以有多个“完整的数字”（在CapabilityRelationshipCompleteNumber 中捕获）。

“数字”和“完整数字”背后的概念本质上是杜威十进制系统。完整数量的 1.2.3.4 将有 4 个级别的 Capability 对象，其中 1 是最顶层（即根节点），4 是叶。因为我在上面列出的结构是一个有向无环图而不是真正的树，所以一个节点实际上可以有多个这些“完整”数字，因为从任何给定节点到它的根节点都可以有多个路径。

如果此描述没有说明，请告诉我，我可以在 Paint 中模拟一些东西。

我将覆盖CapabilityRelationship.save() 方法，因为每次更改关系时我都需要重新计算完整的数字，因为number 可能已经更改。所以我要做的就是简单地计算新的完整数字，删除所有旧的完整数字，然后保存新的数字。

我遇到的问题是我根本无法删除旧的完整号码，这让我感到莫名其妙。我想知道是否有一些关于覆盖CapabilityRelationship.save() 的东西我根本没有得到。例如：

def save(self, *args, **kwargs):
    complete_numbers = self._get_complete_numbers()

    print('complete numbers: {}'.format(complete_numbers))

    super().save(*args, **kwargs)  # Call the "real" save() method.

    print('before delete: {}'.format(self.capabilityrelationshipcompletenumber_set.all()))
    self.capabilityrelationshipcompletenumber_set.all().delete()
    print('after delete: {}'.format(self.capabilityrelationshipcompletenumber_set.all()))

    for complete_number in complete_numbers:
        CapabilityRelationshipCompleteNumber.objects.create(capability_relationship=self,
                                                            complete_number=complete_number)

    print('after save: {}'.format(self.capabilityrelationshipcompletenumber_set.all()))

如果我访问管理站点并将叶节点的 number 设置为 1，保存它，然后将其修改为 2，我会得到以下输出：

complete numbers: {'1.2.1.1', '2.2.1.1', '1.3.1.1', '1.1.1.1'}
before delete: <QuerySet []>
after delete: <QuerySet []>
after save: <QuerySet [<CapabilityRelationshipCompleteNumber: 1.1.1.1>, <CapabilityRelationshipCompleteNumber: 1.2.1.1>, <CapabilityRelationshipCompleteNumber: 1.3.1.1>, <CapabilityRelationshipCompleteNumber: 2.2.1.1>]>
complete numbers: {'1.1.1.2', '1.2.1.2', '2.2.1.2', '1.3.1.2'}
before delete: <QuerySet [<CapabilityRelationshipCompleteNumber: 1.1.1.1>, <CapabilityRelationshipCompleteNumber: 1.2.1.1>, <CapabilityRelationshipCompleteNumber: 1.3.1.1>, <CapabilityRelationshipCompleteNumber: 2.2.1.1>]>
after delete: <QuerySet []>
after save: <QuerySet [<CapabilityRelationshipCompleteNumber: 1.1.1.2>, <CapabilityRelationshipCompleteNumber: 1.2.1.2>, <CapabilityRelationshipCompleteNumber: 1.3.1.2>, <CapabilityRelationshipCompleteNumber: 2.2.1.2>]>

现在，所有这些看起来都很棒，但是当我访问管理站点以获取所有完整数字的列表时，我看到了到目前为止计算的所有 8 个完整数字，而不是更改后当前正确的 4 个number 到 2。如果我打开一个 Python shell 并列出完整的数字，我会看到到目前为止创建的所有数字：

> ./manage.py shell
Python 3.6.4 (default, Dec 19 2017, 15:24:51)
[GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.39.2)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
(InteractiveConsole)
>>> from src.apps.api.models.capability import *
>>> CapabilityRelationshipCompleteNumber.objects.all()
<QuerySet [<CapabilityRelationshipCompleteNumber: 1.1.1.1>, <CapabilityRelationshipCompleteNumber: 1.1.1.2>, <CapabilityRelationshipCompleteNumber: 1.2.1.1>, <CapabilityRelationshipCompleteNumber: 1.2.1.2>, <CapabilityRelationshipCompleteNumber: 1.3.1.1>, <CapabilityRelationshipCompleteNumber: 1.3.1.2>, <CapabilityRelationshipCompleteNumber: 2.2.1.1>, <CapabilityRelationshipCompleteNumber: 2.2.1.2>]>

如果我使用 psql 直接查看数据库，我会看到同样的情况。

显然，无论出于何种原因，delete 调用实际上并没有发生。我尝试过CapabilityRelationshipCompleteNumber.objects.filter(capability_relationship=self).delete()、CapabilityRelationshipCompleteNumber.objects.all().delete()，并使用connection.cursor() 发出原始SQL DELETE FROM api_capabilityrelationshipcompletenumber;。似乎什么都不起作用。我不明白发生了什么事。我已阅读有关删除 QuerySet 和覆盖 save() 的 Django 文档，但没有看到任何可以帮助我诊断问题的内容。

有人知道这里发生了什么吗？任何帮助是极大的赞赏。请让我知道我是否可以澄清这些。

Answer 1

好的，经过调试和在互联网上搜索了一整天后，我想通了。据我所知，问题似乎在于 Django 如何保存多对多关系。正如https://stackoverflow.com/a/1925784/1269634 所讨论的：

当您通过管理表单保存模型时，它不是原子事务。首先保存主对象（以确保它具有 PK），然后 M2M 被清除 并将新值设置为表单中出现的任何值。因此，如果您在主对象的 save() 中，则您处于 M2M 尚未更新的机会窗口中。事实上，如果你尝试对 M2M 做点什么，更改将被 clear() 清除。大约一年前我遇到了这个问题。

虽然这篇文章最初可以追溯到 2009 年，但这种烦恼显然在今天的 Django 2.0 中仍然存在。读完后，我相信这就是造成奇怪的原因。不幸的是，据我所知，这在 Django 文档中的任何地方都没有记录，这就是为什么要花这么长时间来追踪和修复的原因。

事实证明，这个问题的修复相对简单。我有效地将CapabilityRelationship.save() 重命名为CapabilityRelationship.update_complete_numbers()。然后，我修改了Capability admin 实现以覆盖ModelAdmin.save_related()。管理实现现在看起来像这样：

@admin.register(Capability)
class CapabilityAdmin(nested_admin.NestedModelAdmin):
    list_display = ['name']
    search_fields = ['name']
    inlines = [CapabilityRelationshipInlineAdmin]

    def save_related(self, request, form, formsets, change):
        super().save_related(request, form, formsets, change)

        for capability_relationship in form.instance.to_set.all():
            capability_relationship.update_complete_numbers()

这行得通！神奇的是，模型保存完全完成后会更新完整的数字（super().save_related(request, form, formsets, change) 返回时是这样）。

感谢大家的帮助！我希望这最终可以对其他人有所帮助。