vr-shopxo-plugin/docs/PLAN_GHOST_SPEC_FIX.md

Plan: 幽灵 Spec 问题修复

日期：2026-04-20 | Issue: #15 + #16 | 状态：待实施

---

问题概览

Issue	问题	优先级	根因文件
#15	save_thing_end 脏数据写回	P1	AdminGoodsSaveHandle.php
#16	GetGoodsViewData 单模板模式	P1	SeatSkuService.php

---

Issue #15 — save_thing_end 脏数据写回

根因

AdminGoodsSaveHandle.php 第 60-65 行：DB 值优先（$goodsRow['vr_goods_config']），前端过滤后的值（$data['vr_goods_config']）只是 fallback。脏 config 块（含已删除场馆的 template_id）被写回 DB，幽灵 spec 累积。

修复计划

步骤 1：调换读取优先级（主要修复）

文件：shopxo/app/plugins/vr_ticket/hook/AdminGoodsSaveHandle.php 行号：第 60-65 行

// 修改前
$rawConfig = is_array($goodsRow) ? ($goodsRow['vr_goods_config'] ?? '') : '';
if (empty($rawConfig)) {
    $rawConfig = $data['vr_goods_config'] ?? '';
}

// 修改后（主要修复）
// 前端已过滤无效 template_id，优先使用 data。若无前端数据再 fallback 到 DB
if (!empty($data['vr_goods_config'])) {
    $rawConfig = $data['vr_goods_config'];
} else {
    $rawConfig = is_array($goodsRow) ? ($goodsRow['vr_goods_config'] ?? '') : '';
}

验证：

• 提交后 git diff 确认读取顺序调换
• git status 确认只有 AdminGoodsSaveHandle.php 被修改

步骤 2：无效 config 块移除（防御层）

文件：同上 行号：第 88-89 行

  if (empty($template)) {
-     continue;
+     unset($configs[$i]);  // 移除无效 config 块
+     continue;
  }

行号：第 145 行（unset($config); 之后）

$configs = array_values($configs);  // 重排数组索引

验证：

• grep 确认 unset($configs[$i]) 在第 89 行附近
• grep 确认 array_values 在 unset($config) 之后

步骤 3：写回前判空（防御层）

行号：第 148 行之前

+ if (!empty($configs)) {
  Db::name('Goods')->where('id', $goodsId)->update([
      'vr_goods_config' => json_encode($configs, JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES),
  ]);
+ }

---

Issue #16 — GetGoodsViewData 单模板模式

根因

SeatSkuService.php 第 368 行：只取 $vrGoodsConfig[0]（第一个配置块）。多模板商品时其余配置块被完全忽略；写回时只写 [$config] 单元素，可能覆盖其他有效配置块。

修复计划

步骤 1：过滤有效配置块

文件：shopxo/app/plugins/vr_ticket/service/SeatSkuService.php 行号：第 365-373 行（在 $config = $vrGoodsConfig[0]; 之前）

// 过滤有效配置块
$validConfigs = [];
foreach ($vrGoodsConfig as $cfg) {
    $tid = intval($cfg['template_id'] ?? 0);
    if ($tid <= 0) continue;
    $tpl = Db::name(self::table('seat_templates'))->where('id', $tid)->find();
    if (!empty($tpl)) {
        $validConfigs[] = $cfg;
    }
}
if (empty($validConfigs)) {
    return ['vr_seat_template' => null, 'goods_spec_data' => [], 'goods_config' => null];
}
$config = $validConfigs[0];  // 取第一个有效配置块用于前端展示

步骤 2：修改写回逻辑

行号：第 386-388 行（原写回 [$config]）

- Db::name('Goods')->where('id', $goodsId)->update([
-     'vr_goods_config' => json_encode([$config], JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES),
- ]);
+ Db::name('Goods')->where('id', $goodsId)->update([
+     'vr_goods_config' => json_encode($validConfigs, JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES),
+ ]);

验证：

• grep 确认 validConfigs 被用于写回
• grep 确认不再有 [$config] 写回模式

---

实施顺序

1. Issue #15（AdminGoodsSaveHandle.php）— 三步修改
2. Issue #16（SeatSkuService.php）— 两步修改
3. 验证：grep + git diff 确认修改正确
4. commit + push

---

禁止事项

• 不改其他文件（尤其是 Admin.php）
• 不改 Hook.php 或其他无关文件
• commit 前执行 git status 确认只有目标文件