|
96
|
6
|
3
|
5
|
|
0
|
claude/fervent-kirch-e2a254 -> origin/claude/fe claude/fervent-kirch-e2a254 -> origin/claude/fervent-kirch-e2a254...
|
## 模块治理摘要
- 模块：
- 战区：
- lane：
- 阶段：
- 分类：`模 ## 模块治理摘要
- 模块：
- 战区：
- lane：
- 阶段：
- 分类：`模块专项通过 / 工作区漂移 / 阻断`
- 结论：
- 战役卡：
- 验收卡：
- runner evidence bundle：
## 验证命令
| 命令 | 结果 | 备注 |
| --- | --- | --- |
| | | |
| | | |
| | | |
## 审计命中
- `audit:tenant`：
- `audit:events`：
- `audit:route-contract`：
- `audit:docs-single-source`：
- 如未跑全量，请说明原因：
## 前端 / 页面验收
- canonical 入口：
- smoke / auth-scope：
- alias / compat 回归：
## 残余风险
- 风险 1：
- 风险 2：
## 文档同步
- [ ] `CLAUDE.md`
- [ ] `AGENTS.md`
- [ ] `README.md`
- [ ] `GEMINI.md`
- [ ] 治理经验库 / 专项记录
## 口径确认
- [ ] 本 PR 只宣称模块级通过，不把模块结果外推为工作区全绿
- [ ] 若存在工作区漂移，已明确标记且未误记到当前模块
- [ ] 若存在热修 lane，已与全域补证据 / build-out lane 隔离
...
|
0
|
0
|
1
|
1
|
1
|
|
0
|
1777597467
|
1779328857
|
1778625303
|
0
|
0
|
0
|
Edit
Delete
|
|
46
|
18
|
18
|
1
|
|
0
|
合并到主分支
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1775806102
|
1775806218
|
1775806125
|
0
|
0
|
0
|
Edit
Delete
|
|
44
|
6
|
1
|
5
|
|
0
|
test(finance): add finance mainline smoke and hard test(finance): add finance mainline smoke and harden invoice detail formatting...
|
## Summary
- add a finance-role mainline smoke tha ## Summary
- add a finance-role mainline smoke that covers invoices, AR ledger, and finance dashboard
- treat reconciliation 403 as the current permission boundary for the finance fixture
- harden invoice detail amount formatting so null values no longer trip the page
## Validation
- E2E_SKIP_WEB_SERVER=true E2E_USE_SYSTEM_CHROME=true pnpm -C e2e exec playwright test tests/auth/finance-mainline-flow-smoke.spec.ts --project=chromium-no-auth
- result: 1 passed (9.4s)
## Note
- git fetch/clone against the current Gitea main is currently failing with upload-pack not our ref, so this patch was replayed through the Gitea contents API on top of the live main history....
|
0
|
0
|
1
|
1
|
1
|
|
0
|
1774950058
|
1779328857
|
1778625303
|
0
|
0
|
0
|
Edit
Delete
|
|
32
|
18
|
7
|
1
|
|
0
|
合并
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1774348913
|
1778573643
|
1778573643
|
0
|
0
|
0
|
Edit
Delete
|
|
30
|
18
|
5
|
1
|
|
0
|
新分支提交
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1774348869
|
1774348877
|
1774348877
|
0
|
0
|
0
|
Edit
Delete
|
|
29
|
18
|
4
|
1
|
|
0
|
0331合并到主分支
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1774346560
|
1774346571
|
1774346571
|
0
|
0
|
0
|
Edit
Delete
|
|
26
|
18
|
1
|
1
|
|
0
|
合并111
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1774345739
|
1774345823
|
1774345753
|
0
|
0
|
0
|
Edit
Delete
|
|
21
|
2
|
21
|
1
|
|
0
|
11111
|
测试
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1773910676
|
1773910796
|
1773910698
|
0
|
0
|
0
|
Edit
Delete
|
|
892
|
116
|
13
|
5
|
|
0
|
chore(reports): adopt Runtime static originals at chore(reports): adopt Runtime static originals at dc13b515 and clarify evidence snapshot...
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793674166
|
1793710134
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
881
|
116
|
5
|
5
|
|
0
|
ci(diagnostic): 只读核对现有runner的维护边界
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793581907
|
1793596652
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
871
|
83
|
18
|
5
|
|
0
|
feat(personnel): D-13 组织类型字段与宿主 Schema 受控通道（PKG-00 feat(personnel): D-13 组织类型字段与宿主 Schema 受控通道（PKG-006）...
|
> **叠在 #17 之上（#17 又叠在 #13 之上），请按 #13 → #17 → 本 > **叠在 #17 之上（#17 又叠在 #13 之上），请按 #13 → #17 → 本 PR 的顺序合入。** 本 PR 自身只有最后 2 个提交：`4fa64f1`（D-13）、`6912ab7`（报告回绑）。
## 背景
- F1 `OrganizationUnitMoved` 要求 `unit_type` 必填（CHG-061 Q5），而 HR 组织原本没有类型字段。
- 02 已于 10-27 定下词表（D73）：采用 CHG-061 的三值，02 侧分别映射为 `BUSINESS_UNIT` / `DEPARTMENT` / `TEAM`。
- 加字段要改 D3 冻结的宿主 `schema.prisma`。用户 2026-10-28 裁定 D-13：开受控通道并实现。
## 内容（对接蓝图 §4.17）
- **D3 受控通道**（`scripts/lib/d3-protection.mjs`）：`packageUpdates` 只多放行一个文件，即宿主 `apps/api-nestjs/prisma/schema.prisma`。既有迁移、`migration_lock.toml` 和已退役的 Fastify schema 仍不得登记；结构变更一律以新迁移追加。
- **字段**：组织新增可空的 `unitType`（`ORGANIZATION` / `DEPARTMENT` / `TEAM`）。迁移 `20261028090000_organization_unit_type` 只追加这一列；已有组织的值为 null，不做推断。
- **contracts**（台账 PKG-006）：
- 创建时可选，缺省为 null；
- 编辑时省略即保留原值，显式传 null 则清除；
- 命令视图与组织摘要必带该字段；
- 双时间快照读取该字段，D-13 之前的快照读为 null。
- **服务**：创建和编辑写入该字段，仍沿用原有乐观锁和 outbox；当前读取与双时间读取两条路径都输出该字段。
- **Web**：组织增改表单新增「组织类型」，组织详情显示类型。
- **未做**：
- OS 产品副本还没有这一列，登记为 H29。0.19.6 尚未进入 OS main，0.19.7 现在注册不了，等它进入后再随产品版本补上。
- F1 本身：等 contracts 发版后，与 F2–F4 一起实现。
- 已有组织的类型补录：需要业务按实际情况登记。
## 验证（本地，报告绑定 `4fa64f1`、clean）
- `d3-protection` 27/27，新增 2 例；2 个变异探针（去掉放行、放宽到整个 prisma 目录）都判红。
- contracts 232/232，新增 5 例。
- 真实 DB 的 HTTP 测试 3/3，覆盖：带类型 / 不带类型创建；编辑时省略保留、修改、清除；词表外取值返回 400 且不落库；用 `knownAt` 能回看到旧类型。另有 3 个服务层变异探针都判红。
- `check:runtime` 482/482，`check:ui` 46/46（在原有组织增改场景里加入了类型选择和读回），用一次性 PostgreSQL / Redis 跑，跑完已删。
- 静态检查：D3 / schema / migrations / RLS / write-guard 等门禁全部通过；静态链在 O22 处如实停止，`check:governance` 只剩 O22。
- 远端 CI 见本页。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793194416
|
1793194416
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
870
|
116
|
4
|
5
|
|
0
|
docs(release): R1 运行器与分支保护变更规格（rc.8 合规发布前置）
|
## 内容
给现役 Gitea / runner 维护者的逐项变更规格，作为 `@juhai/ ## 内容
给现役 Gitea / runner 维护者的逐项变更规格，作为 `@juhai/*` 发布列车 `1.0.0-rc.8` 走合规通道（同源 main 合格 Platform CI → clean tag → `.gitea/workflows/publish-packages.yml`）的前置（R1）。
- 新增 `docs/R1运行器与分支保护变更规格-2026-10-28.md`：
1. Gitea 升到含上游 PR #37119（ActionRunAttempt）的正式版，`MAX_RERUN_ATTEMPTS ≥ 2`，升级前一致性备份与隔离恢复；
2. act_runner 升到下发真实 `GITHUB_RUN_ATTEMPT` 的版本，保持 `ubuntu-latest`、Docker、github.com 访问与 fork Secret 隔离；
3. main 保护 9 个字段逐一对应 `governance/lib/release-ci.mjs` 第 30–31 行；
4. 交付后验收顺序与 rc.8 发布步骤。
- `docs/正式发布材料待办-2026-10-10.md` 顶部加指针。
## 依据（2026-10-28 PDT 只读实测）
- main 最近 15 次 Platform run 全部 failure，均在第一步「Candidate tag」因 `GITHUB_RUN_ATTEMPT` 缺失失败关闭（例：run 26610 @ `485db734d`，`run_attempt=0`，`attempts/1` → 404）；runner act_runner v0.2.13；版本 API 仅返回 `development`。
- `branch_protections` 为空列表。
## 影响提示
启用 main 保护后禁止直推（10-21 起 main 首父链 492 个提交中约 435 个为直推），所有会话须改走 PR，且每个 PR 要通过完整 Platform CI；规格要求先有一个 PR 全绿再落规则，避免锁死 main。
## 边界
只改文档；发布门不变，未恢复本地旁路（G-12），未打 tag、未写 Registry、未改服务。`pnpm check:doc-claims` 通过。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793191041
|
1793191041
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
861
|
83
|
17
|
5
|
|
0
|
feat(contracts): P1 前置片——CHG-061 三条 HR 事实的载荷与信封 Sc feat(contracts): P1 前置片——CHG-061 三条 HR 事实的载荷与信封 Schema（PKG-005）...
|
> **叠在 #13 之上，请先合 #13。** 本分支已合入更新后的 #13（#13 已含 > **叠在 #13 之上，请先合 #13。** 本分支已合入更新后的 #13（#13 已含 main 上的 #16）。#13 合入后，这里的差异只剩 P1 前置片与报告回绑。
## 为什么重建（2026-10-28）
PR #16 已在 main 上以 PKG-004 落地 V-04（来源域按事实类型；注册表条目的 `source_domain` 须为字面量）。本 PR 原先也用 PKG-004 实现了另一版 V-04，与 #16 冲突。重建时：
- contracts 与台账取 main 版本；
- 去掉本分支自己的 `sourceDomain` 放宽；
- 改按 #16 的注册表类型重做，台账改为 **PKG-005**，蓝图记为 **§4.16**。
## 内容（对接蓝图 §4.16，P1 / H5）
- **contracts**（`packages/contracts/src/enterprise-fact.ts`，台账 PKG-005）
- 新增 `organizationCanonicalIdSchema`（`org_`）。
- 新增 `EmploymentStarted`、`EmploymentTerminated`、`OrganizationUnitStatusChanged` v1 的载荷 Schema，逐字段对应 04 `contracts/schemas/facts/*.v1.json`（CHG-061）。组织状态载荷另要求 `change_kind` 与 `state_after` 一致。
- 三份信封 Schema 保持 ZodObject，`source_domain` 为字面量（hr / organization），满足 #16 的注册表类型（typecheck 已证明），升 pin 后可以直接登记。
- `buildEnterpriseFactEnvelope` 在注册表 Schema 解析之后统一校验两条：载荷里的实体 ID 等于 `entity_id`；组织事实的 `valid_from` 等于 `effective_at`。校验失败抛 ZodError，Relay 记为 `FACT_SCHEMA_INVALID` 并进死信。这条同样约束已登记的 `EmploymentTransferred`；HR 写入时两者本来就相同。
- **刻意不做**：出站注册表、`facts.owns`、生效事务写 Intent。原因是 M6 只认 pinned `@juhai/contracts` 的 Catalog，而 Registry 上最新的 rc.7 只有 11 条事实。发版已交给单独的 04 会话推进，真实发布前须用户同意；HR 侧的请求草稿是 `docs/domain/request-draft-04-contracts-release-chg061-2026-10-28.md`。
- **F1 `OrganizationUnitMoved`**：组织类型词表已由 02 D73 定为 CHG-061 三值（`organization` / `department` / `team`）。HR 加这个字段要改 D3 冻结的宿主 Prisma 和产品迁移，已登记为待裁决 D-13（用户已同意开受控通道并实现，另起 PR）。
- **文档**：蓝图 §4.16、H5 / P1 行，矩阵 #19 按 02 D72 订正（Site / Space / SiteOperator 的现用写点改为 EES 物包），新增 D-13；同步更新 CLAUDE.md G23、CHG 草案和发版请求草稿。
## 验证（本地，报告绑定 `b8f85d3`、clean）
- contracts 227/227（enterprise-fact 24 例，其中新增 8 例）。
- 5 个变异探针各判红：去掉实体 ID 一致、去掉状态一致、去掉 `valid_from` 一致、封装不调用一致性校验、组织事实来源域改为 hr。
- `check:d3` 通过；去掉 PKG-005 后判红（6 项）。
- `check:app-manifest`、`check:contract-consumers`、`check:validation`、lint、typecheck 均通过。
- `check:runtime` 474/474、`check:ui` 46/46，用一次性 PostgreSQL / Redis 跑完已删。静态检查链如实停在 O22，`check:governance` 只剩 O22。
- 远端 CI 结果见本页。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793153681
|
1793194416
|
0
|
0
|
1
|
0
|
Edit
Delete
|
|
858
|
83
|
15
|
5
|
|
0
|
feat(contracts): 出站事实来源域按事实类型收紧（V-04，PKG-004）
|
CHG-061 独立核验 V-04：通用 Intent 记录 Schema 把 `sourceDom CHG-061 独立核验 V-04：通用 Intent 记录 Schema 把 `sourceDomain` 写死为 `hr`，CHG-061 登记的组织事实（Catalog 与 04 Scope 要求 `source_domain = organization`）即使登记进注册表也无法出站。
## 改动（D-12 台账 PKG-004，`packages/*` 冻结字节链式放行）
- 通用记录 Schema 的 `sourceDomain` 只校验非空字符串；来源域由注册表条目的信封 Schema 按事实类型收紧，`ENTERPRISE_FACT_ENVELOPE_SCHEMAS` 的类型约束要求每个条目 `source_domain` 为字面量。
- `resolveEnterpriseFactEnvelopeSchema` / `buildEnterpriseFactEnvelope` 增加默认为生产注册表的可选注册表参数；relay 调用不变。
- 生产注册表仍只有 `EmploymentTransferred@1`，行为不变；未登记类型仍以 `FACT_TYPE_UNREGISTERED` 失败关闭。
- contracts 追加 4 例（测试用注册表下 organization 来源域封装通过、hr 来源域被拒；未登记组织事实失败关闭；EmploymentTransferred 仍只接受 hr）。负向：记录 Schema 改回 `z.literal("hr")`，新增 2 例判红。
- 文档：对接蓝图 H1 / H21 / P1 按 CHG-061 落地订正并新增 §4.15；`CLAUDE.md` 真源地图与 G23 动态区同步。
## 验证（本地工作区，clean `11adf0f`，一次性 PostgreSQL / Redis）
- `check:d3` 0 问题；contracts 219/219；`check:app-manifest` 通过；lint、typecheck 通过。
- `check:runtime` 466 / 0（地板 452）；`check:ui` 46 / 46；报告回绑提交 `e51fd28`。
- `check:governance` 只剩 main 上既有的 O22（OS main 不含 0.19.6，H12 / D-8）。远端 CI 未查验。
## 未包含
生产注册表登记新事实、`app-manifest.json` 的 `facts.owns`、`@juhai/contracts` 升 pin（rc.7 的 Catalog 为 11 条）、组织 / 用工事实的 producer 与生效事务写意图。
🤖 Generated with [Claude Code](https://claude.com/claude-code)
...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793150819
|
1793150819
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
857
|
83
|
14
|
5
|
|
0
|
含五件事：H20 台账收敛的记录、H11 回灌、PKG-002 vitest 升级、D-9，以及这次 含五件事：H20 台账收敛的记录、H11 回灌、PKG-002 vitest 升级、D-9，以及这次的冻结测试放行加 PKG-003。...
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合当前生命周期规范：既有 Personnel Core 保持 `/api/personnel/*`，HR v2 领域使用 `/api/v2/{domain}/*`，身份协议使用标准 OIDC 端点
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 涉及页面或浏览器行为时，`pnpm check:ui` 与 `pnpm check:ui:fastify` 均通过，未只验证一个后端
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793147366
|
1793147366
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
856
|
83
|
13
|
5
|
|
0
|
docs(integration): 蓝图补服务级对接矩阵，按 04 CHG-061 订正镜像与 H docs(integration): 蓝图补服务级对接矩阵，按 04 CHG-061 订正镜像与 H1/H5/H21...
|
## 内容
- 对接蓝图新增 §2.5 服务级对接矩阵：HR 与 02/03/04 共 20 个对 ## 内容
- 对接蓝图新增 §2.5 服务级对接矩阵：HR 与 02/03/04 共 20 个对接点，HR 侧写到 file:line，并标明 dev 实际跑通过的只有 3 条：事实写入、Scope 来源、OS 装载产品包。
- 按 04 `e50aad416`（CHG-061）订正：
- 应用清单镜像已合到 0.19.6，工作区镜像剩 5 处差异（H1）；
- 4 条 HR 事实已在 Catalog 登记为 draft，H5 的 Catalog 半边完成；
- Scope 改按 Catalog producer 接受 `juhai.hr` 组织事实，H21 部分关闭，剩载荷约束与投影器现场摄取。
- 新增缺口：
- H25：OS 清单契约仍强制双后端，HR 清单里的 Fastify 声明是占位，不能单方删除；
- H26：门店归属事实没有生产者；
- H27：IdP 令牌 `aud` 是生成的 UUID，不是 `juhai.hr`；
- H28：数字智能层自带一份 0.19.5 副本。
- §2.5 补本机现场读数（2026-10-28 00:35 UTC 前后）：
- HR API 未运行；
- 04 runtime 开了事实受理但缺 `FACT_APPLICATION_BINDINGS_JSON`，IdP 签发的机器令牌会被拒；
- Scope 来源 65281 不可达；
- OS 0.19.6 分支只在本机、未推 Gitea。
- 已确认三项：
- client-fact rc.3 与 rc.8 摘要实现逐行相同（H7 只剩发布）；
- EES 运行时以 `READ_ONLY_PROJECTION` 拒绝已撤回人域命令；
- Scope 证据 `owner: juhai.hr` 由 04 按 HR 签名组装。
- H22：本机开发基座容器已于 00:40 UTC 重建为只绑回环，数据卷沿用。
- **CI 修复**（`a174e46`）：本 PR 首次运行 26581 停在「Build locked OS compatibility sources」，原因是兼容锁指向 D3 后的 OS `1b8ae0d7`，该步却仍写死 `--filter=api-fastify`。
- 现改为经 `scripts/lib/os-host-backends.mjs` 读锁定 OS 的 `backend-profile.json` 决定宿主范围；
- `check:docs-truth` 新增 `ci-os-host-backends` 断言防回潮，对旧工作流负向判红。
- 同步落地状态：CLAUDE.md G23 / G14、两份 CHG 草稿、人域对照表。
- 已合入 `main` `46da429`：CLAUDE.md 冲突以 main 为准，只叠加本分支的 H21 订正。
## 验证（本地，PR 头 `25aecd7`，报告绑定 `a174e46`、clean）
- 通过：`check:runtime` 462/462（地板 452）、`check:ui` 46/46。用的是一次性 PostgreSQL 16 / Redis 7，只绑回环，跑完已删。
- 静态链如实停在 O22（OS main 尚无 0.19.6）。其后 governance-docs、docs-truth、lint、typecheck 通过。`check:governance` 只剩 O22，`reportProvenanceViolations` 0。
- CI 构建步骤：本地覆盖 5 种情形（nestjs-production、D3 前双宿主、缺 profile、非法 profile、profile 与 Fastify 冲突），以及锁定 OS 快照上的 turbo dry-run。
- 远端 CI 见本页检查结果。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793145750
|
1793194416
|
0
|
0
|
1
|
0
|
Edit
Delete
|
|
851
|
83
|
8
|
5
|
|
0
|
feat(integration): Web/开发基座只绑回环、D-12 packages/* 受控 feat(integration): Web/开发基座只绑回环、D-12 packages/* 受控通道与 H20 出站事实注册表...
|
## 改了什么
本 PR 有三部分；已合入 main 的 PR #9（D-10 API 回环）与 ## 改了什么
本 PR 有三部分；已合入 main 的 PR #9（D-10 API 回环）与 PR #10（H23 根与 apps/* 依赖）均已并入本分支。
### 1. Web / 开发基座只绑回环（蓝图 §4.9）
- Web `next dev` / `next start` 与 UI 验收的 `next start` 加 `-H 127.0.0.1`；开发基座 `apps/api-nestjs/docker-compose.yml` 的 PostgreSQL / Redis 端口只发布到回环（此前发布到全部网卡，PG 口令 `app`、Redis 无口令）。已在运行的基座容器须 `docker compose up -d` 重建才生效。
- 在 #9 的 `scripts/lib/listen-host-guard.mjs` 内新增 `exposureViolations`（同一守卫），接入 `check:dual-backend`；单测 9/9。
- 文档：owner-matrix 去掉 D3 前「双后端 / Fastify」措辞；Scope 来源接入文档改正 IdentityLink 归属（enterprise-idp 拥有）。
### 2. D-12：`packages/*` 受控升级通道（关 H24，蓝图 §4.10）
- D3 保护此前逐字节冻结全部 `packages/*`（215 个受保护文件），台账只放行产品目录。现在同一台账 `docs/post-d3-product-updates.json` 增加 `packageUpdates`：编号 `PKG-NNN` 递增，须写来源与理由，仅限 D3 冻结清单内的 `packages/*` 文件，逐文件从冻结 hash 链式放行；任一条目不合规即整体不放行。
- `check:d3` 30/30：新增登记后放行、未登记 / 台账缺失判红，以及断链、前后相同、非 hash、缺理由、缺来源、编号格式错、编号不递增、空文件、包外路径、非冻结文件、字段类型错均判红。变异探针：删掉「理由与来源」或「链式校验」规则，测试各判红 1 项，已还原。
### 3. H20：出站事实注册表（蓝图 §4.10）
- contracts 新增 `ENTERPRISE_FACT_ENVELOPE_SCHEMAS`（`factType@schemaVersion` → 信封 Schema，当前只有 `EmploymentTransferred@1`）与 `EnterpriseFactTypeUnregisteredError`；`buildEnterpriseFactEnvelope` 按注册表分派。EmploymentTransferred v1 的 Schema 不变。
- relay 去掉全部类型强转；未登记类型或版本记 `FACT_TYPE_UNREGISTERED`，立即进死信、不调用 Publisher（部署补登记后可经既有重放恢复）。
- contracts 两个文件经台账 **PKG-001** 放行（登记前 `check:d3` 对这两个文件判红）。
- `check:app-manifest` M6 新增注册表与 `facts.owns` 双向一致检查（3 个负例）。
- 测试：contracts 新增 4 例；新文件 `apps/api-nestjs/test/enterprise-fact-relay-registry.test.ts` 真实 DB 3 例。原 `enterprise-fact-relay.test.ts` 受 D3 冻结未改，12 例仍全绿。
## 验证（本地，2026-10-27，干净提交 `effed90`）
- `check:runtime` 462/462、`check:ui` 46/46（一次性 PostgreSQL / Redis，跑完删除）
- typecheck、lint、`check:d3`、`check:dual-backend`、`check:app-manifest`、`check:docs-truth`、`check:governance-docs`、`check:migrations`、`check:database-rls` 等静态门禁通过
- `check:governance` 剩 2 项，与 main 相同：O22（D-8，等 OS 写入授权）、`osProductReleaseTargetRemotes` 2→1（D-9）
## 未覆盖
- 远端 CI 未查验（本机无 Gitea API 令牌）
- 新事实（`OrganizationUnitMoved` 等）仍待 Catalog 登记；H21（Scope 消费端写死来源应用）仍开放
- `packages/*` 内的 vitest 2.1.9 与 `client-fact` rc.3 现可经台账升级，本 PR 未做
- 00 框架同一监听默认值（他层仓）；README 第 3 步提到的 `.env.example` 不存在（既有不一致）
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793111425
|
1793124194
|
0
|
0
|
3
|
0
|
Edit
Delete
|
|
850
|
83
|
7
|
5
|
|
0
|
ci: per-run docker PostgreSQL/Redis for acceptance ci: per-run docker PostgreSQL/Redis for acceptance, with reports rebound to e8591c7...
|
Two commits:
- e8591c7 ci: the runtime job no l Two commits:
- e8591c7 ci: the runtime job no longer uses `services:` (never started on this host-mode runner; 127.0.0.1:5432 reached an unrelated PostgreSQL on the runner host in run 26519). scripts/ci-acceptance-services.sh fails without docker; otherwise it starts per-run PostgreSQL/Redis containers on docker-assigned high ports with a random masked password and removes them in an if: always() step. G14 updated.
- 8cf50a6 chore(reports): runtime, UI and static reports rerun on the clean final source e8591c7.
Local evidence (isolated containers, recreated database):
- check:runtime 452/452 (floor 452), frozen oracle 12, outbox 1 scenario / 0 duplicates, outage health 2.
- check:ui 46/46.
- pnpm check with DIGITAL_EMPLOYEE_OS_ROOT at e12d0f2: exit 0, provenance violations 0.
- Script checks: start/connect/stop with no leftovers, repeated stop, and failures for missing docker, a non-prefixed database name and an invalid run id.
Not yet verified: docker on the runner host and its ability to pull postgres:16 / redis:7 (override with CI_POSTGRES_IMAGE / CI_REDIS_IMAGE repository variables).
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793102531
|
1793190011
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
842
|
116
|
3
|
5
|
|
0
|
fix(channel-sdk): bound response bodies and reject fix(channel-sdk): bound response bodies and reject redirected results...
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1792411267
|
1792411267
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
841
|
116
|
2
|
5
|
|
0
|
fix(credential-sdk): harden bounded responses and fix(credential-sdk): harden bounded responses and redirect cleanup...
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1792411245
|
1792411245
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
822
|
124
|
16
|
5
|
|
0
|
docs: 开发默认值口头确认，并刷新验收矩阵
|
- required-business-decisions.md：开发默认值表标为“2026-10- - required-business-decisions.md：开发默认值表标为“2026-10-09 已口头确认，待正式审批记录”，包括秒杀迟到支付转人工退款核对、转写中断片段不能校对两条补充规则；仍待定的部分（门店地址随 BD01、暂锁到期是否通知、可校对岗位、真实开票能力）单独写明。
- 这只是口头确认：代码里的策略仍是 approved:false，要等正式批准记录到位后再改；没有启用任何真实支付、设备、平台、开票或语音识别能力。
- 重新生成验收矩阵：#12/#13 合并后 main 上的矩阵已过期，重跑 903/903 通过，--check 恢复为 0。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1791588968
|
1791588968
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
817
|
124
|
11
|
5
|
|
0
|
test: dedicated evidence for 14 P1/P2 requirements test: dedicated evidence for 14 P1/P2 requirements and strict licence dates...
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1791559480
|
1791559480
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
806
|
121
|
4
|
5
|
|
0
|
untracked files on codex/resource-health-axis: 5ef untracked files on codex/resource-health-axis: 5ef0d78 feat(integration): prepare trusted external SalesOrder fact inbox...
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1790675336
|
1790675336
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
805
|
121
|
3
|
5
|
|
0
|
untracked files on codex/resource-health-axis: f0f untracked files on codex/resource-health-axis: f0ff3df docs(scope): 重梳 V1.0 三图与31章开发缺口...
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1790675329
|
1790675330
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
804
|
121
|
2
|
5
|
|
0
|
feat(work): gate Task on latest confirmed Outcome
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1790675319
|
1790675319
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
803
|
121
|
1
|
5
|
|
0
|
wip(K3.3): PolicyDefinition — preserved snapshot o wip(K3.3): PolicyDefinition — preserved snapshot of uncommitted work (base 0.71, not for merge)...
|
Preservation snapshot (R0, 2026-09-29) of work tha Preservation snapshot (R0, 2026-09-29) of work that only existed as uncommitted changes in a /tmp worktree.
Nothing here was verified by this commit: no tests were ever written for it and it was not run against a database.
What the snapshot contains:
- sql/072_policy_definition.sql: governance.policy_definition_version (EXECUTION_AUTHORIZATION with maxTtlSeconds),
a ten-state lifecycle, immutable content, one ACTIVE version per tenant and type, FORCE RLS; also replaces the
authority_assignment action CHECK with governance.manage_policy / governance.approve_policy added. The file header
is a copy of the K3.2 header.
- src/contexts/policy-definition.js: the PolicyDefinition commands (create, validate, request review, approve,
reject, publish, activate, deprecate, retire, revoke) with self-approval denial and authority checks.
- registrations in commands.js, owner-roles.js, rejection-target.js, kernel.js, catalog and the payload / event
contracts.
Known state when preserved:
- Based on 0490b8e (0.71). Main is 0.103.x with migrations up to 095; 072 already exists there
(072_task_case_correlation_immutable), so the migration must be renumbered, and kernel.js / catalog / contracts
will need a real rebase.
- Missing for the slice: tests, contract document, smoke, and the binding of a GovernanceDecision to the active
policy version with drift rejection (K3.3b).
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1790675308
|
1790675308
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
773
|
85
|
8
|
4
|
|
0
|
按省市统计设备月度新增上线数量报表
|
## 功能概述
在 PC 管理后台「数据报表」模块，按省市统计各 KTV 设备每月新增上线数量，支 ## 功能概述
在 PC 管理后台「数据报表」模块，按省市统计各 KTV 设备每月新增上线数量，支持全国省份分布视图 + 城市下钻。
## 完整产物链路
| 阶段 | 产物 | 链接 | 状态 |
|------|------|------|------|
| 需求意图 | Intent 文档 | PR #1 | ✅ 已合并 |
| 设计原型 | Design 文档 | PR #3 | ✅ 已合并 |
| 技术方案（后台） | Plan 文档 | PR #6 | ✅ 已合并 |
| 开发实现（后台） | admin 仓库代码 | admin PR #1 | ✅ 已合并 |
| 开发实现（Web前端） | admin 仓库代码 | admin PR #2 | ✅ 已合并 |
| 测试验收 | — | 本 Issue | 🧪 待测试 |
## 验收标准
- 后台 API 返回省份/城市维度的月度新增上线设备数，分页正确
- PC 管理后台菜单「数据报表 > 设备上线分布」可正常点击进入
- 省份视图正常渲染（横向柱状图），点击下钻到城市视图
- 城市视图面包屑可返回全国
- 月份切换后数据刷新正确
- 无数据月份显示空状态，不报错...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1789379887
|
1789380277
|
0
|
0
|
2
|
0
|
Edit
Delete
|
|
731
|
57
|
2
|
5
|
|
0
|
chore: add material factory model weights
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1786603028
|
1786603028
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
691
|
50
|
1
|
5
|
|
0
|
fix(web): update Lyra product labels
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1783327279
|
1783431501
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
673
|
32
|
1
|
1
|
|
0
|
222
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1781158798
|
1781158798
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
668
|
23
|
36
|
1
|
|
0
|
🔍 代码审查报告：admin-260616 - 生成小程序码
|
## 自动代码审查报告
**分支**: admin-260616
**提交**: `d7e82ca ## 自动代码审查报告
**分支**: admin-260616
**提交**: `d7e82ca5e26517b2e5ceaef844232aa1a7546ef4`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 18:14:17
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：业务意图清晰，核心流程（生成、缓存、返回小程序码）已实现。但存在明显的逻辑隐患（二进制流与 JSON 混处理）、性能瓶颈（重复加载模型、日志记录不当）及框架规范偏离。缺乏必要的异常处理与并发控制，需重点修复。
- **风险等级**：中（存在脏数据写入、日志膨胀及潜在运行时警告风险）
> 📌 **框架说明**：提供的目录结构与代码风格高度吻合 **CodeIgniter 3**。若 `phpci` 为内部定制框架且继承自 CI3，以下建议完全适用；若为独立架构，请根据实际生命周期调整依赖加载方式。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `create_qrcode()` 方法 | 微信 `getUnlimited` 接口成功时返回**二进制图片流**，失败时返回 JSON。代码直接对返回值执行 `json_decode`，成功时返回 `null` 导致逻辑“侥幸”通过；且 `var_export` 记录二进制数据会导致日志文件暴增，甚至触发内存溢出。 | 先判断返回值是否为 JSON 格式（或检查 HTTP 状态/错误码），成功则直接上传，失败才解析并记录日志。避免对二进制数据执行序列化。 | `if (str_starts_with(trim($wxacode), '{')) { $err = json_decode($wxacode, true); if (isset($err['errcode']) && $err['errcode'] > 0) { do_log(...); return ''; } }` |
| 🔴 严重 | `get_qrcode()` 方法 | `$merchant_data['_business_model']` 未做空值校验。若商家不存在或查询返回空数组，将触发 `Undefined index` 警告，并可能绕过业务模型校验直接生成码。 | 增加 `empty()` 或 `isset()` 防御性判断，确保数据结构完整后再访问字段。 | `if (empty($merchant_data) || $merchant_data['_business_model'] != self::BUSINESS_MODEL_SELF_SERVICE) { return ''; }` |
| 🟠 警告 | 文件顶部 (1-3行) | 全局作用域使用 `$CI = &get_instance();` 违反 MVC 模型规范。在 CI/类 CI 框架中，模型文件被 `include` 时即执行，易引发作用域污染、重复实例化及内存泄漏。 | 移除顶部全局赋值，依赖加载应移至类构造函数或使用 `$this->load`。 | 删除顶部两行，在 `__construct()` 中调用 `$this->load->model('Simple_model');`（若父类未自动加载） |
| 🟠 警告 | `update_qrcode()` 方法 | 存在**并发竞态条件**。高并发下多个请求可能同时通过 `empty($data['_qrcode'])` 检查，导致重复调用微信接口、重复上传 OSS 及数据库重复插入/覆盖。 | 数据库 `_family_server_id` 字段添加唯一索引，代码改用原子操作（如 `INSERT ... ON DUPLICATE KEY UPDATE` 或框架 `upsert`）。 | `$this->db->set($insert)->where($where)->on_duplicate_key_update(['_qrcode' => $qrcode])->insert();` |
| 🟠 警告 | `create_qrcode()` 方法 | 阿里云 OSS 上传 `alioss_internal_addObject_by_content` 未做返回值校验或异常捕获。若上传失败，仍会拼接 URL 写入数据库，产生大量无效脏数据。 | 增加上传结果校验，失败时记录明确日志并中断流程，避免写入无效路径。 | `if (!alioss_internal_addObject_by_content($file_url, $buffer)) { do_log('OSS upload failed', 'qrcode_error'); return ''; }` |
| 🟡 建议 | 全局/类定义 | 类名 `Ahead_room_renewal_mini_qrcode_model` 使用蛇形命名，不符合 PSR-12 规范；魔法数字 `2` 硬编码；`$room_data` 数组键未做类型/存在性校验。 | 类名改为大驼峰 `AheadRoomRenewalMiniQrcodeModel`；提取业务模型常量；增加输入参数校验。 | `class AheadRoomRenewalMiniQrcodeModel extends Simple_model { const BUSINESS_MODEL_SELF_SERVICE = 2; ... }` |
| 🟡 建议 | `get_qrcode()` 方法 | 方法内部动态加载 `$this->load->model('ahead_yc_merchant_model')`，每次调用都会触发框架加载器解析，增加不必要的 I/O 开销。 | 移至构造函数中统一加载，提升执行效率。 | `public function __construct() { parent::__construct(); $this->load->model('ahead_yc_merchant_model'); }` |
| 🟡 建议 | `create_qrcode()` 方法 | 硬编码 OSS 域名 `https://vodonline.g-hi.com/` 及路径拼接逻辑，不利于多环境（测试/生产）切换；`DEBUG_VERSION` 若未严格过滤可能存在路径穿越隐患。 | 将域名与基础路径抽离至配置文件，使用框架配置函数读取。 | `$oss_domain = config_item('oss_public_domain'); $file_url = rtrim($oss_domain, '/') . '/' . $path;` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复微信接口返回值处理逻辑**：严格区分二进制流与 JSON 错误响应，移除对二进制数据的 `json_decode` 和 `var_export`，防止日志爆炸与误判。
2. **增加空值防御与上传校验**：在 `get_qrcode` 中校验 `$merchant_data` 是否存在；在 `create_qrcode` 中校验 OSS 上传结果，失败时阻断数据库写入。
3. **消除全局 `$CI` 实例化**：移除文件顶部的 `get_instance()`，遵循框架模型生命周期规范，避免隐式内存泄漏。
### 🛠 后续重构与优化方向
- **并发安全设计**：为 `ahead_room_renewal_mini_qrcode` 表的 `_family_server_id` 添加唯一索引，将 `get_one` + `insert/update` 替换为数据库层面的 `UPSERT` 操作，彻底解决竞态条件。
- **依赖注入与配置化**：将硬编码的常量、域名、业务模型阈值抽离至 `config/` 目录；考虑使用构造函数注入依赖模型，提升单元测试友好度。
- **日志规范化**：使用结构化日志（如 JSON 格式）替代 `var_export`，仅记录关键标识符与错误码，避免敏感数据或二进制内容污染日志系统。
- **输入校验层**：在 Controller 层或 Model 入口处对 `$room_data` 进行类型断言与必填字段校验（如 `_id`, `_merchant_id` 必须为整型），防止脏数据流入核心逻辑。
> 💡 **提示**：若 `phpci` 框架对模型加载、数据库操作有特定封装（如内置 `upsert` 或统一响应对象），请优先查阅官方文档替换原生 SQL/CI 写法，以保持架构一致性。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781086457
|
1781086457
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
667
|
23
|
35
|
1
|
|
0
|
🔍 代码审查报告：admin-260616 - 小程序续费二维码
|
## 自动代码审查报告
**分支**: admin-260616
**提交**: `7cc3245 ## 自动代码审查报告
**分支**: admin-260616
**提交**: `7cc3245c843739ac94cf83deb882fbe15bca93d5`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 18:08:11
---
## 1. 审查摘要
- **代码质量评分**：6.5/10
- **总体评价**：代码实现了小程序码生成与缓存的核心流程，但存在明显的框架生命周期误用、二进制/JSON混合处理缺陷、数据库竞态条件及硬编码问题。整体逻辑可跑通，但健壮性、可维护性与安全性存在较大优化空间。
- **风险等级**：高
> 📌 **注**：提供的目录结构与 `$CI = &get_instance()` 用法高度符合 **CodeIgniter 3** 规范。以下审查基于 CI3 生命周期与 PHP 最佳实践。若 `phpci` 为内部定制框架，请对照其官方文档调整组件加载机制。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 全局/第2行 | `$CI = &get_instance();` 在类外部直接调用，会在文件被 `include/require` 时立即执行，破坏框架初始化生命周期，且在 CLI 或单元测试中极易引发 `Call to undefined function` 错误。 | 移除全局调用。模型应通过 `$this->load->model()` 加载依赖，或在 `__construct()` 中初始化。 | `public function __construct() { parent::__construct(); $this->load->model('ahead_yc_merchant_model'); }` |
| 🔴 严重 | `create_qrcode()` | 微信接口成功时返回**二进制图片流**，失败时返回 **JSON**。代码使用 `$buffer = $wxacode = getUnlimitedWxacode(...)` 混合赋值，随后对二进制数据执行 `json_decode()` 会产生 Warning，且未校验 OSS 上传结果即更新数据库，可能导致脏数据。 | 明确区分成功/失败分支；增加 OSS 上传返回值校验；避免对二进制数据使用 `var_export` 记录日志。 | 见下方重构示例 |
| 🟠 警告 | `update_qrcode()` | 仅使用 `_family_server_id` 作为唯一查询条件。若同一服务器下存在多个包厢，会导致数据覆盖或误更新。且 `SELECT` 后 `INSERT/UPDATE` 存在典型的 **TOCTOU 竞态条件**。 | 使用复合唯一键（如 `_family_server_id` + `_room_id`）；改用 `INSERT ... ON DUPLICATE KEY UPDATE` 或框架的 `replace()` 方法。 | `$this->db->replace($this->table_name, $insert_data);` |
| 🟠 警告 | `get_qrcode()` | 方法内部动态加载模型 `$this->load->model()`，若高频调用会导致重复解析与实例化开销；且未对 `$room_data` 数组键进行防御性校验，缺失键将触发 `Undefined index` 通知。 | 依赖加载移至构造函数；使用 `isset()` 或空合并运算符 `??` 进行参数校验。 | `if (!isset($room_data['_merchant_id'], $room_data['_id'])) { return ''; }` |
| 🟡 建议 | 全局/类名 | 类名 `Ahead_room_renewal_mini_qrcode_model` 使用下划线分隔，不符合 PSR-12 的 `PascalCase` 规范，影响自动加载与团队协作。 | 重命名为 `AheadRoomRenewalMiniQrcodeModel`，并全局同步更新引用路径。 | `class AheadRoomRenewalMiniQrcodeModel extends Simple_model` |
| 🟡 建议 | `create_qrcode()` | 硬编码路径 `DEBUG_VERSION . 'ktv/...'` 与域名 `https://vodonline.g-hi.com/`，不利于多环境（开发/测试/生产）切换与配置管理。 | 提取至配置文件（如 `config/qrcode.php`），通过 `$this->config->item()` 动态读取。 | `$oss_path = $this->config->item('qrcode_oss_path'); $domain = $this->config->item('qrcode_cdn_domain');` |
| 🟡 建议 | `get_qrcode()` | 魔法数字 `2` 表示“自助商家”业务模式，语义不清晰，后期维护易出错。 | 定义类常量替代硬编码，提升可读性。 | `const BUSINESS_MODEL_SELF_SERVICE = 2;`<br>`if ($merchant_data['_business_model'] !== self::BUSINESS_MODEL_SELF_SERVICE)` |
### 🔧 核心方法重构参考 (`create_qrcode`)
```php
public function create_qrcode(array $room_data): string
{
// 1. 参数防御
$required = ['_shop_id', '_id', '_merchant_id', '_family_server_id'];
foreach ($required as $key) {
if (!isset($room_data[$key])) {
do_log("Missing key: {$key}", 'renewal_qrcode_error');
return '';
}
}
$scene = "type=2&shop_id={$room_data['_shop_id']}&room_id={$room_data['_id']}";
$wxacodeParams = ['scene' => $scene, 'page' => self::MINI_PAGE];
// 2. 调用微信接口
$response = getUnlimitedWxacode($room_data['_merchant_id'], $wxacodeParams);
// 3. 区分成功(二进制)与失败(JSON)
if (is_string($response) && json_validate($response)) {
$errData = json_decode($response, true);
do_log("WX API Error: " . json_encode($errData), 'renewal_qrcode_error');
return '';
}
// 4. 上传 OSS 并校验
$filename = "{$room_data['_family_server_id']}_{$room_data['_id']}";
$ossKey = $this->config->item('qrcode_oss_path') . "renewal/{$filename}.jpg";
$uploadResult = alioss_internal_addObject_by_content($ossKey, $response);
if (!$uploadResult) {
do_log("OSS Upload Failed for: {$filename}", 'renewal_qrcode_error');
return '';
}
$fileUrl = $this->config->item('qrcode_cdn_domain') . '/' . $ossKey;
$this->update_qrcode($room_data, $fileUrl);
return $fileUrl;
}
```
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **移除全局 `$CI` 调用**：将依赖加载移至 `__construct()`，确保符合框架生命周期，避免 CLI/异步任务崩溃。
2. **修复微信接口返回值处理**：明确区分二进制流与 JSON 错误响应，禁止对非 JSON 数据执行 `json_decode`，并增加 OSS 上传成功校验。
3. **消除数据库竞态条件**：将 `update_qrcode` 中的 `SELECT → INSERT/UPDATE` 逻辑替换为原子操作（如 `REPLACE INTO` 或 `INSERT ... ON DUPLICATE KEY UPDATE`），防止并发请求导致数据错乱。
### 🛠 后续优化方向
- **配置外置化**：将所有环境相关路径、域名、业务状态码抽离至 `config/` 目录，支持多环境无缝切换。
- **类型约束与文档**：为方法参数添加 `array` 类型提示，补充 `@throws` 异常说明，启用 PHP 7.4+ 严格模式（`declare(strict_types=1);`）。
- **缓存策略升级**：当前逻辑每次缺失都重新生成并请求微信接口。建议引入 Redis 缓存层，设置合理的 TTL（如 7 天），降低微信 API 调用频率与数据库 IO。
- **单元测试覆盖**：针对 `create_qrcode` 的失败分支（API 报错、OSS 失败、参数缺失）编写 Mock 测试，确保异常路径可观测。
> 💡 **框架适配提示**：若 `phpci` 框架对模型加载或数据库操作有特定封装（如强制使用 Repository 模式或特定 Query Builder），请优先遵循其官方文档。上述建议基于通用 PHP/CI3 最佳实践，可直接平滑迁移。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781086091
|
1781086091
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
666
|
22
|
83
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 订单退款api
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `fbf39c180 ## 自动代码审查报告
**分支**: app-260616
**提交**: `fbf39c1808c9d230b929c3e80aadf2d78042a9f2`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 16:23:08
---
## 1. 审查摘要
- **代码质量评分**：4.5 / 10 分
- **总体评价**：代码呈现典型的历史遗留系统特征，业务逻辑高度耦合，存在多处严重的安全隐患（硬编码密钥、明文密码比对、SQL 拼接风险）与架构反模式（上帝控制器、构造函数过重、直接操作超全局变量）。整体可维护性、扩展性与安全性均不达标，需进行系统性重构。
- **风险等级**：🔴 高
> 📌 **框架说明**：从 `CI_Controller`、`$this->load->`、`defined('BASEPATH')` 等特征判断，当前代码基于 **CodeIgniter 3** 架构。若 `phpci` 为贵司内部定制框架，以下安全与架构规范同样适用。具体组件调用请以 `phpci` 官方文档为准。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `MerchantAppServer.php`<br>`case '0005'` | **硬编码敏感凭证**：科大讯飞 TTS 的 `APISecret`、`APIKey` 直接暴露在业务逻辑中，极易通过版本库或反编译泄露。 | 移至独立配置文件或环境变量，通过框架配置类读取。禁止在代码中明文存储密钥。 | `// config/keys.php\n$config['xfyun'] = ['APPID' => '...', 'APISecret' => '...'];\n// 控制器中\n$xfyun = $this->config->item('xfyun');` |
| 🔴 严重 | `MerchantAppServer.php`<br>`case '00064'` | **明文密码存储与比对**：`_discount_pwd` 疑似明文存储，且直接与 `$_old_password` 比对，违反基础安全规范。 | 使用 `password_hash()` 加密存储，`password_verify()` 验证。 | `if (!password_verify($_old_password, $data['_discount_pwd'])) { ... }\n$hash = password_hash($_new_password1, PASSWORD_DEFAULT);` |
| 🔴 严重 | `Ahead_yc_order_model.php`<br>多处查询方法 | **SQL 注入风险**：`$addsql`、`$shop_ids`、`$_start_date` 等变量直接拼接入 SQL 字符串，未使用参数绑定。若上游传入恶意数据将导致注入。 | 全面改用 CI Query Builder 或严格使用 `?` 占位符。禁止拼接外部传入的 SQL 片段。 | `$this->db->where_in('_shop_id', explode(',', $shop_ids));\n$this->db->where('_timestamp >=', $start_date);` |
| 🔴 严重 | `Ahead_pay_log_model.php`<br>`update_refund_amount()` | **原始 WHERE 条件拼接**：`$where = '_relation_id="' . $relation_id . '" ...'` 存在注入风险且易引发语法错误。 | 使用 CI 数组条件或 Query Builder 方法构建查询。 | `$where = ['_relation_id' => $relation_id, '_type' => $type, '_status' => [1, 4]];` |
| 🟠 警告 | `Api.php`<br>`__construct()` & `jsonEcho()` | **输出缓冲滥用 & 未设响应头**：直接操作 `ob_*` 易触发 Warning，且未设置 `Content-Type: application/json`，可能导致客户端解析异常。 | 移除冗余缓冲操作，使用 CI 的 `output` 类统一响应。 | `$this->output->set_content_type('application/json')->set_output(json_encode($result, JSON_UNESCAPED_UNICODE));` |
| 🟠 警告 | `MerchantAppServer.php`<br>`__construct()` & `index()` | **构造函数过重 & 上帝方法**：构造函数执行鉴权、日志、配置加载；`index()` 包含数十个 `case`，严重违反单一职责原则，难以测试与维护。 | 将鉴权/日志移至基类控制器或中间件；按业务域拆分控制器，利用 CI 路由分发。 | 拆分为 `AuthController`、`OrderController`、`PrinterController` 等独立控制器。 |
| 🟠 警告 | `Api.php`<br>`selfChangeRoom()` | **直接修改 CI 超对象属性**：`$CI->merchant_id = ...` 破坏框架封装，易引发请求间状态污染与并发安全问题。 | 通过参数传递上下文数据至 Model，或使用 Session/Request 对象管理状态。 | `$this->load->model('order_model');\n$this->order_model->change_room($merchant_id, $admin_data, $params);` |
| 🟡 建议 | 全局多处 | **浮点数处理金额**：使用 `float` 进行金额加减（如 `bcsub` 未全面覆盖），在 PHP 中可能导致精度丢失（如 `0.1+0.2=0.30000000000000004`）。 | 金额统一以“分”为单位（整数）存储计算，或全面使用 `BCMath` 扩展。 | `$actual_pay = bcsub($pay, $refund, 2);\n$refund_amount = (int)round($refund * 100); // 转为分` |
| 🟡 建议 | 全局多处 | **模型重复加载**：在方法内部频繁 `$this->load->model()`，增加 I/O 开销且不符合 CI 最佳实践。 | 移至构造函数统一加载，或配置 `autoload.php` 自动加载高频模型。 | `public function __construct() { parent::__construct(); $this->load->model('vip_model'); }` |
## 3. 总结与行动建议
### 🚨 优先修复的关键问题（P0）
1. **移除硬编码密钥**：立即将 `MerchantAppServer.php` 中的 `xfyun_tts_config` 迁移至配置文件或密钥管理服务（如 Vault/Env），并轮换已泄露的密钥。
2. **修复密码安全漏洞**：对 `_discount_pwd` 字段执行一次性哈希迁移脚本，后续所有密码比对必须使用 `password_verify()`。
3. **封堵 SQL 注入入口**：全面审查 `$addsql`、`$shop_ids`、`$relation_id` 等变量的来源，替换所有字符串拼接 SQL 为 Query Builder 或参数绑定查询。
### 🛠 后续重构与优化方向
1. **架构解耦与路由规范化**：
- 废弃 `switch ($request['function'])` 的伪路由模式，改用 CI 原生路由配置（`config/routes.php`）映射到独立控制器。
- 将鉴权、日志记录、参数校验等横切关注点抽离至 `MY_Controller` 基类或中间件，保持业务控制器轻量。
2. **统一输入/输出处理**：
- 废弃直接读取 `$_POST`/`$_GET`，统一使用 `$this->input->post()`、`$this->input->get()` 或 `$this->input->raw_input_stream`，利用 CI 内置的 XSS/过滤机制。
- 封装统一的 `ApiResponse` 类，替代 `jsonEcho()`，自动处理 Header、状态码与 JSON 序列化。
3. **财务计算精度保障**：
- 建立全局金额处理规范，禁止使用 `float` 进行财务运算。引入 `BCMath` 或整数（分）计算，并在入库/出库时进行严格校验。
4. **代码规范与可维护性提升**：
- 遵循 PSR-12 规范，统一命名风格（如 `AplicationController` 拼写修正、模型类名大小写统一）。
- 消除魔法数字，将 `1, 2, 3, 15, 16` 等状态码/支付类型提取为类常量或枚举。
- 补充关键方法的 PHPDoc 注释与类型声明（PHP 7.4+ 推荐），提升 IDE 提示与静态分析能力。
> 💡 **提示**：由于提供的代码片段存在截断，部分全局函数（如 `throwError`、`request_frequency`）及基类 `Simple_model` 的实现未完全展示。建议在完整代码库中结合静态分析工具（如 `PHPStan`、`SonarQube`）进行全量扫描，以覆盖潜在的类型不匹配与未捕获异常。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781079788
|
1781079788
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
665
|
18
|
203
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 1
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `d7e5996ab6 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `d7e5996ab65bb821f278a19b971a31f3a3f1c4b9`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 16:18:15
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10
- **总体评价**：代码实现了退款数据的查询、关联与格式化逻辑，基础功能完整。但存在明显的架构反模式（如全局作用域加载、方法内重复加载依赖）、硬编码魔法数字、JSON 解析缺乏容错、以及数据获取与视图格式化严重耦合。整体可维护性、健壮性与性能有较大优化空间。
- **风险等级**：🟠 中（主要隐患在于 JSON 解析异常导致崩溃、硬编码维护成本高、潜在的性能损耗及框架生命周期管理不规范）
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 文件顶部 (全局作用域) | 在类外部使用 `$CI =& get_instance();` 并加载 `Simple_model`。此写法破坏面向对象封装，易引发依赖冲突、内存泄漏，且不符合现代 PHP 框架规范。 | 移除全局 `$CI` 调用。基类 `Simple_model` 应由框架自动加载或继承时自动解析，无需手动 `load`。 | `// 删除顶部代码\n// $CI =& get_instance();\n// $CI->load->model('Simple_model');` |
| 🔴 严重 | `get_refund_data` / `get_refund_log` 循环内 | `json_decode()` 未做严格类型校验。若数据库字段值为 `"null"`、`"false"` 或损坏 JSON，`json_decode` 将返回 `null`，后续 `foreach` 会触发 `Warning: Invalid argument supplied for foreach()`。 | 增加返回值类型强校验，或使用 `is_array()` 兜底。 | `$data = json_decode($json, true);\n$v['refund_amount_info'] = is_array($data) ? $data : [];` |
| 🟠 警告 | `get_refund_data` / `get_refund_log` 方法内 | 在业务方法中频繁调用 `$this->load->model()` 与 `$this->config->load()`。每次调用均触发框架 Loader 检查，造成冗余 I/O 与性能损耗。 | 将依赖加载统一移至 `__construct()` 中，或使用框架的自动加载/依赖注入机制。 | `public function __construct() {\n parent::__construct();\n $this->load->model('ahead_yc_order_refund_infos_model');\n $this->load->model('ahead_shop_config_model');\n $this->load->model('ahead_yc_merchant_user_model');\n $this->config->load('merchant', TRUE);\n}` |
| 🟠 警告 | `get_refund_data` & `get_refund_log` | 硬编码支付平台 ID 数组 `[17, 18, 19, 20, 23, 24, 25, 26, 27, 28]` 重复出现。业务变更时需多处修改，极易遗漏且可读性差。 | 提取为类常量或独立配置文件，使用 `in_array($id, self::CUSTOM_PAY_IDS, true)` 提升安全性。 | `const CUSTOM_PAY_PLATFORMS = [17, 18, 19, 20, 23, 24, 25, 26, 27, 28];\n// 使用时\nif (in_array($vv['pay_platform'], self::CUSTOM_PAY_PLATFORMS, true)) { ... }` |
| 🟠 警告 | `get_refund_data` 方法内 | `$this->setTableName()` 修改表名后，若中间逻辑抛出异常，将导致后续所有查询使用错误的表名（状态未回滚）。 | 使用 `try...finally` 确保表名必定恢复，或封装为独立查询方法避免污染全局状态。 | `try {\n $this->setTableName($this->table_name.' a');\n // ... 查询逻辑\n} finally {\n $this->setTableName($table_name);\n}` |
| 🟠 警告 | `get_refund_log` 循环内 | `$total_refund_amount += $v['refund_amount'];` 未进行类型安全转换。若数据库返回字符串类型金额，可能触发 PHP 警告或浮点精度丢失。 | 累加前强制转换为浮点数，并处理空值。 | `$total_refund_amount += (float) ($v['refund_amount'] ?? 0);` |
| 🟡 建议 | 类定义行 | 类名 `Ahead_yc_order_refund_model` 使用蛇形命名，不符合 PSR-12 规范（类名应使用大驼峰 PascalCase）。 | 重命名为 `AheadYcOrderRefundModel`，并全局同步更新引用。 | `class AheadYcOrderRefundModel extends Simple_model` |
| 🟡 建议 | 方法返回值 | `return $refund_info ?$refund_info : array();` 存在语法空格不规范，且三元表达式冗余。 | 使用空合并运算符简化，提升可读性与执行效率。 | `return $refund_info ?? [];` |
| 🟡 建议 | 架构设计 | 模型层承担了过多视图格式化职责（如拼接 `【套餐配送】`、日期格式化、金额字符串拼接）。违反单一职责原则 (SRP)。 | 将数据格式化逻辑剥离至 `Service` 层或 `ViewModel`，模型仅负责纯净的数据查询与返回。 | `// 模型返回原始数组\n// Service/ViewModel 层负责格式化\n$formatter = new RefundDataFormatter();\nreturn $formatter->format($rawData);` |
| 🟡 建议 | PHPDoc 注释 | 方法注释缺少 `@return` 类型声明及参数类型提示，不利于 IDE 静态分析与团队协作。 | 补充完整 PHPDoc，明确参数与返回值类型。 | `/**\n * @param int $order_id\n * @param string|int $order_type\n * @param int $shop_id\n * @return array\n */` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **移除全局 `$CI` 调用**：彻底清理文件顶部的 `$CI =& get_instance();`，避免破坏框架依赖树。
2. **JSON 解析容错**：对所有 `json_decode` 结果进行 `is_array()` 校验，防止脏数据导致循环崩溃。
3. **依赖加载前置**：将 `load->model()` 与 `config->load()` 统一收敛至构造函数，消除运行时重复加载开销。
4. **硬编码提取**：将支付平台 ID 列表提取为类常量或配置文件，降低后续维护成本。
### 🛠 后续重构与优化方向
- **架构分层**：当前模型混合了 `数据查询` 与 `展示层格式化`。建议引入 `Service` 层处理业务编排，或使用 `DTO/ViewModel` 处理前端展示所需的字符串拼接与格式化，保持 Model 的纯粹性。
- **状态安全管理**：`setTableName()` 属于框架级状态修改，务必配合 `try...finally` 或封装为闭包查询，防止异常中断导致全局状态污染。
- **类型声明升级**：若运行环境为 PHP 7.4+，建议为方法参数与返回值添加类型声明（如 `public function get_refund_data(int $order_id, string $order_type = '', int $shop_id = 0): array`），提升代码健壮性。
- **框架适配说明**：代码结构高度契合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请确认其 Loader 机制、配置缓存策略与 CI3 是否完全一致。对于不确定的生命周期行为，建议查阅 `phpci` 官方文档中关于 `Model 初始化` 与 `Config 缓存` 的最佳实践。
> 💡 **局限性提示**：本次审查基于提供的单文件代码。由于未提供 `Simple_model` 基类实现、`$this->select()` 底层 SQL 构建逻辑及完整框架配置，部分安全性（如底层是否自动参数绑定防注入）与性能评估基于通用框架经验推断。建议结合完整项目上下文进行集成测试验证。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781079495
|
1781079495
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
664
|
18
|
202
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 小程序用户退款记录
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `0d7e1c9d23 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `0d7e1c9d23083f6aa8f7e8061e7300a8fa1aa5ff`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 16:16:42
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：代码实现了核心业务逻辑，但存在明显的遗留框架使用习惯（如全局获取实例、状态化表名切换）、性能瓶颈（循环内字符串拼接、重复加载配置）及现代 PHP 规范缺失。整体可运行，但在高并发、PHP 8+ 环境及长期维护下存在隐患。
- **风险等级**：🟠 中（存在潜在 SQL 注入风险、JSON 解析异常未处理、模型状态污染可能引发数据错乱）
> 📌 **框架说明**：代码结构高度符合 CodeIgniter 3 规范。若 `phpci` 为内部定制框架，请结合其官方文档确认 `Simple_model` 的查询构造器实现机制及生命周期钩子。以下建议基于 CI 架构与现代 PHP (7.4+/8.0+) 标准。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 顶部全局 | 在类外部使用 `$CI =& get_instance(); $CI->load->model('Simple_model');` 违反 OOP 原则与框架生命周期，易导致模型重复加载或状态污染。 | 移除顶部代码，依赖注入或移至构造函数加载。若 `Simple_model` 为父类，直接 `extends` 即可。 | `public function __construct() { parent::__construct(); $this->load->model('ahead_shop_config_model'); }` |
| 🔴 严重 | `get_refund_data` / `get_refund_log` | `$where` 数组直接传入自定义 `select()`，若底层未使用预处理/参数绑定，存在 **SQL 注入** 风险。 | 确保 `Simple_model::select()` 内部使用 `$this->db->where()` 或 PDO 预处理。对外部输入进行类型强转。 | `$where['a._order_id'] = (int) $order_id;`<br>`$where['_unique_key'] = (string) $unique_key;` |
| 🟠 警告 | 多处 `json_decode` | `json_decode()` 未处理非法 JSON 字符串。在 PHP 8+ 中若传入非字符串或格式错误，可能触发 Warning 或返回 `null`，导致后续数组访问报错。 | 增加类型校验与安全回退，或使用 `json_decode($str, true) ?? []`。 | `$data = is_string($json) ? (json_decode($json, true) ?? []) : [];` |
| 🟠 警告 | `get_refund_data` 循环内 | 使用 `$goods_info[...] .= ',' . ...` 进行字符串拼接，频繁分配内存且需 `trim()` 处理末尾逗号，性能较差。 | 改用数组收集，循环结束后使用 `implode()` 合并。 | `$goods_list[] = "{$v['_goods_name']}({$v['_refund_quantity']}{$v['_goods_unit_name']})";`<br>`$goods_info[$id] = implode(',', $goods_list);` |
| 🟠 警告 | `get_refund_log` | `$admin_ids = array_column(...)` 可能为空数组，但仍执行 `get_data_by_ids()` 查询，浪费数据库连接。 | 增加空值判断，提前返回或跳过查询。 | `if (empty($admin_ids)) { $admin_data = []; } else { $admin_data = $this->...->get_data_by_ids(...); }` |
| 🟠 警告 | `setTableName()` 调用 | 通过 `$this->setTableName()` 动态修改模型内部表名状态。在并发请求或同一实例多次调用时，极易引发 **状态污染** 与数据错乱。 | 避免修改实例状态。改用查询构造器直接指定表名/别名，或每次查询前克隆实例。 | `$this->db->select($fields)->from($this->table_name.' a')->join(...)->get()->result_array();` |
| 🟡 建议 | 全局方法签名 | 缺乏类型声明（Type Hints），不符合现代 PHP 规范，降低 IDE 提示与静态分析能力。 | 为参数与返回值添加严格类型声明（PHP 7.4+/8.0+）。 | `public function get_refund_data(int $order_id, string $order_type = '', int $shop_id = 0): array` |
| 🟡 建议 | 硬编码数组 | `[17, 18, 19, 20, 23, 24, 25, 26, 27, 28]` 与魔法数字 `10`, `14`, `1`, `3` 重复出现，语义不明且维护困难。 | 提取为类常量或配置文件，`in_array` 增加严格模式 `true`。 | `private const CUSTOM_PAY_IDS = [17, 18, 19, 20, 23, 24, 25, 26, 27, 28];`<br>`in_array($id, self::CUSTOM_PAY_IDS, true)` |
| 🟡 建议 | 配置加载 | `$this->config->load('merchant', TRUE);` 在方法内重复调用，增加 I/O 开销。 | 移至构造函数加载一次，或缓存至类属性。 | `private $merchantConfig;`<br>`$this->merchantConfig = $this->config->item('merchant');` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **消除全局实例加载**：立即移除文件顶部的 `$CI =& get_instance();`，将依赖模型加载收敛至 `__construct()` 或使用 CI 的 `autoload.php`。
2. **防御 SQL 注入**：确认 `Simple_model::select()` 底层是否使用参数绑定。若未使用，必须对所有外部传入的 `$order_id`、`$unique_key` 进行 `(int)` 或 `(string)` 强转，或改用框架原生 Query Builder。
3. **修复 JSON 解析隐患**：统一替换 `json_decode` 调用，增加 `is_string()` 校验与 `?? []` 安全回退，避免 PHP 8+ 环境下的隐式类型错误。
4. **解除状态污染**：废弃 `setTableName()` 动态切换表名的做法。改为在查询时显式传递表名/别名，或封装独立的查询方法，确保模型实例无状态（Stateless）。
### 🛠 后续重构与优化方向
- **性能优化**：将 `get_refund_data` 中的商品明细查询改为 `JOIN` 关联查询，避免分步查询带来的网络往返开销；使用 `implode()` 替代循环字符串拼接。
- **架构规范化**：
- 遵循 PSR-12 规范，统一类名（建议 `AheadYcOrderRefundModel`）、方法命名与缩进。
- 引入 PHP 8 特性：使用 `readonly` 属性、枚举（`enum`）替代魔法数字、`match` 表达式优化支付平台映射逻辑。
- **可维护性提升**：将硬编码的支付平台 ID、订单类型、日志类型抽离至 `config/` 目录或数据库字典表，实现业务逻辑与配置解耦。
- **单元测试覆盖**：针对 `get_refund_data` 和 `get_refund_log` 编写 PHPUnit 测试用例，重点覆盖空结果集、非法 JSON、边界金额及并发调用场景。
> 💡 **提示**：若 `phpci` 框架对模型生命周期或查询构造器有特殊封装，请优先查阅其官方文档中关于 `Model` 状态管理与 `DB` 驱动绑定的章节，以确保重构方案与框架底层兼容。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781079402
|
1781079402
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
663
|
18
|
201
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - Jh_community_shop_revenues_de 🔍 代码审查报告：pc-260616 - Jh_community_shop_revenues_detail...
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `230bfa5c0f ## 自动代码审查报告
**分支**: pc-260616
**提交**: `230bfa5c0f0903d47edda3edd8f2585a60f34596`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 14:59:24
---
## 1. 审查摘要
- **代码质量评分**：6.0 / 10 分
- **总体评价**：代码实现了社区商家营收明细的查询、过滤与导出功能，业务逻辑基本完整。但存在明显的 SQL 注入风险、N+1 查询性能瓶颈、分页统计逻辑缺陷以及多处不符合现代 PHP/CI 规范的写法。整体可维护性与安全性有待提升。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 第 138-145 行 | **SQL 注入漏洞**：`$pay_platform_where` 数组通过字符串拼接直接构造 SQL 片段，若 `$params['pay_platform_arr']` 来源不可信，将导致严重注入风险。 | 使用查询构建器的参数绑定机制，或强制类型转换后使用框架提供的 `or_where` / `where_in` 组合。 | ```php<br>// 安全写法示例<br>$this->db->group_start();<br>foreach ($params['pay_platform_arr'] as $pp) {<br> $parts = explode('_', $pp);<br> $this->db->or_where('a._pay_platform', (int)$parts[0]);<br> if (!empty($parts[1])) {<br> $this->db->or_where('a._second_pay_platform', (int)$parts[1]);<br> }<br>}<br>$this->db->group_end();<br>``` |
| 🔴 严重 | 第 1 行 | **全局作用域执行实例化**：`$CI = &get_instance();` 放在类外部，文件被 `include/require` 时即执行，破坏框架生命周期，易引发内存泄漏或上下文污染。 | 移除全局代码，将模型加载移至类构造函数中。 | ```php<br>public function __construct()<br>{<br> parent::__construct();<br> $this->load->model('Report_model');<br>}<br>``` |
| 🟠 警告 | 第 108-122 行 | **分页统计逻辑缺陷**：`$count` 与 `$sum_data` 仅在 `$params['page'] == '1'` 时计算。翻至第 2 页时变量未定义，虽用 `??` 兜底，但会导致分页总数与总金额显示为 0，破坏业务体验。 | 移除 `if ($params['page'] == '1')` 条件，始终执行统计查询；或引入缓存机制避免重复计算。 | ```php<br>// 始终计算统计值<br>$count = $this->count($where);<br>$sum_data = $this->get_one($where, 'sum(...) as total_amount');<br>``` |
| 🟠 警告 | 第 158-165 行 | **N+1 查询性能瓶颈**：在 `foreach ($data as &$v)` 循环内调用 `$this->ahead_book_order_model->get_one()`，数据量大时将产生大量数据库往返请求。 | 提取所有关联订单 ID，使用批量查询（如 `get_data_by_ids`）后通过数组映射回填。 | ```php<br>$book_ids = array_unique(array_column($data, 'order_id'));<br>$book_orders = $this->ahead_book_order_model->get_data_by_ids($book_ids, '_id,_shop_name,_arrival_time,_end_time', '_id');<br>// 循环内直接 $v['book_info'] = $book_orders[$book_order_id] ?? [];``` |
| 🟠 警告 | 第 118-120 行 | `json_decode` 未指定关联数组参数，且未处理解析失败情况。若传入非法 JSON，后续逻辑可能静默失败。 | 添加 `true` 参数并配合 `json_last_error()` 或 `JSON_THROW_ON_ERROR` 进行校验。 | ```php<br>$arr = json_decode($params['order_type_arr'], true);<br>if (!is_array($arr)) { $arr = []; }<br>``` |
| 🟡 建议 | 第 1 行 | **类名不符合 PSR-12 规范**：`Jh_community_shop_revenues_detail_model` 使用下划线命名，现代 PHP 推荐 PascalCase。 | 重命名为 `JhCommunityShopRevenuesDetailModel`，并同步更新自动加载与调用处。 | `class JhCommunityShopRevenuesDetailModel extends Report_model` |
| 🟡 建议 | 第 95-98 行 | **日期参数未校验**：`strtotime($params['start_time'])` 若传入非法字符串将返回 `false`，导致查询条件变为 `0` 或 `-1`。 | 增加日期格式校验与默认值回退机制。 | ```php<br>$start = strtotime($params['start_time'] ?? date('Y-m-d'));<br>$end = strtotime($params['end_time'] ?? date('Y-m-d', strtotime('+1 day')));<br>``` |
| 🟡 建议 | 多处 | **模型重复加载**：`$this->load->model()` 在多个方法内重复调用，增加框架解析开销。 | 统一在 `__construct()` 中加载，或使用依赖注入容器管理。 | 见 🔴 严重第 1 行示例 |
> 📌 **框架适配说明**：代码呈现典型的 **CodeIgniter 3** 架构特征（如 `$CI = &get_instance()`、`$this->load->model()`、自定义 `Report_model` 查询构建器）。若 `phpci` 为内部定制框架，请确保 `enforce_con_db()`、`count()`、`select()` 等方法与框架底层 DB 驱动兼容，并查阅官方文档确认查询数组 `$where` 是否支持原生 SQL 片段注入。
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **立即修复 SQL 注入**：第 138-145 行的字符串拼接查询必须改为参数化查询或框架安全构建器，这是最高优先级安全漏洞。
2. **修正分页统计逻辑**：移除 `page == 1` 的条件判断，确保任何分页状态下都能正确返回 `count` 与 `total_amount`。
3. **消除 N+1 查询**：将循环内的单条查询改为批量查询，预计可将该接口响应时间降低 60%~80%（尤其在数据量 > 50 条时）。
### 🛠 后续重构与优化方向
- **架构规范化**：将全局 `$CI` 引用移至构造函数，遵循 PSR-12 命名规范，逐步引入 PHP 8 类型声明（如 `public function get_community_revenues_list(int $merchant_id, array $params, bool $export = false): array`）。
- **查询构建器抽象**：当前 `$where` 数组结构高度定制化，建议封装为独立的 `QueryBuilder` 类或使用框架原生 Active Record，避免手动拼接 `join`、`where`、`like` 导致维护困难。
- **配置与常量分离**：`ORDER_TYPE`、`revenues_pay_platform_arr` 等硬编码数据建议移至 `config/` 目录或数据库字典表，便于运营动态调整。
- **防御性编程**：对 `$params` 增加统一校验层（如使用 `Form_validation` 或自定义 DTO），确保 `start_time`、`end_time`、`shop_id` 等关键字段类型与范围合法后再进入业务逻辑。
> 💡 若需针对 `phpci` 框架的特定查询构建器或缓存机制进行深度适配，请提供 `Report_model` 核心方法签名或框架官方文档链接，以便输出更精准的底层优化方案。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781074764
|
1781074764
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
662
|
18
|
200
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 需求-修改包厢停用文案
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `3f2788382e ## 自动代码审查报告
**分支**: pc-260616
**提交**: `3f2788382e253594920d1fecb1914bc69d2beca9`
**提交人**: caihongyuchy (1091045324@qq.com)
**时间**: 2026-06-10 13:21:30
---
## 📋 审查摘要
- **变更文件数**: 0
- **严重问题**: 0
- **高危问题**: 0
- **中危问题**: 0
- **建议优化**: 0
## 🐛 发现的问题
*(注：您提供的输入中 `## 变更文件内容` 部分为空，未检测到任何可审查的代码片段。请补充具体代码后重新提交。)*
## ✅ 代码亮点
*(待代码提供后补充)*
## 📝 总体建议
您已提供了 CodeIgniter 3 的核心系统文件结构（`system/` 目录下的 helpers、libraries、database 驱动等），但 **`## 变更文件内容` 部分为空**。
请补充您需要审查的具体业务代码（如控制器、模型、自定义库或视图）。收到代码后，我将立即严格按照您的要求执行以下深度审查：
1. 🔴 **跨文件引用验证**：逐行核对 `$this->load->model()`、`$this->load->library()`、`$this->load->helper()` 及自定义类/方法调用，确保拼写正确且文件真实存在。
2. 🔴 **CI 框架规范**：验证模型命名是否严格对应 `Xxx_model.php`，控制器是否对应 `Xxx.php`，检查 CI 3.x 特有语法兼容性。
3. 🛡️ **安全与逻辑**：重点扫描 SQL 注入（未使用查询绑定）、XSS（未过滤输出）、未定义变量、类型错误、资源未释放及越权访问风险。
4. 📝 **代码质量**：检查硬编码、重复逻辑、错误处理缺失及命名规范。
请粘贴变更代码，我将为您生成符合指定格式的完整审查报告。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781068890
|
1781068890
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
661
|
22
|
82
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 预订管理
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `e6fd8b8b7 ## 自动代码审查报告
**分支**: app-260616
**提交**: `e6fd8b8b7e1699eeac1d338194fc5d8217f67ea7`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 11:21:00
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：业务逻辑覆盖较全面，能处理预订、关联账单、权限校验及短信通知等复杂场景。但代码存在**严重的 SQL 注入风险**、**事务管理不规范**、**N+1 查询性能瓶颈**以及**MVC 职责严重越界**等问题。整体架构偏向老旧的 CodeIgniter 3 写法，缺乏现代 PHP 的类型约束、单一职责划分与防御性编程意识。
- **风险等级**：🔴 高
> 📌 **框架说明**：提交代码的语法特征（`&get_instance()`、`$this->load->model()`、`$this->db->trans_start()` 等）高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，以下建议基于 CI3 核心机制与现代 PHP 最佳实践，架构原则通用。
---
## 2. 问题详情
| 严重程度 | 文件/方法 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_book_list` / `update_book` | **SQL 注入风险**：直接将用户输入 `$params['book_no']` 拼接到 `LIKE` 语句，且使用 `implode(",", $ids)` 拼接 `IN` 条件，未做任何转义或参数绑定。 | 使用查询构造器（Query Builder）或 `$this->db->escape()` / `escape_like_str()` 进行安全处理。 | `$this->db->like('book._book_no', $params['book_no']);`<br>`$this->db->where_in('_id', $ids);` |
| 🔴 严重 | `add_book` | **事务管理混乱**：混用 `trans_start()`、手动 `trans_rollback()` 与 `try-catch`。若 `catch` 中直接 `return` 而未调用 `trans_complete()`，可能导致连接池事务未释放或死锁。 | 遵循框架标准事务流：移除手动 `trans_rollback()`，统一在 `trans_complete()` 后通过 `trans_status()` 判断结果。 | 见下方 `事务重构示例` |
| 🟠 警告 | `get_book_list` | **N+1 查询性能瓶颈**：在 `foreach ($list['rows'] as &$row)` 循环内多次调用 `get_one()`（查用户、查兑换记录、查VIP等）。数据量 >100 时将引发严重性能雪崩。 | 收集所有关联 ID，使用 `WHERE IN` 批量查询，在内存中通过 `array_column` 映射数据。 | 见下方 `批量查询优化示例` |
| 🟠 警告 | `add_book` / `update_book` | **MVC 职责越界**：Model 层直接处理短信发送、房态推送、操作日志记录、权限校验及业务规则判断。违反单一职责原则，难以测试与维护。 | 将非持久化逻辑抽离至 `Service` 层或控制器，Model 仅负责数据读写。复杂流程可改用事件/队列解耦。 | 模型仅保留 `$this->insert()`，业务逻辑移至 `BookService::create()` |
| 🟠 警告 | 文件顶部 | **全局作用域加载模型**：`$CI = &get_instance(); $CI->load->model('Simple_model');` 在类外部执行，易导致重复加载、内存泄漏或 CLI 环境报错。 | 移除全局代码，在 `__construct()` 中按需加载，或依赖框架自动加载机制。 | `public function __construct() { parent::__construct(); }` |
| 🟡 建议 | 全局 | **缺乏类型声明与防御性编程**：方法无参数类型/返回类型约束；`strtotime()` 未校验格式；`throwError()` 为全局函数，可能暴露堆栈信息。 | 添加 PHP 7+ 类型提示；对时间/手机号等关键参数做前置校验；使用框架标准异常类。 | `public function add_book(int $merchantId, int $uid, array $params): array` |
| 🟡 建议 | `add_book` | **魔法数字与硬编码**：大量使用 `1, 2, -1, 7, 11, 22, 34` 等状态码/模板ID，可读性差且易出错。 | 提取为类常量或配置项，如 `const SMS_TEMPLATE_BOOK_SUCCESS = 34;` | `const SMS_TEMPLATE_BOOK_ADMIN = 22;` |
### 🔧 核心代码重构示例
**1. 事务安全写法（替代原 `try-catch` + 手动回滚）**
```php
$this->db->trans_begin(); // 或 trans_start()
// 执行所有数据库操作...
$result = $this->insert($addData);
if ($result === false) {
$this->db->trans_rollback();
return ['success' => false, 'msg' => '预订失败'];
}
// 其他更新操作...
$this->db->trans_commit(); // 或 trans_complete()
if ($this->db->trans_status() === FALSE) {
$this->db->trans_rollback();
return ['success' => false, 'msg' => '数据库事务异常'];
}
// 事务成功后执行非DB操作（短信、通知等）
```
**2. N+1 查询优化（`get_book_list` 循环内）**
```php
// 1. 收集所有需要查询的ID
$userIds = array_unique(array_filter(array_column($list['rows'], 'ahead_user_id')));
$exchangeIds = array_unique(array_filter(array_column($list['rows'], 'group_platform_order_id')));
// 2. 批量查询
$userMap = $exchangeMap = [];
if ($userIds) {
$users = $this->ahead_user_model->get_list(['_id' => $userIds], '_id,_mobile');
$userMap = array_column($users, '_mobile', '_id');
}
if ($exchangeIds) {
$exchanges = $this->ahead_tuangou_exchange_log_model->get_list(['_id' => $exchangeIds], '_id,_platform_voucher_code');
$exchangeMap = array_column($exchanges, '_platform_voucher_code', '_id');
}
// 3. 循环内直接内存映射
foreach ($list['rows'] as &$row) {
if (empty($row['customer_contact']) && isset($userMap[$row['ahead_user_id']])) {
$row['customer_contact'] = $userMap[$row['ahead_user_id']];
}
// ... 其他逻辑同理
}
```
---
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **彻底修复 SQL 注入**：全局搜索 `$params` 直接拼接 SQL 的位置，统一替换为 `$this->db->where()`、`$this->db->like()` 或 `$this->db->query($sql, $bindings)`。
2. **规范事务生命周期**：移除 `try-catch` 中的手动 `trans_rollback()`，严格遵循 `trans_begin() -> 业务逻辑 -> trans_commit() -> trans_status()` 流程。
3. **消除循环查库**：对 `get_book_list` 中的关联数据查询实施批量加载（Batch Fetching），预计可提升列表接口响应速度 **60%~80%**。
### 🛠 后续重构方向
1. **架构分层（Service 模式）**：当前 Model 承担了 Controller 和 Service 的职责。建议创建 `BookService`，将权限校验、短信发送、房态同步、操作日志等逻辑移出 Model，Model 仅保留 `CRUD` 与基础数据校验。
2. **引入现代 PHP 特性**：
- 添加严格类型声明：`declare(strict_types=1);`
- 使用 PHP 8 属性/类型提示：`public function add_book(int $merchantId, int $uid, array $params): array`
- 替换全局函数 `throwError()` 为 `throw new \InvalidArgumentException('...')` 或框架标准异常。
3. **配置与常量管理**：将硬编码的短信模板 ID、状态映射、业务阈值抽离至 `config/book.php` 或类常量中，便于多环境部署与后期维护。
4. **代码格式化**：使用 `PHP-CS-Fixer` 或 `Prettier` 统一应用 PSR-12 规范，拆分超过 150 行的方法（如 `add_book` 可拆分为 `validate()`, `prepareData()`, `executeTransaction()`, `notify()`）。
> ⚠️ **局限性说明**：提供的代码在 `update_book` 方法末尾处截断（`if (isset($params['welcome_minutes']) && $params['welcome_minutes']) {`），未能审查完整逻辑。若该方法后续包含类似拼接或事务操作，请同步按上述标准修复。建议补充完整代码或进行二次审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781061660
|
1781061660
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
660
|
21
|
307
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - 关房失败记录日志
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `c4228b887 ## 自动代码审查报告
**分支**: pay-260616
**提交**: `c4228b88720d75994636e692a6eb1a6bfc8c07b6`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 10:53:38
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码实现了预订下单、支付回调、退款及消息通知等核心业务，但存在明显的架构反模式。事务管理机制混乱、存在 SQL 注入隐患、循环内执行数据库查询导致性能瓶颈，且单个方法过长严重违反单一职责原则（SRP）。整体安全性、可维护性与执行效率亟待重构。
- **风险等级**：🔴 高
> 📌 **框架说明**：根据 `$CI = &get_instance()`、`$this->load->model()`、`$this->db->trans_start()` 等特征，代码高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请核对底层事务计数器、配置加载及查询构造器 API 的差异。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `refund_by_notify` 方法内 | **SQL 注入风险**：使用字符串拼接构造 `$log_where` 条件，若 `$order_data['_id']` 来源不可控，将导致注入。 | 使用框架查询构造器或参数绑定，禁止直接拼接 SQL 片段。 | `$this->db->where('_relation_id', $order_data['_id'])->where('_status', 1)->where_in('_type', [5, 13]);`<br>`$this->ahead_pay_log_model->up($log_up, $this->db->get_compiled_where());` |
| 🔴 严重 | `check_notify` 方法内 | **事务管理混乱**：混用 `trans_start()` 与手动 `trans_rollback()`。CI 的 `trans_start()` 依赖内部计数器，手动回滚会破坏状态，且提前 `return` 未调用 `trans_complete()` 可能导致连接池锁死。 | 统一使用 `trans_begin()` 配合手动 `trans_commit()`/`trans_rollback()`，或完全依赖 `trans_start()` + `trans_complete()` 自动机制。 | `$this->db->trans_begin();`<br>`try { /* 业务 */ $this->db->trans_commit(); } catch(\Exception $e) { $this->db->trans_rollback(); }` |
| 🟠 警告 | `get_list` 方法内 | **N+1 查询性能瓶颈**：在 `foreach` 循环中逐条查询商户信息，数据量大时将导致严重性能衰减。 | 批量查询或使用 `WHERE IN` 一次性获取，在内存中映射。 | `$ids = array_column($order_info, 'merchant_id');`<br>`$merchants = $this->ahead_merchant_model->get_many(['_id' => $ids], '_id,_business_model');`<br>`$map = array_column($merchants, '_business_model', '_id');` |
| 🟠 警告 | 文件顶部 & `send_success_msg` | **全局实例化与重复加载**：文件顶部 `$CI = &get_instance();` 在类外执行，易在框架未完全初始化时报错；方法内频繁 `$this->load->model()` 增加 I/O 开销。 | 移除顶部实例化；将高频使用的 Model 移至 `__construct()` 或配置自动加载；配置加载改用 `$this->load->config()`。 | `public function __construct() { parent::__construct(); $this->load->model('ahead_merchant_model'); }` |
| 🟠 警告 | `refund_by_notify` 金额计算 | **浮点数精度丢失**：使用 `floatval()` 处理金额，PHP 浮点运算易产生 `0.0000000001` 级误差，导致对账失败。 | 金额统一以“分”为单位使用整数运算，或使用 `bcmath` 扩展。 | `$wx_pay_amount = bcsub($wx_pay_amount, $change_total_pay[1]['total_pay'] ?? '0', 2);` |
| 🟡 建议 | 全文多处 | **代码规范与可维护性**：混合使用 `array()` 与 `[]`；存在大量注释死代码；魔法数字泛滥（如 `-1, 1, 2, 14, 56`）；日志函数名不统一（`doLog` vs `do_log`）。 | 遵循 PSR-12；统一短数组语法；提取业务常量；清理注释代码；统一日志调用入口。 | `const STATUS_PENDING = -1; const STATUS_PAID = 1;`<br>`$this->logger->info('msg');` |
| 🟡 建议 | `create_community_shop_book_order` 末尾 | **代码截断**：文件在 `if ($this->tuangou->verify_token) {` 处突然结束，逻辑不完整，无法评估后续分支。 | 补充完整代码或说明缺失部分。当前审查仅基于已提供片段。 | 无 |
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **修复 SQL 注入**：立即将 `refund_by_notify` 中的字符串拼接条件替换为查询构造器或参数化查询。
2. **规范事务流程**：将 `check_notify` 中的 `$this->db->trans_start()` 改为 `$this->db->trans_begin()`，确保所有异常分支和提前返回均正确调用 `trans_rollback()`，成功分支调用 `trans_commit()`。
3. **消除 N+1 查询**：重构 `get_list` 方法，将循环内的单条查询改为批量查询，预计可提升列表接口 50%~80% 的响应速度。
### 🛠 后续重构与优化方向
1. **拆分“上帝模型”**：当前 `Ahead_book_order_model` 承担了订单创建、支付回调、退款、消息推送、库存扣减、财务对账等职责。建议按 **领域驱动设计 (DDD)** 或 **服务层模式** 拆分：
- `OrderService`：处理下单、退款业务流
- `PaymentNotifyHandler`：专注支付回调与幂等控制
- `NotificationService`：封装微信模板消息与短信发送
- `InventoryService`：处理套餐/包厢库存扣减与回滚
2. **统一金额处理规范**：全项目金额字段建议统一转为 `int`（单位：分），入库/出库时使用 `bcadd`/`bcsub`/`bcmul`/`bcdiv`，彻底杜绝浮点精度问题。
3. **清理技术债务**：
- 移除所有 `//` 注释的废弃代码，使用 Git 版本控制追溯历史。
- 将魔法数字提取为类常量或配置文件（如 `config/book_status.php`）。
- 统一日志函数调用（建议封装为 `Logger` 类或统一使用 `log_message()`）。
4. **框架适配确认**：若 `phpci` 非标准 CI3，请重点核对 `$this->db->trans_*` 系列方法的行为差异，以及 `$this->load->config()` 的命名空间隔离机制，必要时查阅官方文档进行 API 对齐。
> 💡 **审查局限性说明**：由于提供的代码在 `create_community_shop_book_order` 方法末尾截断，无法完整评估团购券核销、组合支付尾款处理等后续逻辑。建议补充完整文件后再次进行针对性审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781060018
|
1781060018
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
659
|
21
|
306
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - 消息加包厢信息
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `47c710f82 ## 自动代码审查报告
**分支**: pay-260616
**提交**: `47c710f82f528f960f1f040bd219ca307b697e8d`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 10:24:03
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码实现了基础的消息通知与分表路由逻辑，业务意图清晰。但存在未定义变量、魔法数字泛滥、重复代码过多、模型重复加载等典型问题。部分逻辑缺乏边界防御，在并发或异常数据场景下易引发运行时警告或静默失败。整体可维护性与健壮性有较大提升空间。
- **风险等级**：🟠 中（存在明确运行时隐患与性能瓶颈，需优先修复）
> 📌 **框架说明**：代码结构、加载方式及目录规范与 **CodeIgniter 3** 高度一致。以下审查基于 CI3/PHPCI 通用最佳实践。若为深度定制框架，请以官方文档为准。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `after_order` (~138行) | `$admin_data` 变量从未定义，直接访问将触发 `PHP Warning`，导致 `_nickname` 始终为空字符串。 | 明确数据来源。若为当前操作管理员，应通过方法参数传入或从 Session/鉴权服务获取。 | `$data['_nickname'] = $admin_data['_admin_name'] ?? '';`<br>改为：`$data['_nickname'] = $admin_name ?? '';`（由调用方传入） |
| 🔴 严重 | `add_call_pay_order` (~108行) | `$content` 仅在 `_pay_platform` 为 `4` 或 `5` 时赋值。若传入其他支付类型，后续 `$insert_data['_content'] = $content;` 将报未定义变量错误。 | 补充 `else` 默认分支或初始化 `$content = ''`，确保变量始终存在。 | `else { $content = '未知支付类型订单待处理'; }` |
| 🟠 警告 | 全局 (~9行) | `$CI = &get_instance();` 置于类外部，违反框架生命周期规范。在部分 PHP 版本或 CLI 环境下会导致上下文丢失或 Fatal Error。 | 移除全局实例化。模型内部应直接使用 `$this->load->...` 或 `$this->config->...`。 | 删除顶部 `$CI = &get_instance();` 及 `$CI->load->model('Sub_table_model');` |
| 🟠 警告 | `push_notice` (~155行) | `$notice_menu_type[$jsondata['type']]['menu_id']` 未做键存在性校验。若配置缺失或 `_type` 值越界，将抛出 `Undefined index` 并中断推送。 | 使用空合并运算符或 `isset()` 提供安全降级逻辑。 | `$jsondata['menu_id'] = $notice_menu_type[$jsondata['type']]['menu_id'] ?? 0;` |
| 🟠 警告 | `insert` / 多处 | 方法内频繁调用 `$this->load->model()`。CI 框架中重复加载会增加内存与 I/O 开销，且破坏依赖注入原则。 | 统一移至 `__construct()` 中加载，或配置自动加载（autoload）。 | `public function __construct() { parent::__construct(); $this->load->model(['ahead_family_servers_model', 'ahead_shop_model', 'Ahead_merchant_model']); }` |
| 🟡 建议 | 全文多处 | 大量魔法数字硬编码（如 `_type=8`, `_php_version>190220`, `_pay_platform` 等），业务语义不透明，后期维护成本极高。 | 提取为类常量（Constants），集中管理业务枚举与阈值。 | `const TYPE_STOCK_UP = 8; const VERSION_SHARDING_THRESHOLD = 190220; const PAY_CASH = 4;` |
| 🟡 建议 | `insert` (~58行) | 每次插入都执行两次 `get_one` 查询。若在循环/批量场景中调用，将引发严重的 **N+1 查询性能瓶颈**。 | 建议由调用方补全 `_room_name`/`_shop_name`，或引入缓存层（如 CI Cache/Redis）避免重复查库。 | `if (empty($arr['_room_name']) && !empty($arr['_room_id'])) { ... }` |
| 🟡 建议 | 全文 | 多个 `add_*_notice` 方法数据结构高度重复，违反 DRY 原则；存在拼写错误 `merchang_data`；未遵循 PSR-12 规范（如 `array()` vs `[]`、缺少类型声明）。 | 抽取私有方法 `buildNoticeData()` 统一组装数据；修正拼写；补充 PHP 7+ 类型提示。 | 见下方重构示例 |
---
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **修复未定义变量**：立即处理 `after_order` 中的 `$admin_data` 与 `add_call_pay_order` 中的 `$content` 未初始化问题，避免生产环境日志污染与数据丢失。
2. **移除全局 `$CI` 实例化**：将 `$CI = &get_instance();` 从类外部移除，改为在构造函数或方法内部按需使用 `$this->load->...`。
3. **防御性编程**：为 `push_notice` 中的配置数组访问添加 `??` 或 `isset()` 保护，防止因配置缺失导致推送服务崩溃。
### 🛠 后续重构与优化方向
1. **提取公共数据组装逻辑**：当前 7 个通知方法结构高度相似，建议封装为受保护方法，减少重复代码并统一字段校验。
```php
protected function buildNoticeData(array $baseData, array $overrides = []): array
{
$defaults = [
'_status' => 0,
'_createdtime' => time(),
'_speech' => 0,
'_extended_field' => 0,
];
return array_merge($defaults, $baseData, $overrides);
}
```
2. **集中管理魔法数字**：将业务枚举与分表阈值定义为类常量，提升可读性。
```php
class Ahead_yc_notice_model extends Sub_table_model
{
const TYPE_ORDER_PAID = 1;
const TYPE_ORDER_UNPAID = 2;
const TYPE_STOCK_UP = 8;
const TYPE_BOOKING = 12;
const SHARDING_VERSION_THRESHOLD = 190220;
// ...
}
```
3. **优化分表路由逻辑**：当前 `set_table()` 在多个方法中重复调用，且 `insert()` 内部也会调用。建议统一在 `insert()` 或父类钩子中处理，避免重复执行或状态覆盖。
```php
public function insert(array $data)
{
// 统一处理分表逻辑
if (isset($data['_merchant_id'])) {
$this->set_table($data['_merchant_id']);
}
// 补充缺失的名称（建议改为缓存或调用方传入）
$data = $this->fillMissingNames($data);
return parent::insert($data);
}
```
4. **规范 HTTP 请求调用**：`curlRequest()` 为全局辅助函数，硬编码参数 `1, 1, 1, 1, false` 语义不明。建议封装为独立服务类或使用框架提供的 `CURL` 库/Guzzle，便于单元测试与超时重试策略管理。
5. **遵循 PSR-12 与类型安全**：逐步替换 `array()` 为 `[]`，为方法参数与返回值添加类型声明（如 `array $data`, `bool`, `int`），开启 `declare(strict_types=1);` 提升代码健壮性。
> 💡 **提示**：若 `phpci` 框架对模型生命周期、自动加载或数据库驱动有特殊约定，请优先查阅其官方文档。上述建议基于现代 PHP (7.4+) 与主流 MVC 框架最佳实践，可直接平滑迁移。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781058243
|
1781058243
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
658
|
21
|
305
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - Merge branch 'pay-260616 🔍 代码审查报告：pay-260616 - Merge branch 'pay-260616' of https://gitea.g-hi.co...
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `3eb99cf2e ## 自动代码审查报告
**分支**: pay-260616
**提交**: `3eb99cf2e9b7a293aaff344582d130a791ed700a`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 09:54:52
---
## 📋 审查摘要
- **变更文件数**: 5
- **严重问题**: 3
- **高危问题**: 5
- **中危问题**: 2
- **建议优化**: 3
## 🐛 发现的问题
### <font color="red">[语法错误] PHP 文件顶部错误调用 get_instance() 导致 Fatal Error</font>
- **严重程度**: <font color="red">严重</font>
- **文件**: `application/models/Ahead_yc_order_model.php`
- **行号**: 约第 3-4 行
- **问题描述**: 在类定义外部直接调用 `$CI = &get_instance();` 和 `$CI->load->model('Simple_model');`。在 CodeIgniter 框架中，`get_instance()` 仅在类实例化后（如构造函数或方法内部）可用。在文件解析阶段调用会导致 `Call to undefined function get_instance()` 或 `Fatal error`，模型根本无法加载。
- **修复建议**: 将模型加载移至类的构造函数 `__construct()` 中：
```php
class Ahead_yc_order_model extends Simple_model {
public function __construct() {
parent::__construct();
$this->load->model('Simple_model'); // 若父类未自动加载
}
// ...
}
```
### <font color="red">[语法错误] PHP 文件末尾代码截断/不完整</font>
- **严重程度**: <font color="red">严重</font>
- **文件**: `application/models/Ahead_yc_order_model.php`
- **行号**: 文件末尾（约第 680 行）
- **问题描述**: `get_timing_order` 方法未闭合，代码在 `return ['success' => false, 'msg' => '订单` 处突然中断，缺少闭合的字符串引号、数组括号、方法大括号及文件结束符。直接导致 PHP 解析失败。
- **修复建议**: 补全缺失的代码结构：
```php
return ['success' => false, 'msg' => '订单不存在'];
}
} // 闭合 get_timing_order
} // 闭合 class
```
### <font color="red">[跨文件调用] 引用了未定义的函数/常量 (throwError, DEFAULTIMG, minToStr, send_wx_pay_order)</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `application/models/Ahead_yc_order_model.php`
- **行号**: 约第 230, 115, 265, 615 行
- **问题描述**: 代码中直接调用了 `throwError('订单不存在')`、`DEFAULTIMG`、`minToStr(...)` 和 `send_wx_pay_order(...)`。这些均非 PHP 原生函数或 CI 内置方法，也未在当前文件或提供的结构中定义。若未正确引入对应 Helper 或全局文件，将触发 `Fatal error: Uncaught Error: Call to undefined function` 或 `Use of undefined constant`。
- **修复建议**:
1. 确认这些函数/常量所在的 Helper 文件（如 `application/helpers/common_helper.php`）是否已自动加载或手动 `require`。
2. 若为自定义全局函数，建议在文件顶部显式引入或改用 CI 的 `show_error()` / `log_message()`。
3. 常量建议通过 `defined('DEFAULTIMG') or define('DEFAULTIMG', '...');` 或配置项统一管理。
### <font color="red">[跨文件调用] 加载的模型文件未在项目中提供/命名不规范</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `application/models/Ahead_yc_order_model.php`
- **行号**: 多处（如 118, 124, 228, 315, 610 等）
- **问题描述**: 代码中大量使用 `$this->load->model()` 加载模型，但提供的文件结构中均未包含对应文件。部分命名不符合 CI 规范：
- `$this->load->model("ahead_open_room_log");` 缺少 `_model` 后缀，CI 默认期望文件名为 `ahead_open_room_log_model.php`。
- 大小写混用：`$this->load->model('Ahead_yc_order_infos_model');` 与 `$this->load->model("ahead_yc_order_infos_model");` 混用，在 Linux 严格区分大小写的服务器上可能导致加载失败。
- **修复建议**:
1. 确保所有被加载的模型文件存在于 `application/models/` 目录下。
2. 统一命名规范：文件名必须为 `xxx_model.php`，类名必须为 `Xxx_model`。
3. 统一使用小写加载：`$this->load->model('ahead_open_room_log_model');`。
### [安全隐患] SQL 注入风险（字符串拼接查询条件）
- **严重程度**: 高危
- **文件**: `application/models/Ahead_yc_order_model.php`
- **行号**: 约第 310 行
- **问题描述**: 在 `get_bill_goods_info` 方法中，使用 `$sql = '_unique_key="' . $unique_key . '" AND ...'` 直接拼接字符串作为查询条件传入 `$this->select($sql)`。若 `$unique_key` 来源于用户输入或外部接口，未进行转义或参数化绑定，将导致严重的 SQL 注入漏洞。
- **修复建议**: 使用 CI 查询构建器的数组条件或参数绑定：
```php
$where = [
'_unique_key' => $unique_key,
'_status' => [1, 4],
'_timestamp>' => time() - 7 * 86400
];
// 或使用 $this->db->where() 链式调用，避免手动拼接 SQL 字符串
$order_data = $this->select($where);
```
### [逻辑 BUG] JS 空指针异常风险 (wx.getStorageSync)
- **严重程度**: 高危
- **文件**: `web/Hi-Zan/Hi-Zan/pages/community-reserve/order-detail/order-detail.js`
- **行号**: 约第 45 行
- **问题描述**: `const uid = wx.getStorageSync('userInfo').uid || ''`。若本地缓存中不存在 `userInfo` 键，`wx.getStorageSync` 返回 `undefined` 或 `null`，直接访问 `.uid` 会抛出 `TypeError: Cannot read properties of undefined (reading 'uid')`，导致页面白屏崩溃。
- **修复建议**: 增加安全访问判断：
```javascript
const userInfo = wx.getStorageSync('userInfo') || {};
const uid = userInfo.uid || '';
```
### [逻辑 BUG] JS 竞态条件/未初始化数据访问
- **严重程度**: 高危
- **文件**: `web/Hi-Zan/Hi-Zan/pages/community-reserve/order-detail/order-detail.js`
- **行号**: 约第 115, 140, 160 行（`onApplyClick`, `scanToOpen`, `onContinueBookClick` 等）
- **问题描述**: 页面 `onShow` 中调用 `getOrderDetail()` 或 `getMyRoomOrderDetail()` 获取数据，这些是异步请求。但多个按钮点击事件（如申请退款、扫码开门、续费）直接读取 `this.data.order_detail.xxx`。若用户点击过快或网络延迟，`order_detail` 仍为初始空对象 `{}`，导致 `undefined` 属性访问或逻辑判断失效。
- **修复建议**: 在操作前增加数据加载状态校验：
```javascript
if (!this.data.order_detail.id) {
wx.showToast({ title: '订单信息加载中，请稍候', icon: 'none' });
return;
}
// 或添加 loading 状态锁
```
### [代码质量] 模型内部冗余调用自身实例
- **严重程度**: 中危
- **文件**: `application/models/Ahead_yc_order_model.php`
- **行号**: 约第 275 行
- **问题描述**: `$before_order_info_data = $this->ahead_yc_order_model->get_one(...)`。在模型内部调用自身，CI 会尝试重新加载该模型并创建新实例，不仅浪费资源，还可能引发循环依赖或属性覆盖。当前类本身已继承 `Simple_model`，应直接使用 `$this->get_one()`。
- **修复建议**: 替换为 `$before_order_info_data = $this->get_one(array('_id' => $order_info['before_order_id']));`
### [代码质量] 函数过长且职责不单一
- **严重程度**: 中危
- **文件**: `application/models/Ahead_yc_order_model.php`
- **行号**: `get_bill_goods_info` 方法（约 300+ 行）
- **问题描述**: 该方法承担了订单查询、账单计算、商品合并、退款统计、格式化输出等过多职责，嵌套层级深，难以维护和测试。
- **修复建议**: 拆分为多个私有方法，如 `_calculate_totals()`, `_merge_goods_list()`, `_format_bill_output()`，提升可读性与可测试性。
## ✅ 代码亮点
1. **前端回调封装规范**：`order.js` 中统一使用 `success` 和 `error` 回调处理请求结果，结构清晰，便于后续统一拦截或日志上报。
2. **状态机设计合理**：`order-detail.js` 中通过 `bigType` 区分 `book` 和 `room` 订单，并分别调用对应接口，业务边界清晰。
3. **防御性编程意识**：JS 中多处使用 `|| ''` 或 `|| 0` 提供默认值，减少了部分 `undefined` 渲染问题。
## 📝 总体建议
1. **优先修复致命语法错误**：PHP 文件顶部的 `get_instance()` 调用和末尾的代码截断必须立即修复，否则服务将直接 500 报错。
2. **统一跨文件依赖管理**：建议建立依赖清单或使用 IDE 的引用检查工具，确保所有 `load->model()` 和 `import` 的文件真实存在且命名符合框架规范。
3. **加强安全与健壮性**：PHP 端严禁字符串拼接 SQL，必须使用参数化查询；JS 端所有异步数据消费前必须校验加载状态，避免竞态条件引发崩溃。
4. **代码重构规划**：`get_bill_goods_info` 方法已严重违反单一职责原则，建议在下一个迭代周期进行拆分重构，并补充单元测试覆盖核心计算逻辑。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781056492
|
1781056492
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
657
|
21
|
304
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - 立即开房申请退款
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `992fc0cc4 ## 自动代码审查报告
**分支**: pay-260616
**提交**: `992fc0cc44f968c62e20e2681838a93533da9108`
**提交人**: linyangrui (yangruilin888@gmail.com)
**时间**: 2026-06-09 19:40:32
---
## 📋 审查摘要
- **变更文件数**: 4
- **严重问题**: 1
- **高危问题**: 3
- **中危问题**: 3
- **建议优化**: 4
> ⚠️ **框架说明**：本次审查的代码为 **微信小程序 JavaScript** 前端代码，非 PHP CodeIgniter 后端代码。因此 CI 框架特定的模型加载规范（如 `$this->load->model()`）不适用，但已严格按照最高优先级对前端跨文件引用、类/方法存在性进行了等效验证。
## 🐛 发现的问题
### <font color="red">[语法错误] 回调参数与内部变量同名导致重复声明</font>
- **严重程度**: <font color="red">严重</font>
- **文件**: `web/Hi-Zan/Hi-Zan/pages/community-reserve/order-detail/order-detail.js`
- **行号**: 约 108 行 (`getOrderDetail` 方法内)
- **问题描述**: 在 `reserveModel.getBookOrderDetail` 的箭头函数回调 `(res) => { ... }` 中，内部使用了 `const res = wx.getStorageSync('handleOpenMachineResultRes')`。在 ES6 严格模式下，同一块级作用域内重复声明 `const` 变量会直接抛出 `SyntaxError: Identifier 'res' has already been declared`，导致页面白屏崩溃。即使不报错，也会覆盖外部传入的 API 响应对象 `res`，导致后续 `res.result` 访问失败。
- **修复建议**: 将内部存储变量重命名，避免与作用域参数冲突：
```javascript
// 修复前
const res = wx.getStorageSync('handleOpenMachineResultRes')
// 修复后
const storageRes = wx.getStorageSync('handleOpenMachineResultRes')
this.handleOpenMachineResult(storageRes)
```
### <font color="red">[跨文件调用] 引用了未提供的模型类及方法，无法验证存在性</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `web/Hi-Zan/Hi-Zan/pages/community-reserve/order-detail/order-detail.js` 及 `apply-refund.js`
- **行号**: 约 4~6 行, 11~13 行, 22 行
- **问题描述**: 代码中通过 `import` 引入了 `ReserveModel`、`CabinetModel` 和 `HTTP`，并调用了大量未在当前变更文件中定义的方法（如 `reserveModel.applyBookRefund`, `reserveModel.openMachine`, `cabinetModel.reOpenCabinetDoor`, `this.request` 等）。由于未提供 `models/reserve.js`、`models/cabinet.js`、`utils/http.js` 的源码，无法确认这些类是否被正确 `export`，以及方法签名是否匹配。若缺失，将直接引发 `ReferenceError` 或 `TypeError`。
- **修复建议**:
1. 确保依赖文件路径正确且存在。
2. 检查 `utils/http.js` 是否导出了 `HTTP` 基类，且包含 `request` 方法。
3. 检查 `models/reserve.js` 和 `models/cabinet.js` 是否完整导出对应类及所有被调用的方法。建议补充这些文件以便进行完整静态分析。
### [安全隐患] 未校验本地存储数据直接访问属性
- **严重程度**: 高危
- **文件**: `web/Hi-Zan/Hi-Zan/pages/community-reserve/order-detail/order-detail.js`
- **行号**: 约 95 行 (`onLoad` 方法内)
- **问题描述**: `const uid = wx.getStorageSync('userInfo').uid || ''`。若用户未登录、缓存被清理或 `userInfo` 为 `null`/`undefined`，直接调用 `.uid` 会抛出 `TypeError: Cannot read properties of null (reading 'uid')`，导致页面初始化失败。
- **修复建议**: 使用可选链或安全解构：
```javascript
const userInfo = wx.getStorageSync('userInfo') || {};
const uid = userInfo.uid || '';
```
### [逻辑 BUG] 对象属性重复定义与关键拼写错误
- **严重程度**: 高危
- **文件**: `web/Hi-Zan/Hi-Zan/pages/community-reserve/order-detail/order-detail.js`
- **行号**: 约 238 行 (`handleOpenMachineResult` 方法内)
- **问题描述**: `showCancelBtn: false` 在 `setData` 对象中连续定义了两次。虽然 JS 引擎会取最后一次的值，但属于冗余代码且极易引发维护误解。此外，`avilable_room_list` 存在明显拼写错误（应为 `available`），虽前后一致不影响运行，但严重降低代码可读性，且易导致后续 WXML 绑定或后端对接时出现字段不一致问题。
- **修复建议**: 删除重复的 `showCancelBtn: false`。全局搜索 `avilable_room_list` 并统一修正为 `available_room_list`，同步更新 JS 逻辑与 WXML 模板。
### [代码质量] 使用弱相等比较符及魔法数字硬编码
- **严重程度**: 中危
- **文件**: `web/Hi-Zan/Hi-Zan/pages/community-reserve/order-detail/order-detail.js` 等多处
- **行号**: 约 100, 101, 113, 156, 178 等
- **问题描述**: 大量使用 `==` 进行类型不安全的比较（如 `this.data.bigType == 'book'`）。同时，业务状态码和场景码（如 `status == 5`, `type == 1`, `operational_scene == 2`）直接硬编码在逻辑中，缺乏语义化，后期新增状态时极易遗漏或写错。
- **修复建议**:
1. 统一替换为 `===` 严格相等。
2. 提取为常量枚举文件，例如：
```javascript
export const ORDER_STATUS = { IN_PROGRESS: 5 };
export const SCENE_TYPE = { ROOM: 1, TABLE: 2, CARD: 4 };
```
### [代码质量] 生产环境保留 console.log 且错误处理缺失
- **严重程度**: 中危
- **文件**: `web/Hi-Zan/Hi-Zan/models/order.js`
- **行号**: 约 15, 28, 40, 54, 66 行
- **问题描述**: 所有网络请求的 `error` 回调仅使用 `console.log(err)` 打印日志，未向用户展示任何错误提示，也未上报至前端监控系统。在生产环境中，`console` 语句可能泄露接口调试信息，且用户遇到网络异常或接口报错时无任何交互反馈，体验极差。
- **修复建议**: 移除裸 `console.log`。封装统一的错误处理函数，例如：
```javascript
error: (err) => {
wx.showToast({ title: '请求失败，请重试', icon: 'none' });
// 可在此处接入 Sentry/神策等错误监控 SDK
}
```
## ✅ 代码亮点
1. **模块化设计清晰**：采用 `Model` 层封装网络请求，页面逻辑与数据请求分离，符合微信小程序 MVC/MVVM 最佳实践。
2. **生命周期使用合理**：在 `onShow` 中根据 `bigType` 动态拉取对应订单详情，避免了 `onLoad` 中一次性加载过多数据导致的性能问题。
3. **用户体验细节到位**：在 `getMyRoomOrderDetail` 中使用了 `complete: () => { wx.hideLoading() }`，确保无论请求成功或失败都会关闭 Loading，防止界面卡死。
## 📝 总体建议
1. **补充依赖文件审查**：当前审查受限于未提供 `models/reserve.js`、`models/cabinet.js` 和 `utils/http.js`。强烈建议在合并前补充这些文件，或使用 TypeScript 定义接口契约，彻底杜绝跨文件调用不一致的风险。
2. **强化类型与常量管理**：建议引入 TypeScript 或 JSDoc 注释，对 `order_detail`、`res.result` 等复杂数据结构进行类型定义。将硬编码的 `status`、`type`、`operational_scene` 抽离为全局常量枚举。
3. **统一错误与 Loading 管理**：当前各页面手动调用 `wx.showLoading()`/`wx.hideLoading()` 较为分散，建议在 `HTTP` 基类中封装全局拦截器，统一处理 Loading 状态、Token 失效跳转及错误提示。
4. **修复关键语法与空指针**：优先解决 `const res` 重复声明问题及 `wx.getStorageSync` 空指针风险，这两项是导致线上白屏的高频原因。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781005232
|
1781005232
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
656
|
21
|
303
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - 申请退款
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `9b97b3156 ## 自动代码审查报告
**分支**: pay-260616
**提交**: `9b97b3156adface57571495d2eb22c652b5e440f`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-09 19:35:24
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：业务逻辑覆盖较全面，但存在明显的安全隐患（SQL注入、弱加密签名）、严重的性能瓶颈（N+1查询）以及框架生命周期使用不规范的问题。核心方法 `get_bill_goods_info` 过于臃肿，混合了数据查询、聚合计算与视图格式化逻辑，缺乏事务保护与常量复用。
- **风险等级**：🔴 高
> 📌 **注**：项目目录结构及 `$CI = &get_instance()` 语法高度符合 **CodeIgniter 3** 特征。若 `phpci` 为内部定制框架，请核对底层加载机制。以下审查基于标准 CI3/MVC 最佳实践与 PSR-12 规范。代码末尾存在截断，审查仅基于已提供片段。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_yc_order_model.php`<br>`get_bill_goods_info` 方法 | **SQL 注入漏洞**：直接使用字符串拼接 `$unique_key` 构建 SQL 条件，未进行转义或使用查询构造器。 | 使用 CI3 查询构造器 `$this->db->where()` 自动转义，或显式调用 `$this->db->escape()`。 | `$this->db->where('_unique_key', $unique_key);`<br>`$this->db->where_in('_status', [1, 4]);` |
| 🔴 严重 | `Ahead_yc_order_model.php`<br>L12, L685 | **硬编码密钥与弱加密**：`$encrypt` 为公开属性且硬编码在类中；`encode_group_buying_order` 使用 `md5` 签名，易受碰撞与彩虹表攻击。 | 密钥移至 `config` 或环境变量；签名改用 `hash_hmac('sha256', ...)`。 | `hash_hmac('sha256', $order_id, config_item('order_sign_key'), true)` |
| 🔴 严重 | `Ahead_yc_order_model.php`<br>`get_detail` 方法 | **逻辑缺陷/数据丢失**：`$order_data['before_payment']` 被赋值，但后续未合并至 `$order_info` 且未返回，导致前端无法获取转房前金额。 | 修正变量名，将数据正确挂载到 `$order_info` 数组中。 | `$order_info['before_payment'] = $before_order_info_data['_actual_pay'] ?? '';` |
| 🔴 严重 | `Ahead_yc_order_model.php`<br>`confirm_receipt` 方法 | **缺乏数据库事务**：连续执行两次 `insert` 记录订单状态，若第二次失败或中断，将导致订单状态不一致（卡在“已确认收货”但未“完成”）。 | 使用 CI3 事务机制包裹关键写入操作。 | `$this->db->trans_start();`<br>`// insert 1 & 2`<br>`$this->db->trans_complete();` |
| 🟠 警告 | `Ahead_yc_order_model.php`<br>`get_list` 方法 | **N+1 查询性能瓶颈**：在 `foreach` 循环内频繁 `load->model` 并执行 `get_one` 查询套餐图片。数据量大时将导致严重延迟。 | 提前收集所有 `package_id`，使用 `WHERE IN` 批量查询，或在主 SQL 中使用 `LEFT JOIN`。 | `$ids = array_column($order_info, 'package_id');`<br>`$imgs = $this->db->where_in('_id', $ids)->get('ahead_room_package')->result_array();` |
| 🟠 警告 | `Ahead_yc_order_model.php`<br>L6-L7 | **框架反模式**：模型文件顶部直接调用 `$CI = &get_instance();` 和 `load->model()`。文件被 `include` 时即执行，破坏 CI 生命周期与单例机制。 | 移至 `__construct()` 中，或交由控制器/自动加载器管理。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` |
| 🟠 警告 | `Ahead_yc_order_model.php`<br>多处方法 | **重复加载模型**：各业务方法内频繁调用 `$this->load->model()`，增加不必要的 I/O 与内存开销。 | 统一在 `__construct()` 中加载，或配置 `application/config/autoload.php`。 | `public function __construct() { $this->load->model(['ahead_room_package_model', 'ahead_yc_order_extension_model']); }` |
| 🟠 警告 | `Ahead_yc_order_model.php`<br>多处条件判断 | **魔法数字泛滥**：大量使用 `10, 12, 14` 等硬编码判断支付类型/状态，未复用类顶部已定义的 `const`。 | 全面替换为类常量，提升可读性与可维护性。 | `if ($order['_pay_platform'] == self::ORDER_AFTER_PAY_PAYPLATFORM)` |
| 🟡 建议 | `Ahead_yc_order_model.php`<br>L9 | **命名规范不符 PSR-12**：类名使用下划线 `Ahead_yc_order_model`，不符合 PHP 标准驼峰命名规范。 | 重命名为 `AheadYcOrderModel`，并全局替换引用。 | `class AheadYcOrderModel extends Simple_model` |
| 🟡 建议 | `Ahead_yc_order_model.php`<br>L12-L14 | **属性可见性不当**：`$encrypt`, `$pay_id_arr` 等声明为 `public`，易被外部实例意外修改或暴露。 | 改为 `protected` 或 `private`，通过 Getter 方法访问。 | `protected $encrypt = "Vs!Fs7VT";`<br>`protected $pay_id_arr = [...];` |
| 🟡 建议 | `Ahead_yc_order_model.php`<br>`get_bill_goods_info` | **违背单一职责原则 (SRP)**：该方法超 300 行，混合了 SQL 查询、金额聚合、商品格式化、业务规则判断。 | 拆分为 `fetchBillData()`, `calculateTotals()`, `formatGoodsList()` 等私有方法，或抽离至 `OrderBillService`。 | 将聚合逻辑与视图格式化逻辑分离，降低圈复杂度。 |
---
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **修复 SQL 注入**：立即将 `get_bill_goods_info` 中的字符串拼接改为查询构造器或预处理语句。
2. **补充事务控制**：为 `confirm_receipt`、`close_room_after`、`bindingOrder` 等涉及多表写入的方法添加 `$this->db->trans_start()/trans_complete()`。
3. **修正数据丢失 Bug**：将 `get_detail` 中的 `$order_data['before_payment']` 更正为 `$order_info['before_payment']`。
4. **规范框架加载**：移除文件顶部的 `$CI = &get_instance();`，将模型依赖移至构造函数。
### 🛠 后续重构与优化方向
1. **性能优化**：
- 彻底解决 `get_list` 的 N+1 查询问题，改用批量查询或 SQL `JOIN`。
- `get_bill_goods_info` 中的金额汇总（`price_total`, `amount_total` 等）可考虑下沉至数据库层，使用 `SUM()`, `GROUP BY` 替代 PHP 循环累加，大幅降低内存与 CPU 消耗。
2. **安全加固**：
- 废弃 `md5` 签名，全面升级至 `hash_hmac('sha256', ...)` 或 JWT。
- 敏感配置（如 `$encrypt`）必须移出代码库，使用 `.env` 或 CI 的 `config` 文件管理。
3. **架构与规范**：
- 遵循 PSR-12 重命名类文件，统一常量使用，消除魔法数字。
- 将 `get_bill_goods_info` 等巨型方法按 **CQRS** 或 **Service 层** 思想拆分，模型仅负责数据存取，业务逻辑与格式化交由上层处理。
- 补充全局函数（如 `throwError`, `minToStr`, `DEFAULTIMG`）的依赖声明或替换为框架内置方法，避免隐式依赖导致维护困难。
> 💡 **提示**：由于提供的代码在 `get_timing_order` 方法处截断，若该方法包含关键业务逻辑（如订单状态机流转、定时任务触发等），请补充完整代码以便进行闭环审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781004924
|
1781004924
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
655
|
21
|
302
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - 开房订单申请退款
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `75d502a79 ## 自动代码审查报告
**分支**: pay-260616
**提交**: `75d502a7967d58d46a34c788d574131fe3900efe`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-09 19:26:01
---
## 1. 审查摘要
- **代码质量评分**：5/10 分
- **总体评价**：代码实现了订单创建、支付路由、账单统计等核心业务，但存在明显的架构反模式与安全/性能隐患。大量硬编码、重复的支付逻辑、循环内动态加载模型以及不规范的参数传递方式，严重影响了系统的可维护性与运行效率。整体偏向老旧的 CodeIgniter 3 风格，与现代 PHP 工程规范存在差距。
- **风险等级**：🔴 高（存在 SQL 注入隐患、敏感信息硬编码、越权删除风险及严重性能瓶颈）
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_yc_order_model.php`<br>文件顶部/类外 | **模型依赖加载位置错误**：在类外部直接执行 `$CI = &get_instance(); $CI->load->model('Simple_model');`。这会在文件被 `include/require` 时立即执行，破坏框架生命周期，且可能导致全局状态污染。 | 移除文件顶部的加载逻辑，模型应通过 `extends Simple_model` 继承，并在构造函数中调用 `parent::__construct()`。 | `class Ahead_yc_order_model extends Simple_model { public function __construct() { parent::__construct(); } }` |
| 🔴 严重 | `Ahead_yc_order_model.php`<br>`get_bill_goods_info` 方法内 | **SQL 注入风险**：使用字符串拼接构建查询条件 `$sql = '_unique_key="' . $unique_key . '" AND ...'`，若 `$unique_key` 未经严格过滤直接传入底层查询，将导致注入。 | 全面使用框架查询构造器或参数绑定，禁止手动拼接 SQL 条件字符串。 | `$this->db->where('_unique_key', $unique_key)->where('_status IN (1,4)')->get();` |
| 🔴 严重 | `Ahead_yc_order_model.php`<br>`delete_one` 方法 | **越权删除隐患**：当 `$uid = 0` 时，`if ($uid)` 条件不成立，直接执行软删除，未校验订单归属权。攻击者可传入 `0` 或空值删除任意订单。 | 强制要求传入有效用户 ID，或从 Session/Token 中获取当前用户 ID，禁止依赖外部可控参数。 | `if (empty($uid)) { throwError('用户身份校验失败'); } $where['_ahead_user_id'] = $uid;` |
| 🟠 警告 | `Ahead_yc_order_model.php`<br>`get_list` 方法 | **N+1 查询性能瓶颈**：在 `foreach` 循环内动态 `load->model()` 并执行 `get_one()`。订单量稍大时将导致数据库连接耗尽与响应超时。 | 提前收集所有 `package_id`，使用 `WHERE IN` 一次性批量查询，再通过 PHP 数组映射回填数据。 | 收集 `$ids` → `$this->db->where_in('_id', $ids)->get()->result_array()` → 建立映射表 `$map[$id]` 供循环使用。 |
| 🟠 警告 | `Order.php`<br>`check_params` 方法 | **参数来源不一致**：方法签名接收 `$param`，但在 `else` 分支中直接使用 `$this->param['family_server_id']`。若调用方未同步更新 `$this->param`，将引发 `Undefined index` 或逻辑错乱。 | 统一使用传入的 `$param` 数组，或在方法开头显式合并：`$param = array_merge($this->param, $param);` | `if ($from != '1') { $family_server_id = $param['family_server_id'] ?? ''; }` |
| 🟠 警告 | `Order.php`<br>`buyRenewalPackage` / `createOrder` | **支付路由逻辑高度重复**：微信支付与国通支付（Chinaums）的判断、参数组装、回调地址拼接在两个方法中几乎完全一致，维护成本极高。 | 提取为私有方法 `generatePaymentPayload($orderData, $payPlatform, $shopId)`，统一处理网关路由。 | 封装支付网关调用，控制器仅负责：`$payData = $this->buildPayParams($order, $platform);` |
| 🟠 警告 | `Ahead_yc_order_model.php`<br>类属性定义 | **敏感信息硬编码**：`public $encrypt = "Vs!Fs7VT";` 直接暴露在源码中，违反安全基线。 | 移至 `config/` 配置文件或环境变量，通过 `config_item()` 或 `getenv()` 动态获取。 | `protected $encrypt; public function __construct() { parent::__construct(); $this->encrypt = config_item('order_encrypt_key'); }` |
| 🟡 建议 | `Order.php` / `Ahead_yc_order_model.php`<br>全局 | **魔法数字泛滥**：大量使用 `1, 2, 4, -1, 10` 表示状态/类型。虽已定义 `const`，但业务逻辑中未全面替换，可读性差且易改错。 | 全面使用已定义的类常量，并在新增状态时同步更新常量映射表。 | `if ($order['type'] == self::ORDER_TIMING_OPEN_ROOM_TYPE)` 替代 `== 5` |
| 🟡 建议 | `Order.php`<br>`do_log` 调用处 | **日志记录敏感数据**：`do_log(var_export($order_add_res['order_data'], 1)...)` 直接打印完整订单数组，可能包含用户手机号、支付密钥等 PII/敏感信息。 | 脱敏处理后再记录，或使用框架标准日志函数，仅记录关键标识符。 | `log_message('debug', 'Pay initiated: order=' . $orderId . ', amount=' . $amount);` |
| 🟡 建议 | `Order.php`<br>文件头部 | **非标准父类引入**：使用 `include FCPATH . 'application' ...` 引入父控制器。现代框架应依赖自动加载或命名空间。 | 移除手动 `include`，确保框架自动加载机制已正确配置父类路径。 | 依赖 Composer 或框架 Autoloader 自动解析 `Index` 类。 |
> 📝 **局限性说明**：`Ahead_yc_order_model.php` 末尾的 `get_timing_order` 方法代码被截断，无法完整评估其查询逻辑与异常处理。建议补充完整代码后二次审查。
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复 SQL 注入与越权漏洞**：立即替换 `get_bill_goods_info` 中的字符串拼接查询，并为 `delete_one` 增加强制用户身份校验。
2. **消除 N+1 查询**：重构 `get_list` 方法，将循环内查询改为批量 `IN` 查询或 `JOIN`，预计可降低 80% 以上的数据库 IO 开销。
3. **统一支付逻辑**：将 `buyRenewalPackage` 与 `createOrder` 中的支付网关路由代码抽离为独立服务类或私有方法，遵循 DRY 原则。
### 🛠 后续重构与优化方向
- **架构规范化**：当前代码呈现典型的 CI3 风格。若 `phpci` 为定制框架，请严格遵循其生命周期（如模型构造函数、控制器基类加载）。建议逐步引入依赖注入（DI）容器，替代 `$this->load->model()` 的隐式加载。
- **常量与枚举管理**：将散落在类中的状态数组（如 `$pay_id_arr`, `$type_arr`）统一迁移至独立的 `Enum` 类或配置文件中，业务层仅通过常量引用。
- **日志与安全审计**：建立统一的日志脱敏中间件，禁止在业务代码中直接 `var_export` 敏感数组。支付回调地址建议通过配置中心动态下发，避免硬编码 `PAY_BASE_URL`。
- **测试覆盖**：针对 `check_params` 边界条件、支付状态流转（`pay_status != -1` 分支）、并发关房场景补充单元测试，防止状态机死锁或资损。
> 💡 **框架适配提示**：代码中大量使用 `get_instance()` 与 `$this->load->` 语法，符合 CodeIgniter 3 特征。若 `phpci` 为基于 CI 的二次开发框架，请确认其是否已升级至 PHP 8.x 兼容模式。若为独立微框架，建议逐步替换为 PSR-4 自动加载与标准 MVC 路由，以提升长期可维护性。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781004361
|
1781004361
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
654
|
21
|
301
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - 现场扫码订单
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `f8aa117ea ## 自动代码审查报告
**分支**: pay-260616
**提交**: `f8aa117ea568aecfd6537e9b87159c6681f47c2b`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-09 19:15:57
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：业务逻辑覆盖较为完整，但存在明显的架构与编码缺陷。核心问题集中在 **N+1 查询导致的性能瓶颈**、**SQL 拼接注入风险**、**缺乏事务保护**以及**敏感信息硬编码**。代码风格偏向老旧的 CI2/3 写法，未充分利用现代 PHP 特性与查询构建器，且单一方法职责过重，可维护性较低。
- **风险等级**：🔴 高
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_bill_goods_info` (~L200) | **SQL 注入风险**：使用字符串拼接构造查询条件 `$sql = '_unique_key="' . $unique_key . '" ...'` 并直接传入 `$this->select()`。若 `$unique_key` 来源不可控，将导致严重注入漏洞。 | 废弃字符串拼接，改用框架查询构建器（Query Builder）或参数绑定机制。 | `$this->db->where('_unique_key', $unique_key)->where_in('_status', [1,4])->get($this->table_name)->result_array();` |
| 🔴 严重 | `confirm_receipt` (~L155) | **数据一致性风险**：连续执行两次 `insert` 操作未包裹在数据库事务中。若第二次插入失败，将产生状态不一致的脏数据。 | 使用框架事务机制包裹关键写入操作，失败时自动回滚。 | `见下方事务示例` |
| 🔴 严重 | 类属性定义 (~L10) | **敏感信息硬编码**：加密串 `public $encrypt = "Vs!Fs7VT";` 直接暴露在源码中，易随版本控制泄露，且不利于多环境配置管理。 | 移至配置文件或环境变量，通过配置项动态读取。 | `protected $encrypt; public function __construct(){ parent::__construct(); $this->encrypt = config_item('order_encrypt_key'); }` |
| 🟠 警告 | `get_list` (~L75) | **严重性能瓶颈 (N+1 查询)**：在 `foreach` 循环内动态加载模型并执行单条查询。订单量超 50 时将引发数据库连接池耗尽与响应超时。 | 提前收集所有 `package_id`，使用 `WHERE IN` 批量查询，在内存中完成数据映射。 | `见下方批量查询示例` |
| 🟠 警告 | `get_detail` (~L115) | **冗余查询与未使用变量**：`$order_data['before_payment']` 赋值后从未返回或参与逻辑；在自身模型内调用 `$this->ahead_yc_order_model->get_one()` 属于冗余加载。 | 移除无效赋值；直接复用 `$this->get_one()`；合并重复的模型加载。 | `// 移除 $order_data 赋值块<br>// 改为: $before_order = $this->get_one(['_id' => $order_info['before_order_id']]);` |
| 🟠 警告 | `encode_group_buying_order` (~L310) | **弱加密算法**：使用 `md5()` 进行签名校验。MD5 已被证实存在碰撞漏洞，不适用于安全签名或防篡改场景。 | 改用 `hash_hmac('sha256', $data, $key)` 提供强加密签名。 | `return hash_hmac('sha256', $order_id, $this->encrypt);` |
| 🟡 建议 | 文件顶部 (~L4) | **框架生命周期误用**：`$CI = &get_instance();` 在类外部执行，文件被 `include` 时即触发，可能在框架未完全初始化时引发 Fatal Error。 | 移至构造函数中，或直接使用 `$this->load->` 链式调用。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` |
| 🟡 建议 | 全局/多处 | **违反 PSR-12 与单一职责**：类名 `Ahead_yc_order_model` 非驼峰；`get_bill_goods_info` 超 300 行，混合了数据查询、金额计算、格式化与业务分支。 | 类名改为 `AheadYcOrderModel`；将大方法拆分为 `fetchBillData()`、`calculateTotals()`、`formatGoods()`。 | `// 遵循 PSR-12<br>class AheadYcOrderModel extends Simple_model { ... }` |
| 🟡 建议 | `binding_order_check` (~L335) | **未定义属性引用**：使用 `$this->uid`，但类中未声明或初始化，依赖隐式全局或父类，破坏封装性。 | 显式声明属性，或通过方法参数/Session 安全注入。 | `protected $current_uid; public function set_current_uid(int $uid): void { $this->current_uid = $uid; }` |
### 🔧 关键代码修复示例
**1. 数据库事务保护 (`confirm_receipt`)**
```php
public function confirm_receipt($order_info, $aheaduid, $aheaduname, $star = 5)
{
$this->db->trans_start(); // 开启事务
// ... 前置校验逻辑 ...
$data = [
'_order_id' => $order_id,
'_process' => 7,
'_process_msg' => $aheaduname . ' - 已确认收货',
'_process_time' => time(),
'_ahead_user_id' => $aheaduid,
'_ahead_user_name' => $aheaduname,
'_star' => $star
];
$this->ahead_yc_order_process_model->insert($data);
$data['_process'] = 8;
$data['_process_msg'] = '订单完成';
$this->ahead_yc_order_process_model->insert($data);
$this->db->trans_complete(); // 提交事务
if ($this->db->trans_status() === FALSE) {
$this->db->trans_rollback();
return ['code' => false, 'msg' => '确认收货失败，数据已回滚'];
}
// ... 推送消息逻辑 ...
return ['code' => true, 'msg' => '确认收货成功'];
}
```
**2. 消除 N+1 查询 (`get_list` 优化思路)**
```php
// 优化前：循环内查库
// 优化后：批量查询 + 内存映射
$package_ids = array_filter(array_column($order_info, 'package_id'));
$package_imgs = [];
if ($package_ids) {
$this->load->model("ahead_room_package_model");
$this->load->model("ahead_wares_package_model");
// 假设框架支持 where_in 批量查询
$room_pkgs = $this->ahead_room_package_model->select(['_id' => $package_ids, 'where_in' => ['_id', $package_ids]], '_id,_img_url');
$wares_pkgs = $this->ahead_wares_package_model->select(['_id' => $package_ids, 'where_in' => ['_id', $package_ids]], '_id,_img_url');
foreach ($room_pkgs as $pkg) $package_imgs[$pkg['_id']] = $pkg['_img_url'] ?? DEFAULTIMG;
foreach ($wares_pkgs as $pkg) $package_imgs[$pkg['_id']] = $pkg['_img_url'] ?? DEFAULTIMG;
}
foreach ($order_info as &$val) {
$val['img'] = $package_imgs[$val['package_id']] ?? DEFAULTIMG;
// ... 其他字段处理 ...
}
```
---
## 3. 总结与行动建议
### 🚨 优先修复项（P0/P1）
1. **修复 SQL 注入**：立即将 `get_bill_goods_info` 中的字符串拼接替换为参数化查询或 Query Builder。
2. **补充事务控制**：为所有涉及多表写入或状态流转的方法（如 `confirm_receipt`、`bindingOrder`）添加事务包裹。
3. **移除硬编码密钥**：将 `$encrypt` 迁移至 `config/application.php` 或 `.env` 文件，并通过 `config_item()` 读取。
4. **解决 N+1 查询**：重构 `get_list` 与 `get_detail` 中的循环查库逻辑，改为批量查询+内存映射，预计可提升 80% 以上的列表接口响应速度。
### 🛠 后续重构方向
1. **方法职责拆分**：`get_bill_goods_info` 严重违反单一职责原则（SRP）。建议拆分为：
- `fetchRawBillData()`：负责纯数据查询
- `calculateFinancials()`：负责金额、折扣、积分计算
- `formatBillResponse()`：负责视图层数据格式化
2. **统一常量管理**：当前类中同时存在 `public $pay_id_arr` 和 `const ORDER_PAY_PLATFORM_ARR` 等重复定义。建议统一使用 `const` 或 `enum`（PHP 8.1+），并移除魔法数字。
3. **异常处理规范化**：将自定义的 `throwError()` 替换为 PHP 标准异常 `\Exception` 或框架提供的 `show_error()`，便于全局错误捕获与日志记录。
4. **类型声明与严格模式**：在文件头部添加 `declare(strict_types=1);`，并为方法参数与返回值添加类型提示（如 `int`, `array`, `bool`），提升代码健壮性。
> ⚠️ **局限性说明**：提交的代码在 `get_timing_order` 方法处被截断，未能审查完整逻辑。若该方法涉及核心计费或状态机流转，请补充完整代码以便进行二次深度审查。
> 📖 **框架适配提示**：基于代码结构（`get_instance()`、`system/` 目录、`$this->load->model()`），判定为基于 CodeIgniter 3 架构的定制框架（phpci）。上述事务与 Query Builder 语法均兼容 CI3 标准，若 phpci 有自定义封装，请以官方文档的 `DB` 驱动 API 为准。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781003757
|
1781003757
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
653
|
21
|
300
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - 现场扫码退款
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `1de21f249 ## 自动代码审查报告
**分支**: pay-260616
**提交**: `1de21f24942458f96482e04d4bf8b4502dfe44e4`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-09 19:12:04
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：代码实现了较为复杂的预订、支付回调、退款及账单聚合业务，整体业务闭环完整。但存在明显的架构反模式、事务与外部API耦合、SQL注入风险、N+1查询性能瓶颈以及硬编码敏感信息等问题。部分方法逻辑冗长，缺乏常量抽象与错误边界控制。
- **风险等级**：🔴 高（存在致命运行时错误、SQL注入隐患及事务锁表风险）
> 📌 **框架说明**：从 `$CI = &get_instance()`、`$this->load->model()`、`$this->db->trans_start()` 等特征判断，当前代码基于 **CodeIgniter 3** 架构开发。若 `phpci` 为内部定制框架，以下建议仍完全适用，请结合官方文档微调生命周期调用。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_yc_notice_model.php`<br>`insert_after` 方法 | 将布尔属性 `$this->push_notice` 当作方法调用：`return $this->push_notice($arr);`，将直接触发 `Fatal Error: Function name must be a string` 导致服务崩溃。 | 明确区分属性与方法名，或改为条件判断后调用同名方法。 | `if ($this->push_notice === true) { return $this->push_notice($arr); }` |
| 🔴 严重 | `Ahead_yc_order_model.php`<br>`get_bill_goods_info` | 直接字符串拼接 SQL 条件：`$sql = '_unique_key="' . $unique_key . '" ...'`，未做任何过滤或参数绑定，存在高危 SQL 注入漏洞。 | 全面改用 CI 查询构造器或参数绑定，杜绝手动拼接。 | `$this->db->where('_unique_key', $unique_key)->where('_status IN (1,4)')->get($this->table_name)->result_array();` |
| 🔴 严重 | `Ahead_book_order_model.php`<br>`check_notify` & `refund_by_notify` | 数据库事务中包裹了外部支付退款 API 调用（如 `WxPayApi::refund`）。网络请求耗时不可控，极易导致数据库连接池耗尽、长事务锁表或超时回滚失败。 | **事务与外部调用必须解耦**。先更新本地状态并提交事务，再通过异步队列、定时任务或独立流程调用退款接口，最后通过回调对账。 | 移除 `refund_by_notify` 中的 `WxPayApi` 调用，改为记录退款任务表，由 Cron 或消息队列异步执行。 |
| 🟠 警告 | `Ahead_book_order_model.php`<br>`check_notify` | 事务回滚逻辑不完整。当 `refund_by_notify` 返回 `['status'=>false]` 时，方法直接 `return`，未显式执行 `$this->db->trans_rollback()`，依赖框架隐式行为存在脏数据风险。 | 统一使用 `$this->db->trans_begin()`、`trans_commit()`、`trans_rollback()` 显式控制，或在所有异常/失败分支前显式回滚。 | `if (!$res['status']) { $this->db->trans_rollback(); return $res; }` |
| 🟠 警告 | `Ahead_book_order_model.php`<br>`get_list` | 循环内执行数据库查询：`foreach ($order_info as &$v) { $merchant_data = $this->ahead_merchant_model->get_one(...); }`，典型 N+1 查询问题，数据量增长时性能呈指数级下降。 | 提取所有 `merchant_id` 使用 `where_in` 批量查询，或在主查询中使用 `JOIN` 一次性获取。 | `$ids = array_column($order_info, 'merchant_id'); $merchants = $this->ahead_merchant_model->where_in('_id', $ids)->get()->result_array();` |
| 🟠 警告 | `Ahead_yc_order_model.php`<br>类属性定义 | 硬编码加密密钥：`public $encrypt = "Vs!Fs7VT";`。密钥随代码库分发，违反安全规范，一旦仓库泄露将导致历史数据可被解密。 | 移至 `config/config.php` 或 `.env` 环境变量中，通过配置项读取。 | `protected $encrypt_key; public function __construct() { $this->encrypt_key = config_item('encrypt_key'); }` |
| 🟡 建议 | 全局多处 | 大量使用魔法数字（如状态 `-1, 1, 2, 3, 4, 5`，支付平台 `1, 3, 14, 22` 等），业务含义不透明，维护成本高。 | 定义类常量或独立枚举类，提升可读性与可维护性。 | `const STATUS_PENDING = -1; const PAY_PLATFORM_WECHAT = 1;` |
| 🟡 建议 | `Ahead_book_order_model.php`<br>文件顶部 | 顶部直接使用 `$CI = &get_instance();` 加载模型。在 CI 架构中，模型文件被 `include` 时即执行，此时 CI 核心可能未完全初始化，易引发未定义变量或加载失败。 | 移除顶部代码，在类构造函数或具体方法内按需 `$this->load->model()`。 | 删除顶部两行，在 `__construct()` 中调用 `$this->load->model('Simple_model');` |
| 🟡 建议 | `send_success_msg` 等方法 | 存在大量注释掉的废弃代码（如旧版短信发送逻辑、时间判断分支），且变量拼写错误 `$rooom_data`。 | 清理无用注释代码，依赖 Git 管理历史版本；修正拼写错误。 | 删除 `/* ... */` 块，将 `$rooom_data` 改为 `$room_data`。 |
| 🟡 建议 | `Ahead_book_order_model.php`<br>`create_community_shop_book_order` | 代码在 `if ($this->tuangou->verify_token) {` 处被截断，无法评估后续逻辑完整性与事务闭合情况。 | 补充完整代码后重新提交审查，确保 `trans_complete()` 或 `trans_rollback()` 成对出现。 | *(需补充完整代码)* |
---
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **修复致命调用错误**：立即修正 `Ahead_yc_notice_model::insert_after` 中的属性/方法名冲突，避免线上 Fatal Error。
2. **消除 SQL 注入风险**：将 `get_bill_goods_info` 及 `refund_by_notify` 中所有手动拼接的 SQL 条件替换为 CI 查询构造器或参数绑定。
3. **事务与外部调用解耦**：将微信支付/银联退款等网络请求移出数据库事务。采用“本地状态更新 → 提交事务 → 异步退款任务 → 回调对账”的标准支付架构。
### 🛠 后续重构与优化方向
1. **架构规范化**：
- 移除文件顶部的 `$CI = &get_instance();`，遵循 CI 模型生命周期。
- 统一使用 `$this->db->trans_begin()` / `trans_commit()` / `trans_rollback()` 替代 `trans_start()` + `try/catch` 的混合写法，提升事务可控性。
2. **性能与可维护性**：
- 消除 N+1 查询，对列表类接口采用批量查询或 `JOIN` 优化。
- 提取魔法数字为类常量（如 `OrderStatus::PAID`、`PayPlatform::WECHAT`），必要时引入 PHP 8.1+ `enum`。
- 拆分超长方法（如 `get_bill_goods_info` 超过 300 行），按职责拆分为 `calculateTotals()`、`mergeGoods()`、`formatBill()` 等私有方法。
3. **安全与规范**：
- 敏感配置（加密串、API 密钥、模板 ID）全部迁移至配置文件或环境变量。
- 清理历史注释代码，统一数组语法为 `[]`，遵循 PSR-12 命名与缩进规范。
- 日志记录避免直接输出 `$e->getTrace()`，改为记录 `$e->getMessage()` 与 `$e->getFile().':'.$e->getLine()`，防止敏感路径泄露。
> 💡 **提示**：若 `phpci` 为内部封装框架，请确认其事务管理器与 CI3 原生行为是否一致。建议在核心支付链路补充单元测试（PHPUnit）与集成测试，覆盖正常支付、退款失败、并发扣减库存等边界场景。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781003524
|
1781003524
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
652
|
21
|
299
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - Merge remote-tracking branch 🔍 代码审查报告：pay-260616 - Merge remote-tracking branch 'pay/pay-260616' into...
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `e3eea539a ## 自动代码审查报告
**分支**: pay-260616
**提交**: `e3eea539ab8ef712c7d9fcb5b23ccc1418107d13`
**提交人**: zhangjunnan (121158035@qq.com)
**时间**: 2026-06-09 18:42:10
---
## 📋 审查摘要
- **变更文件数**: 5
- **严重问题**: 1
- **高危问题**: 3
- **中危问题**: 2
- **建议优化**: 4
## 🐛 发现的问题
### <font color="red">[语法错误] 文件内容被意外截断导致语法解析失败</font>
- **严重程度**: <font color="red">严重</font>
- **文件**: `application/models/Ahead_shop_group_buying_coupon_model.php`
- **行号**: 文件末尾（约第 100 行）
- **问题描述**: 提供的代码在 `if (!empty($same` 处突然中断，缺少闭合括号、分号及后续逻辑。若直接部署将导致 PHP 解析致命错误（Parse Error）。
- **修复建议**: 补全缺失的代码逻辑，确保语法完整。例如：
```php
if (!empty($same_package_room_type)) {
// 补充完整逻辑
}
// 确保类和方法正确闭合
}
```
### <font color="red">[跨文件调用] 模型加载命名大小写不一致可能导致Linux环境致命错误</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `application/controllers/mini/hz/Book.php`, `application/models/Ahead_book_order_model.php`, `application/models/Ahead_shop_config_second_model.php`
- **行号**: 多处（如 `Book.php` 第 338, 345 行；`Ahead_book_order_model.php` 第 188, 225 行等）
- **问题描述**: CodeIgniter 的 `$this->load->model()` 在 Linux 等大小写敏感的文件系统中，会严格匹配文件名。代码中混用了 `Ahead_xxx_model`（首字母大写）和 `ahead_xxx_model`（全小写）。若实际文件名为小写，加载大写名称将直接抛出 `Unable to locate the model you have specified` 致命错误。
- **修复建议**: 统一模型加载命名规范。建议全部使用小写或遵循 CI 官方推荐的首字母大写+下划线格式，并确保与磁盘文件名完全一致。
```php
// 推荐统一为小写加载（CI会自动处理首字母大写映射）
$this->load->model('ahead_book_invite_model');
$this->load->model('ahead_sms_config_model');
$this->load->model('ahead_ai_audio_player_content_model');
```
### <font color="red">[跨文件调用] 调用了未在当前上下文中验证的方法</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `application/models/Ahead_book_order_operation_log_model.php`
- **行号**: 约第 85 行
- **问题描述**: 调用了 `$this->ahead_book_order_model->get_order_by_bill($bill_data);`。在提供的 `Ahead_book_order_model.php` 代码片段中未找到 `get_order_by_bill` 方法定义。若该方法不存在或签名不匹配，将触发 `Call to undefined method` 错误。
- **修复建议**: 确认 `Ahead_book_order_model` 中是否已定义 `get_order_by_bill` 方法。若未定义，需补充实现；若已定义，请确保方法签名与调用处参数一致。
### [安全隐患] SQL注入风险（Where条件字符串直接拼接）
- **严重程度**: 高危
- **文件**: `application/models/Ahead_book_order_model.php`
- **行号**: 约第 330 行
- **问题描述**: `$log_where = '_relation_id="' . $order_data['_id'] . '" and _status=1 and _type in (5,13)';` 使用字符串拼接构造 SQL 条件。虽然 `$order_data['_id']` 当前来自数据库查询，但若未来数据来源变更或包含特殊字符，极易引发 SQL 注入或语法错误。
- **修复建议**: 使用 CI 查询构造器（Query Builder）安全绑定参数：
```php
$this->db->where('_relation_id', $order_data['_id']);
$this->db->where('_status', 1);
$this->db->where_in('_type', [5, 13]);
$this->db->update('pay_log', $log_up);
```
### [逻辑 BUG] 死代码/无效条件判断导致功能未生效
- **严重程度**: 高危
- **文件**: `application/controllers/mini/hz/Book.php`
- **行号**: 约第 233 行
- **问题描述**: `if (false && $order_id) { ... }` 条件永远为 `false`，导致内部的邀请函跳转逻辑完全失效。这属于典型的调试遗留代码或逻辑错误。
- **修复建议**: 移除硬编码的 `false`，恢复正确的业务判断条件：
```php
// 修复前
if (false && $order_id) {
// 修复后
if ($order_id) {
```
### [代码质量] 违反单一职责原则的超长 Switch 语句
- **严重程度**: 中危
- **文件**: `application/models/Ahead_shop_config_second_model.php`
- **行号**: 约第 130 ~ 330 行
- **问题描述**: `get_shop_setting` 方法中的 `switch` 语句超过 200 行，包含大量重复的默认值赋值和场景判断逻辑。代码可读性差，维护成本高，且容易引发合并冲突。
- **修复建议**: 将配置映射提取为类属性数组，或使用策略模式/配置数组映射简化逻辑：
```php
protected $config_defaults = [
'book_trial_time' => 0,
'turn_on_the_ac_early' => 10,
// ... 其他默认值
];
// 在方法中直接返回 $this->config_defaults[$field] ?? $data[$field] ?? '';
```
### [代码质量] 重复加载同一模型
- **严重程度**: 低危
- **文件**: `application/models/Ahead_book_order_model.php`
- **行号**: 约第 208, 210 行
- **问题描述**: 在 `_check_param` 方法中，`$this->load->model('ahead_room_type_book_log_model');` 被连续加载了两次。CI 的 Loader 具有单例缓存机制，重复加载虽不报错，但浪费性能且影响代码整洁度。
- **修复建议**: 删除重复的加载语句，保留一次即可。
### [代码质量] 控制器基类引入方式不规范
- **严重程度**: 低危
- **文件**: `application/controllers/mini/hz/Book.php`
- **行号**: 第 3 行
- **问题描述**: 使用 `include FCPATH . 'application' . DIRECTORY_SEPARATOR ... 'Index.php';` 手动引入父控制器。这破坏了 CI 的自动加载机制，且在 Windows/Linux 路径分隔符处理上存在隐患。
- **修复建议**: 将 `Index` 控制器移至 `application/core/MY_Controller.php` 或 `application/controllers/BaseController.php`，并通过 CI 的自动加载或 `require_once APPPATH.'controllers/mini/hz/Index.php';` 规范引入。
## ✅ 代码亮点
1. **事务管理完善**：在 `Ahead_book_order_model.php` 的支付回调和退款逻辑中，正确使用了 `$this->db->trans_start()`、`trans_complete()` 和 `trans_rollback()`，并配合 `try-catch` 保证了数据一致性。
2. **防刷机制**：在 `Book.php` 的关键下单接口（如 `createCommunityShopBookOrder`、`aiBookChangeRecommend`）中调用了 `request_frequency()`，有效防止恶意并发请求。
3. **配置缓存优化**：`Ahead_shop_config_second_model.php` 使用 `public static $shop_config` 进行门店配置缓存，减少了重复的数据库查询，提升了接口响应性能。
## 📝 总体建议
1. **统一命名规范**：严格遵循 CodeIgniter 的模型/库命名规范（文件 PascalCase，加载小写），避免跨平台部署时的致命错误。
2. **清理调试代码**：全面排查代码中的 `if (false && ...)`、注释掉的旧逻辑及未使用的变量，保持代码库整洁。
3. **安全加固**：所有涉及数据库查询的 `WHERE` 条件必须使用 CI 的 Query Builder 或预处理语句，杜绝字符串拼接。对 `$_POST`/`$this->param` 传入的数据进行严格的类型转换和过滤。
4. **重构臃肿方法**：建议将 `Ahead_shop_config_second_model::get_shop_setting` 拆分为独立的配置读取类或使用配置数组映射，降低圈复杂度（Cyclomatic Complexity）。
5. **补全截断文件**：务必确认 `Ahead_shop_group_buying_coupon_model.php` 的完整代码，当前状态无法通过语法检查，存在上线阻断风险。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781001731
|
1781001731
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
651
|
21
|
298
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - bug
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `24f12a6f6 ## 自动代码审查报告
**分支**: pay-260616
**提交**: `24f12a6f67e6ea31f2764884f8f8c8de8987eb40`
**提交人**: linyangrui (yangruilin888@gmail.com)
**时间**: 2026-06-09 18:16:51
---
## 📋 审查摘要
- **变更文件数**: 2
- **严重问题**: 1
- **高危问题**: 4
- **中危问题**: 2
- **建议优化**: 3
## 🐛 发现的问题
### <font color="red">[语法错误] 未定义的变量 `qrcode` 导致运行时报错</font>
- **严重程度**: <font color="red">严重</font>
- **文件**: `web/Hi-Zan/Hi-Zan/pages/table-tennis/scan-order/scan-order.js`
- **行号**: 约 188 行
- **问题描述**: 在 `openRoomCheckPackageTime` 方法的回调中，调用了 `this.exchange('qr_code', qrcode)`。变量 `qrcode` 在当前作用域中并未定义，根据上下文逻辑，此处应为传入的参数 `qr_code`。该错误将直接触发 `ReferenceError`，阻断扫码兑换流程。
- **修复建议**: 将 `qrcode` 修正为 `qr_code`。
```javascript
// 错误
this.exchange('qr_code', qrcode)
// 正确
this.exchange('qr_code', qr_code)
```
### <font color="red">[跨文件调用] 引用的模型/行为文件未在提供的项目结构中定义</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `package.js`, `scan-order.js`
- **行号**: 约 3-6 行, 3-8 行
- **问题描述**: 代码中通过 `import` 引用了 `../../../models/billiards`、`../../../models/room`、`../../../models/reward`、`../../../models/reserve`、`../../../models/user` 及 `../../../behaviors/themeBehavior`。但提供的「项目结构」仅包含 PHP 系统文件，**未包含任何 JS 模型或配置文件**。无法验证这些模块是否存在，也无法验证其导出的方法（如 `getRoomPackageList`, `tuanGouExchange`, `openRoomCheckPackageTime` 等）签名是否匹配。若文件缺失或导出方式不符，将导致模块加载失败。
- **修复建议**: 请确保对应相对路径下的 `.js` 文件已正确创建，且使用 `export class` 或 `export default` 正确导出。建议补充前端模型文件清单以便进行完整交叉验证。
### [逻辑 BUG] `setData` 异步特性导致 `onHourTap` 读取到旧数据引发崩溃
- **严重程度**: 高危
- **文件**: `web/Hi-Zan/Hi-Zan/pages/table-tennis/package/package.js`
- **行号**: 约 108 行
- **问题描述**: 在 `getRoomPackageList` 回调中，调用 `this.setData` 更新 `hour_list` 后，立即同步调用了 `this.onHourTap(...)`。微信小程序的 `setData` 是异步的，此时 `this.data.hour_list` 尚未更新（仍为空数组）。`onHourTap` 内部执行 `const hour = this.data.hour_list[index].hour` 时，会尝试读取 `undefined` 的属性，抛出 `TypeError`。
- **修复建议**: 使用 `setData` 的回调函数确保数据更新后再执行依赖逻辑，或直接使用局部变量计算。
```javascript
this.setData({
hour_list: res.result.hour_list,
// ...其他字段
}, () => {
// 确保 setData 完成后再调用
this.onHourTap({ currentTarget: { dataset: { index: 0, item: hourList[0] } } })
})
```
### [逻辑 BUG] 直接修改 `data` 对象属性违反小程序规范
- **严重程度**: 高危
- **文件**: `web/Hi-Zan/Hi-Zan/pages/table-tennis/scan-order/scan-order.js`
- **行号**: 约 158 行
- **问题描述**: `toggleRuleInfo` 方法中直接执行了 `this.data.package_coupon_list[index].openRules = !...`。微信小程序官方明确禁止直接修改 `data` 对象，这会导致视图层与逻辑层状态不同步，且在某些基础库版本中会触发警告或渲染异常。
- **修复建议**: 使用 `setData` 配合动态路径更新，或深拷贝后整体替换。
```javascript
const key = `package_coupon_list[${index}].openRules`
this.setData({ [key]: !this.data.package_coupon_list[index].openRules })
```
### [安全隐患] URL 参数拼接未进行编码，存在路由解析异常风险
- **严重程度**: 高危
- **文件**: `package.js`, `scan-order.js`
- **行号**: 多处（如 `package.js` 约 135 行, `scan-order.js` 约 125 行）
- **问题描述**: 多个 `wx.navigateTo` 使用字符串拼接 `+ this.data.room_id +` 等方式传递参数。若业务数据中包含 `&`, `?`, `=`, `#` 或空格等特殊字符，将破坏 URL 结构，导致目标页面 `options` 解析错乱或路由拦截失败。
- **修复建议**: 统一使用模板字符串配合 `encodeURIComponent()` 进行安全编码。
```javascript
url: `/pages/community-reserve/pay/pay?room_id=${encodeURIComponent(this.data.room_id)}&package_id=${encodeURIComponent(packageItem.id)}&...`
```
### <font color="red">[未定义变量] 状态字段 `agreement` 未在 `data` 中初始化</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `web/Hi-Zan/Hi-Zan/pages/table-tennis/scan-order/scan-order.js`
- **行号**: 约 244 行
- **问题描述**: `onGetPhoneNumber` 中调用了 `this.setData({ agreement: true, ... })`，但页面初始 `data` 中并未声明 `agreement` 字段。虽然小程序允许动态添加，但会导致状态管理混乱，且极易与已有的 `agreeMobileAuth` 字段产生语义冲突。
- **修复建议**: 在 `data` 初始化中显式声明 `agreement: false`，或确认业务意图是否应为更新 `agreeMobileAuth`。
### [代码质量] 多处使用 `==` 而非 `===` 进行类型比较
- **严重程度**: 中危
- **文件**: `package.js`, `scan-order.js`
- **行号**: 多处（如 `item.status == '-1'`, `res.result.enough_time == -1`, `type == 'agreement'`）
- **问题描述**: JavaScript 中 `==` 会进行隐式类型转换，在复杂业务逻辑中可能引发难以排查的边界条件错误。
- **修复建议**: 全局替换为严格相等运算符 `===`，提升代码健壮性。
## ✅ 代码亮点
1. **结构清晰**：页面逻辑与数据模型分离良好，通过 `import` 引入独立 Model 类，符合模块化开发规范。
2. **交互体验完善**：合理使用了微信小程序的 `wx.navigateTo`、`wx.scanCode`、`wx.getPhoneNumber` 等 API，并配合弹窗（`showCrossTimePop`, `showAuthPop`）处理边界状态。
3. **动态 Tab 控制**：`package.js` 中根据接口返回的 `package_list` 和 `hour_list` 动态渲染 Tab 列表，提升了页面的灵活性。
## 📝 总体建议
1. **框架上下文说明**：提供的「项目结构」为 PHP CodeIgniter 框架目录，但变更文件为微信小程序前端 JS 代码。本次审查已针对小程序特性进行深度分析。若需进行 PHP/CI 后端审查，请补充对应的 Controller/Model 文件。
2. **异步编程规范**：小程序中 `setData` 为异步操作，后续依赖该数据的逻辑务必放入 `setData` 的回调函数中，或使用 `async/await` 封装 Promise 化请求。
3. **统一错误处理**：当前所有 Model 回调均未处理网络异常或业务失败状态（如 `res.result.status !== '1'` 或 `res.code !== 200`）。建议封装统一的请求拦截器，或在每个回调中补充 `else` 分支的 `wx.showToast` 提示。
4. **常量管理**：`operational_scene` 的枚举值（`1ktv 2台球 3棋牌 4酒馆`）硬编码在注释中，建议提取至 `config.js` 或独立常量文件，避免魔法字符串散落。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781000211
|
1781000211
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
650
|
21
|
297
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - 预订退款判断
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `7e2cd1572 ## 自动代码审查报告
**分支**: pay-260616
**提交**: `7e2cd1572cdc8a693aadf5d382c6ea720a3b5aaa`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-09 17:44:22
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：该 Model 承载了大量门店配置读取与业务规则处理逻辑，功能覆盖完整。但存在明显的**性能瓶颈**（循环内重复加载模型与查询）、**框架生命周期误用**（文件顶部全局获取实例）、以及**可维护性隐患**（巨型 `switch`、硬编码长字段串、变量未初始化）。整体代码偏向“过程式”堆砌，缺乏面向对象设计与批量处理思维。
- **风险等级**：🔴 高（主要源于高并发下的数据库连接耗尽风险、静态缓存潜在串扰、以及异常处理缺失）
> 📌 **框架说明**：根据目录结构（`system/`, `application/models/`）及 `$CI = &get_instance()` 用法，推断项目基于 **CodeIgniter 3** 或其衍生框架。若 `phpci` 为内部定制框架，请对照其官方文档调整生命周期与 Query Builder 用法。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `renewal_audio_broadcast` 方法内 | **N+1 查询与循环内加载模型**：在 `foreach` 中反复调用 `$this->load->model()` 和 `get_end_time_room()`，数据量稍大即会导致数据库连接池耗尽、脚本超时。 | 1. 将模型加载移至循环外；<br>2. 收集所有 `shop_id`/`room_id` 后使用 `WHERE IN` 批量查询；<br>3. 合并业务逻辑，减少嵌套循环。 | 见下方 `renewal_audio_broadcast` 优化示例 |
| 🔴 严重 | 文件顶部 (第 3-4 行) | **违反框架生命周期**：在类外部执行 `$CI = &get_instance(); $CI->load->model(...)`。每次文件被 `include` 时都会执行，造成性能损耗，且在 CLI/定时任务中可能引发 `get_instance()` 未初始化错误。 | 移除文件顶部代码。在方法内部直接使用 `$this->load->model()` 或 `$this->db`。CI 框架会自动处理实例注入。 | `// 删除顶部两行<br>class Ahead_shop_config_second_model extends Simple_model { ... }` |
| 🟠 警告 | `deal_audio_content_params` 方法 | **未定义变量警告**：`$total_time` 仅在 `if (!empty($room_data['_open_id']))` 分支内赋值，若条件不满足将触发 `Undefined variable` 警告。且循环内重复加载模型。 | 1. 提前初始化 `$total_time = '';`；<br>2. 将 `$this->load->model()` 移至 `foreach` 外部。 | `$total_time = ''; // 提前声明<br>$this->load->model('ahead_open_room_log_model');<br>foreach ($params_map as &$param) { ... }` |
| 🟠 警告 | `get_shop_setting` 方法 | **巨型 Switch 与隐式变量**：超 150 行的 `switch` 难以维护；部分分支未赋值 `$result`，依赖末尾 `?? ''` 兜底，易引发逻辑遗漏。静态缓存 `self::$shop_config` 在长驻进程（如 Swoole/RoadRunner）中会导致多请求数据串扰。 | 1. 在 `switch` 前初始化 `$result = '';`；<br>2. 若运行环境为 PHP-FPM 可保留静态缓存，否则建议改用 CI Cache 驱动或请求级缓存；<br>3. 考虑将转换逻辑抽离为私有方法。 | `public function get_shop_setting(...) {<br> $result = ''; // 显式初始化<br> switch ($field) { ... }<br> return $result;<br>}` |
| 🟠 警告 | `get_wechat_group_config` 方法 | **全局状态污染风险**：使用 `$this->set_table_name()` 临时修改表名，若后续查询抛出异常，表名无法恢复，将影响同一进程内的后续请求。 | 使用 `try...finally` 确保状态恢复，或优先使用 CI Query Builder 的 `from()` 与 `join()` 避免修改模型内部状态。 | `try {<br> $this->set_table_name($table_name . ' a');<br> $res = $this->select($where, $fields);<br>} finally {<br> $this->set_table_name($table_name);<br>}` |
| 🟡 建议 | `get_one` 方法参数 | **拼写错误**：参数名 `$fileds` 拼写错误，应为 `$fields`。 | 修正拼写，保持代码规范与 IDE 提示准确性。 | `public function get_one($where, $fields = '*', $order = '')` |
| 🟡 建议 | 多处使用 `throwError()` | **非标准异常处理**：依赖全局函数 `throwError()` 不符合现代 PHP 规范，且可能直接暴露堆栈信息或中断正常流程。 | 改用标准 PHP 异常或返回统一错误结构体，交由框架错误处理器统一拦截。 | `throw new \InvalidArgumentException('门店ID不能为空');` |
| 🟡 建议 | 全局代码规范 | **命名与缩进**：方法名使用 `snake_case`（CI3 历史习惯），不符合 PSR-12 推荐的 `camelCase`；缩进混用 Tab 与空格；缺少类型声明。 | 若团队强制沿用 CI3 规范可保留，但建议统一为 4 空格缩进。新增方法建议采用 `camelCase`，并补充 PHP 7+ 类型提示。 | `public function getShopSetting(int $merchantId, int $shopId, string $field): string` |
---
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **重构 `renewal_audio_broadcast` 方法**：当前实现是典型的“循环查库”反模式。必须将模型加载外置，并采用批量查询（如收集所有 `shop_id` 后一次性 `get_community_bill_shop`，或使用 `JOIN` 关联查询账单与房间）。
2. **清理文件顶部全局代码**：立即删除 `$CI = &get_instance();` 及外部 `load->model()`，避免框架生命周期冲突与内存泄漏。
3. **修复未初始化变量**：`deal_audio_content_params` 中的 `$total_time` 必须提前声明，否则在严格错误级别下会导致脚本中断。
### 🛠 后续重构与优化方向
1. **拆分巨型 `switch`**：`get_shop_setting` 承担了过多职责。建议：
- 将纯映射类字段（如直接返回 `$data['xxx'] ?? default`）提取为配置数组或动态属性访问。
- 将需要复杂计算/关联查询的字段（如 `currency_symbol`, `book_hour_options`）拆分为独立的私有方法 `private function resolveCurrencySymbol($data)`，在 `switch` 中仅做路由。
2. **引入请求级缓存**：若项目未来可能迁移至 PHP-FPM 以外的运行环境（如 Swoole、Workerman），`self::$shop_config` 静态缓存将引发严重的数据安全问题。建议改用 CI 的 `$this->cache->save()` 或注入 `Request` 对象进行单次请求缓存。
3. **统一异常与错误处理**：逐步废弃 `throwError()` 全局函数，建立统一的 `AppException` 或返回 `['code' => -1, 'msg' => '...']` 结构，提升 API 健壮性。
4. **数据库查询优化**：`$this->set_table_name()` 属于非标准 CI 用法。建议全面迁移至 `$this->db->from()->join()->get()` 链式调用，避免隐式状态修改。
### ⚠️ 审查局限性说明
- 本次审查仅基于提供的单个 Model 文件。`Simple_model` 父类实现、`hourToTime()`/`timeToHour()` 等全局助手函数、以及 `throwError()` 的具体行为未提供，部分逻辑（如跨天时间判断、缓存机制）的准确性依赖于这些外部组件的实现。
- 若 `phpci` 为高度定制的内部框架，部分 Query Builder 或生命周期建议需以官方文档为准。
> 💡 **附：`renewal_audio_broadcast` 核心优化思路示例**
> ```php
> public function renewal_audio_broadcast()
> {
> // 1. 模型外置
> $this->load->model(['ahead_bill_model', 'ahead_family_servers_model', 'Ahead_ai_audio_player_content_model']);
>
> $time = strtotime(date('Y-m-d H:i:00'));
> $business_date = get_business_date($time)['business_day'];
>
> // 2. 批量获取门店（避免逐店查询）
> $shops = $this->ahead_bill_model->get_community_bill_shop($time);
> if (empty($shops)) return true;
>
> $shop_ids = array_column($shops, 'shop_id');
> $merchant_ids = array_column($shops, 'merchant_id');
>
> // 3. 批量获取账单/房间数据（示例：使用 WHERE IN 或 JOIN）
> // 实际需根据 ahead_bill_model 的 API 调整，此处展示批量处理思想
> $all_bills = $this->ahead_bill_model->get_end_time_rooms_batch($merchant_ids, $shop_ids, $business_date, $time);
>
> foreach ($all_bills as $bill) {
> // 4. 统一处理播报逻辑，消除嵌套循环
> $this->ahead_family_servers_model->operational_scene_audio_broadcast(
> $bill['merchant_id'], $bill['shop_id'], $bill['room_id'], 'renewal',
> ['remaining_time' => $bill['renewal_time'], 'ai_audio' => $bill['is_ai']]
> );
> }
> return true;
> }
> ```
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780998262
|
1780998262
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
649
|
21
|
296
|
1
|
|
0
|
🔍 代码审查报告：pay-260616 - Merge branch 'pay-260616 🔍 代码审查报告：pay-260616 - Merge branch 'pay-260616' of https://gitea.g-hi.co...
|
## 自动代码审查报告
**分支**: pay-260616
**提交**: `6280ae8dc ## 自动代码审查报告
**分支**: pay-260616
**提交**: `6280ae8dcba40d7f0e7ed8d6e14cea5efb25ed32`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-09 17:41:23
---
## 📋 审查摘要
- **变更文件数**: 2
- **严重问题**: 3
- **高危问题**: 4
- **中危问题**: 3
- **建议优化**: 4
## 🐛 发现的问题
### <font color="red">[语法错误] PHP文件顶部直接调用 get_instance() 导致框架初始化失败</font>
- **严重程度**: <font color="red">严重</font>
- **文件**: `application/models/Ahead_shop_config_second_model.php`
- **行号**: 第 3 行
- **问题描述**: 在类定义外部直接执行 `$CI = &get_instance();`。CodeIgniter 框架的超级对象（Super Object）仅在控制器实例化后才会通过 `get_instance()` 返回。在模型文件加载阶段（通常早于控制器实例化）调用此函数会返回 `null` 或直接抛出 `Fatal error: Call to undefined function get_instance()`，导致整个应用崩溃。
- **修复建议**: 删除文件顶部的 `$CI = &get_instance();` 和 `$CI->load->model('Simple_model');`。在 CI 模型中，应直接使用 `$this->load->model()` 加载依赖，或在具体方法内部按需加载。
### <font color="red">[跨文件调用] 调用了未定义的自定义辅助函数</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `application/models/Ahead_shop_config_second_model.php`
- **行号**: 第 338, 352, 353, 355, 446 行等
- **问题描述**: 代码中多处直接调用了 `throwError()`, `hourToTime()`, `timeToHour()`, `minToStr()`, `get_business_date()` 等函数。这些并非 PHP 内置函数或 CI 标准辅助函数。若未在 `autoload.php` 中全局加载或未在当前文件 `require/include`，将触发 `Fatal error: Uncaught Error: Call to undefined function`。
- **修复建议**:
1. 确认这些函数是否定义在自定义 Helper 中，并在控制器或 `autoload.php` 中通过 `$this->load->helper('your_helper')` 加载。
2. 若为全局函数，建议在文件顶部显式引入或改用类方法封装，避免隐式依赖。
### <font color="red">[跨文件调用] JS 模型文件引用路径及类名未经验证</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `web/Hi-Zan/Hi-Zan/pages/community-reserve/order-list/order-list.js`
- **行号**: 第 2-5 行
- **问题描述**: 引入了 `ReserveModel`, `PublicModel`, `OrderModel`, `RoomModel`。根据微信小程序规范，需确保 `../../../models/reserve.js` 等文件真实存在，且内部正确 `export` 了对应类。若文件缺失或导出名称不匹配，页面加载时将直接白屏报错。
- **修复建议**: 核对项目目录结构，确保 `models/` 目录下存在对应 `.js` 文件，且导出语法为 `export class ReserveModel { ... }`。建议在开发环境开启严格模式验证模块依赖。
### [逻辑BUG] Array.find() 未处理未匹配情况导致运行时 TypeError
- **严重程度**: 高危
- **文件**: `web/Hi-Zan/Hi-Zan/pages/community-reserve/order-list/order-list.js`
- **行号**: 第 118 行
- **问题描述**: `this.data.status_list.find(item => item.value == this.data.status).name`。当 `this.data.status` 传入的值不在 `status_list` 中时（例如脏数据或新增状态未同步），`find()` 返回 `undefined`，紧接着访问 `.name` 会抛出 `TypeError: Cannot read properties of undefined (reading 'name')`，导致页面崩溃。
- **修复建议**: 增加安全判断或使用可选链：
```javascript
const matchedStatus = this.data.status_list.find(item => item.value == this.data.status);
this.setData({
status_name: matchedStatus ? matchedStatus.name : '全部状态'
});
```
### [安全隐患] URL 参数拼接未全面使用 encodeURIComponent
- **严重程度**: 高危
- **文件**: `web/Hi-Zan/Hi-Zan/pages/community-reserve/order-list/order-list.js`
- **行号**: 第 245, 263, 270, 277 行等
- **问题描述**: 多处使用字符串拼接构造跳转 URL，如 `'?order_id=' + this.data.oper_order.id + '&type=1...'`。虽然当前业务字段多为数字，但若未来 `shop_id`、`room_id` 或 `shelf_name` 包含特殊字符（如 `&`, `=`, `?`, 中文），将破坏 URL 结构，导致参数解析错误或路由跳转失败。
- **修复建议**: 对所有动态参数统一使用 `encodeURIComponent()` 包裹，或使用微信官方推荐的 `wx.navigateTo` 参数对象化写法（若框架支持）。
### [逻辑BUG] PHP 静态缓存未处理空值且缺乏清理机制
- **严重程度**: 中危
- **文件**: `application/models/Ahead_shop_config_second_model.php`
- **行号**: 第 108, 468, 508 行
- **问题描述**: 使用 `self::$shop_config[$shop_id]` 进行请求级缓存。若 `$this->get_one()` 查询不到数据返回 `false` 或 `null`，该空值会被缓存。后续同一请求中再次调用将直接返回缓存的空值，可能导致业务逻辑误判（如认为配置已存在但实际为空）。此外，静态变量在 CLI 脚本或长连接服务中不会自动释放，可能引发内存泄漏或脏数据。
- **修复建议**: 缓存前判断结果有效性，或提供 `clearCache($shop_id = null)` 方法。建议改用 CI 的 `Cache` 库或 Redis 进行分布式缓存。
### [代码质量] PHP 模型中 switch 语句过长，违反单一职责原则
- **严重程度**: 中危
- **文件**: `application/models/Ahead_shop_config_second_model.php`
- **行号**: 第 110-320 行
- **问题描述**: `get_shop_setting()` 方法包含超过 50 个 `case` 分支，代码行数超 200 行。该函数承担了数据查询、类型转换、默认值处理、业务规则计算（如时间取整、状态映射）等多重职责，可读性差且极难维护。
- **修复建议**: 采用配置映射表（Config Map）或策略模式重构。例如将默认值和转换逻辑抽离为独立数组或配置文件，通过 `array_key_exists` 和闭包函数动态处理，将函数长度控制在 50 行以内。
### [代码质量] PHP 模型加载命名大小写不一致，存在跨平台兼容性风险
- **严重程度**: 中危
- **文件**: `application/models/Ahead_shop_config_second_model.php`
- **行号**: 第 440, 485 行
- **问题描述**: 混用了 `$this->load->model('Ahead_ai_audio_player_content_model')`（首字母大写）和 `$this->load->model('ahead_shop_model')`（全小写）。CodeIgniter 在 Windows 下不区分文件名大小写，但在 Linux/生产环境中严格区分。若实际文件名与加载字符串大小写不一致，将导致 `Unable to locate the model you have specified` 错误。
- **修复建议**: 统一遵循 CI 规范：模型文件名全小写（如 `ahead_ai_audio_player_content_model.php`），加载时统一使用小写字符串。
## ✅ 代码亮点
1. **JS 状态管理清晰**：小程序页面数据（`data`）结构划分明确（预订单、门店、酒水订单分区），配合 `tabIndex` 和 `loading` 状态控制，交互逻辑完整。
2. **PHP 缓存优化意识**：在 `get_shop_setting` 和 `check_is_cleaned_for_open_room` 中使用了静态变量 `self::$shop_config` 避免重复查询数据库，提升了单次请求性能。
3. **防御性编程**：JS 中多处使用 `e.currentTarget.dataset.item` 安全获取事件参数，PHP 中大量使用 `??` 空值合并运算符设置默认值，降低了空指针风险。
## 📝 总体建议
1. **修复致命依赖问题**：优先处理 PHP 文件顶部的 `get_instance()` 调用和未声明的辅助函数。这是导致应用无法启动或运行时报错的核心原因。
2. **统一跨文件引用规范**：建立项目级的模型/Helper 命名与加载规范。PHP 端严格遵循 `小写文件名 + 小写加载`；JS 端确保 `import` 路径与导出类名一致，可考虑引入 ESLint 进行静态检查。
3. **重构超长 Switch**：建议将 `get_shop_setting` 拆分为 `基础配置获取` + `业务规则处理器`。可引入配置中心或 JSON 映射表，使新增业务场景时无需修改核心模型代码。
4. **增强异常边界处理**：JS 端对网络请求返回值增加 `Array.isArray()` 和 `res.code === 200` 校验；PHP 端对 `throwError` 等自定义异常捕获机制进行统一封装，避免错误堆栈直接暴露给前端。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780998083
|
1780998083
|
0
|
0
|
0
|
0
|
Edit
Delete
|