From c2053d7017625f270577bc0f6c523d04f52db6ea Mon Sep 17 00:00:00 2001
From: 云 <2163098428@qq.com>
Date: 星期五, 24 七月 2026 14:20:29 +0800
Subject: [PATCH] refactor(crm): 重构文件上传功能，替换原有的fileUrls字段为统一的StorageAttachmentApi
---
yudao-module-crm/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/followup/CrmFollowUpRecordController.java | 58 +++++++++++++++++++++++++++++++++++++++++++++++++---------
1 files changed, 49 insertions(+), 9 deletions(-)
diff --git a/yudao-module-crm/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/followup/CrmFollowUpRecordController.java b/yudao-module-crm/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/followup/CrmFollowUpRecordController.java
index 7946aea..f09026a 100644
--- a/yudao-module-crm/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/followup/CrmFollowUpRecordController.java
+++ b/yudao-module-crm/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/followup/CrmFollowUpRecordController.java
@@ -11,9 +11,13 @@
import cn.iocoder.yudao.module.crm.dal.dataobject.business.CrmBusinessDO;
import cn.iocoder.yudao.module.crm.dal.dataobject.contact.CrmContactDO;
import cn.iocoder.yudao.module.crm.dal.dataobject.followup.CrmFollowUpRecordDO;
+import cn.iocoder.yudao.module.crm.enums.common.CrmBizTypeEnum;
+import cn.iocoder.yudao.module.crm.enums.permission.CrmPermissionLevelEnum;
import cn.iocoder.yudao.module.crm.service.business.CrmBusinessService;
import cn.iocoder.yudao.module.crm.service.contact.CrmContactService;
import cn.iocoder.yudao.module.crm.service.followup.CrmFollowUpRecordService;
+import cn.iocoder.yudao.module.crm.service.permission.CrmPermissionService;
+import cn.iocoder.yudao.module.system.api.storage.StorageAttachmentApi;
import cn.iocoder.yudao.module.system.api.user.AdminUserApi;
import cn.iocoder.yudao.module.system.api.user.dto.AdminUserRespDTO;
import io.swagger.v3.oas.annotations.Operation;
@@ -25,12 +29,17 @@
import org.springframework.web.bind.annotation.*;
import java.util.ArrayList;
+import java.util.List;
import java.util.Map;
+import java.util.stream.Stream;
+import static cn.iocoder.yudao.framework.common.exception.util.ServiceExceptionUtil.exception;
import static cn.iocoder.yudao.framework.common.pojo.CommonResult.success;
import static cn.iocoder.yudao.framework.common.util.collection.CollectionUtils.convertSet;
import static cn.iocoder.yudao.framework.common.util.collection.CollectionUtils.convertSetByFlatMap;
import static cn.iocoder.yudao.framework.security.core.util.SecurityFrameworkUtils.getLoginUserId;
+import static cn.iocoder.yudao.module.crm.enums.ErrorCodeConstants.CRM_PERMISSION_DENIED;
+import static cn.iocoder.yudao.module.crm.enums.ErrorCodeConstants.FOLLOW_UP_RECORD_NOT_EXISTS;
@Tag(name = "绠＄悊鍚庡彴 - 璺熻繘璁板綍")
@@ -45,9 +54,13 @@
private CrmContactService contactService;
@Resource
private CrmBusinessService businessService;
+ @Resource
+ private CrmPermissionService permissionService;
@Resource
private AdminUserApi adminUserApi;
+ @Resource
+ private StorageAttachmentApi storageAttachmentApi;
@PostMapping("/create")
@Operation(summary = "鍒涘缓璺熻繘璁板綍")
@@ -68,7 +81,16 @@
@Parameter(name = "id", description = "缂栧彿", required = true, example = "1024")
public CommonResult<CrmFollowUpRecordRespVO> getFollowUpRecord(@RequestParam("id") Long id) {
CrmFollowUpRecordDO followUpRecord = followUpRecordService.getFollowUpRecord(id);
- return success(BeanUtils.toBean(followUpRecord, CrmFollowUpRecordRespVO.class));
+ if (followUpRecord == null) {
+ throw exception(FOLLOW_UP_RECORD_NOT_EXISTS);
+ }
+ if (!permissionService.hasPermission(followUpRecord.getBizType(), followUpRecord.getBizId(),
+ getLoginUserId(), CrmPermissionLevelEnum.READ)) {
+ throw exception(CRM_PERMISSION_DENIED, CrmBizTypeEnum.getNameByType(followUpRecord.getBizType()));
+ }
+ CrmFollowUpRecordRespVO respVO = BeanUtils.toBean(followUpRecord, CrmFollowUpRecordRespVO.class);
+ respVO.setAttachmentList(storageAttachmentApi.listAttachments("crm_follow_up_record", id));
+ return success(respVO);
}
@GetMapping("/page")
@@ -77,22 +99,40 @@
PageResult<CrmFollowUpRecordDO> pageResult = followUpRecordService.getFollowUpRecordPage(pageReqVO);
// 1.1 鏌ヨ鑱旂郴浜哄拰鍟嗘満
Map<Long, CrmContactDO> contactMap = contactService.getContactMap(
- convertSetByFlatMap(pageResult.getList(), item -> item.getContactIds().stream()));
+ convertSetByFlatMap(pageResult.getList(), item -> {
+ List<Long> contactIds = item.getContactIds();
+ return contactIds != null ? contactIds.stream() : Stream.empty();
+ }));
Map<Long, CrmBusinessDO> businessMap = businessService.getBusinessMap(
- convertSetByFlatMap(pageResult.getList(), item -> item.getBusinessIds().stream()));
+ convertSetByFlatMap(pageResult.getList(), item -> {
+ List<Long> businessIds = item.getBusinessIds();
+ return businessIds != null ? businessIds.stream() : Stream.empty();
+ }));
// 1.2 鏌ヨ鐢ㄦ埛
Map<Long, AdminUserRespDTO> userMap = adminUserApi.getUserMap(
- convertSet(pageResult.getList(), item -> Long.valueOf(item.getCreator())));
+ convertSet(pageResult.getList(), item -> {
+ String creator = item.getCreator();
+ return creator != null && !creator.isEmpty() ? Long.valueOf(creator) : null;
+ }));
// 2. 鎷兼帴鏁版嵁
PageResult<CrmFollowUpRecordRespVO> voPageResult = BeanUtils.toBean(pageResult, CrmFollowUpRecordRespVO.class, record -> {
// 2.1 璁剧疆鑱旂郴浜哄拰鍟嗘満淇℃伅
record.setBusinesses(new ArrayList<>()).setContacts(new ArrayList<>());
- record.getContactIds().forEach(id -> MapUtils.findAndThen(contactMap, id, contact ->
- record.getContacts().add(new CrmBusinessRespVO().setId(contact.getId()).setName(contact.getName()))));
- record.getBusinessIds().forEach(id -> MapUtils.findAndThen(businessMap, id, business ->
- record.getBusinesses().add(new CrmBusinessRespVO().setId(business.getId()).setName(business.getName()))));
+ List<Long> contactIds = record.getContactIds();
+ if (contactIds != null) {
+ contactIds.forEach(id -> MapUtils.findAndThen(contactMap, id, contact ->
+ record.getContacts().add(new CrmBusinessRespVO().setId(contact.getId()).setName(contact.getName()))));
+ }
+ List<Long> businessIds = record.getBusinessIds();
+ if (businessIds != null) {
+ businessIds.forEach(id -> MapUtils.findAndThen(businessMap, id, business ->
+ record.getBusinesses().add(new CrmBusinessRespVO().setId(business.getId()).setName(business.getName()))));
+ }
// 2.2 璁剧疆鐢ㄦ埛淇℃伅
- MapUtils.findAndThen(userMap, Long.valueOf(record.getCreator()), user -> record.setCreatorName(user.getNickname()));
+ String creator = record.getCreator();
+ if (creator != null && !creator.isEmpty()) {
+ MapUtils.findAndThen(userMap, Long.valueOf(creator), user -> record.setCreatorName(user.getNickname()));
+ }
});
return success(voPageResult);
}
--
Gitblit v1.9.3