SQLite Web: gitea.db

id

Primary key. INTEGER NOT NULL

repo_id

INTEGER

index

INTEGER

poster_id

INTEGER

original_author

TEXT

original_author_id

INTEGER

name

TEXT

content

## 自动代码审查报告

**分支**: admin-260519
**提交**: `2d4d2b55bfb87cabc2bc226b8c7fcbf22183ce22`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-20 18:39:49

---

## 1. 审查摘要
- **代码质量评分**：6.0 / 10 分
- **总体评价**：代码完整实现了预购订单查询、导出、退款及支付流水管理等核心业务逻辑，整体结构清晰。但存在**严重的 SQL 注入风险**、**核心退款流程缺失事务控制**、以及多处违反 MVC 分层原则与 CI3/现代 PHP 规范的反模式代码。若直接上线，极易引发数据不一致与安全漏洞。
- **风险等级**：🔴 高

> 📌 **框架说明**：代码结构、语法及调用方式（如 `$CI = &get_instance()`、`$this->load->model()`、`$this->db->trans_start()`）高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请结合其官方文档对底层 ORM/DB 驱动的差异进行适配。以下审查基于 CI3 最佳实践。

---

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_songs_sales_pay_log_model.php` ~第 64 行 | **SQL 注入漏洞**：`$title` 参数未经过滤直接拼接到 `LIKE` 语句中，攻击者可构造恶意输入破坏查询或拖库。 | 使用框架查询构建器或参数化查询替代字符串拼接。 | `$this->db->group_start();` `$this->db->like('log._title', $title);` `$this->db->or_like('book._group_package_name', $title);` `$this->db->or_like('book._group_platform_name', $title);` `$this->db->group_end();` |
| 🔴 严重 | `Ahead_preorder_order_model.php` `refund()` 方法 (约 133-210 行) | **缺失数据库事务**：退款涉及主订单状态更新、会员余额回滚、多表明细插入、支付流水更新等 10+ 次 DB 操作。若中途失败，将导致资金账目不平、状态不一致。 | 使用 `$this->db->trans_start()` 包裹所有写操作，失败时 `trans_rollback()`。 | `$this->db->trans_start();` `// ... 所有 update/insert 操作 ...` `if ($this->db->trans_status() === false) {` ` $this->db->trans_rollback();` ` return ['status'=>false, 'msg'=>'退款失败'];` `}` `$this->db->trans_complete();` |
| 🟠 警告 | `Ahead_songs_sales_pay_log_model.php` ~第 238 行 | **动态 SQL 拼接风险**：`$log_where` 使用字符串拼接构造 `WHERE` 条件，未做类型校验或转义，存在注入隐患且难以维护。 | 改用数组条件或查询构建器链式调用。 | `$log_where = [` ` '_merchant_id' => $pay_log['_merchant_id'],` ` '_shop_id' => $pay_log['_shop_id'],` ` '_relation_id' => $pay_log['_order_id'],` ` '_status' => 1,` ` '_type IN' => [1, 5, 13]` `];` |
| 🟠 警告 | `Ahead_preorder_order_model.php` ~第 149-152 行 | **JSON 解析未校验**：`json_decode($order['_mult_pay'], true)` 若字段为空或格式错误将返回 `null`，后续 `foreach` 会触发 `Warning: Invalid argument supplied for foreach()`。 | 增加类型判断与默认值处理。 | `$mult_pay = json_decode($order['_mult_pay'], true);` `if (is_array($mult_pay)) {` ` foreach ($mult_pay as $pay) { ... }` `}` |
| 🟠 警告 | `Ahead_preorder_order_model.php` ~第 100 行 | **模型层直接终止脚本**：`exit('成功');` 破坏了 MVC 架构，导致控制器无法统一封装响应格式（如 JSON/HTTP 状态码），且不利于单元测试。 | 移除 `exit`，由控制器统一处理输出与响应。 | `// 模型中直接 return 结果` `return ['status' => true, 'msg' => '导出成功'];` `// 控制器中处理` `$this->output->set_content_type('application/json')->set_output(json_encode($res));` |
| 🟡 建议 | 两个文件顶部 (第 3-4 行) | **类外获取 CI 实例**：`$CI = &get_instance();` 放在类定义外部会在文件被 `include` 时立即执行，不符合 OOP 规范，且可能引发加载顺序问题。 | 移至构造函数或按需调用。 | `public function __construct() {` ` parent::__construct();` ` $this->load->model('Simple_model');` `}` |
| 🟡 建议 | `Ahead_preorder_order_model.php` ~第 63 行 | **数组键未定义检查**：`$this->pay_platform_arr[$v['pay_platform']]` 若数据库值不在映射表中会触发 `Undefined index` 警告。 | 使用空合并运算符或 `isset()` 防御。 | `$v['pay_platform_name'] = $this->pay_platform_arr[$v['pay_platform']] ?? '未知支付';` |
| 🟡 建议 | 多处 `up()` 调用 (如 165, 268, 298 行) | **原始 SQL 字符串更新**：`'_account=_account+' . $vip_pay_amount` 依赖自定义 `up()` 方法解析，易引发注入且丧失框架查询构建器的类型安全与日志追踪能力。 | 优先使用 `$this->db->set()` 或严格类型转换 `(float)`。 | `$this->db->set('_account', '_account + ' . (float)$vip_pay_amount, FALSE);` `$this->db->where(['_merchant_id' => $merchant_id, '_vip_card' => $order['_vip_card']]);` `$this->db->update('vip_table');` |

---

## 3. 总结与行动建议

### 🚨 优先修复项（P0）
1. **修复 SQL 注入**：立即替换 `Ahead_songs_sales_pay_log_model.php` 中所有字符串拼接的 `WHERE` 与 `LIKE` 条件，改用 CI3 查询构建器或参数绑定。
2. **补充事务控制**：为 `Ahead_preorder_order_model::refund()` 添加完整的事务包裹。当前 `Ahead_songs_sales_pay_log_model::refund()` 虽开启了事务，但调用的子模型退款方法无事务，一旦子方法内部报错或返回失败，外层事务可能无法正确回滚关联数据。
3. **移除模型层 `exit`**：将 `export()` 方法中的 `exit('成功')` 改为返回状态码，由 Controller 统一输出，保持架构纯净。

### 🛠 后续重构与优化方向
1. **统一输入校验**：所有来自 `$param` 的时间、金额、ID 等字段，应在入口处进行类型强转与合法性校验（如 `strtotime()` 返回 `false` 时应拦截或赋默认值）。
2. **封装全局函数**：代码中大量使用 `throwError()`、`check_export_datetime()`、`change_number_format()`、`wx_refund()` 等全局函数。建议将其迁移至 `application/helpers/` 或 `application/libraries/`，并通过 `$this->load->helper()` 或 `$this->load->library()` 按需加载，提升可测试性与命名空间隔离。
3. **优化模型加载策略**：频繁在方法内调用 `$this->load->model()` 会增加运行时开销。建议在 `__construct()` 中预加载高频模型，或启用 CI3 的 `autoload.php` 自动加载机制。
4. **金额计算精度**：涉及财务退款与余额回滚的逻辑（如 `_room_account` 与 `_account` 拆分退款），建议统一使用 `bcmath` 扩展或严格保留 2 位小数的浮点运算，避免 PHP 浮点数精度丢失导致的账目偏差。

> 💡 **局限性说明**：本次审查基于提供的两个 Model 文件。由于未提供 `Simple_model` 底层实现、控制器调用链路及全局函数定义，部分自定义方法（如 `up()`、`select()`、`get_one()`）的底层安全性与性能表现需结合实际框架源码进一步评估。建议结合 CI Profiler 或 Xdebug 进行全链路压测与 SQL 审计。

---
*此 Issue 由代码审查服务自动创建*

TEXT

milestone_id

INTEGER

priority

INTEGER

is_closed

INTEGER

is_pull

INTEGER

num_comments

INTEGER

ref

TEXT

deadline_unix

INTEGER

created_unix

INTEGER

updated_unix

INTEGER

closed_unix

INTEGER

is_locked

INTEGER NOT NULL (default 0

content_version

INTEGER NOT NULL (default 0

time_estimate

INTEGER NOT NULL (default 0

Update row 256 in issue