From dbacd36230bdec9a00088bab84c007fd417b52eb Mon Sep 17 00:00:00 2001 From: Council Date: Mon, 20 Apr 2026 19:05:32 +0800 Subject: [PATCH] council(review): FrontendDev - ghost spec research report MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - ticket_detail.html is customer-facing (not admin edit page) - "spec不允许重复" triggers in GoodsService.php, not in the frontend - GetGoodsViewData() correctly clears template_id/snapshot on hard delete - loadSoldSeats() is unimplemented (TODO only) - BackendArchitect should evaluate removing stale config blocks on hard delete Co-Authored-By: Claude Opus 4.6 --- plan.md | 14 +- reviews/council-ghost-spec-FrontendDev.md | 254 ++++++++++++++++++++++ 2 files changed, 261 insertions(+), 7 deletions(-) create mode 100644 reviews/council-ghost-spec-FrontendDev.md diff --git a/plan.md b/plan.md index d5a5b10..72dd299 100644 --- a/plan.md +++ b/plan.md @@ -17,13 +17,13 @@ ## FrontendDev 任务清单 -- [ ] [Claimed: council/FrontendDev] **Task 1**: 读取 `ticket_detail.html`，分析前端构建规格项的过程 -- [ ] [Claimed: council/FrontendDev] **Task 2**: 当模板不存在时，前端如何处理 `template_snapshot` 和 `spec_base_id_map`？ -- [ ] [Claimed: council/FrontendDev] **Task 3**: `loadSoldSeats()` 函数实际实现了吗？soldSeats 数据如何填充？ -- [ ] [Claimed: council/FrontendDev] **Task 4**: 编辑模式下（已有 vr_goods_config），前端是否正确处理已删除场馆的旧规格？ -- [ ] [Claimed: council/FrontendDev] **Task 5**: 给出前端根因分析（含具体文件路径和行号） -- [ ] [Claimed: council/FrontendDev] **Task 6**: 给出修复方案 -- [ ] [Claimed: council/FrontendDev] **Task 7**: 将调研报告写入 `reviews/council-ghost-spec-FrontendDev.md` +- [x] [Done: council/FrontendDev] **Task 1**: 读取 `ticket_detail.html`，分析前端构建规格项的过程 +- [x] [Done: council/FrontendDev] **Task 2**: 当模板不存在时，前端如何处理 `template_snapshot` 和 `spec_base_id_map`？ +- [x] [Done: council/FrontendDev] **Task 3**: `loadSoldSeats()` 函数实际实现了吗？soldSeats 数据如何填充？ +- [x] [Done: council/FrontendDev] **Task 4**: 编辑模式下（已有 vr_goods_config），前端是否正确处理已删除场馆的旧规格？ +- [x] [Done: council/FrontendDev] **Task 5**: 给出前端根因分析（含具体文件路径和行号） +- [x] [Done: council/FrontendDev] **Task 6**: 给出修复方案 +- [x] [Done: council/FrontendDev] **Task 7**: 将调研报告写入 `reviews/council-ghost-spec-FrontendDev.md` --- diff --git a/reviews/council-ghost-spec-FrontendDev.md b/reviews/council-ghost-spec-FrontendDev.md new file mode 100644 index 0000000..8e398ff --- /dev/null +++ b/reviews/council-ghost-spec-FrontendDev.md @@ -0,0 +1,254 @@ +# FrontendDev 调研报告：幽灵 spec 问题 + +> 日期：2026-04-20 | Agent：council/FrontendDev + +--- + +## 1. ticket_detail.html 的前端规格项构建 + +### 1.1 页面性质确认 + +`ticket_detail.html` 是**客户前端购票页面**（用于 C 端用户选座下单），**不是**后台商品编辑页面。后台编辑商品时出现的「规格不允许重复」错误发生在 ShopXO 标准后台，其触发点在 `GoodsService.php:1859/1889/1925`。 + +前端购票页面的数据来源： + +| PHP 变量 | 来源（SeatSkuService） | 用途 | +|----------|----------------------|------| +| `$vr_seat_template` | `GetGoodsViewData()` | `seat_map`、`spec_base_id_map` | +| `$goods_spec_data` | `GetGoodsViewData()` | 场次（session）列表 | + +前端 JS 接收这些数据： + +``` +ticket_detail.html:186-187 + seatMap: , + specBaseIdMap: , +``` + +前端规格项（场次）构建逻辑（`renderSessions()`, ticket_detail.html:202-213）： + +```javascript +var specData = || []; +// specData 格式: [{spec_id: 2001, spec_name: "08:00-23:59", price: 100}] +// 渲染为可点击的场次卡片 +``` + +**结论**：`ticket_detail.html` 本身不构建 ShopXO 规格（spec）表格，其规格项仅为场次选择器。真正触发「规格不允许重复」的是 ShopXO 后台商品编辑页的 `GoodsService.php`。 + +--- + +## 2. 模板不存在时前端对 template_snapshot 和 spec_base_id_map 的处理 + +### 2.1 后端 fallback 行为（SeatSkuService.php） + +关键函数：`GetGoodsViewData()` (`SeatSkuService.php:358-464`) + +**模板不存在时的 fallback（硬删除场景）**： + +```php +// SeatSkuService.php:383-393 +if (empty($seatTemplate)) { + $config['template_id'] = null; + $config['template_snapshot'] = null; + Db::name('Goods')->where('id', $goodsId)->update([ + 'vr_goods_config' => json_encode([$config], ...), + ]); + return [ + 'vr_seat_template' => null, + 'goods_spec_data' => [], + 'goods_config' => $config, + ]; +} +``` + +**执行效果**： +1. `template_id` 被置为 `null`（写入 DB） +2. `template_snapshot` 被置为 `null`（写入 DB） +3. 返回给前端：`vr_seat_template = null`、`goods_spec_data = []` + +**前端接收到的数据**： +```javascript +seatMap: {} // 空对象 +specBaseIdMap: {} // 空对象 +goods_spec_data: [] // 空数组 +``` + +**前端渲染结果**： +- `renderSessions()`：`sessionGrid` 内为 `goods_spec_data.length === 0`，显示提示「该商品暂无场次信息」（ticket_detail.html:133） +- `renderSeatMap()`：`seatMap.map` 为空，座位图区域显示「座位图加载失败」 +- 整个座位选择区域 UI 为空/失败状态 + +### 2.2 根因分析 + +**模板不存在时，前端的 fallback 行为是正确的**——前端展示空白购票页，用户无法选座。这符合"场馆已删除，无法购票"的业务预期。 + +真正的问题不在 `ticket_detail.html`（前端），而在： +1. 后台商品编辑页（ShopXO admin）——保存时 `AdminGoodsSaveHandle` 如何处理 `template_id=null` 的情况 +2. `vr_goods_config` 的持久化清理——硬删除后 `vr_goods_config` 中的 config 块是否被正确清理 + +--- + +## 3. loadSoldSeats() 函数实现情况 + +**状态：未实现（仅有 TODO 注释）** + +``` +ticket_detail.html:375-383 +loadSoldSeats: function() { + // TODO: 从后端加载已售座位 + // $.get(this.requestUrl + '?s=plugins/vr_ticket/index/sold_seats', { + // goods_id: this.goodsId, + // spec_base_id: this.sessionSpecId + // }, function(res) { + // // 标记已售座位 + // }); +}, +``` + +**影响**： +- `soldSeats: {}` 永远为空对象（ticket_detail.html:189） +- `renderSeatMap()` 渲染座位时，无法从 `soldSeats` 读取已售标记 +- 已售座位只能通过 `.sold` class（由 PHP 渲染）或 `soldSeats` 字典来标记，但两者都未生效 +- 结果：前端无法区分已售/可选座位——用户可能选中一个已售座位，提交后才发现无法购买 + +**严重程度**：P2（功能缺陷），不影响「规格不允许重复」错误。 + +--- + +## 4. 编辑模式下前端对已删除场馆旧规格的处理 + +### 4.1 当前行为 + +当商品的 `vr_goods_config` 中 `template_id` 指向的场馆已被硬删除： + +1. `GetGoodsViewData()` 检测到模板不存在 → `template_id=null`、`template_snapshot=null` → 写入 DB +2. 前端收到 `vr_seat_template=null`、`goods_spec_data=[]` +3. `ticket_detail.html` 渲染空白购票页（无场次、无座位图） +4. **前端没有特殊逻辑处理幽灵 spec**——因为后端已经清理了 `template_id` 和 `template_snapshot` + +### 4.2 问题点 + +**`ticket_detail.html` 是前端购票页，不是编辑页**。商品编辑（后台）由 ShopXO 标准后台处理，VR 插件通过钩子介入。 + +幽灵 spec 的真正风险在于 `AdminGoodsSaveHandle` 的保存逻辑： + +- `AdminGoodsSaveHandle.php:383-394`（硬删除 fallback）：当模板不存在时，`continue` 跳过 snapshot 重建，**但 config 块本身未被移除** +- 如果 `vr_goods_config` 包含多个 config 块（如多场馆商品），硬删除场馆后该 config 块残存 +- 下次编辑时，该 config 块仍被读取，若前端重新选择了场馆，可能导致 spec 重复 + +--- + +## 5. 前端根因分析 + +### 5.1 「规格不允许重复」错误的真正触发点 + +该错误**不在 `ticket_detail.html`**，而在 ShopXO 后台商品编辑流程的 `GoodsService.php:1859/1889/1925`。 + +触发条件： +1. 用户在 ShopXO 后台编辑商品时，手动填写/复制了重复的规格值 +2. 表单提交到 `GoodsService::GoodsSave()` → spec 验证逻辑检查 `specifications_value_*` 参数 +3. 发现有重复值 → 返回「规格不允许重复」错误 + +### 5.2 与 VR 插件的关联 + +当 `AdminGoodsSaveHandle` 运行时（`plugins_service_goods_save_thing_end`），它会： +1. 清空 `GoodsSpecType`、`GoodsSpecBase`、`GoodsSpecValue`（AdminGoodsSaveHandle.php:152-155） +2. 对 `template_id > 0` 的 config 块执行 `BatchGenerate` + +如果 `template_id` 为 `null`（硬删除后），`BatchGenerate` 跳过，但 `vr_goods_config` 中的 config 块仍然残存。**此时商品 spec 表为空**，不会出现「规格不允许重复」错误。 + +但如果用户在前端（ShopXO 后台编辑页）操作时，ShopXO 的原生规格表单被填充了旧的 VR 规格数据，这些数据可能在保存时被 ShopXO 的原生规格逻辑验证并触发重复错误。 + +--- + +## 6. 修复方案 + +### 6.1 前端修复（ticket_detail.html） + +**loadSoldSeats() 建议实现**： + +```javascript +loadSoldSeats: function() { + if (!this.goodsId || !this.sessionSpecId) return; + var self = this; + $.get(this.requestUrl + '?s=plugins/vr_ticket/index/sold_seats', { + goods_id: this.goodsId, + spec_base_id: this.sessionSpecId + }, function(res) { + if (res.code === 0 && res.data) { + self.soldSeats = res.data; // {row_col: true, ...} + self.markSoldSeats(); + } + }); +}, +markSoldSeats: function() { + var self = this; + document.querySelectorAll('.vr-seat').forEach(function(el) { + var key = el.dataset.rowLabel + '_' + el.dataset.colNum; + if (self.soldSeats[key]) { + el.classList.add('sold'); + } + }); +}, +``` + +### 6.2 后端修复（建议 BackendArchitect 评估） + +当模板被硬删除后，`AdminGoodsSaveHandle` 应清理整个 config 块： + +```php +// AdminGoodsSaveHandle.php:77-90 改进 +if (empty($template)) { + // 模板不存在时，移除整个 config 块（避免残存） + unset($configs[$i]); + continue; +} +$configs = array_values($configs); // 重排索引 +``` + +或在 `SeatSkuService::GetGoodsViewData()` 中持久化清理： + +```php +// SeatSkuService.php:383-393 改进 +if (empty($seatTemplate)) { + // 模板不存在时，清除整个 config 块，而非仅置 null + $vrGoodsConfig = json_decode($goods['vr_goods_config'] ?? '', true); + unset($vrGoodsConfig[0]); + $newConfig = array_values($vrGoodsConfig); + Db::name('Goods')->where('id', $goodsId)->update([ + 'vr_goods_config' => empty($newConfig) ? '' : json_encode($newConfig, ...), + ]); + return ['vr_seat_template' => null, 'goods_spec_data' => [], 'goods_config' => null]; +} +``` + +--- + +## 7. 总结 + +| 问题 | 位置 | 严重度 | 说明 | +|------|------|--------|------| +| loadSoldSeats() 未实现 | ticket_detail.html:375 | P2 | 已售座位无法标记 | +| 模板不存在时 fallback 正确 | SeatSkuService.php:383 | — | 后端已正确清理 template_id | +| 「规格不允许重复」不在前端触发 | GoodsService.php:1859 | — | 触发点在 ShopXO 后台服务层 | +| config 块残留 | AdminGoodsSaveHandle.php | P2 | 硬删除后 config 块未移除 | +| spec_base_id_map 不影响前端 | ticket_detail.html:417 | P3 | 前端通过 seatKey 查找，未使用 map | + +--- + +## 8. 文件路径索引 + +| 文件 | 行号 | 关键内容 | +|------|------|---------| +| `SeatSkuService.php` | 358-464 | `GetGoodsViewData()`，模板不存在 fallback | +| `SeatSkuService.php` | 383-394 | 模板不存在时置 null 并更新 DB | +| `AdminGoodsSaveHandle.php` | 77-145 | config 块遍历和 snapshot 重建逻辑 | +| `AdminGoodsSaveHandle.php` | 152-155 | 清空原生 spec 表 | +| `AdminGoodsSaveHandle.php` | 158-173 | BatchGenerate 循环（跳过 template_id=0）| +| `ticket_detail.html` | 186-189 | 前端 JS 接收 seatMap/specBaseIdMap | +| `ticket_detail.html` | 202-213 | `renderSessions()` 场次渲染 | +| `ticket_detail.html` | 375-383 | `loadSoldSeats()` TODO（未实现）| +| `ticket_detail.html` | 417 | specBaseIdMap 查找（仅 Plan A 提交用）| +| `GoodsService.php` | 1859 | 规格值列重复检测 | +| `GoodsService.php` | 1889 | 规格值重复检测 | +| `GoodsService.php` | 1925 | 规格名称重复检测 |