diff --git a/plan.md b/plan.md index 7b99191..ea9288e 100644 --- a/plan.md +++ b/plan.md @@ -1,31 +1,25 @@ -# Plan — 调试 "Undefined array key 'id'" PHP 错误 +# Plan — 调研「场馆删除后编辑商品出现规格重复错误」问题 > 版本:v1.0 | 日期:2026-04-20 | Agent:council/BackendArchitect -> 关联提交:bbea35d83(feat: 保存时自动填充 template_snapshot) +> 关联:vr_goods_config 全链路数据流追踪 --- ## 任务概述 -调试 ShopXO 后台编辑票务商品(goods_id=118)保存时报错: -``` -Undefined array key "id" -``` - -根因代码位于 `bbea35d83` 新增的 `AdminGoodsSaveHandle.php` save_thing_end 时机。 +当票务商品的场馆模板被硬删除后,编辑商品时出现「规格不允许重复」错误。 +根因:旧的幽灵 spec(来自已删除场馆的 `spec_base_id_map`)仍混入表单,提交时触发去重逻辑。 --- -## 任务清单 +## BackendArchitect 任务清单 -- [x] [Done: council/BackendArchitect] **Task 1**: 根因定位 — 逐行分析所有 "id" 访问位置 -- [x] [Done: council/BackendArchitect] **Task 2**: Db::name() 表前缀问题 — ShopXO 插件表前缀行为确认 -- [x] [Done: council/BackendArchitect] **Task 3**: 根因 1 — `$r['id']` 空安全(AdminGoodsSaveHandle 第 79 行) -- [x] [Done: council/BackendArchitect] **Task 4**: 根因 2 — `find()` 返回 null 的空安全(AdminGoodsSaveHandle 第 72 行) -- [x] [Done: council/BackendArchitect] **Task 5**: 根因 3 — `$config['template_id']` / `selected_rooms` 数据类型问题 -- [x] [Done: council/BackendArchitect] **Task 6**: SeatSkuService::BatchGenerate 类似问题审计 -- [x] [Done: council/BackendArchitect] **Task 7**: 修复方案汇总 + 建议修复优先级 -- [x] [Done: council/BackendArchitect] **Task 8**: 将修复方案写入 `reviews/BackendArchitect-on-Issue-13-debug.md` +- [ ] [Claimed: council/BackendArchitect] **Task 1**: 读取 AdminGoodsSaveHandle.php,分析 vr_goods_config 的读取和解析逻辑 +- [ ] [ ] **Task 2**: 找出 spec_base_id_map 如何被转换成规格项(spec items) +- [ ] [ ] **Task 3**: 分析 SeatSkuService.php GetGoodsViewData,模板不存在时的 fallback 行为 +- [ ] [ ] **Task 4**: 找出幽灵 spec 的产生环节,确认保存时是否有过滤 +- [ ] [ ] **Task 5**: 商品保存时规格去重逻辑位置;vr_goods_config 中多个规格项 spec_base_id 相同时的行为 +- [ ] [ ] **Task 6**: 根因分析报告(含具体文件路径和行号)+ 修复方案 --- @@ -33,61 +27,24 @@ Undefined array key "id" | 阶段 | 内容 | |------|------| -| **Draft** | ✅ Task 1-6:根因定位(只读,不修改代码) | -| **Review** | ✅ Task 7:汇总所有根因,给出修复建议 | -| **Finalize** | ✅ Task 8:输出评审报告到 reviews/ | - ---- - -## 依赖关系 - -- Task 2、3、4、5 互不依赖,可并行分析 -- Task 6 依赖 Task 1 的结论(确认 BatchGenerate 中类似位置) -- Task 7 依赖 Task 1-6 的结论 -- Task 8 依赖 Task 7 - ---- - -## 执行顺序 - -``` -并行: - Task 1: 读 AdminGoodsSaveHandle.php → 分析所有 $r['id'] / $template['seat_map'] / $r['id'] 访问 - Task 2: 读 BaseService::table() + ShopXO Db::name() 前缀逻辑 - Task 3: Task 1 完成后,确认 $r['id'] 空安全 - Task 4: Task 2 完成后,确认 find() null 处理 - Task 5: Task 1 完成后,确认 selected_rooms 类型匹配 -串行: - Task 6: 读 SeatSkuService.php → 检查 ensureAndFillVrSpecTypes / BatchGenerate 类似问题 - Task 7: 汇总根因,写修复建议 - Task 8: 输出 reviews/BackendArchitect-on-Issue-13-debug.md -``` +| **Draft** | Task 1-5:全链路数据流追踪 | +| **Review** | Task 6:输出调研报告至 reviews/council-ghost-spec-BackendArchitect.md | +| **Finalize** | 汇总至 reviews/council-ghost-spec-summary.md | --- ## 关键文件(只读) -| 文件 | 关注点 | -|------|--------| -| `shopxo/app/plugins/vr_ticket/hook/AdminGoodsSaveHandle.php` | save_thing_end 逻辑,template_snapshot 填充代码 | -| `shopxo/app/plugins/vr_ticket/service/SeatSkuService.php` | BatchGenerate、ensureAndFillVrSpecTypes | -| `shopxo/app/plugins/vr_ticket/service/BaseService.php` | table() 前缀方法 | -| `docs/VR_GOODS_CONFIG_SPEC.md` | vr_goods_config v3.0 JSON 格式 | -| `docs/PHASE2_PLAN.md` | 项目背景 | +- `shopxo/app/plugins/vr_ticket/hook/AdminGoodsSaveHandle.php` +- `shopxo/app/plugins/vr_ticket/service/SeatSkuService.php` +- `shopxo/app/plugins/vr_ticket/view/goods/ticket_detail.html` +- `shopxo/app/plugins/vr_ticket/admin/Admin.php` +- `shopxo/app/admin/hook/AdminGoodsSave.php` --- -## 根因结论(已验证) +## 执行顺序 -1. **Primary(99%)**: `AdminGoodsSaveHandle.php:77` — `$r['id']` 无空安全,rooms 中缺少 id key 时崩溃 -2. **Secondary(5%)**: `AdminGoodsSaveHandle.php:71` — `find()` 返回 null 后直接访问 `$template['seat_map']` -3. **Tertiary(静默)**: `AdminGoodsSaveHandle.php:77` — `selected_rooms` 类型不匹配,`in_array` 永远 false -4. **已排除**: 表前缀问题 — `Db::name()` 和 `BaseService::table()` 均查询 `vrt_vr_seat_templates`,等价 -5. **已排除**: SeatSkuService::BatchGenerate — 第 100 行已有 `!empty()` 空安全 fallback - -## 修复记录 - -- [x] **Fix Commit**: `804d465d0` — P1+P2 空安全修复已合并到 main - - `AdminGoodsSaveHandle.php:71-73`:新增 `if (empty($template)) { continue; }` - - `AdminGoodsSaveHandle.php:80`:`$r['id']` 前加 `isset()` 空安全 - - 合并到 main:`49930844f` +1. Task 1 → 2 → 3:并行读取关键文件 +2. Task 4 → 5:基于读取结果追踪 spec 去重逻辑 +3. Task 6:输出带行号的根因分析报告