From 2dfaaa8a923b33fb58b8789925ba5b7b2df21f3a Mon Sep 17 00:00:00 2001
From: tuqin <tuqin@local>
Date: 星期五, 06 十一月 2026 14:34:37 +0800
Subject: [PATCH] fix(hrm): 调岗申请人以员工档案的 user_id 为准（前端可能传员工主键）
---
yudao-module-hrm/src/main/java/cn/iocoder/yudao/module/hrm/service/approval/HrmTransferApplicationServiceImpl.java | 120 ++++++++++++++++++++++++++++++++++++++++++++++++++---------
1 files changed, 101 insertions(+), 19 deletions(-)
diff --git a/yudao-module-hrm/src/main/java/cn/iocoder/yudao/module/hrm/service/approval/HrmTransferApplicationServiceImpl.java b/yudao-module-hrm/src/main/java/cn/iocoder/yudao/module/hrm/service/approval/HrmTransferApplicationServiceImpl.java
index d63e58a..3671410 100644
--- a/yudao-module-hrm/src/main/java/cn/iocoder/yudao/module/hrm/service/approval/HrmTransferApplicationServiceImpl.java
+++ b/yudao-module-hrm/src/main/java/cn/iocoder/yudao/module/hrm/service/approval/HrmTransferApplicationServiceImpl.java
@@ -2,6 +2,8 @@
import cn.hutool.core.collection.CollUtil;
import cn.hutool.core.util.ObjUtil;
+import cn.hutool.core.util.StrUtil;
+import cn.hutool.json.JSONUtil;
import cn.iocoder.yudao.framework.common.pojo.PageResult;
import cn.iocoder.yudao.framework.common.util.object.BeanUtils;
import cn.iocoder.yudao.module.bpm.api.task.BpmProcessInstanceApi;
@@ -13,12 +15,15 @@
import cn.iocoder.yudao.module.hrm.controller.admin.approval.vo.HrmTransferApplicationRespVO;
import cn.iocoder.yudao.module.hrm.controller.admin.approval.vo.HrmTransferApplicationSaveReqVO;
import cn.iocoder.yudao.module.hrm.dal.dataobject.approval.HrmTransferApplicationDO;
+import cn.iocoder.yudao.module.hrm.dal.dataobject.employee.HrmEmployeeDO;
import cn.iocoder.yudao.module.hrm.dal.mysql.approval.HrmTransferApplicationMapper;
import cn.iocoder.yudao.module.hrm.dal.redis.HrmNoRedisDAO;
import cn.iocoder.yudao.module.hrm.enums.HrmAuditStatusEnum;
import cn.iocoder.yudao.module.hrm.enums.HrmTransferTypeEnum;
+import cn.iocoder.yudao.module.hrm.service.employee.HrmEmployeeService;
import cn.iocoder.yudao.module.system.api.dept.DeptApi;
import cn.iocoder.yudao.module.system.api.dept.PostApi;
+import cn.iocoder.yudao.module.system.api.storage.StorageAttachmentApi;
import cn.iocoder.yudao.module.system.api.dept.dto.DeptRespDTO;
import cn.iocoder.yudao.module.system.api.dept.dto.PostRespDTO;
import cn.iocoder.yudao.module.system.api.user.AdminUserApi;
@@ -47,7 +52,7 @@
/**
* 璋冨矖鐢宠 Service 瀹炵幇绫�
*
- * @author 鑺嬮亾婧愮爜
+ * @author 瓒呯骇绠＄悊鍛�
*/
@Service
@Validated
@@ -65,41 +70,103 @@
@Resource
private AdminUserApi adminUserApi;
@Resource
+ private HrmEmployeeService employeeService;
+ @Resource
private BpmProcessInstanceApi bpmProcessInstanceApi;
@Resource
private BpmCategoryService bpmCategoryService;
+ @Resource
+ private StorageAttachmentApi storageAttachmentApi;
@Resource
private BpmProcessDefinitionService bpmProcessDefinitionService;
@Override
@Transactional(rollbackFor = Exception.class)
public Long createTransferApplication(HrmTransferApplicationSaveReqVO createReqVO) {
- // 1. 鏍￠獙鐩爣閮ㄩ棬涓庡師閮ㄩ棬涓嶈兘鐩稿悓
+ // 1. 瑙ｆ瀽鐢宠浜� & 琛ュ叏鍘熼儴闂�/鍘熷矖浣�
+ // 鍓嶇琛ㄥ崟鍙鐢ㄦ埛閫夋嫨銆岃璋冨矖鍛樺伐銆嶏紝涓嶇洿鎺ユ彁浜� currentDeptId/currentPostId锛�
+ // 鑰岃繖涓や釜瀛楁鍦ㄥ簱閲屾槸 NOT NULL锛岀己鐪佷細鐩存帴鎶� SQL 寮傚父锛團ield 'current_dept_id' doesn't have a default value锛�
+ Long userId = createReqVO.getUserId();
+ Long employeeId = createReqVO.getEmployeeId();
+ HrmEmployeeDO employee = null;
+ if (employeeId != null) {
+ employee = employeeService.getEmployee(employeeId);
+ }
+ if (employee == null && userId != null) {
+ // 绾﹀畾 user_id 瀛� system_users.id锛涗絾鍘嗗彶涓婂墠绔篃鏈夋妸銆屽憳宸ヤ富閿�嶅杩� userId 鐨勬儏鍐碉紝涓ょ閮藉吋瀹�
+ employee = employeeService.getEmployeeByUserId(userId);
+ if (employee == null) {
+ employee = employeeService.getEmployee(userId);
+ }
+ }
+ // 浠ュ憳宸ユ。妗堜笂鐨� user_id 涓哄噯锛堝墠绔彲鑳芥妸鍛樺伐涓婚敭濉炲湪 userId 瀛楁閲岋紝鐩存帴鐢ㄤ細鍐欓敊鐢宠浜猴級
+ if (employee != null && employee.getUserId() != null) {
+ userId = employee.getUserId();
+ }
+ if (userId == null && employee != null) {
+ userId = employee.getUserId();
+ }
+ if (userId == null) {
+ userId = getCurrentUserId();
+ if (employee == null) {
+ employee = employeeService.getEmployeeByUserId(userId);
+ }
+ }
+ if (adminUserApi.getUser(userId) == null) {
+ userId = getCurrentUserId();
+ }
+ if (createReqVO.getCurrentDeptId() == null && employee != null) {
+ createReqVO.setCurrentDeptId(employee.getDeptId());
+ }
+ if (createReqVO.getCurrentPostId() == null && employee != null) {
+ createReqVO.setCurrentPostId(parseFirstPostId(employee.getPostIds()));
+ }
+ if (createReqVO.getCurrentDeptId() == null) {
+ throw exception(TRANSFER_APPLICATION_CURRENT_DEPT_REQUIRED);
+ }
+ if (createReqVO.getCurrentPostId() == null) {
+ throw exception(TRANSFER_APPLICATION_CURRENT_POST_REQUIRED);
+ }
+ // 2. 鏍￠獙鐩爣閮ㄩ棬涓庡師閮ㄩ棬涓嶈兘鐩稿悓锛堟斁鍦ㄨˉ鍏ㄤ箣鍚庯紝鍚﹀垯琛ュ叏鍑烘潵鐨勫師閮ㄩ棬缁曡繃浜嗘牎楠岋級
if (Objects.equals(createReqVO.getTargetDeptId(), createReqVO.getCurrentDeptId())) {
throw exception(TRANSFER_APPLICATION_SAME_DEPT);
}
- // 2. 鏍￠獙鐩爣宀椾綅涓庡師宀椾綅涓嶈兘鐩稿悓
+ // 3. 鏍￠獙鐩爣宀椾綅涓庡師宀椾綅涓嶈兘鐩稿悓
if (Objects.equals(createReqVO.getTargetPostId(), createReqVO.getCurrentPostId())) {
throw exception(TRANSFER_APPLICATION_SAME_POST);
}
- // 3. 鐢熸垚鐢宠鍗曞彿
+ // 4. 鐢熸垚鐢宠鍗曞彿
String no = generateUniqueNo();
- // 4. 璁剧疆鐢宠浜轰俊鎭�
- Long userId = createReqVO.getUserId();
- if (userId == null) {
- userId = getCurrentUserId();
- }
- AdminUserRespDTO user = adminUserApi.getUser(userId);
- if (user == null) {
- userId = getCurrentUserId();
- }
// 5. 鎻掑叆
HrmTransferApplicationDO application = BeanUtils.toBean(createReqVO, HrmTransferApplicationDO.class);
application.setNo(no);
application.setUserId(userId);
application.setStatus(HrmAuditStatusEnum.DRAFT.getStatus());
transferApplicationMapper.insert(application);
+ storageAttachmentApi.bindAttachments("file", "hrm_transfer_application",
+ application.getId(), createReqVO.getBlobIds());
return application.getId();
+ }
+
+ /**
+ * 鍙栧憳宸ュ矖浣嶅垪琛ㄤ腑鐨勭涓�涓綔涓恒�屽師宀椾綅銆�
+ *
+ * <p>hrm_employee.post_ids 浠� JSON 鏁扮粍瀛楃涓插瓨鍌紝渚嬪 {@code [20,21]}
+ *
+ * @param postIds 宀椾綅缂栧彿 JSON 瀛楃涓�
+ * @return 绗竴涓矖浣嶇紪鍙凤紱瑙ｆ瀽涓嶅嚭鏉ユ椂杩斿洖 {@code null}
+ */
+ private Long parseFirstPostId(String postIds) {
+ if (StrUtil.isBlank(postIds)) {
+ return null;
+ }
+ try {
+ List<Long> ids = JSONUtil.toList(postIds, Long.class);
+ return CollUtil.isEmpty(ids) ? null : ids.get(0);
+ } catch (Exception ex) {
+ log.warn("[parseFirstPostId][postIds({}) 瑙ｆ瀽澶辫触]", postIds, ex);
+ return null;
+ }
}
private Long getCurrentUserId() {
@@ -129,6 +196,7 @@
}
HrmTransferApplicationDO updateObj = BeanUtils.toBean(updateReqVO, HrmTransferApplicationDO.class);
transferApplicationMapper.updateById(updateObj);
+ storageAttachmentApi.updateAttachments("file", "hrm_transfer_application", updateReqVO.getId(), updateReqVO.getBlobIds());
}
@Override
@@ -137,6 +205,7 @@
if (!Objects.equals(application.getStatus(), HrmAuditStatusEnum.DRAFT.getStatus())) {
throw exception(TRANSFER_APPLICATION_UPDATE_FAIL_NOT_DRAFT);
}
+ storageAttachmentApi.deleteAttachmentsByRecord("hrm_transfer_application", id);
transferApplicationMapper.deleteById(id);
}
@@ -161,15 +230,21 @@
}
HrmTransferApplicationRespVO respVO = BeanUtils.toBean(application, HrmTransferApplicationRespVO.class);
if (application.getUserId() != null) {
- AdminUserRespDTO user = adminUserApi.getUser(application.getUserId());
- if (user != null) {
- respVO.setUserName(user.getNickname());
+ // user_id 姝ｅ父瀛� system_users.id锛涘巻鍙叉暟鎹篃鏈夊瓨銆屽憳宸ヤ富閿�嶇殑锛屼袱绉嶉兘鍏煎
+ HrmEmployeeDO employee = employeeService.getEmployeeByUserId(application.getUserId());
+ if (employee == null) {
+ employee = employeeService.getEmployee(application.getUserId());
+ }
+ if (employee != null) {
+ respVO.setUserName(employee.getName());
+ respVO.setEmployeeId(employee.getId());
}
}
fillDeptNames(respVO, application);
fillPostNames(respVO, application);
respVO.setTransferTypeName(HrmTransferTypeEnum.getNameByStatus(application.getTransferType()));
respVO.setStatusName(HrmAuditStatusEnum.getNameByStatus(application.getStatus()));
+ respVO.setAttachmentList(storageAttachmentApi.listAttachments("hrm_transfer_application", id));
return respVO;
}
@@ -207,7 +282,14 @@
.map(HrmTransferApplicationDO::getUserId)
.filter(Objects::nonNull)
.collect(Collectors.toSet());
- Map<Long, AdminUserRespDTO> userMap = adminUserApi.getUserMap(userIds);
+ // userId 瀛樼殑鏄憳宸D锛屾寜鍛樺伐琛ㄦ煡璇㈠鍚�
+ Map<Long, String> employeeNameMap = new HashMap<>();
+ for (Long userId : userIds) {
+ HrmEmployeeDO employee = employeeService.getEmployee(userId);
+ if (employee != null) {
+ employeeNameMap.put(userId, employee.getName());
+ }
+ }
Set<Long> deptIds = pageResult.getList().stream()
.flatMap(app -> Stream.of(app.getCurrentDeptId(), app.getTargetDeptId()))
.filter(Objects::nonNull)
@@ -220,8 +302,8 @@
Map<Long, PostRespDTO> postMap = postApi.getPostMap(postIds);
List<HrmTransferApplicationRespVO> list = pageResult.getList().stream().map(app -> {
HrmTransferApplicationRespVO vo = BeanUtils.toBean(app, HrmTransferApplicationRespVO.class);
- if (app.getUserId() != null && userMap.containsKey(app.getUserId())) {
- vo.setUserName(userMap.get(app.getUserId()).getNickname());
+ if (app.getUserId() != null && employeeNameMap.containsKey(app.getUserId())) {
+ vo.setUserName(employeeNameMap.get(app.getUserId()));
}
if (app.getCurrentDeptId() != null && deptMap.containsKey(app.getCurrentDeptId())) {
vo.setCurrentDeptName(deptMap.get(app.getCurrentDeptId()).getName());
--
Gitblit v1.9.3