Django change_view中如何编辑和删除ProductImage关联的旧多张图片

问题原因 #

你当前的change_view存在两个核心问题：

1. POST请求处理时给ImageFormSet指定的queryset = ProductImage.objects.none()，表单集无法关联到当前商品已有的旧图片实例，只会将所有提交的内容识别为新增记录，不会触发旧数据的更新和删除逻辑
2. 没有开启表单集的删除功能，也没有处理图片文件的物理删除逻辑（Django默认只会删除数据库记录，不会删除存储路径里的实际图片文件）

具体修改方案 #

1. 调整change_view逻辑 #

修改views.py中的change_view代码，开启表单集删除权限、关联正确的查询集、优化保存逻辑：

def change_view(request, id):
    obj = get_object_or_404(Product, id = id)
    # 新增can_delete=True，开启表单的删除选项
    ImageFormSet = modelformset_factory(ProductImage, form = ProductImageForm, extra = 4, can_delete=True)
    form = ProductForm(instance = obj)
    if request.method == 'POST':
        form = ProductForm(request.POST, files = request.FILES, instance = obj)
        # POST时不要用none作为查询集，关联当前商品的已有图片
        formset = ImageFormSet(request.POST, files = request.FILES, queryset = ProductImage.objects.filter(product=obj))
        if form.is_valid() and formset.is_valid():
            obj = form.save()
            # 保存表单集，自动处理新增、更新、删除数据库记录
            instances = formset.save(commit=False)
            # 处理被勾选删除的图片，先物理删除文件
            for obj in formset.deleted_objects:
                if obj.image:
                    obj.image.delete(save=False)
                obj.delete()
            # 保存新增/更新的图片
            for instance in instances:
                instance.product = obj
                instance.save()
            return redirect('shop.detail', obj.id)
    else:
        formset = ImageFormSet(queryset = ProductImage.objects.filter(product=obj))
    context = {
        'form': form, 'formset': formset
    }
    return render(request, 'shop/products/create.html', context)

2. 可选：全量替换旧图片（不需要保留原有图片的场景） #

如果你的需求是只要提交新图片就删除所有旧图片，不需要用户手动勾选删除，可以在保存新图片前主动删除所有旧记录和文件：

# 放在保存新图片逻辑之前
old_images = ProductImage.objects.filter(product=obj)
for img in old_images:
    if img.image:
        img.image.delete(save=False)
old_images.delete()

3. 模板适配 #

不需要修改现有模板代码，{{ form.as_p }}会自动渲染每个图片表单自带的删除复选框，用户勾选后提交就会触发删除逻辑。

内容的提问来源于stack exchange，提问作者Dragon-LV