WooCommerce插件函数内动态获取当前产品ID失效排查

你的两段代码都存在可导致运行异常的问题，具体如下：

• 固定ID版本本身存在变量名笔误：你实例化WC_Product得到的对象赋值给了$get_product，后续调用相册获取方法时却使用了未定义的$product变量，之前能正常运行是因为运行环境中恰好存在同名全局变量匹配了对应产品，并非代码逻辑正确。
• 动态版本直接调用全局$product失效，是因为该全局变量仅在WooCommerce前台产品单页的主循环内会被自动初始化，你在插件的PDF生成逻辑、后台页面、自定义接口等场景调用函数时，该变量未被赋值，直接调用类方法会触发致命错误。
• 两段代码共同的语法问题：wp_get_attachment_image的第二个参数medium未加单引号，PHP会将其识别为未定义常量，高版本PHP环境下会直接抛出运行错误。

优先推荐显式传入产品ID的实现，不依赖全局变量，兼容性更强：

function print_gallery_pdf( $product_id = 0 ){
    // 未传入产品ID时，尝试从全局变量获取兜底
    if ( empty( $product_id ) ) {
        global $product;
        if ( ! ( $product instanceof WC_Product ) ) {
            // 无法获取有效产品对象时返回空数组，避免阻断流程
            return array();
        }
    } else {
        // 传入ID时手动获取产品对象
        $product = wc_get_product( $product_id );
        if ( ! $product ) {
            return array();
        }
    }

    $gallery_links = array();
    $attachment_ids = $product->get_gallery_image_ids();
    foreach ( $attachment_ids as $attachment_id ){
        // 图片尺寸参数为字符串，需要加引号
        $gallery_image = wp_get_attachment_image( $attachment_id, 'medium' );
        $gallery_links[] = $gallery_image;
    }
    return $gallery_links;
}

• 所有非产品单页场景（包括PDF生成接口、后台操作、AJAX请求、定时任务等）必须显式传入目标产品ID调用，示例：print_gallery_pdf( $target_pid );
• 仅当函数在WooCommerce产品单页主循环内执行时，可以不传参，函数会自动读取全局产品对象。

注意：全局$product的上下文依赖性极强，生产环境不建议直接依赖全局变量做业务逻辑，显式传参是最稳定的实现方式。

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