python - 从 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 是叶子。因为我在上面布置的结构是一个 DAG 而不是一棵树，一个节点实际上可以有多个这些“完整”数字，因为从任何给定节点到它的根可以有多条路径。

如果此描述不成立，请告诉我，我可以在 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 直接查看数据库，我会看到同样的事情。

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

有人知道这是怎么回事吗？任何帮助是极大的赞赏。如果我可以澄清任何一个，请告诉我。

Answer 1

好吧，我调试了一整天在互联网上翻来覆去地想出来了。据我所知，问题似乎在于 Django 如何保存 ManyToMany 关系。正如在 https://stackoverflow.com/a/1925784/1269634 中讨论的那样:

When you save a model via admin forms it's not an atomic transaction. The main object gets saved first (to make sure it has a PK), then the M2M is cleared and the new values set to whatever came out of the form. So if you are in the save() of the main object you are in a window of opportunity where the M2M hasn't been updated yet. In fact, if you try to do something to the M2M, the change will get wiped out by the clear(). I ran into this about a year ago.

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

事实证明，此问题的修复相对简单。我有效地将 CapabilityRelationship.save() 重命名为 CapabilityRelationship.update_complete_numbers()。然后，我修改了 Capability 管理实现以覆盖 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) 返回时为真) ).

感谢大家的帮助!我希望这最终可以帮助其他人。