From f55ba36e7b1ff407a4d19f4d5317fa64b153944f Mon Sep 17 00:00:00 2001 From: Council Date: Wed, 15 Apr 2026 20:45:09 +0800 Subject: [PATCH] council(draft): BackendArchitect - Q2 editor research findings Plugins_view_admin_goods_save is an injection point, not a replacement point. Save() accepts standard POST. Hook can intercept save flow. Final recommendation: hook injection + JSON editor. Co-Authored-By: Claude Opus 4.6 --- council-output/EDITOR_RESEARCH.md | 197 ++++++++++++++++++++++++++++++ 1 file changed, 197 insertions(+) create mode 100644 council-output/EDITOR_RESEARCH.md diff --git a/council-output/EDITOR_RESEARCH.md b/council-output/EDITOR_RESEARCH.md new file mode 100644 index 0000000..f5d949d --- /dev/null +++ b/council-output/EDITOR_RESEARCH.md @@ -0,0 +1,197 @@ +# vr-shopxo-plugin 编辑器方案调研报告 + +> 版本：v1.0 | 日期：2026-04-15 | Agent：BackendArchitect (Q2) + FrontendDev (Q1) + +## Q2：商品发布页替换方案（邪门方案）可行性 — BackendArchitect + +### 核心代码路径 + +| 文件 | 作用 | +|------|------| +| `shopxo/app/admin/controller/Goods.php:82-177` | SaveInfo() 方法 | +| `shopxo/app/admin/controller/Goods.php:187-192` | Save() 方法 | +| `shopxo/app/service/GoodsService.php:1549-1565` | plugins_service_goods_save_handle 钩子 | +| `shopxo/app/admin/view/default/goods/saveinfo.html:505-510` | 钩子渲染位置 | + +--- + +### Q2-A: 钩子调用位置分析 + +**`plugins_view_admin_goods_save` 在 SaveInfo() 中的位置**（Goods.php:159-167）： + +```php +$hook_name = 'plugins_view_admin_goods_save'; +$assign[$hook_name.'_data'] = MyEventTrigger($hook_name, [ + 'hook_name' => $hook_name, + 'is_backend' => true, + 'goods_id' => isset($params['id']) ? $params['id'] : 0, + 'data' => &$data, + 'params' => &$params, +]); +// 紧接着: +MyViewAssign($assign); +return MyView(); // 渲染 saveinfo.html +``` + +**结论：钩子在模板渲染之前被调用，结果存入 `$assign['plugins_view_admin_goods_save_data']` 供模板使用。** + +--- + +### Q2-B: 能否完全替换页面内容？ + +**关键发现：NO — 钩子仅是注入点，不是替换点。** + +查看 `saveinfo.html` 模板结构（简化）： +``` +ModuleInclude('public/header') +
+
+ [商品名称输入框] + [商品分类选择器] + [nav_switch_btn: base/spec/parameters/photos/content/video/seo/use_guide] + +
+ +
+ plugins_view_admin_goods_save ← 钩子在此注入 +
+
+ [SEO信息tab] + [popup submit按钮] +
+
+``` + +钩子注入位置在第 505-510 行： +```html +{{if !empty($plugins_view_admin_goods_save_data) and is_array(...)}} + {{foreach $plugins_view_admin_goods_save_data as $hook}} + {{$hook|raw}} + {{/foreach}} +{{else /}} + {{:ModuleInclude('public/not_data')}} +{{/if}} +``` + +**注入内容受限于 `
` 容器内**，外层 `
`、Tab 导航、商品名称/分类等核心字段无法被替换。 + +### 替代方案：模板文件覆盖 + +`MyView()` 函数（common.php:984-991）支持主题覆盖插件文件： +```php +if(substr($view, 0, 16) == '../../../plugins') { + $plugins_view_file = APP_PATH.$group.DS.'view'.DS.$theme.DS.'plugins'.DS.str_replace(...); + if(@file_exists($plugins_view_file)) { + $view = $plugins_view_file; // 主题文件覆盖插件文件 + } +} +``` +但这只对 `plugins` 控制器的路径生效。SaveInfo() 是 `admin/goods` 路径，无法利用此机制。 + +**唯一可行的完全替换路径**：将 `saveinfo.html` 复制到 `app/admin/view/default/goods/saveinfo.html`（ShopXO 默认主题目录），然后修改。但这是覆盖核心文件，升级 ShopXO 时会丢失。 + +**推荐：不要完全替换页面。** 改为在 base tab 内注入 ticket 专属表单，或添加新的 tab 项。 + +--- + +### Q2-C: Save() 数据接收方式 + +**Goods::Save()（Goods.php:187-192）**： +```php +public function Save() { + $params = $this->data_request; // ← ThinkPHP 标准请求数据（$_POST） + $params['admin'] = $this->admin; + return ApiService::ApiDataReturn(GoodsService::GoodsSave($params)); +} +``` + +数据源是 ThinkPHP 的 `$this->data_request`，等价于标准 `$_POST`。**任何自定义表单都可以 POST 到 `admin/goods/save`，只要字段名符合 GoodsService 期望。** + +**GoodsService::GoodsSave() 中钩子位置（GoodsService.php:1549-1565）**： +```php +// 构建 $data 数组（从 $params 提取 title, category_ids 等） +$data['title'] = $params['title'] ?? ''; +// ... 更多字段 ... + +// 商品保存处理钩子 — 在事务启动之前 +$ret = EventReturnHandle(MyEventTrigger('plugins_service_goods_save_handle', [ + 'params' => &$params, // 引用：可修改 + 'data' => &$data, // 引用：可修改（影响最终 INSERT/UPDATE） + 'spec' => &$specifications['data'], + 'goods_id' => isset($params['id']) ? intval($params['id']) : 0, +])); +if(isset($ret['code']) && $ret['code'] != 0) { + return $ret; // ← 钩子可提前返回，阻止标准保存 +} +``` + +**关键能力**： +1. `$data` 数组通过引用传入，插件可修改后影响最终 `INSERT/UPDATE` +2. 钩子返回 `['code'=>0, 'msg'=>'...', 'data'=>...]` 可阻止标准流程（直接返回） +3. 但 `$params['title']`、`$params['category_ids']` 等字段在钩子调用前已被提取进 `$data` + +**两条可行路径**： +- **路径A（推荐）**：在 `$data` 中填入最小必需字段（title、category_ids 等），让标准 INSERT 继续执行，插件在钩子内完成票务数据保存 +- **路径B**：钩子直接 `Db::startTrans()` 自己处理票务数据，然后 `return ['code'=>0]` 阻止标准流程 + +--- + +### Q2-D: 插件视图文件路径可行性 + +目前 `plugin.json` 中未注册 `plugins_view_admin_goods_save` 钩子（只有 `onOrderPaid`）。需要两步启用： + +1. **注册钩子**：在 `plugin.json` 添加： + ```json + "backend_hook": { + "plugins_view_admin_goods_save": ["\\app\\plugins\\vr_ticket\\hook\\AdminGoodsSave"] + } + ``` + +2. **实现 AdminGoodsSave.php**：返回 HTML 字符串，注入到 saveinfo.html 的 base tab + +**插件视图文件路径**：`plugins/vr_ticket/view/admin/goods/ticket_save.html`（如果走模板覆盖方案） + +--- + +### item_type 字段验证 + +- `goods` 表已存在 `item_type` 字段（EventListener.php:129），值包括 `'normal'` / `'ticket'` / `'physical'` +- 前台 `app/index/controller/Goods.php:139` 已有 `item_type == 'ticket'` 判断 +- **后台 `Goods.php`（admin控制器）中不存在 `item_type` 判断逻辑**——任务描述中关于此判断存在于后台的说法需要更正 + +--- + +## Q1：JSON 编辑器复杂度评估（FrontendDev） + +> [待 FrontendDev 填写] + +--- + +## 最终推荐 + +### 推荐方案：**插件钩子注入 + JSON Schema 编辑器** + +**理由**： + +1. **完全替换方案不可行**：`plugins_view_admin_goods_save` 是注入点而非替换点，位于 base tab 的 `
` 内。要完全替换需覆盖核心 `saveinfo.html`，代价是失去 ShopXO 升级兼容性。 + +2. **注入 + 隐藏策略可行但脆弱**：通过钩子注入 HTML + JS 隐藏标准字段能实现视觉替换，但依赖 DOM 操作，维护成本高。 + +3. **Save() 数据流完全可控**：`plugins_service_goods_save_handle` 钩子在事务前以引用方式接收 `$data`，插件有两条清晰路径（填最小字段走标准流，或自行处理返回）完成数据保存。 + +4. **与 JSON 编辑器方案互补**：插件钩子注入 ticket 专属表单 + JSON Schema 编辑器处理 `seat_map` 嵌套数据，可以完整覆盖票务商品编辑场景。 + +### 备选：走插件独立路由 + +即 `admin/goods/saveinfo` → 重定向到插件自己的控制器方法（如 `plugins/vr_ticket/admin/goods/Save`），完全独立实现票务编辑器。不经过 ShopXO 的 `Goods::SaveInfo()` 和 `Goods::Save()`，干净隔离但需要开发独立的菜单和控制器。 + +--- + +## 总结 + +| 维度 | 完全替换 | 钩子注入（推荐） | 独立路由 | +|------|----------|----------------|----------| +| 实现复杂度 | 高（需覆盖核心模板） | 中 | 高（重写全套） | +| 升级兼容性 | 差 | 好 | 好 | +| 数据保存可控性 | 好 | 好 | 好 | +| 开发工作量 | ~2人天 | ~1人天 | ~3人天 |