From 8706151e23b2b0a92958a3ee2283f2538d91fee8 Mon Sep 17 00:00:00 2001 From: chengma Date: Thu, 23 Jul 2026 09:36:00 +0800 Subject: [PATCH] fix(product-suite): finalize sorting after drag --- app/gui/tabs/product_suite.py | 57 +++++++++++++++++++++---- docs/04-architecture.md | 2 +- docs/routes.md | 2 +- docs/tasks/T-693.md | 8 +++- tests/test_product_suite_gui.py | 76 ++++++++++++++++++++++++++++++--- 5 files changed, 127 insertions(+), 18 deletions(-) diff --git a/app/gui/tabs/product_suite.py b/app/gui/tabs/product_suite.py index 01002cd..073b10b 100644 --- a/app/gui/tabs/product_suite.py +++ b/app/gui/tabs/product_suite.py @@ -310,6 +310,8 @@ class ProductOriginalList(QListWidget): self.setDragDropMode(QListWidget.InternalMove) self.setDefaultDropAction(Qt.MoveAction) self._dragged_item = None + self._drag_start_ids = None + self._pending_order = None self.setContextMenuPolicy(Qt.CustomContextMenu) self.customContextMenuRequested.connect(self._show_menu) self.itemChanged.connect(lambda item: self.checkedAssetsChanged.emit(self.checked_asset_ids())) @@ -398,31 +400,68 @@ class ProductOriginalList(QListWidget): if event.source() is not self: event.ignore() return - self._move_dragged_item_to(event.position().toPoint()) + if not self._move_dragged_item_to(event.position().toPoint()): + event.ignore() + return event.setDropAction(Qt.MoveAction) event.accept() def startDrag(self, supported_actions): item = self.currentItem() + if not self._begin_internal_drag(item): + return + try: + super().startDrag(Qt.MoveAction) + finally: + self._finish_internal_drag() + + def _begin_internal_drag(self, item): if ( not self._reordering_enabled or item is None or item.data(Qt.UserRole) is None ): - return + return False + asset_ids = self.asset_ids() + if len(asset_ids) != len(set(asset_ids)): + return False self._dragged_item = item - try: - super().startDrag(Qt.MoveAction) - finally: - self._dragged_item = None + self._drag_start_ids = asset_ids + self._pending_order = None + return True + + def _finish_internal_drag(self): + pending_order = self._pending_order + drag_start_ids = self._drag_start_ids + self._dragged_item = None + self._drag_start_ids = None + self._pending_order = None + if pending_order is None or drag_start_ids is None: + return False + current_ids = self.asset_ids() + if ( + current_ids != pending_order + or len(pending_order) != len(set(pending_order)) + or len(pending_order) != len(drag_start_ids) + or set(pending_order) != set(drag_start_ids) + ): + return False + self.orderChanged.emit(list(pending_order)) + return True def _move_dragged_item_to(self, position): item = self._dragged_item - if item is None: + drag_start_ids = self._drag_start_ids + if item is None or drag_start_ids is None: return False source_row = self.row(item) before_ids = self.asset_ids() - if source_row < 0 or len(before_ids) != len(set(before_ids)): + if ( + source_row < 0 + or len(before_ids) != len(set(before_ids)) + or len(before_ids) != len(drag_start_ids) + or set(before_ids) != set(drag_start_ids) + ): return False target = self.itemAt(position) if target is None: @@ -446,7 +485,7 @@ class ProductOriginalList(QListWidget): return False if after_ids == before_ids: return False - self.orderChanged.emit(after_ids) + self._pending_order = after_ids return True def resizeEvent(self, event): diff --git a/docs/04-architecture.md b/docs/04-architecture.md index 0287035..0d3fe8c 100644 --- a/docs/04-architecture.md +++ b/docs/04-architecture.md @@ -461,7 +461,7 @@ data/images///__new. # AI 生成的新 - 套图比例输出固定映射为 `1:1 -> 1024x1024`、`3:4/9:16 -> 1024x1536`、`4:3/16:9 -> 1536x1024`。`image_studio_assets.aspect_ratio` 保持用户原选项，新增 `requested_output_size`、`rendered_width`、`rendered_height` 记录请求尺寸与真实像素；旧记录保持空值，SQLite 迁移仅附加且幂等。服务返回近似比例结构化结果，T-679b 在确认界面呈现。 - 商品套图的生图来源在 T-679 组全部验收前不得进入发布版本；图片理解「AI帮写」仍仅使用默认网关 `vision_alias`。 - T-658a 后商品套图异步生图提交统一使用 `images` 数组，单图也使用单元素 `{"image_base64": ...}`，不再提交顶层 `image_base64`。提交层最多传 8 张本地图片，单图原文件上限10MiB、编码后总输入上限32MiB；超限在请求前以中文错误阻断，后续多图参考的资产快照和提示词语义由 T-658b/T-658c 负责。 -- T-658b 后 `image_studio_jobs.reference_asset_ids` 以有序 JSON 图片 ID 列表冻结每个商品套图 job 的参考图；新一轮未勾选逐图主图时，第1张为主图、后续最多7张写入快照，勾选时写空数组。历史 `NULL` 行继续按单图任务处理。商品原图 UI 的第1张明确显示「主图」，内部排序由列表记录单一拖拽源、按 drop 坐标计算插入行，并以 `takeItem()` + `insertItem()` 移动同一个项；不得调用 Qt 默认的“投放到图片上”复制路径。最终 ID 顺序唯一且确有变化时才由事务更新 `source_order`，排序只影响之后新建的一轮 job，不回写既有 job 快照。外部文件 URL 继续走导入。恢复或重试只读 job 快照，不回读当前原图列表；参考图资产或本地文件缺失时提交前失败，不静默减少提交数量。 +- T-658b 后 `image_studio_jobs.reference_asset_ids` 以有序 JSON 图片 ID 列表冻结每个商品套图 job 的参考图；新一轮未勾选逐图主图时，第1张为主图、后续最多7张写入快照，勾选时写空数组。历史 `NULL` 行继续按单图任务处理。商品原图 UI 的第1张明确显示「主图」，内部排序由列表记录单一拖拽源、按 drop 坐标计算插入行，并以 `takeItem()` + `insertItem()` 移动同一个项；不得调用 Qt 默认的“投放到图片上”复制路径。`dropEvent()` 只记录待提交顺序，必须等 `QDrag` 返回并清理拖拽上下文后才发送一次排序信号，避免写库后的列表重建与仍在运行的拖拽事务重入。最终 ID 顺序唯一且确有变化时才由事务更新 `source_order`，排序只影响之后新建的一轮 job，不回写既有 job 快照。外部文件 URL 继续走导入。恢复或重试只读 job 快照，不回读当前原图列表；参考图资产或本地文件缺失时提交前失败，不静默减少提交数量。 - T-658d 后逐图主图 checkbox 右侧显示可换行中文 helper，说明多SKU应逐图生成、同商品多角度应作为参考图一同提交。常规生成确认框从最终 specs 计算主图/参考图数量；未开启逐图主图且可用原图超过8张时，明确第1张主图加前7张参考图的上限和忽略数量。确认总点数仍只按 job 请求数计算，不因单请求图片数量重复计算。 - T-647 后商品套图中的「AI帮写」不是②标题生成的复用入口：它使用 `vision_alias` 调 `POST /api/v1/analyze/images`，把当前项目中1至8张可用本地原图按 `source_order` 在一次请求中发送；超过8张时明确只使用前8张，原图勾选状态不改变该输入集合。视觉提示词将这些图片定义为同一商品项目的多角度、细节、包装或场景证据集，要求产出一份商品级联合分析，不按“图1/图2”逐图罗列；结果按商品概述、可确认卖点、适用人群与场景、套图画面要求、待确认或避免编造的信息组织，并把可见差异明确为待确认项，不强行合并为事实。单图上限10MiB、总计上限32MiB，缺图、未下载、超限或未配置别名时不发请求且保留用户现有卖点。读取等待固定120秒、连接等待沿用设置；读超时或网络中断只提示“结果未确认”，不自动重发。成功只显示图片张数、扣点和余额，不显示路径、base64、接口URL、完整提示词或上游原始响应；取消只在请求前/返回后协作生效，不强杀网络线程。 - `product_suite.render_product_suite_prompt()` 是弹窗预览和真实生成的唯一 renderer；`build_job_specs()` 建立本轮 specs 前只读取一次用户模板并冻结，每个 job 保存最终完整 prompt，运行中修改模板只影响下一轮。新模板使用 `{生成目标}`，其值为内置分类固定目标描述或含实际名称的“生成自定义分类图片：分类名称。”；同时保留 `{套图名称}`、`{补充描述}` 及平台/地区/语言/比例、可选商品ID/主参考图序号、参考图规则、商品卖点和四个只读规则。旧模板必须同时包含 `{套图名称}` 与 `{补充描述}` 才能按兼容路径继续使用；必需变量缺失、未知/未闭合花括号、只读规则变量未独占一行都视为无效。模板校验仅针对模板文本；替换值是业务数据，商品卖点与要求、分类名等可原样包含 `{}`，不再二次作为变量解析。`product_suite_reference_rule()` 是 `{参考图规则}` 的唯一事实来源，变量值都以 `参考图规则:` 开头：逐图主图开启时为单图主参考规则；未开启时固定为“使用第一張上傳圖作為主商品圖，其餘圖片只作為參考。”，不随实际参考图数量变化。提示词弹窗与 job 共用 renderer；实际提交的参考图数量仍由冻结的 `reference_asset_ids` 决定。默认模板独占一行，不额外重复标签或序号。四个只读规则覆盖尺寸与长图（含禁止多宫格拼接）、政治标识、价格和尺码；商品主体一致性与禁止编造并入参考图规则。比例仍同时传入 `image_studio_generation.run_jobs(aspect_ratio=...)`，进入 cmhub 请求与输出资产元数据。 diff --git a/docs/routes.md b/docs/routes.md index 0748354..08422fc 100644 --- a/docs/routes.md +++ b/docs/routes.md @@ -217,7 +217,7 @@ - 二级套图任务标签使用独立紧凑样式，不继承主模块 Tab 的大尺寸点击区。上下文栏左侧集中「历史生成 / 添加图片」，右侧集中账号、商品 ID 和拉取入口；常见 11～13 位商品 ID 不得裁切，长账号可通过 tooltip 查看完整名称。内部生成目录不提供顶部直达入口，正式取图统一通过历史窗口的「导出本轮」选择外部目录。 - 项目仍以 `账号别名 + 商品ID` 唯一，复用 `image_studio_projects/assets/jobs`。商品ID输入框只接受数字；提交时清除不可见格式字符，失焦、拉取、添加原图和生成共用提交校验。非数字输入只提示并保留在输入框，不覆盖当前已绑定项目；合法切换商品继续弹窗确认。`suite_settings_json` 保存平台、国家地区、语言、比例、逐图主图模式和分类数量；卖点文本继续使用 `draft_prompt`，用户停止输入约 500ms 后自动保存，切换任务、关闭任务或程序前同步补保存。未创建项目时只保留在当前任务内存，不因输入文字自动创建临时草稿。 - 「商品卖点与要求」关闭内部横向和纵向滚动条，输入框按完整换行内容自然增高；文本删除后可缩回最低高度，页面过长时统一由左侧 `suiteConfigScroll` 滚动。 -- 商品原图最多16张。前6个槽位固定显示主图与参考1～5；列表关闭内部滚动条，按可用宽度换行并自然向下展开，由左侧配置区统一滚动。支持文件选择、外部拖入、剪贴板粘贴和列表内排序；内部排序只移动当前拖拽的同一个图片项，不走 Qt 默认的投放复制行为。第1张以「主图」文字、粗体和语义色标识，拖动任意图片到第一位即可设为主图，排序成功提示“已更新商品原图顺序，第一张图片为主图”。最终顺序必须唯一且实际变化才回写；外部文件拖入只添加图片。历史失效远程图不占有效名额；第1张是主参考图。 +- 商品原图最多16张。前6个槽位固定显示主图与参考1～5；列表关闭内部滚动条，按可用宽度换行并自然向下展开，由左侧配置区统一滚动。支持文件选择、外部拖入、剪贴板粘贴和列表内排序；内部排序只移动当前拖拽的同一个图片项，不走 Qt 默认的投放复制行为。第1张以「主图」文字、粗体和语义色标识，拖动任意图片到第一位即可设为主图，排序成功提示“已更新商品原图顺序，第一张图片为主图”。拖放期间只记录待提交顺序，待 Qt 拖拽完全结束后才写库并刷新，连续换主图时不得清除仍被拖拽持有的列表项。最终顺序必须唯一且实际变化才回写；外部文件拖入只添加图片。历史失效远程图不占有效名额；第1张是主参考图。 - 每张真实原图左上角提供独立勾选框，标题行显示「已选 N 张 / 全选 / 反选」；添加占位图不参与选择。勾选只在当前任务界面内临时保留，普通刷新和排序按资产 ID 保留，切换任务或删除成功后清空。右键或 Delete 可批量移除，确认框说明准确数量、主图变化及非破坏性边界；生成中或勾选项仍在下载时整批阻断。删除会继续保护终选直接引用、运行中和可继续查询/重试 job 的主图及参考图；已成功且不可重试的历史 job 不阻断删除，删除后可按剩余原图开始新一轮生成。移除只删除当前项目的本地资产记录，不删除本地源文件或蝦皮线上图片。 - 「拉取蝦皮主图」复用只读 CDP，读取 URL 后由最多2个下载 worker 后台落盘；不改标题/封面、不拖拽、不点击更新。拉取、下载期间其余界面和其他任务仍可操作，但商品原图列表仅锁定内部拖拽排序，并用 tooltip 说明“正在拉取蝦皮主图，完成后可调整图片顺序”；文件拖入、预览等非冲突操作不因此整页禁用。 - 套图按分类配置生成图片，固定分类顺序为白底图、场景图、模特场景图、细节说明图、卖点图，之后是自定义分类；默认数量为白底图1、场景图2、模特场景图0、细节说明图0、卖点图2。新增分类默认数量为0，用户未主动配置时不增加生成任务或点数消耗。自定义分类名称非空、无空格、最多10字且不可重名。逐图主图开启后，白底图只生成一次，其余分类按每张有效原图展开。 diff --git a/docs/tasks/T-693.md b/docs/tasks/T-693.md index 47cb715..0a1f37b 100644 --- a/docs/tasks/T-693.md +++ b/docs/tasks/T-693.md @@ -3,7 +3,7 @@ id: T-693 title: 商品套图连续拖放丢失显示修复 phase: 7 deps: [T-692] -status: TODO +status: DONE created: 2026-07-23 --- @@ -58,4 +58,8 @@ T-692 已避免 Qt 默认内部投放复制同一图片项，但用户连续把 ## 执行记录 -- 待实现。 +- 2026-07-23 完成。 +- `ProductOriginalList` 新增单轮拖拽生命周期状态：开始时冻结原图 ID 集合，`dropEvent()` 仅以受控移动更新列表并保存待提交顺序，无效投放直接拒绝；`super().startDrag()` 返回后先清除旧拖拽项引用，再发送一次 `orderChanged`。因此页面写库和 `_refresh_originals()` 清空重建列表时，Qt 已不再持有上一轮拖拽源。 +- 提交前同时校验当前列表、待提交顺序和拖拽开始快照的数量、唯一性与 ID 集合；原地释放、无拖拽源、重复 ID、列表中途变化和排序锁定均不提交。SQLite 服务层的完整集合与重复 ID 校验保持不变。 +- 扩展列表级测试，确认移动发生时不会提前发信号，拖拽结束才发送一次；新增项目级连续换主图回归，依次把4张参考图移到第一位，每轮确认5张原图全部保留、标签连续、数据库在结束拖拽前不变且结束后与界面一致。同步更新 `docs/04-architecture.md` 与 `docs/routes.md`。 +- 验证通过：`py -3.10 -m unittest tests.test_product_suite_gui tests.test_image_studio tests.test_product_suite`（109项）、`py -3.10 -m unittest discover -s tests`（676项）、`py -3.10 -m ruff check app tests main.py`、`py -3.10 -m compileall app main.py`。离屏 PySide6 仍有字体目录和插件能力告警，但无测试失败。 diff --git a/tests/test_product_suite_gui.py b/tests/test_product_suite_gui.py index d33d04a..4adeafd 100644 --- a/tests/test_product_suite_gui.py +++ b/tests/test_product_suite_gui.py @@ -2934,37 +2934,103 @@ class ProductSuiteGuiTests(TempDirMixin, unittest.TestCase): changes = [] original_list.orderChanged.connect(changes.append) - original_list._dragged_item = original_list.item(1) + self.assertTrue(original_list._begin_internal_drag(original_list.item(1))) first_rect = original_list.visualItemRect(original_list.item(0)) self.assertTrue(original_list._move_dragged_item_to(first_rect.center())) self.assertEqual([12, 11, 13], original_list.asset_ids()) + self.assertEqual([], changes) + self.assertTrue(original_list._finish_internal_drag()) self.assertEqual([[12, 11, 13]], changes) self.assertEqual(3, len(set(original_list.asset_ids()))) - original_list._dragged_item = original_list.item(0) + self.assertTrue(original_list._begin_internal_drag(original_list.item(0))) append_position = original_list.viewport().rect().bottomRight() - QPoint(2, 2) self.assertTrue(original_list._move_dragged_item_to(append_position)) self.assertEqual([11, 13, 12], original_list.asset_ids()) + self.assertEqual([[12, 11, 13]], changes) + self.assertTrue(original_list._finish_internal_drag()) self.assertEqual([[12, 11, 13], [11, 13, 12]], changes) - original_list._dragged_item = original_list.item(1) + self.assertTrue(original_list._begin_internal_drag(original_list.item(1))) same_rect = original_list.visualItemRect(original_list.item(1)) self.assertFalse(original_list._move_dragged_item_to(same_rect.center())) + self.assertFalse(original_list._finish_internal_drag()) self.assertEqual([11, 13, 12], original_list.asset_ids()) self.assertEqual([[12, 11, 13], [11, 13, 12]], changes) - original_list._dragged_item = None self.assertFalse(original_list._move_dragged_item_to(first_rect.center())) self.assertEqual([11, 13, 12], original_list.asset_ids()) duplicate = QListWidgetItem("重复原图") duplicate.setData(Qt.UserRole, 12) original_list.addItem(duplicate) - original_list._dragged_item = original_list.item(0) + self.assertFalse(original_list._begin_internal_drag(original_list.item(0))) self.assertFalse(original_list._move_dragged_item_to(first_rect.center())) self.assertEqual(4, original_list.count()) self.assertEqual([[12, 11, 13], [11, 13, 12]], changes) + original_list.takeItem(3) + original_list.set_reordering_enabled(False) + self.assertFalse(original_list._begin_internal_drag(original_list.item(0))) + + def test_original_list_repeated_primary_moves_keep_every_asset_and_label(self): + with self.make_temp_dir() as temp_dir: + config = self._config(temp_dir) + project, assets = self._create_project_with_assets(temp_dir, config, 5) + tab = ProductSuiteTab(config=config, db_path=config["db_path"]) + self.addCleanup(tab.close) + state = tab._displayed_state + state.account_alias = "alias-a" + state.item_id = "51100639510" + state.project_id = project.id + tab._load_state(state) + tab.resize(1180, 760) + tab.show() + self.app.processEvents() + + original_ids = {asset.id for asset in assets} + expected_labels = ["主图", "参考1", "参考2", "参考3", "参考4"] + for asset in assets[1:]: + row = tab.original_list.asset_ids().index(asset.id) + self.assertTrue( + tab.original_list._begin_internal_drag(tab.original_list.item(row)) + ) + first_rect = tab.original_list.visualItemRect(tab.original_list.item(0)) + self.assertTrue( + tab.original_list._move_dragged_item_to(first_rect.center()) + ) + database_ids_before_finish = [ + stored.id + for stored in image_studio.list_assets( + project.id, + kind=image_studio.ASSET_KIND_ORIGINAL, + path=config["db_path"], + ) + ] + self.assertNotEqual(tab.original_list.asset_ids(), database_ids_before_finish) + + self.assertTrue(tab.original_list._finish_internal_drag()) + visible_ids = tab.original_list.asset_ids() + database_ids = [ + stored.id + for stored in image_studio.list_assets( + project.id, + kind=image_studio.ASSET_KIND_ORIGINAL, + path=config["db_path"], + ) + ] + self.assertEqual(5, tab.original_list.count()) + self.assertEqual(original_ids, set(visible_ids)) + self.assertEqual(5, len(set(visible_ids))) + self.assertEqual(asset.id, visible_ids[0]) + self.assertEqual(visible_ids, database_ids) + self.assertEqual( + expected_labels, + [tab.original_list.item(row).text() for row in range(5)], + ) + + self.assert_removed(temp_dir) + def test_temporary_draft_allows_local_work_but_blocks_shopee_pull_and_recovers(self): with self.make_temp_dir() as temp_dir: config = self._config(temp_dir)