diff --git a/app/gui/tabs/product_suite.py b/app/gui/tabs/product_suite.py index 8baddd4..01002cd 100644 --- a/app/gui/tabs/product_suite.py +++ b/app/gui/tabs/product_suite.py @@ -309,6 +309,7 @@ class ProductOriginalList(QListWidget): self.setDropIndicatorShown(True) self.setDragDropMode(QListWidget.InternalMove) self.setDefaultDropAction(Qt.MoveAction) + self._dragged_item = None self.setContextMenuPolicy(Qt.CustomContextMenu) self.customContextMenuRequested.connect(self._show_menu) self.itemChanged.connect(lambda item: self.checkedAssetsChanged.emit(self.checked_asset_ids())) @@ -366,6 +367,10 @@ class ProductOriginalList(QListWidget): if event.mimeData().hasUrls(): event.acceptProposedAction() return + if event.source() is self and self._reordering_enabled and self._dragged_item is not None: + event.setDropAction(Qt.MoveAction) + event.accept() + return super().dragEnterEvent(event) def dragMoveEvent(self, event): @@ -375,6 +380,10 @@ class ProductOriginalList(QListWidget): if not self._reordering_enabled: event.ignore() return + if event.source() is self and self._dragged_item is not None: + event.setDropAction(Qt.MoveAction) + event.accept() + return super().dragMoveEvent(event) def dropEvent(self, event): @@ -386,12 +395,59 @@ class ProductOriginalList(QListWidget): if not self._reordering_enabled: event.ignore() return - internal_move = event.source() is self - if internal_move: - event.setDropAction(Qt.MoveAction) - super().dropEvent(event) - if internal_move and event.isAccepted() and event.dropAction() == Qt.MoveAction: - self.orderChanged.emit(self.asset_ids()) + if event.source() is not self: + event.ignore() + return + self._move_dragged_item_to(event.position().toPoint()) + event.setDropAction(Qt.MoveAction) + event.accept() + + def startDrag(self, supported_actions): + item = self.currentItem() + if ( + not self._reordering_enabled + or item is None + or item.data(Qt.UserRole) is None + ): + return + self._dragged_item = item + try: + super().startDrag(Qt.MoveAction) + finally: + self._dragged_item = None + + def _move_dragged_item_to(self, position): + item = self._dragged_item + if item 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)): + return False + target = self.itemAt(position) + if target is None: + target_row = self.count() + else: + target_row = self.row(target) + if position.x() > self.visualItemRect(target).center().x(): + target_row += 1 + if target_row > source_row: + target_row -= 1 + moved_item = self.takeItem(source_row) + target_row = max(0, min(target_row, self.count())) + self.insertItem(target_row, moved_item) + self.setCurrentItem(moved_item) + moved_item.setSelected(True) + after_ids = self.asset_ids() + if len(after_ids) != len(set(after_ids)): + self.takeItem(target_row) + self.insertItem(source_row, moved_item) + self.setCurrentItem(moved_item) + return False + if after_ids == before_ids: + return False + self.orderChanged.emit(after_ids) + return True def resizeEvent(self, event): super().resizeEvent(event) @@ -3389,9 +3445,7 @@ class ProductSuiteTab(QWidget): for index, asset in enumerate(assets, 1): label = "主图" if index == 1 else "参考%d" % (index - 1) item = QListWidgetItem(label) - item.setFlags( - item.flags() | Qt.ItemIsDragEnabled | Qt.ItemIsDropEnabled - ) + item.setFlags(item.flags() | Qt.ItemIsDragEnabled) if index == 1: font = QFont(item.font()) font.setBold(True) diff --git a/docs/04-architecture.md b/docs/04-architecture.md index 3cc5a91..0287035 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张明确显示「主图」，每个真实原图项同时具备内部拖动和放置能力，列表强制使用 `Qt.MoveAction`；用户拖动图片到第一位后由事务更新 `source_order`，排序只影响之后新建的一轮 job，不回写既有 job 快照。内部 drop 未被 Qt 接受或属于外部文件 URL 时不得触发排序回写，外部 URL 继续走导入。恢复或重试只读 job 快照，不回读当前原图列表；参考图资产或本地文件缺失时提交前失败，不静默减少提交数量。 +- 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-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 2da983b..0748354 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；列表关闭内部滚动条，按可用宽度换行并自然向下展开，由左侧配置区统一滚动。支持文件选择、外部拖入、剪贴板粘贴和列表内排序；每张真实图片都是内部拖放的合法放置目标，列表固定以移动方式排序。第1张以「主图」文字、粗体和语义色标识，拖动任意图片到第一位即可设为主图，排序成功提示“已更新商品原图顺序，第一张图片为主图”。未被接受的内部 drop 不回写顺序；外部文件拖入只添加图片。历史失效远程图不占有效名额；第1张是主参考图。 +- 商品原图最多16张。前6个槽位固定显示主图与参考1～5；列表关闭内部滚动条，按可用宽度换行并自然向下展开，由左侧配置区统一滚动。支持文件选择、外部拖入、剪贴板粘贴和列表内排序；内部排序只移动当前拖拽的同一个图片项，不走 Qt 默认的投放复制行为。第1张以「主图」文字、粗体和语义色标识，拖动任意图片到第一位即可设为主图，排序成功提示“已更新商品原图顺序，第一张图片为主图”。最终顺序必须唯一且实际变化才回写；外部文件拖入只添加图片。历史失效远程图不占有效名额；第1张是主参考图。 - 每张真实原图左上角提供独立勾选框，标题行显示「已选 N 张 / 全选 / 反选」；添加占位图不参与选择。勾选只在当前任务界面内临时保留，普通刷新和排序按资产 ID 保留，切换任务或删除成功后清空。右键或 Delete 可批量移除，确认框说明准确数量、主图变化及非破坏性边界；生成中或勾选项仍在下载时整批阻断。删除会继续保护终选直接引用、运行中和可继续查询/重试 job 的主图及参考图；已成功且不可重试的历史 job 不阻断删除，删除后可按剩余原图开始新一轮生成。移除只删除当前项目的本地资产记录，不删除本地源文件或蝦皮线上图片。 - 「拉取蝦皮主图」复用只读 CDP，读取 URL 后由最多2个下载 worker 后台落盘；不改标题/封面、不拖拽、不点击更新。拉取、下载期间其余界面和其他任务仍可操作，但商品原图列表仅锁定内部拖拽排序，并用 tooltip 说明“正在拉取蝦皮主图，完成后可调整图片顺序”；文件拖入、预览等非冲突操作不因此整页禁用。 - 套图按分类配置生成图片，固定分类顺序为白底图、场景图、模特场景图、细节说明图、卖点图，之后是自定义分类；默认数量为白底图1、场景图2、模特场景图0、细节说明图0、卖点图2。新增分类默认数量为0，用户未主动配置时不增加生成任务或点数消耗。自定义分类名称非空、无空格、最多10字且不可重名。逐图主图开启后，白底图只生成一次，其余分类按每张有效原图展开。 diff --git a/docs/tasks/T-692.md b/docs/tasks/T-692.md index 60511a1..b64d822 100644 --- a/docs/tasks/T-692.md +++ b/docs/tasks/T-692.md @@ -3,7 +3,7 @@ id: T-692 title: 商品套图原图拖放重复项修复 phase: 7 deps: [T-691] -status: TODO +status: DONE created: 2026-07-23 --- @@ -54,4 +54,9 @@ T-691 为使参考图可以拖到第一位，给每个 `QListWidgetItem` 添加 ## 执行记录 -- 待实现。 +- 2026-07-23 完成。 +- 移除 T-691 对真实原图项追加的 `ItemIsDropEnabled`，避免 Qt 将拖到缩略图上的动作走默认复制/插入路径。`ProductOriginalList` 现在在 `startDrag()` 记录单一拖拽项，在内部 `dragEnter/dragMove/drop` 中只接受本列表来源的移动。 +- 内部 drop 不再调用 `super().dropEvent()`：按目标缩略图和坐标计算插入行，以 `takeItem()` + `insertItem()` 移动原 `QListWidgetItem`；移动前后均检查 asset ID 唯一性，仅顺序实际变化时发送一次 `orderChanged`。原地释放、无拖拽源、跨列表和已存在异常重复 ID 都不回写排序。 +- 外部文件 URL 拖入继续只发送 `filesDropped`；拉取或生成中的锁定逻辑、SQLite 服务层重复 ID 最后防线、主图刷新样式和历史 job 快照均未改动。已同步 `docs/04-architecture.md` 与 `docs/routes.md`。 +- 新增 GUI 回归覆盖第二张移到第一张、向后移动、原地释放、无拖拽源和已有重复 ID 防御；现有项目级测试继续验证 SQLite 排序、主图标签和外部文件导入隔离。 +- 验证通过：`py -3.10 -m unittest tests.test_product_suite_gui tests.test_image_studio tests.test_product_suite`（108项）、`py -3.10 -m unittest discover -s tests`（675项）、`py -3.10 -m ruff check app tests main.py`、`py -3.10 -m compileall app main.py`、`git diff --check`。离屏 PySide6 有字体目录/插件告警，但无测试失败。 diff --git a/tests/test_product_suite_gui.py b/tests/test_product_suite_gui.py index 17467b3..d33d04a 100644 --- a/tests/test_product_suite_gui.py +++ b/tests/test_product_suite_gui.py @@ -2890,7 +2890,7 @@ class ProductSuiteGuiTests(TempDirMixin, unittest.TestCase): item = tab.original_list.item(row) self.assertIsNotNone(item.data(ORIGINAL_CHECK_STATE_ROLE)) self.assertTrue(item.flags() & Qt.ItemIsDragEnabled) - self.assertTrue(item.flags() & Qt.ItemIsDropEnabled) + self.assertFalse(item.flags() & Qt.ItemIsDropEnabled) self.assertEqual(Qt.MoveAction, tab.original_list.defaultDropAction()) tab.select_all_originals_button.click() @@ -2921,6 +2921,50 @@ class ProductSuiteGuiTests(TempDirMixin, unittest.TestCase): self.assert_removed(temp_dir) + def test_original_list_moves_dragged_item_without_duplicate_asset_ids(self): + original_list = ProductOriginalList() + self.addCleanup(original_list.close) + original_list.setFixedSize(360, 120) + for asset_id in (11, 12, 13): + item = QListWidgetItem("原图%d" % asset_id) + item.setData(Qt.UserRole, asset_id) + original_list.addItem(item) + original_list.show() + self.app.processEvents() + changes = [] + original_list.orderChanged.connect(changes.append) + + original_list._dragged_item = 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([[12, 11, 13]], changes) + self.assertEqual(3, len(set(original_list.asset_ids()))) + + original_list._dragged_item = 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], [11, 13, 12]], changes) + + original_list._dragged_item = 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.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._move_dragged_item_to(first_rect.center())) + self.assertEqual(4, original_list.count()) + self.assertEqual([[12, 11, 13], [11, 13, 12]], changes) + 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)