material_manage_system

zhangwenrui/material_manage_system

Fork 0

Commit Graph

Author	SHA1	Message	Date
文睿张	5d508cd2bb	fix: 给不存在的仓库建库位时误报创建成功 CreateLocation 的 INSERT...SELECT 匹配不到仓库时影响 0 行，MySQL 不报错： err 为 nil、LastInsertId() 返回 0。此前只看 err，接口回 201 "Location created successfully" 且 data.id 为 0，而库里一行都没写进去—— 刷新后库位消失，重试还是"成功"。改为以 RowsAffected 为准（INSERT 的 RowsAffected 就是实际插入行数，不受 clientFoundRows 影响），影响 0 行返回 ErrWarehouseNotFound，接口回 400 "仓库不存在，可能已被删除或不属于当前组织"。与既有的 execExpectingRow、 role_permissions 插入保持同一套路。实测（docker，重建后端镜像后）：不存在的仓库 999 -> HTTP 400，warehouse_locations 1 行 -> 1 行真实仓库 1 -> HTTP 201 id=3，1 行 -> 2 行，库中确有此行新增 warehouse_test.go 覆盖两个分支，变异验证：去掉 affected==0 判断后 "影响 0 行"用例失败（返回 <nil>，正是线上症状）。	2026-09-22 18:21:31 +08:00
文睿张	696e90893b	fix: 主数据编辑的两类静默失败 UPDATE 漏列： - code_rules 的 SET 补上 category_id。此前改分类提示"保存成功"但不落库，是唯一用户可达的活缺陷（同页的分类下拉没有 disabled，且规则只能改不能删，没有"删了重建"这条退路） - code_rules 有 uk_code_rules_org_category，补列后改到已占用的分类会撞唯一键，新建与更新统一翻成中文的 ErrDuplicateCodeRule - 分类/单位/仓库/库位/伙伴的编码定为不可变：分类编码是编码规则前缀的来源（init.sql 的种子数据直接取 material_categories.code），伙伴编码是对外业务编号，改掉会让历史数据脱节且查不回来。service 的 Update* 不再要求编码非空 ——原先是"必须传、传了却被丢弃"；新建侧的必填校验保留 - 顺带补齐两处同款漏网：organization.go 的 code 与 partner.go 的 partner_code （前者 service 本就不校验编码，自洽，无需改）记录不存在时的静默成功： - DSN 开启 clientFoundRows，让 RowsAffected 返回"匹配"行数而非"实际改变"行数。否则"编辑时一个字段都没改"和"记录不存在"都返回 0，无法区分，直接加检查会把正常保存误判成记录不存在（真 MySQL 8.0 实测） - 新增 repository.execExpectingRow，7 个 Update* 影响 0 行时返回 ErrRecordNotFound；伙伴走事务内的 SELECT ... FOR UPDATE，只需翻译 sql.ErrNoRows - 既有 4 处 RowsAffected 用法逐处核对过，均不受该 flag 影响测试：补 6 个 Update* 的存在性检查、编码必填约束的两端、唯一键翻译、DSN 拼接；全部做过变异验证（抽掉实现后对应测试确实变红）。	2026-09-21 17:26:53 +08:00
文睿张	a103549854	拆分主数据和仓库数据	2026-09-16 16:29:04 +08:00

Author

SHA1

Message

Date

文睿张

5d508cd2bb

fix: 给不存在的仓库建库位时误报创建成功

CreateLocation 的 INSERT...SELECT 匹配不到仓库时影响 0 行，MySQL 不报错：
err 为 nil、LastInsertId() 返回 0。此前只看 err，接口回 201
"Location created successfully" 且 data.id 为 0，而库里一行都没写进去——
刷新后库位消失，重试还是"成功"。

改为以 RowsAffected 为准（INSERT 的 RowsAffected 就是实际插入行数，不受
clientFoundRows 影响），影响 0 行返回 ErrWarehouseNotFound，接口回 400
"仓库不存在，可能已被删除或不属于当前组织"。与既有的 execExpectingRow、
role_permissions 插入保持同一套路。

实测（docker，重建后端镜像后）：
  不存在的仓库 999  ->  HTTP 400，warehouse_locations 1 行 -> 1 行
  真实仓库 1        ->  HTTP 201 id=3，1 行 -> 2 行，库中确有此行

新增 warehouse_test.go 覆盖两个分支，变异验证：去掉 affected==0 判断后
"影响 0 行"用例失败（返回 <nil>，正是线上症状）。

2026-09-22 18:21:31 +08:00

文睿张

696e90893b

fix: 主数据编辑的两类静默失败

UPDATE 漏列：
- code_rules 的 SET 补上 category_id。此前改分类提示"保存成功"但不落库，
  是唯一用户可达的活缺陷（同页的分类下拉没有 disabled，且规则只能改不能删，
  没有"删了重建"这条退路）
- code_rules 有 uk_code_rules_org_category，补列后改到已占用的分类会撞唯一键，
  新建与更新统一翻成中文的 ErrDuplicateCodeRule
- 分类/单位/仓库/库位/伙伴的编码定为不可变：分类编码是编码规则前缀的来源
  （init.sql 的种子数据直接取 material_categories.code），伙伴编码是对外业务
  编号，改掉会让历史数据脱节且查不回来。service 的 Update* 不再要求编码非空
  ——原先是"必须传、传了却被丢弃"；新建侧的必填校验保留
- 顺带补齐两处同款漏网：organization.go 的 code 与 partner.go 的 partner_code
  （前者 service 本就不校验编码，自洽，无需改）

记录不存在时的静默成功：
- DSN 开启 clientFoundRows，让 RowsAffected 返回"匹配"行数而非"实际改变"行数。
  否则"编辑时一个字段都没改"和"记录不存在"都返回 0，无法区分，直接加检查会把
  正常保存误判成记录不存在（真 MySQL 8.0 实测）
- 新增 repository.execExpectingRow，7 个 Update* 影响 0 行时返回 ErrRecordNotFound；
  伙伴走事务内的 SELECT ... FOR UPDATE，只需翻译 sql.ErrNoRows
- 既有 4 处 RowsAffected 用法逐处核对过，均不受该 flag 影响

测试：补 6 个 Update* 的存在性检查、编码必填约束的两端、唯一键翻译、DSN 拼接；
全部做过变异验证（抽掉实现后对应测试确实变红）。

2026-09-21 17:26:53 +08:00

文睿张

a103549854

拆分主数据和仓库数据

2026-09-16 16:29:04 +08:00

3 Commits