WordPress代码移除指定顶级父类的祖先分类无效问题排查

我的文章分类层级为：Adverts → Agriculture and Gardening → Agricultural tools，需求是仅保留以Adverts为顶级父类的最末级子分类（即只留Agricultural tools），移除所有祖先分类。手动在后台取消父类勾选再保存文章能正常实现，但通过functions.php添加save_post钩子代码执行后，error_log显示已修改为仅保留子分类，可前后台仍显示父类，这是怎么回事？

原代码 #

add_action( 'save_post', 'remove_ancestor_terms' );
function remove_ancestor_terms( $post_id ) {
    // 检查是否是修订版或自动保存
    if( wp_is_post_revision( $post_id ) || wp_is_post_autosave( $post_id ) ) {
        return;
    }

    // 目标顶级分类
    $ancestor_slug = 'adverts';
    $ancestor_term = get_term_by( 'slug', $ancestor_slug, 'category' );
    $ancestor_id   = $ancestor_term ? $ancestor_term->term_id : false;

    // 获取文章关联的分类ID
    $post_terms = wp_get_post_terms( $post_id, 'category', array( 'fields' => 'ids' ) );

    // 复制原始分类列表用于筛选
    $keep_terms = $post_terms;

    // 遍历每个分类，检查祖先
    foreach( $post_terms as $term_id ) {
        $term_ancestors = get_ancestors( $term_id, 'category' );

        // 跳过无祖先或不包含目标顶级分类的项
        if( empty( $term_ancestors ) || ! in_array( $ancestor_id, $term_ancestors ) ) { continue; }

        // 移除祖先分类，仅保留子分类
        $keep_terms = array_diff( $keep_terms, $term_ancestors );
    }

    /* 日志输出：
    before: Array([0] => 83 // Adverts, [1] => 125 // Agricultural tools, [2] => 100 // Agriculture and Gardening)
    return: Array([0] => 125)
    after: Array([0] => 125)
    */

    // 更新文章分类
    $return = wp_set_post_categories( $post_id, $keep_terms );
}

问题根源 #

1. 钩子执行时机冲突：默认save_post钩子优先级为10，WordPress后台保存分类的逻辑可能在之后执行，导致你修改后的分类被后台表单提交的原始数据覆盖。
2. 递归触发问题：调用wp_set_post_categories会再次触发save_post钩子，虽然加了自动保存检查，但仍可能出现逻辑混乱，甚至修改被递归调用冲掉。
3. 后台分类显示逻辑：WordPress后台编辑页的分类勾选框会自动勾选父类（即使数据库仅存储子分类），但手动取消后能正常保存，说明核心问题还是钩子时机与递归问题。

修复后的代码 #

add_action( 'save_post', 'remove_ancestor_terms', 99 );
function remove_ancestor_terms( $post_id ) {
    // 自动保存直接返回
    if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) {
        return;
    }
    // 跳过修订版
    if( wp_is_post_revision( $post_id ) ) {
        return;
    }
    // 检查用户权限，避免非法操作
    if ( ! current_user_can( 'edit_post', $post_id ) ) {
        return;
    }
    // 防止递归调用：标记当前正在执行，避免wp_set_post_categories再次触发钩子
    static $is_running = false;
    if ( $is_running ) {
        return;
    }
    $is_running = true;

    // 获取目标顶级分类ID
    $ancestor_slug = 'adverts';
    $ancestor_term = get_term_by( 'slug', $ancestor_slug, 'category' );
    $ancestor_id   = $ancestor_term ? $ancestor_term->term_id : false;

    // 目标分类不存在直接返回
    if ( !$ancestor_id ) {
        $is_running = false;
        return;
    }

    // 获取文章当前关联的分类ID
    $post_terms = wp_get_post_terms( $post_id, 'category', array( 'fields' => 'ids' ) );
    $keep_terms = $post_terms;

    foreach( $post_terms as $term_id ) {
        $term_ancestors = get_ancestors( $term_id, 'category' );
        // 跳过无祖先或不包含目标顶级分类的项
        if( empty( $term_ancestors ) || !in_array( $ancestor_id, $term_ancestors ) ) {
            continue;
        }
        // 移除所有祖先分类，仅保留子分类
        $keep_terms = array_diff( $keep_terms, $term_ancestors );
    }

    // 只有当分类确实需要修改时才执行更新，减少不必要操作
    if ( $keep_terms !== $post_terms ) {
        wp_set_post_categories( $post_id, $keep_terms );
    }

    $is_running = false;
}

修复说明 #

• 提升钩子优先级到99，确保代码在WordPress默认分类保存逻辑之后执行，避免修改被覆盖。
• 添加static $is_running变量，防止wp_set_post_categories触发递归调用，避免逻辑混乱。
• 增加权限检查，确保只有有权限的用户才能触发逻辑，提升安全性。
• 增加分类变化判断，仅在需要时执行更新，减少冗余操作。

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