如何取消 django 信号中的删除

Question

有没有办法使用 django pre_delete 信号取消记录删除？

示例：

def on_delete(sender,**kwargs):
  if not <some condition>:
    #cancel the deletion
 # else continue with the deletion
pre_delete.connect(on_delete,sender=MyModel)

另一个问题是有没有办法对模型说“在更改文件之前先删除原始文件”，因为现在这就是我所做的（参见下面的代码）并且我不确定这是否是最好的方法。

def on_save(sender,**kwargs):
  obj = kwargs['instance']
  try:
    id = obj.pk
    # find the file
    original_file = sender.objects.get(pk=id)
    # delete the original file before uploading a new file
    original_file.file.delete()
  except ....

pre_save.connect(on_save,sender=ModelWithFileUpload)

（在 django 1.2 中，他们在更改或删除时自动删除文件，但在 django 1.3 中，他们删除了此功能）

提前致谢

Answer 1

使用内置 Django 信号时这是不可能的。信号上的“send()”和“send_robust()”方法返回一个二元组列表——（接收者，响应）。因此，如果您有正确的代码来处理来自每个接收器的响应，则您可能可以根据一个信号处理程序的返回值来阻止某些操作。

contrib.comments 应用程序执行此操作，允许任何返回 False 的接收器“取消”信号操作。请参阅 第 111-120 行：

但是， 核心 Django 代码 没有任何这种特殊处理。所有这些信号的作用就是通知接收者发生了一些事情。

Answer 2

我会尝试一些黑客解决方法：

def on_delete(sender,**kwargs):
  if not <some condition>:
    raise Exception('Do not delete')#cancel the deletion
 # else continue with the deletion
pre_delete.connect(on_delete,sender=MyModel)

并且视图

def on_save(sender,**kwargs):
  obj = kwargs['instance']
  try:
    id = obj.pk
    # find the file
    original_file = sender.objects.get(pk=id)
    # delete the original file before uploading a new file
  except ... :
    # oder exceptions 

  try:
    original_file.file.delete()
  except:
    pass #not deleted

pre_save.connect(on_save,sender=ModelWithFileUpload)

在信号中引发异常应该制动delete()方法执行，同时将异常返回到调用它的地方。您可以创建自己的 Exception 子类来仅排除某些类型的异常（您几乎永远不应该使用不带参数的 except）。

Answer 3

我知道我的回答有点晚了，但这个问题的第二部分正是我几天前所需要的。

所以，首先要做的事情是：

1. 有没有办法使用 django pre_delete 信号取消记录删除？

不是真的，除了 dk 提出的那个。老实说，不应该有任何。为什么？因为 pre_delete 意味着在删除对象之前发生的操作。如果你阻止删除，它就不再是pre_delete（注意到恶性循环了吗？）

2. 有没有办法告诉模型在更改文件之前先删除原始文件？

是的，有，而且你说得非常正确。我创建了一个更通用的代码，它将适用于任何具有关联的 File 对象的模型（见下文）。但是，您应该正手 阅读为什么此行为在 Django 1.3 中被删除，并查看它是否以任何方式影响您的逻辑。它主要与您如何处理回滚以及不同模型对同一文件的多次引用有关。

def delete_files_from_instance(instance, field_names):
    for field_name in field_names:
        field_value = getattr(instance, field_name, None)
        if field_value:
            if isinstance(field_value, File):
                try:
                    os.remove(field_value.path)
                except OSError:
                    pass


@receiver(pre_delete)
def on_delete(sender, instance, **kwargs):
    # When an object is deleted, all associated files are also removed
    delete_files_from_instance(instance, sender._meta.get_all_field_names())


@receiver(pre_save)
def on_update(sender, instance, **kwargs):
    # When an object is updated, if any media files are replaced, the old ones should be deleted.
    from_fixture = 'raw' in kwargs and kwargs['raw'] # this prevents errors when loading files from fixtures
    is_valid_app = sender._meta.app_label in VALID_APPS # Define what apps are targeted by your code
    if is_valid_app and not from_fixture:
        try:
            old_instance = sender.objects.filter(pk=instance.id).first()
            if old_instance and old_instance is not None:
                delete_files_from_instance(old_instance, sender._meta.get_all_field_names())
        except LookupError:
            pass

请记住，这假设您的删除/更新操作将会成功。如果失败，您将永久丢失文件。

更好的方法是在 post_save/post_delete 信号中处理文件删除，或者创建一个 cron 作业来定期清理数据库中不再引用的所有文件。