442 lines
15 KiB
SQL
442 lines
15 KiB
SQL
# 代码调整记录 - 2025-01-18 - wuzhen
|
||
|
||
## 2025-01-18 17:35:00 - 优化预算申报统计分析模块初始化逻辑
|
||
|
||
### 🎯 优化目标
|
||
优化预算申报统计分析模块的初始化逻辑，界面打开时默认查询所有年份的申报数据，并将预算年度选项从2025年开始。
|
||
|
||
### 🔍 原有实现问题
|
||
**原逻辑**：
|
||
1. ❌ 界面打开时默认查询2024年的数据
|
||
2. ❌ 年度选择下拉框包含2024年选项
|
||
3. ❌ 用户需要手动选择"全部年度"才能看到完整数据
|
||
|
||
**问题影响**：
|
||
- 用户体验不佳，需要额外操作才能看到全量数据
|
||
- 年度选项包含过时的2024年，不符合当前业务需求
|
||
|
||
### 🚀 优化方案
|
||
**新逻辑**：
|
||
1. ✅ 界面打开时默认查询所有年份的申报数据（budgetYear = ''）
|
||
2. ✅ 年度选择从2025年开始，移除2024年选项
|
||
3. ✅ 增加更多年份选项（2025-2028年）
|
||
4. ✅ 重置功能保持一致，重置后也是查询所有年份
|
||
|
||
### 🔧 技术实现
|
||
|
||
#### 修改文件
|
||
**文件路径**：`web/src/views/cw/statistics/index.vue`
|
||
|
||
#### 核心变更内容
|
||
|
||
1. **修改默认查询参数**（第244行）
|
||
```javascript
|
||
// 修改前
|
||
queryParams: {
|
||
pageNum: 1,
|
||
pageSize: 10,
|
||
budgetYear: '2024' // 设置默认年度为2024
|
||
},
|
||
|
||
// 修改后
|
||
queryParams: {
|
||
pageNum: 1,
|
||
pageSize: 10,
|
||
budgetYear: '' // 设置默认为空，查询所有年份数据
|
||
},
|
||
```
|
||
|
||
2. **更新年度选择选项**（第7-12行）
|
||
```html
|
||
<!-- 修改前 -->
|
||
<el-select v-model="queryParams.budgetYear" placeholder="请选择预算年度" clearable style="width: 200px">
|
||
<el-option label="全部年度" value="" />
|
||
<el-option label="2024年" value="2024" />
|
||
<el-option label="2025年" value="2025" />
|
||
<el-option label="2026年" value="2026" />
|
||
</el-select>
|
||
|
||
<!-- 修改后 -->
|
||
<el-select v-model="queryParams.budgetYear" placeholder="请选择预算年度" clearable style="width: 200px">
|
||
<el-option label="全部年度" value="" />
|
||
<el-option label="2025年" value="2025" />
|
||
<el-option label="2026年" value="2026" />
|
||
<el-option label="2027年" value="2027" />
|
||
<el-option label="2028年" value="2028" />
|
||
</el-select>
|
||
```
|
||
|
||
3. **重置逻辑验证**（第438-440行）
|
||
```javascript
|
||
/** 重置按钮操作 */
|
||
resetQuery() {
|
||
this.queryParams.budgetYear = '' // ✅ 重置为空，查询所有年份
|
||
this.handleQuery()
|
||
}
|
||
```
|
||
|
||
### 📊 预期效果
|
||
- ✅ **用户体验优化**：界面打开即可看到所有年份的完整统计数据
|
||
- ✅ **数据展示完整**：默认展示全量数据，避免数据遗漏
|
||
- ✅ **年度选项更新**：移除过时的2024年，从2025年开始
|
||
- ✅ **操作一致性**：重置功能与初始化逻辑保持一致
|
||
- ✅ **前瞻性设计**：增加未来年份选项，便于长期使用
|
||
|
||
### 🚨 后续操作要求
|
||
1. **无需重新编译**：仅前端静态资源修改，刷新页面即可生效
|
||
2. **功能测试验证**：
|
||
- 打开预算申报统计分析页面，验证默认显示所有年份数据
|
||
- 测试年度筛选功能，确认2025-2028年选项正常
|
||
- 测试重置功能，确认重置后显示全部年度数据
|
||
- 验证各个统计图表和数据表格正常加载
|
||
|
||
### 💡 技术要点
|
||
- **前端状态管理**：通过修改data中的默认值实现初始化优化
|
||
- **用户界面优化**：年度选项从当前年度开始，提升用户体验
|
||
- **数据查询逻辑**：空字符串作为查询参数时，后端返回所有年份数据
|
||
- **一致性保证**：初始化、重置、查询逻辑保持统一
|
||
|
||
---
|
||
|
||
## 2025-01-18 17:20:00 - 优化财务审批院领导选择逻辑（基于权限角色）
|
||
|
||
### 🎯 优化目标
|
||
将财务审批模块中选择院领导的逻辑从"按部门查找"优化为"按权限角色查找"，确保选择的用户具有实际的院领导审批权限。
|
||
|
||
### 🔍 原有实现问题
|
||
**原逻辑**：查询部门名称包含"院领导"的部门，返回该部门下的所有正常状态用户
|
||
**问题分析**：
|
||
1. ❌ 依赖部门名称，不够灵活
|
||
2. ❌ 可能包含非审批人员（部门内其他用户）
|
||
3. ❌ 没有权限验证，可能选择到无权限用户
|
||
4. ❌ 不符合RBAC权限模型设计理念
|
||
|
||
### 🚀 优化方案
|
||
**新逻辑**：基于权限角色查找具有 `cw:approval:leader` 权限的所有用户
|
||
**优势**：
|
||
- ✅ 权限导向，确保选择的用户有审批权限
|
||
- ✅ 不依赖部门结构，更灵活
|
||
- ✅ 符合RBAC权限模型
|
||
- ✅ 便于后续权限管理
|
||
- ✅ 保留部门查询作为备选方案
|
||
|
||
### 🔧 技术实现
|
||
|
||
#### 修改文件
|
||
**文件路径**：`backend/ruoyi-modules/ruoyi-cw/src/main/java/org/dromara/cw/service/impl/CwApprovalRecordServiceImpl.java`
|
||
**修改方法**：`getAvailableApprovers(Long applicationId)` (第1126行)
|
||
|
||
#### 核心变更内容
|
||
|
||
1. **添加JdbcTemplate依赖**
|
||
```java
|
||
@Autowired
|
||
private JdbcTemplate jdbcTemplate;
|
||
```
|
||
|
||
2. **权限查询SQL**
|
||
```sql
|
||
SELECT DISTINCT
|
||
u.user_id, u.user_name, u.nick_name,
|
||
COALESCE(d.dept_name, '未分配部门') as dept_name,
|
||
COALESCE(d.dept_id, 0) as dept_id
|
||
FROM sys_user u
|
||
LEFT JOIN sys_dept d ON u.dept_id = d.dept_id
|
||
INNER JOIN sys_user_role ur ON u.user_id = ur.user_id
|
||
INNER JOIN sys_role r ON ur.role_id = r.role_id
|
||
INNER JOIN sys_role_menu rm ON r.role_id = rm.role_id
|
||
INNER JOIN sys_menu m ON rm.menu_id = m.menu_id
|
||
WHERE m.perms = 'cw:approval:leader'
|
||
AND u.status = '0'
|
||
AND u.del_flag = '0'
|
||
AND r.status = '0'
|
||
ORDER BY u.user_name
|
||
```
|
||
|
||
3. **双重保障机制**
|
||
- **主方案**：基于权限查询具有 `cw:approval:leader` 权限的用户
|
||
- **备选方案**：如果权限查询无结果，使用原有的部门查询逻辑
|
||
|
||
#### 实现逻辑流程
|
||
```
|
||
1. 执行权限查询SQL → 查找具有cw:approval:leader权限的用户
|
||
2. 转换查询结果 → 组装返回Map格式数据
|
||
3. 结果检查 → 如果权限查询为空，启用部门查询备选方案
|
||
4. 返回结果 → 具有审批权限的院领导列表
|
||
```
|
||
|
||
### 📊 预期效果
|
||
- ✅ **精确权限控制**：只返回真正具有院领导审批权限的用户
|
||
- ✅ **系统一致性**：与RBAC权限模型保持一致
|
||
- ✅ **灵活性提升**：不依赖特定部门名称
|
||
- ✅ **向后兼容**：保留部门查询作为备选，确保系统稳定
|
||
- ✅ **日志完善**：详细的查询过程日志，便于问题排查
|
||
|
||
### 🚨 后续操作要求
|
||
1. **重新编译后端服务**：`mvn clean compile`
|
||
2. **重启Spring Boot应用**
|
||
3. **功能测试验证**：
|
||
- 财务人员审批时选择院领导功能
|
||
- 验证只显示具有 `cw:approval:leader` 权限的用户
|
||
- 测试备选方案是否正常工作
|
||
|
||
### 💡 技术要点
|
||
- **权限查询**：通过多表关联查询用户权限
|
||
- **NULL值处理**：使用COALESCE处理部门信息为空的情况
|
||
- **异常处理**：查询失败时返回空列表，不影响主流程
|
||
- **日志记录**：完整的查询过程日志，便于调试和监控
|
||
|
||
---
|
||
|
||
## 2025-01-18 16:45:00 - 修复财务和院领导审批"我已审批"查询问题
|
||
|
||
### 🚨 问题描述
|
||
财务审批和院领导审批模块的"我已审批"功能无法查询到已审批的记录，用户反馈看不到自己审批过的内容。
|
||
|
||
### 🔍 问题分析
|
||
通过深度分析发现两个核心问题：
|
||
1. **财务已审批查询状态不匹配**：
|
||
- 审批时更新状态为：`approved/rejected`
|
||
- 查询时使用状态为：`completed/rejected`
|
||
- 导致状态不匹配，查询不到记录
|
||
|
||
2. **院领导已审批查询状态不匹配**：
|
||
- 审批时更新状态为：`approved/rejected`
|
||
- 查询时使用状态为：`complete`
|
||
- 导致状态不匹配，查询不到记录
|
||
|
||
### 🔧 修复方案
|
||
**方案一：修正查询状态（已采用）**
|
||
|
||
#### 修复内容1：财务已审批查询
|
||
**文件**：`backend/ruoyi-modules/ruoyi-cw/src/main/resources/mapper/CwApprovalRecordMapper.xml`
|
||
**位置**：第146行
|
||
```xml
|
||
<!-- 修复前 -->
|
||
and ar.node_status in ('completed', 'rejected')
|
||
|
||
<!-- 修复后 -->
|
||
and ar.node_status in ('approved', 'rejected')
|
||
```
|
||
|
||
#### 修复内容2：院领导已审批查询
|
||
**文件**：`backend/ruoyi-modules/ruoyi-cw/src/main/java/org/dromara/cw/service/impl/CwApprovalRecordServiceImpl.java`
|
||
**位置**：第467行
|
||
```java
|
||
// 修复前
|
||
return baseMapper.selectByApproverAndNode(currentUserId, "leader_approve", "complete");
|
||
|
||
// 修复后
|
||
return baseMapper.selectByApproverAndNode(currentUserId, "leader_approve", "approved");
|
||
```
|
||
|
||
### 📊 修复效果预期
|
||
- ✅ 财务人员能在"财务已审批"中看到自己审批的记录
|
||
- ✅ 院领导能在"院领导已审批"中看到自己审批的记录
|
||
- ✅ 通用"我已审批"功能正常工作（之前已修复approverId问题）
|
||
- ✅ 三个查询功能相互独立，互不影响
|
||
|
||
### 🚨 后续操作要求
|
||
1. **重新编译后端服务**：`mvn clean compile`
|
||
2. **重启Spring Boot应用**
|
||
3. **功能测试验证**：用财务和院领导账号分别测试审批和查询功能
|
||
|
||
---
|
||
|
||
## 📋 本次变更概述
|
||
**变更时间**: 2025-01-18 15:30:00
|
||
**变更人员**: wuzhen
|
||
**变更类型**: 🔧 审批流程状态统一修复
|
||
**影响范围**: 财务预算申报审批流程
|
||
|
||
## 🎯 问题背景
|
||
发现审批流程中存在**状态不统一**和**逻辑错误**问题：
|
||
|
||
### 原有问题
|
||
1. **节点状态不统一**：
|
||
- 部门审批通过：`"approved"`
|
||
- 财务审批通过：`"completed"`
|
||
- 分管领导审批通过：`"completed"`
|
||
|
||
2. **分管领导审批逻辑错误**：
|
||
- 审批通过后状态设为 `"finance_approving"`
|
||
- 应该是最终完成状态 `"approved"`
|
||
|
||
3. **"我已审批"查询逻辑不准确**：
|
||
- 使用排除法 `.notIn("waiting", "pending")`
|
||
- 可能包含错误状态
|
||
|
||
## 🔧 修改内容
|
||
|
||
### 1. 统一财务审批节点状态
|
||
**文件**: `CwApprovalRecordServiceImpl.java`
|
||
**位置**: 第1302行
|
||
**修改**: `"completed"` → `"approved"`
|
||
|
||
```java
|
||
// 修改前
|
||
updateRecord.setNodeStatus("approve".equals(approveAction) ? "completed" : "rejected");
|
||
|
||
// 修改后
|
||
updateRecord.setNodeStatus("approve".equals(approveAction) ? "approved" : "rejected");
|
||
```
|
||
|
||
### 2. 统一分管领导审批节点状态
|
||
**文件**: `CwApprovalRecordServiceImpl.java`
|
||
**位置**: 第1202行
|
||
**修改**: `"completed"` → `"approved"`
|
||
|
||
```java
|
||
// 修改前
|
||
updateRecord.setNodeStatus("approve".equals(approveAction) ? "completed" : "rejected");
|
||
|
||
// 修改后
|
||
updateRecord.setNodeStatus("approve".equals(approveAction) ? "approved" : "rejected");
|
||
```
|
||
|
||
### 3. 修正分管领导审批的申报状态逻辑
|
||
**文件**: `CwApprovalRecordServiceImpl.java`
|
||
**位置**: 第1224-1229行
|
||
**修改**: `"finance_approving"` → `"approved"`
|
||
|
||
```java
|
||
// 修改前
|
||
if ("approve".equals(approveAction)) {
|
||
// 审批通过，状态改为财务审批中
|
||
newStatus = "finance_approving";
|
||
log.info("✅ 分管领导审批通过，提交财务审批：applicationId={}", applicationId);
|
||
// TODO: 这里可以添加创建财务审批记录的逻辑
|
||
|
||
// 修改后
|
||
if ("approve".equals(approveAction)) {
|
||
// 分管领导审批通过，整个审批流程完成
|
||
newStatus = "approved";
|
||
log.info("✅ 分管领导审批通过，整个审批流程完成：applicationId={}", applicationId);
|
||
```
|
||
|
||
### 4. 修正"我已审批"查询逻辑
|
||
**文件**: `CwApprovalRecordServiceImpl.java`
|
||
**位置**: 第381行
|
||
**修改**: 排除法 → 精确匹配法
|
||
|
||
```java
|
||
// 修改前
|
||
queryWrapper.eq(CwApprovalRecord::getApproverId, currentUserId)
|
||
.notIn(CwApprovalRecord::getNodeStatus, Arrays.asList("waiting", "pending"))
|
||
.orderByDesc(CwApprovalRecord::getEndTime);
|
||
|
||
// 修改后
|
||
queryWrapper.eq(CwApprovalRecord::getApproverId, currentUserId)
|
||
.in(CwApprovalRecord::getNodeStatus, Arrays.asList("approved", "rejected", "returned"))
|
||
.orderByDesc(CwApprovalRecord::getEndTime);
|
||
```
|
||
|
||
## 📊 修改后的状态体系
|
||
|
||
### 双层状态管理
|
||
1. **申报级别状态** (`CwProjectApplication.approval_status`):
|
||
- `"approved"` = 整个流程完成 ✅
|
||
- `"approving"` = 审批进行中 ⏳
|
||
- `"rejected"` = 审批被拒绝 ❌
|
||
- `"returned"` = 被退回修改 🔄
|
||
|
||
2. **审批节点状态** (`CwApprovalRecord.node_status`):
|
||
- `"approved"` = 节点通过（统一标准）
|
||
- `"rejected"` = 节点拒绝
|
||
- `"returned"` = 节点退回
|
||
- `"waiting"` = 等待审批
|
||
|
||
### 正确的流程逻辑
|
||
```
|
||
提交申报 → approval_status = "submitted"
|
||
↓
|
||
部门审批 → node_status = "approved" + approval_status = "approving"
|
||
↓
|
||
财务审批 → node_status = "approved" + approval_status = "approving"
|
||
↓
|
||
分管领导审批 → node_status = "approved" + approval_status = "approved" ✅最终完成
|
||
```
|
||
|
||
## 🎯 预期效果
|
||
|
||
1. **状态统一**：所有审批节点通过状态统一为 `"approved"`
|
||
2. **流程正确**：分管领导审批通过后，整个流程完成
|
||
3. **用户体验**：通过 `approval_status` 清晰知道审批进度
|
||
4. **功能修复**："我已审批"功能能正确显示审批记录
|
||
|
||
## ⚠️ 注意事项
|
||
|
||
1. **需要重新编译和启动服务**以生效
|
||
2. **现有数据兼容性**：历史数据中的 `"completed"` 状态需要考虑兼容
|
||
3. **前端显示**：确保前端能正确显示新的状态逻辑
|
||
4. **测试验证**：需要完整测试审批流程的各个环节
|
||
|
||
## 📝 后续工作
|
||
|
||
1. 重启后端服务
|
||
2. 测试完整的审批流程
|
||
3. 验证"我已审批"功能
|
||
4. 检查前端状态显示是否正确
|
||
5. 考虑是否需要数据迁移脚本处理历史数据
|
||
|
||
---
|
||
|
||
## 🔧 财务审批"我已审批"功能修复 - 2025-01-18 22:30:00
|
||
|
||
### 🚨 **问题描述**
|
||
- **问题现象**: 财务审批通过后，该申请在"我已审批"列表中看不到
|
||
- **具体案例**: YS202508049627申请财务审批通过后，在财务人员的"我已审批"中不显示
|
||
- **影响范围**: 所有财务审批通过的记录都无法在"我已审批"中查看
|
||
|
||
### 🔍 **根本原因**
|
||
财务审批方法(`financeApproval`)中缺少关键的`approverId`字段更新：
|
||
|
||
**问题代码**:
|
||
```java
|
||
// 财务审批时只更新了状态，但没有更新实际审批人ID
|
||
updateRecord.setNodeStatus("approved");
|
||
updateRecord.setApproveAction("approve");
|
||
// ❌ 缺少：updateRecord.setApproverId(currentUserId);
|
||
```
|
||
|
||
**"我已审批"查询逻辑**:
|
||
```java
|
||
// 根据approverId查询，如果approverId不匹配就查不到
|
||
queryWrapper.eq(CwApprovalRecord::getApproverId, currentUserId)
|
||
```
|
||
|
||
### ✅ **修复方案**
|
||
在`CwApprovalRecordServiceImpl.financeApproval()`方法中添加缺失的审批人信息更新：
|
||
|
||
```java
|
||
// 🔧 修复：更新实际审批人信息（解决"我已审批"查询不到的问题）
|
||
updateRecord.setApproverId(currentUserId);
|
||
|
||
// 更新审批人姓名（保持数据一致性）
|
||
SysUserVo currentUser = sysUserService.selectUserById(currentUserId);
|
||
if (currentUser != null) {
|
||
updateRecord.setApproverName(currentUser.getNickName());
|
||
}
|
||
```
|
||
|
||
### 📁 **修改文件**
|
||
- `backend/ruoyi-modules/ruoyi-cw/src/main/java/org/dromara/cw/service/impl/CwApprovalRecordServiceImpl.java`
|
||
- 修改位置：第1304行后
|
||
- 修改内容：添加approverId和approverName字段更新
|
||
|
||
### 🎯 **预期效果**
|
||
修复后：
|
||
- ✅ 财务审批通过的记录能在"我已审批"中正确显示
|
||
- ✅ 显示正确的审批人信息
|
||
- ✅ 与分管领导审批记录保持一致的数据结构
|
||
- ✅ 历史数据需要手动修复（如果需要）
|
||
|
||
### 🔄 **后续操作**
|
||
1. 重启后端服务
|
||
2. 测试财务审批功能
|
||
3. 验证"我已审批"显示是否正常
|
||
|
||
---
|
||
|
||
**✅ 修改完成时间**: 2025-01-18 15:45:00
|
||
**🔄 状态**: 待重启服务验证 |