Skip to content

fix: 修复库存操作按钮无响应 - #69

Open
dajiaohuang wants to merge 1 commit into
zhtyyx:mainfrom
dajiaohuang:main
Open

fix: 修复库存操作按钮无响应#69
dajiaohuang wants to merge 1 commit into
zhtyyx:mainfrom
dajiaohuang:main

Conversation

@dajiaohuang

@dajiaohuang dajiaohuang commented Jun 19, 2026

Copy link
Copy Markdown

调整说明

根据维护建议,本 PR 已收敛为一个独立改动:仅修复库存列表的入库、出库、调整按钮无响应。原 PR 中的商品导入导出、颜色/尺码管理、退货流程和 UserProfile 改动均已移除,后续如继续处理会分别讨论和提交。

修复内容

  • 将三个 href="#" 按钮连接到对应库存操作页面,并携带当前商品的 product_id
  • 入库和出库页面与已有调整页面保持一致,自动预选当前商品。
  • 补齐三个操作视图此前引用但缺失的表单模板,避免点击后返回 500。
  • 增加回归测试,覆盖三个按钮链接、三个操作页面、模板渲染和商品预选。

验证

  • python manage.py test inventory.tests.test_views.InventoryViewTest --verbosity 2:4 项通过。
  • python manage.py check:通过;仅有仓库既有的 static/ 目录缺失警告。
  • python manage.py makemigrations --check --dry-run:无迁移变化。
  • 完整 46 项测试:42 项通过,4 项既有失败;这 4 项已在未修改的最新 upstream/main 上单独复现,涉及盘点权限夹具、销售创建夹具和旧商品模板路径断言,与本 PR 无关。

Refs #34

@zhtyyx

zhtyyx commented Jun 21, 2026

Copy link
Copy Markdown
Owner

@dajiaohuang 方便加个联系方式聊一下吗, zhtyyx@gmail.com

@zhtyyx

zhtyyx commented Jun 21, 2026

Copy link
Copy Markdown
Owner

感谢提交 PR,这里面确实有一些有价值的修复,比如库存入库/出库/调整按钮、商品导入导出、退货、用户扩展信息等方向都能对应现有 issue。
但这个 PR 目前不适合直接合并,主要原因是改动范围太大,把多个互不相关的功能和修复放在了一起:Issue #34、Issue #23、退货流程、UserProfile、颜色/尺码管理、商品导入导出等都在同一个 PR 里,后续 review、回滚和定位问题都会比较困难。

另外其中有几处高风险点还需要进一步处理:

  1. 退货会修改库存、会员余额、积分、消费次数和销售单状态,属于财务/库存核心路径,目前缺少对应测试,也缺少更明确的权限控制,不能只依赖登录态。
  2. 商品导入页里的示例 CSV 字段和实际校验字段不一致,用户按示例导入可能会失败。
  3. 新增的颜色/尺码管理、商品导入导出、退货等写操作需要补齐权限边界,避免普通登录用户也能操作敏感功能。
  4. 这个 PR 没有 CI 或测试结果支撑,改动文件较多,直接合并风险较高。

建议把这个 PR 拆成几个更小的 PR:

  • 一个 PR 只修库存按钮无响应;
  • 一个 PR 做商品导入导出,并修正 CSV 示例和测试;
  • 一个 PR 做颜色/尺码管理;
  • 一个 PR 单独做退货流程,补充库存、余额、积分相关测试;
  • UserProfile 扩展信息单独讨论和提交。

这样每个 PR 都能独立 review、测试和合并,也更容易保证主分支稳定。

@dajiaohuang dajiaohuang changed the title fix: #34 功能需求 + UserProfile 扩展信息 fix: 修复库存操作按钮无响应 Aug 13, 2026
@dajiaohuang

Copy link
Copy Markdown
Author

感谢详细反馈。我已按建议把这个 PR 收敛为只修复“库存入库/出库/调整按钮无响应”,并基于最新 main 重建了分支,冲突现已解决。

本次提交 26b977a 做了以下调整:

  • 移除原 PR 中的商品导入导出、颜色/尺码管理、退货和 UserProfile 等其他主题;
  • 将库存列表的三个按钮连接到对应操作页并携带当前商品;
  • 让入库、出库、调整页面预选当前商品;
  • 补齐操作视图缺失的表单模板,避免按钮跳转后出现 500;
  • 增加链接和表单预选的回归测试。

验证结果:聚焦的 InventoryViewTest 4 项全部通过,Django system check 通过(仅保留仓库已有的 static/ 目录警告),且无迁移变化。完整 46 项测试为 42 项通过、4 项既有失败;后者已在未修改的最新上游分支上逐项复现,和本次库存按钮改动无关。

联系方式可以通过您留下的邮箱;为了方便 review 和后续追踪,本次代码调整与验证结果也完整保留在这里。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants