diff --git a/docs/tasks/T-694.md b/docs/tasks/T-694.md new file mode 100644 index 0000000..962bdd8 --- /dev/null +++ b/docs/tasks/T-694.md @@ -0,0 +1,62 @@ +--- +id: T-694 +title: 商品套图拖放源项二次删除修复 +phase: 7 +deps: [T-693] +status: TODO +created: 2026-07-23 +--- + +## 问题 / 背景 + +T-693 将排序信号延后到 `super().startDrag(Qt.MoveAction)` 返回后,解决了拖拽期间同步刷新列表的问题,但真实 GUI 连续把参考图拖到第一位时,仍会按照拖动顺序逐张从商品原图区域消失,最终只剩下未被拖动的主图。 + +现场 SQLite 再次确认原图记录、状态和本地文件均完整,列表消失没有写入数据库。真实根因是当前代码同时使用两套移动所有权:自定义 `dropEvent()` 已通过 `takeItem()` + `insertItem()` 手动移动源项,而 `QAbstractItemView.startDrag()` 在 `MoveAction` 完成后还会按 Qt 内建 model/view 语义清理移动源项。一次拖拽因此发生“手动移动一次 + Qt 源端删除一次”,每轮少一个 GUI 项。 + +T-693 的测试直接调用 `_begin_internal_drag()`、`_move_dragged_item_to()`、`_finish_internal_drag()`,绕过了真实的 `super().startDrag()`,所以没有覆盖 Qt 返回 MoveAction 后的源端清理,形成了错误的全绿结果。 + +## 方案 + +### 1. 单一拖拽所有权 + +- `ProductOriginalList.startDrag()` 不再调用 `super().startDrag(Qt.MoveAction)`。 +- 由列表自行创建 `QDrag`,source 固定为当前列表;MIME 只承载当前单张原图的内部拖拽数据,拖拽缩略图作为可选视觉反馈,并以 `Qt.MoveAction` 执行。 +- 自定义 `dropEvent()` 继续是唯一改变列表顺序的位置,只调用一次受控 `takeItem()` + `insertItem()`;不交给 Qt 默认 model drop,也不允许 Qt 再删除移动源项。 +- `QDrag.exec()` 返回后清理拖拽上下文,再按 T-693 的快照规则提交一次唯一顺序。 + +### 2. 失败自动恢复 + +- 拖拽开始快照、待提交顺序和结束时列表必须数量相等、ID 唯一且集合一致。 +- 若拖拽结束时列表数量或 ID 集合异常,不写库;发送独立恢复信号,让 `ProductSuiteTab` 从 SQLite 重新加载当前商品原图,并显示“原图排序未完成,已恢复原顺序,请重试”。 +- 恢复信号不得复用 `orderChanged`,避免把残缺或旧顺序误写数据库;切换任务、数据库异常或页面已关闭时保持无破坏失败。 +- SQLite `reorder_original_assets()` 的完整集合、重复 ID 和项目归属校验继续作为服务层最后防线。 + +### 3. 覆盖真实 startDrag 路径 + +- 测试必须调用 `ProductOriginalList.startDrag()`,用可控 `QDrag` 替身在 `exec()` 中触发一次内部移动;验证没有调用基类 `startDrag()` 的二次删除语义,结束后图片数量和 ID 集合不变且只发送一次 `orderChanged`。 +- 连续把参考1、参考2、参考3、参考4移到第一位,每轮都经过完整自定义 `startDrag()` 生命周期,并验证 SQLite 顺序、缩略图数量和“主图 / 参考N”连续标签。 +- 模拟 `QDrag.exec()` 返回前列表被意外移除一项,验证不发送 `orderChanged`、发送一次恢复信号,页面从 SQLite 恢复全部原图。 +- 保留原地释放、取消拖拽、重复 ID、排序锁定和外部文件 URL 拖入回归。 + +## 验收要点 + +- 连续把多张参考图拖到第一位,商品原图数量始终不变,不再按拖动顺序消失。 +- 每次有效拖拽只移动一个现有项,不复制、不删除;结束后 GUI 与 SQLite 的图片 ID 集合及顺序一致。 +- 拖拽过程中若出现任何数量或 ID 异常,界面自动恢复数据库顺序,不把异常状态保存。 +- 原图不少于两张时始终存在“参考1”,所有参考图按当前位置连续编号。 +- 自动验证: + - `py -3.10 -m unittest tests.test_product_suite_gui tests.test_image_studio tests.test_product_suite` + - `py -3.10 -m unittest discover -s tests` + - `py -3.10 -m ruff check app tests main.py` + - `py -3.10 -m compileall app main.py` + - `git diff --check` + +## 边界(不改什么) + +- 不改 SQLite schema、图片文件、原图删除功能、生成任务、提示词、cmhub、蝦皮 CDP 或①至⑤模块。 +- 不改外部文件拖入和剪贴板导入;内部 MIME 只用于当前列表单张图片排序,不接受跨列表或跨商品拖放。 +- 不以 `CopyAction` 伪装移动来规避 Qt 清理,不使用固定延时,不放宽服务层数据完整性校验。 + +## 执行记录 + +- 待实现。