WooCommerce使用BACS/COD支付时订单付款状态显示不一致问题排查

当前自定义代码存在三类致命缺陷，直接导致了同支付方式订单Paid标记不统一的异常：

• 存在PHP逻辑语法错误，判断条件完全失效
  原代码中判断支付方式的表达式 ( get_post_meta($order->id, '_payment_method', true) == 'bacs' || 'cod' ) 存在运算符优先级问题：==优先级高于||，且'cod'没有配套的比较逻辑，作为独立布尔值永远为真，导致整个判断括号的结果无论实际支付方式是什么都恒为true。只要订单处于on-hold/pending状态，就会被强制触发状态修改，完全没有按照预设的“仅BACS、COD订单”规则执行。
• 状态修改逻辑不符合WooCommerce的Paid标记规则
  WooCommerce的“Paid”标签不与订单状态直接绑定，只有当订单元数据_paid_date存在时才会显示该标记，而这个元数据只会在订单调用payment_complete()方法、走官方支付完成流程时自动写入。当前代码直接调用update_status('processing')只会修改订单的状态字符串，不会触发Paid标记的写入逻辑，自然会出现“状态是Processing，但不显示Paid”的情况。部分订单显示Paid，是因为这些订单后续走了正常的支付完成流程（比如后台手动确认到账），被写入了_paid_date，和脚本修改的状态形成了不一致。
• 代码存在版本兼容和重复触发问题
  原代码使用WooCommerce 3.0版本后就废弃的$order->id属性获取订单ID，新版本下该属性经常返回空值，导致不同订单的状态判断、修改流程执行结果不一致，对应随机出现的“同支付方式部分订单正常、部分异常”现象。同时代码在通用woocommerce_thankyou钩子和COD专属的woocommerce_thankyou_cod钩子上重复挂载了功能几乎一致的状态修改逻辑，会导致同一订单在单次请求中被重复修改状态，进一步放大异常。

• 抽取同是BACS支付的两类异常订单（显示Paid/不显示Paid），对比订单元数据中_paid_date的差异，验证不显示Paid的订单是否确实没有写入该字段
• 暂时停用当前的自定义代码，观察后续新生成的BACS订单的状态和Paid标记是否恢复统一，确认自定义逻辑的影响范围

直接移除原有错误的自定义代码，替换为符合WooCommerce规范的实现：

add_action( 'woocommerce_thankyou', 'custom_auto_process_bacs_cod_orders', 10, 1 );
function custom_auto_process_bacs_cod_orders( $order_id ) {
    if ( ! $order_id ) return;
    // 使用官方方法获取订单对象，避免版本兼容问题
    $order = wc_get_order( $order_id );
    if ( ! $order ) return;

    // 严格判断支付方式，避免逻辑短路
    $payment_method = $order->get_payment_method();
    if ( ! in_array( $payment_method, ['bacs', 'cod'] ) ) return;
    // 仅处理还未进入处理流程的订单
    if ( ! in_array( $order->get_status(), ['pending', 'on-hold'] ) ) return;

    // 注意：调用payment_complete()会自动写入_paid_date、标记订单为Paid，同时将订单状态设为Processing
    // 若BACS场景下未实际收到转账即标记Paid，存在财务对账风险，请根据业务实际需求确认是否启用
    $order->payment_complete();
}

• 不要使用update_status()直接修改订单状态来模拟已支付效果，该方法不会触发任何支付完成的钩子、不会写入支付时间，始终无法正常显示Paid标记
• 不要在通用支付钩子和某一支付方式的专属钩子上重复挂载相同功能的逻辑，避免同一请求多次触发状态修改
• BACS的默认流程是下单后订单为On-Hold状态，待财务确认到账后再手动标记Paid，如无特殊业务需求，不建议自动将BACS订单标记为已支付，避免对账混乱。

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