WooCommerce基于订单数自动变更用户角色报错求助

你的代码出现内部服务器错误，核心原因是调用了两个WordPress/WooCommerce不存在的原生函数：get_user_role()和get_user_order_count()，同时还有逻辑冗余和未过滤订单状态的问题，以下是修复后的完整代码及问题说明：

错误点分析 #

• get_user_role() 不存在：WordPress没有这个原生函数，获取用户角色需要通过get_userdata()获取用户对象后读取角色属性。
• get_user_order_count() 不存在：需要自行实现函数，过滤出"处理中"和"已完成"状态的订单并计数。
• 重复调用email_exists()：多次查询数据库获取用户ID，浪费资源且冗余。
• 未处理边界情况：未判断订单对象是否有效、用户是否存在，可能引发空值报错。
• 订单状态未过滤：原代码没有筛选"处理中/已完成"的订单，不符合需求。

修复后的完整代码 #

add_action('woocommerce_new_order', 'change_user_role_based_on_order_count', 10, 1);

function change_user_role_based_on_order_count($order_id) {
    // 获取订单对象并检查有效性
    $order = wc_get_order($order_id);
    if (!$order) return;

    // 获取订单关联的邮箱
    $user_email = $order->get_billing_email();
    if (empty($user_email)) return;

    // 获取用户ID，避免重复查询
    $user_id = email_exists($user_email);
    if (!$user_id) return;

    // 排除管理员
    if (user_can($user_id, 'administrator')) return;

    // 获取用户当前角色
    $user = get_userdata($user_id);
    $current_roles = $user->roles;
    // 取用户的第一个角色（WordPress用户可能有多个角色，这里取主角色）
    $current_user_role = reset($current_roles);

    // 目标角色
    $desired_role = 'your_desired_role';

    // 检查用户是否已拥有目标角色，避免重复操作
    if (in_array($desired_role, $current_roles)) return;

    // 获取用户的处理中/已完成订单数量
    $valid_order_count = get_valid_user_order_count($user_id);

    // 满足条件则更新角色
    if ($valid_order_count >= 10) {
        wp_update_user(array('ID' => $user_id, 'role' => $desired_role));
    }
}

// 自定义函数：获取用户处理中/已完成的订单数量
function get_valid_user_order_count($user_id) {
    $args = array(
        'customer_id' => $user_id,
        'status' => array('processing', 'completed'), // 过滤目标状态
        'return' => 'count',
        'limit' => -1,
    );
    return wc_get_orders($args);
}

关键修复说明 #

1. 新增订单有效性检查：先判断$order是否存在，避免空对象调用方法报错。
2. 优化用户ID查询：只调用一次email_exists()，减少数据库查询。
3. 正确获取用户角色：通过get_userdata()获取用户对象，读取roles属性，处理多角色场景。
4. 实现订单计数函数：用wc_get_orders()筛选"processing"和"completed"状态的订单，返回计数。
5. 提前终止无效流程：在每一步边界条件（如无邮箱、无用户ID、已有目标角色）时直接返回，避免无效执行。

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