WooCommerce自动删除旧已完成订单代码失效问题排查

你的代码存在以下几个关键问题，导致无法正常运行：

• 日期比较逻辑错误：使用mdy格式的字符串做日期对比会出现判断失误，比如123123（2023-12-31）和010124（2024-01-01），字符串比较时前者会被判定为更大，但实际日期更早。这种格式无法正确处理跨月跨年的日期对比。
• 未定义变量：循环中调用wp_delete_post($order_id,true)，但$order_id从未定义，正确的变量应为$order->ID。
• 数据库查询效率低下：先查询所有已完成订单再循环判断日期，会加载大量不必要的数据。应该直接在SQL语句中筛选出180天前的订单，减少内存占用和处理时间。
• 钩子选择不合理：admin_footer钩子会在后台每个页面加载时触发，不仅效率极低，还会重复执行删除操作，容易导致误删或性能问题。
• 日期计算冗余：通过$today拼接计算过期日期完全多余，直接用strtotime("-180 days")就能得到正确的过期时间戳。
• 未处理WooCommerce关联数据：直接用wp_delete_post只能删除订单帖子，无法清理WooCommerce关联的订单项、订单元数据等，应该使用WooCommerce官方提供的wc_delete_order函数。

修正后的代码 #

function auto_delete_old_completed_orders() {
    global $wpdb;

    // 计算180天前的时间戳并转换为MySQL可识别的日期格式
    $expiration_date = date('Y-m-d H:i:s', strtotime("-180 days"));

    // 直接查询180天前的已完成订单ID
    $order_ids = $wpdb->get_col(
        $wpdb->prepare(
            "SELECT ID FROM $wpdb->posts 
             WHERE post_type = 'shop_order' 
             AND post_status = 'wc-completed' 
             AND post_date < %s",
            $expiration_date
        )
    );

    if (!empty($order_ids)) {
        foreach ($order_ids as $order_id) {
            // 使用WooCommerce官方函数彻底删除订单及所有关联数据
            wc_delete_order($order_id, true);
        }
    }
}

// 添加每日定时任务，避免重复触发
if (!wp_next_scheduled('auto_delete_old_orders_hook')) {
    wp_schedule_event(time(), 'daily', 'auto_delete_old_orders_hook');
}
add_action('auto_delete_old_orders_hook', 'auto_delete_old_completed_orders');

说明 #

1. 改用MySQL标准日期格式做筛选，确保日期判断准确无误。
2. 使用wc_delete_order替代wp_delete_post，完整清理订单的关联订单项、元数据等内容。
3. 通过WordPress定时任务每日执行一次，避免重复触发和不必要的性能损耗。
4. 直接在SQL中筛选目标订单，大幅提升查询和处理效率。

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