Django UpdateView POST请求遇302错误，需同步更新Factura与新增Historico记录

问题根源 #

你的302错误源于重定向逻辑冲突，以及代码中的几个核心逻辑错误：

1. UpdateView的post方法在表单验证通过后会自动返回success_url的重定向响应，但你在重写的post方法末尾又强制return redirect，导致两次触发重定向。
2. if super().post(request, *args, **kwargs)的判断逻辑错误：super().post返回的是HTTP响应对象（成功是302，失败是200），永远为True，导致即使表单验证失败，也会尝试创建Historico记录。
3. historico.factura = isinstance(self.object, Factura) and self.object or None写法冗余且风险高——UpdateView的self.object本身就是Factura实例，无需额外判断。

最优解决方案：重写form_valid方法（Django推荐方式） #

UpdateView提供了form_valid钩子，仅在表单验证通过时执行，完美适配你“更新Factura后创建Historico”的需求，避免重定向冲突：

class ActualizarFactura(UpdateView):
    model = Factura
    form_class = FormActualizarFactura
    template_name = 'facturas/fac_actualizar.html'
    success_url = reverse_lazy('facturas:listaFacturas')
    
    def form_valid(self, form):
        # 先调用父类方法，保存更新后的Factura实例
        response = super().form_valid(form)
        
        # 创建Historico记录（用cleaned_data取数更安全，避免直接操作POST）
        try:
            Historico.objects.create(
                factura=self.object,
                estado=self.object.estado,
                user=self.request.user,
                comentario=form.cleaned_data.get('comentario')
            )
        except Exception as e:
            messages.error(self.request, f'Error al crear registro histórico: {e}')
            # 可选：如果历史记录创建失败，回滚Factura的更新
            # self.object.refresh_from_db()
        
        return response
    
    def form_invalid(self, form):
        # 表单验证失败时的错误提示
        messages.error(self.request, f'Error al actualizar la factura: {self.object.NFactura}')
        return super().form_invalid(form)

备选方案：用TemplateView手动处理表单（不推荐，仅作参考） #

如果一定要用TemplateView，可以手动初始化表单、验证并执行双操作，但代码冗余度更高：

from django.views.generic.base import TemplateView

class ActualizarFacturaView(TemplateView):
    template_name = 'facturas/fac_actualizar.html'
    
    def get_context_data(self, **kwargs):
        context = super().get_context_data(**kwargs)
        # 传递当前Factura实例和表单到模板
        factura = Factura.objects.get(pk=self.kwargs['pk'])
        context['form'] = FormActualizarFactura(instance=factura)
        context['factura'] = factura
        return context
    
    def post(self, request, *args, **kwargs):
        factura = Factura.objects.get(pk=self.kwargs['pk'])
        form = FormActualizarFactura(request.POST, instance=factura)
        
        if form.is_valid():
            form.save()
            # 创建Historico记录
            try:
                Historico.objects.create(
                    factura=factura,
                    estado=factura.estado,
                    user=request.user,
                    comentario=form.cleaned_data['comentario']
                )
                messages.success(request, 'Factura actualizada y historial creado')
            except Exception as e:
                messages.error(request, f'Factura actualizada pero falló el historial: {e}')
            return redirect('facturas:listaFacturas')
        else:
            messages.error(request, f'Error al actualizar la factura: {factura.NFactura}')
            # 验证失败时返回带错误的表单
            context = self.get_context_data(**kwargs)
            context['form'] = form
            return self.render_to_response(context)

关键优化点 #

• 避免重写post方法：UpdateView的form_valid/form_invalid钩子已经封装了验证后的逻辑，无需手动处理请求流转。
• 使用form.cleaned_data：表单验证后的数据更安全，避免直接读取request.POST的脏数据。
• 精准捕获异常：仅在创建Historico时捕获异常，不掩盖表单验证本身的错误。

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