WooCommerce结账后更新用户自定义字段代码失效求助

嘿，我来帮你排查下这段代码的问题～ 你的思路是对的，但几个细节没处理到位，导致代码没生效，我给你调整一下：

首先看你原代码里的几个问题：

• 实例化WC_Order的时候，你给$order_id加了单引号，变成了字符串'$order_id'，而不是传递实际的订单ID变量，这会导致无法正确获取订单对象
• 没有判断用户是否是登录状态，访客订单的user_id是0，这时候调用update_user_meta是无效的
• 新版WooCommerce里，获取订单项产品ID更推荐用官方方法，而不是直接取数组键值，兼容性更好
• woocommerce_thankyou钩子会在所有订单到达感谢页时触发，包括未支付的订单，最好加个订单状态判断，确保只在支付成功后执行

下面是修正后的代码：

add_action( 'woocommerce_thankyou', 'checkout_update_user_meta' );
function checkout_update_user_meta( $order_id ) {
    // 避免重复执行（有些支付网关会多次触发这个钩子）
    if ( get_post_meta( $order_id, '_custom_field_updated', true ) ) {
        return;
    }

    // 获取订单对象，注意不要给$order_id加引号
    $order = wc_get_order( $order_id );
    if ( ! $order ) {
        return;
    }

    // 只处理已支付的订单
    if ( ! $order->is_paid() ) {
        return;
    }

    $user_id = $order->get_user_id();
    // 判断用户是否登录，访客订单跳过
    if ( $user_id == 0 ) {
        return;
    }

    foreach ( $order->get_items() as $item ) {
        // 用官方方法获取产品ID，兼容新版WooCommerce
        $product_id = $item->get_product_id();
        // 替换成你目标产品的ID
        if ( $product_id == 777 ) {
            update_user_meta( $user_id, 'masterclass-1', 'ok' );
            // 标记已更新，避免重复执行
            update_post_meta( $order_id, '_custom_field_updated', 'yes' );
            // 找到目标产品后可以跳出循环，提升效率
            break;
        }
    }
}

我额外加了几个优化点：

• 增加了重复执行的判断，因为有些支付网关会多次触发woocommerce_thankyou钩子，避免重复更新用户字段
• 用wc_get_order()代替直接实例化WC_Order，这是WooCommerce推荐的获取订单对象的方式
• 增加了订单对象有效性检查，防止异常情况
• 找到目标产品后跳出循环，不用遍历所有订单项

你把这段代码替换原来的，应该就能正常工作啦～

备注：内容来源于stack exchange，提问作者5jv