为账户客户设置信用限额时遭遇PHP代码错误求助

问题背景 #

将COD（货到付款）作为信用账户支付方式，为信用账户客户创建了自定义角色credit_account，仅该角色登录用户可见COD选项。通过自定义用户元字段设置信用限额，但代码报错，错误为调用get_cart_subtotal()时$woocommerce->cart对象为Null。

错误原因 #

woocommerce_available_payment_gateways钩子会在前台购物流程和后台特定场景（比如WooCommerce引导任务、支付设置页面）触发。后台场景下购物车对象并未初始化，直接调用$woocommerce->cart的方法就会抛出Null错误。另外原代码还存在两个小问题：

• 用户编辑页面的输入框name属性为code，但更新逻辑中读取的是$_POST['credit_limit']，导致信用限额无法保存
• 条件判断的逻辑分组错误，可能导致不符合预期的网关隐藏行为

修复方案 #

关键修改点 #

1. 先检查购物车对象是否存在：仅当WC()->cart有效时才执行信用限额判断，后台场景直接返回原网关列表
2. 替换购物车金额获取方式：用WC()->cart->subtotal直接获取浮点数值，无需正则处理（get_cart_subtotal()返回的是带货币符号的字符串，而subtotal是纯净的数值）
3. 修正用户元字段的name属性：让输入框name和更新逻辑的key一致，确保数据能正确保存
4. 修复条件判断的逻辑分组：用括号明确分组逻辑，避免逻辑歧义

修复后的完整代码 #

// 添加用户编辑页面的信用限额字段
add_action('show_user_profile', 'custom_user_profile_fields');
add_action('edit_user_profile', 'custom_user_profile_fields');
function custom_user_profile_fields( $user ) {
?>
    <table class="form-table">
        <tr>
            <th>
                <label for="credit_limit"><?php _e( 'Credit Limit' ); ?></label>
            </th>
            <td>
                <!-- 修正name属性为credit_limit，和更新逻辑对应 -->
                <input type="text" name="credit_limit" id="credit_limit" value="<?php echo esc_attr( get_user_meta( $user->ID, 'credit_limit', true ) ); ?>" class="regular-text" />
                <p class="description"><?php _e( 'Set the credit limit for this user (e.g. 500 for $500).' ); ?></p>
            </td>
        </tr>
    </table>
<?php
}

// 更新用户信用限额元字段
add_action( 'personal_options_update', 'update_extra_profile_fields' );
add_action( 'edit_user_profile_update', 'update_extra_profile_fields' );
function update_extra_profile_fields( $user_id ) {
    if ( current_user_can( 'edit_user', $user_id ) && isset($_POST['credit_limit']) ) {
        // 清理输入值，确保是数字
        $credit_limit = floatval($_POST['credit_limit']);
        update_user_meta( $user_id, 'credit_limit', $credit_limit );
    }
}

// 根据用户角色和信用限额控制COD网关显示
function payment_gateway_disable_on_account( $available_gateways ) {
    // 仅在前台购物场景处理，后台直接返回原网关
    if ( is_admin() || !WC()->cart ) {
        return $available_gateways;
    }

    $user = wp_get_current_user();
    $credit_limit = get_user_meta( $user->ID, 'credit_limit', true );

    // 设置默认信用限额
    if ( empty( $credit_limit ) || !is_numeric($credit_limit) ) {
        $credit_limit = 500;
    }

    // 直接获取购物车小计数值，无需正则处理
    $cart_subtotal = WC()->cart->subtotal;
    $credit_remaining = $credit_limit - $cart_subtotal;

    // 修正逻辑分组：如果不是信用账户用户，或者剩余信用不足，则隐藏COD
    if ( isset( $available_gateways['cod'] ) ) {
        if ( !in_array( 'credit_account', $user->roles ) || $credit_remaining < 0 ) {
            unset( $available_gateways['cod'] );
        }
    }

    return $available_gateways;
}
add_filter( 'woocommerce_available_payment_gateways', 'payment_gateway_disable_on_account' );

额外说明 #

• 使用WC()->cart替代全局$woocommerce对象，是WooCommerce官方推荐的调用方式，兼容性更好
• 添加了输入值的数字验证，避免非数字值导致的计算错误
• 增加了后台场景的判断，彻底避免后台触发钩子时的Null错误

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