Spree 2-4-stable：应用新优惠券促销时清除旧促销方案

我之前维护Spree 2-4-stable项目时遇到过完全一样的问题，以下是我验证过的可行方案：

问题根源 #

当你更换优惠券时，旧优惠券对应的Spree::Promotion实例并没有从order.promotions集合中移除，所以再次尝试应用旧优惠券时，promotion_exists_on_order?方法会检测到它已存在，从而抛出提示。核心要做的就是在应用新优惠券前，自动清理掉订单中已有的所有优惠券类型促销。

修改优惠券处理逻辑 #

我们需要修改.../bundler/gems/spree-651f537d0d89/core/app/models/spree/promotion_handler/coupon.rb文件中的代码，添加清理旧优惠券的逻辑，并在应用新优惠券前执行。

步骤1：新增清理旧优惠券的方法 #

在该文件中添加一个专门用于移除订单中已有优惠券促销的方法：

# 移除订单中所有已应用的优惠券类型促销
def remove_existing_coupon_promotions(order)
  # 筛选出所有带优惠券码的促销（区别于无自动触发的无码促销）
  existing_coupon_promos = order.promotions.select do |promo|
    promo.coupon_code.present?
  end

  existing_coupon_promos.each do |promo|
    # 从订单的promotions集合中移除促销
    order.promotions.delete(promo)
    # 同时删除对应的费用调整项，避免订单金额计算异常
    order.adjustments.where(source: promo).destroy_all
  end
end

步骤2：在应用新优惠券前调用清理方法 #

找到文件中的apply方法（这是处理优惠券应用的核心逻辑），在开始应用新优惠券的流程前，先调用上面的清理方法：

def apply
  promotion = Spree::Promotion.find_by(code: coupon_code)

  # 关键操作：先清理订单中已有的优惠券促销
  remove_existing_coupon_promotions(order)

  # 原有的存在性检查（现在订单里已无旧优惠券，此判断主要防止重复提交同一优惠券）
  if promotion_exists_on_order?(order, promotion)
    return false, Spree.t(:coupon_already_applied)
  end

  # 保留原有的促销应用逻辑
  if promotion && promotion.eligible?(order, promotion_code: promotion.coupon_code)
    promotion.activate(order: order, promotion_code: promotion.coupon_code)
    return true, promotion.success_message
  else
    return false, Spree.t(:coupon_not_found)
  end
end

验证效果 #

现在测试你的场景：

1. 应用优惠券AAAA → 订单promotions集合仅包含AAAA对应的促销
2. 更换为优惠券BBBB → 系统自动移除AAAA的促销，再添加BBBB的促销，此时order.promotions仅保留BBBB
3. 再次尝试应用AAAA → 系统检测不到AAAA在集合中，可正常应用（同时会自动移除BBBB的促销）

这样就实现了订单始终只保留一个优惠券促销的需求。

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