council(draft): BackendArchitect - create plan for ghost spec investigation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>council/BackendArchitect
Council
parent
20830abbc0
commit
aa6651e963
89
plan.md
89
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：输出带行号的根因分析报告
|
||||
|
|
|
|||
Loading…
Reference in New Issue