Django用户上传图片重命名问题及递归错误解决求助

问题核心 #

创建新Item实例时，因未生成数据库ID导致文件名含None；使用post_save信号重命名时触发递归循环（RecursionError）。以下是两种可行的优化方案：

方案1：通过post_save信号+自定义标记避免递归 #

步骤1：修改模型（可选，用自定义参数替代额外字段） #

重写save方法支持自定义参数，标记是否跳过信号处理：

from django.db import models
from django.utils.text import slugify

class Item(models.Model):
    name = models.CharField(max_length=100)
    image = models.ImageField(upload_to='temp_items/')  # 先存临时目录

    def save(self, *args, **kwargs):
        # 提取自定义参数，用于跳过信号递归
        self._skip_rename = kwargs.pop('_skip_rename', False)
        super().save(*args, **kwargs)

步骤2：编写信号处理函数 #

在信号中判断实例状态，完成重命名+缩放，并用自定义参数避免递归：

from django.db.models.signals import post_save
from django.dispatch import receiver
from django.conf import settings
from PIL import Image
import os

@receiver(post_save, sender=Item)
def process_item_image(sender, instance, created, **kwargs):
    # 跳过标记为"重命名中"的保存操作，避免递归
    if instance._skip_rename:
        return

    # 仅处理新创建且有图片的实例
    if created and instance.image:
        # 1. 处理图片缩放（示例：缩放到最大宽度800px）
        img_path = instance.image.path
        with Image.open(img_path) as img:
            max_width = 800
            if img.width > max_width:
                ratio = max_width / img.width
                new_height = int(img.height * ratio)
                img = img.resize((max_width, new_height), Image.Resampling.LANCZOS)
                img.save(img_path)

        # 2. 生成目标文件名（slugify处理名称避免特殊字符）
        ext = os.path.splitext(img_path)[1].lower()
        new_filename = f"{slugify(instance.name)}_{instance.id}{ext}"
        
        # 3. 移动并重命名文件到目标目录
        target_dir = os.path.join(settings.MEDIA_ROOT, 'items')
        os.makedirs(target_dir, exist_ok=True)
        new_path = os.path.join(target_dir, new_filename)
        os.rename(img_path, new_path)

        # 4. 更新数据库中图片路径，通过自定义参数跳过信号
        instance.image.name = f"items/{new_filename}"
        instance.save(update_fields=['image'], _skip_rename=True)

方案2：在视图中分两步保存（无需信号） #

直接在创建视图中先保存实例获取ID，再处理图片重命名：

from django.shortcuts import render, redirect
from .forms import ItemForm
from .models import Item
from django.conf import settings
from PIL import Image
import os
from django.utils.text import slugify

def create_item(request):
    if request.method == 'POST':
        form = ItemForm(request.POST, request.FILES)
        if form.is_valid():
            # 第一步：保存实例获取ID（不提交图片的话，这里会自动处理临时存储）
            item = form.save(commit=False)
            item.save()

            # 第二步：处理图片缩放与重命名
            if item.image:
                img_path = item.image.path
                # 缩放逻辑
                with Image.open(img_path) as img:
                    max_width = 800
                    if img.width > max_width:
                        ratio = max_width / img.width
                        new_height = int(img.height * ratio)
                        img = img.resize((max_width, new_height), Image.Resampling.LANCZOS)
                        img.save(img_path)
                
                # 生成新文件名
                ext = os.path.splitext(img_path)[1].lower()
                new_filename = f"{slugify(item.name)}_{item.id}{ext}"
                # 移动到目标目录
                target_dir = os.path.join(settings.MEDIA_ROOT, 'items')
                os.makedirs(target_dir, exist_ok=True)
                new_path = os.path.join(target_dir, new_filename)
                os.rename(img_path, new_path)

                # 更新数据库路径
                item.image.name = f"items/{new_filename}"
                item.save(update_fields=['image'])
            
            return redirect('item_detail', pk=item.pk)
    else:
        form = ItemForm()
    return render(request, 'create_item.html', {'form': form})

额外注意事项 #

• 临时目录清理：可使用django-cleanup库自动清理未被引用的临时图片文件。
• 文件名唯一性：因ID是数据库主键，结合slugify后的名称，可确保文件名无冲突。
• 权限问题：确保Django进程对MEDIA_ROOT目录有读写权限。

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