Django中遍历查询集将布尔字段设为True的问题排查

核心问题 #

1. 模型重写save方法引发异常：你的CartOrder模型中并没有定义payment_status字段，但在CartOrderItem的save方法里尝试访问self.order.payment_status，这会直接抛出AttributeError，导致所有save操作失败——包括你在视图中逐个调用的c.save()。
2. 逐个更新效率低且易出错：视图中循环遍历CartOrderItem逐个保存的方式不仅效率低下，还会因为模型save方法的异常导致更新失效。

修复步骤 #

方案一：直接批量更新（推荐） #

去掉CartOrderItem中重写的save方法，在视图中使用Django的批量更新操作，直接在数据库层面修改字段，无需触发模型的save方法，高效且避免异常：

from django.db import transaction
from django.shortcuts import get_object_or_404, render
from django.contrib.auth.decorators import login_required

@login_required
def PaymentSuccessView(request):
    # 此处省略获取stripe session的代码
    ...
    # 用事务确保操作原子性，避免部分更新成功部分失败
    with transaction.atomic():
        order = get_object_or_404(CartOrder, stripe_payment_intent=session.id)
        # 批量更新所有关联的CartOrderItem的paid字段为True
        CartOrderItem.objects.filter(order=order).update(paid=True)
        
    return render(request, "payment/payment_success.html", {"order": order}) 

方案二：修复模型逻辑（如果需要自动同步订单状态） #

如果你希望通过订单状态自动同步CartOrderItem的paid字段，需要先给CartOrder添加缺失的payment_status字段，再调整模型逻辑：

1. 更新CartOrder模型：

class CartOrder(models.Model):
    vendor = models.ManyToManyField(Vendor)
    buyer = models.ForeignKey(User, on_delete=models.SET_NULL, null=True, related_name="buyer")
    stripe_payment_intent = models.CharField(max_length=200,null=True, blank=True)
    # 添加支付状态字段，设置默认值为未支付
    payment_status = models.CharField(
        max_length=20,
        default="unpaid",
        choices=[("unpaid", "未支付"), ("paid", "已支付")]
    )

2. 执行数据库迁移：

python manage.py makemigrations
python manage.py migrate

3. 调整视图代码，先更新订单状态：

@login_required
def PaymentSuccessView(request):
    ...
    with transaction.atomic():
        order = get_object_or_404(CartOrder, stripe_payment_intent=session.id)
        order.payment_status = "paid"
        order.save()
        # 此时如果需要触发CartOrderItem的save方法，可以遍历保存，或者继续用批量更新
        CartOrderItem.objects.filter(order=order).update(paid=True)
        
    return render(request, "payment/payment_success.html", {"order": order}) 

4. 调整CartOrderItem的save方法（可选，确保后续修改订单状态时自动同步）：

class CartOrderItem(models.Model):
    order = models.ForeignKey(CartOrder, on_delete=models.CASCADE)
    vendor = models.ForeignKey(Vendor, on_delete=models.SET_NULL, null=True)
    paid = models.BooleanField(default=False)
    ....
    
    def save(self, *args, **kwargs):
        # 先判断order是否存在payment_status字段，避免异常
        if hasattr(self.order, 'payment_status') and self.order.payment_status == "paid":
            self.paid = True
        # 调用父类的save方法
        super().save(*args, **kwargs)

排查验证建议 #

• 开启Django的DEBUG模式，查看是否有AttributeError等异常信息，确认问题根源。
• 在视图中添加日志或打印语句，验证session.id与order.stripe_payment_intent是否匹配，以及CartOrderItem.objects.filter(order=order).count()是否大于0，确保有需要更新的条目。

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