code review：数据权限的实现

zhijiantianya@gmail.com
1 parent 9f1b92ab
Showing 50 changed files with 80 additions and 66 deletions
yudao-module-crm/yudao-module-crm-api/src/main/java/cn/iocoder/yudao/module/crm/enums/ErrorCodeConstants.java
yudao-module-crm/yudao-module-crm-api/src/main/java/cn/iocoder/yudao/module/crm/enums/common/TransferTypeEnum.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contact/ContactController.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contact/vo/ContactCreateReqVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contact/vo/ContactExportReqVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contact/vo/ContactPageReqVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contact/vo/ContactRespVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contact/vo/ContactUpdateReqVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contact/vo/CrmTransferContactReqVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contract/ContractController.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contract/package-info.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contract/vo/ContractCreateReqVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contract/vo/ContractExcelVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contract/vo/ContractExportReqVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contract/vo/ContractPageReqVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contract/vo/ContractRespVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contract/vo/ContractUpdateReqVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contract/vo/CrmTransferContractReqVO.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/customer/CrmCustomerController.java
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/customer/vo/CrmCustomerCreateReqVO.java
@@ -34,6 +34,7 @@ public interface ErrorCodeConstants {
     // ========== 客户管理 1_020_006_000 ==========
     ErrorCode CUSTOMER_NOT_EXISTS = new ErrorCode(1_020_006_000, "客户不存在");
+    // TODO @puhui999：权限管理？？？
     // ========== 客户管理 1_020_007_000 ==========
     ErrorCode CRM_PERMISSION_NOT_EXISTS = new ErrorCode(1_020_007_000, "数据权限不存在");
     ErrorCode CRM_PERMISSION_DENIED = new ErrorCode(1_020_007_001, "{}操作失败，原因：没有权限");
@@ -6,6 +6,7 @@ import lombok.Getter;
 import java.util.Arrays;
+// TODO @puhui999：这个可以不用哈
 /**
  * Crm 负责人转移后原负责人的处理方式
  *
@@ -17,6 +18,7 @@ public enum TransferTypeEnum implements IntArrayValuable {
     REMOVE(1, "移除"),
     TEAM(2, "转为团队成员");
+
     public static final int[] ARRAYS = Arrays.stream(values()).mapToInt(TransferTypeEnum::getType).toArray();
     /**
@@ -26,7 +26,7 @@ import static cn.iocoder.yudao.framework.common.pojo.CommonResult.success;
 import static cn.iocoder.yudao.framework.operatelog.core.enums.OperateTypeEnum.EXPORT;
 import static cn.iocoder.yudao.framework.security.core.util.SecurityFrameworkUtils.getLoginUserId;
-@Tag(name = "管理后台 - crm联系人")
+@Tag(name = "管理后台 - CRM 联系人")
 @RestController
 @RequestMapping("/crm/contact")
 @Validated
@@ -5,7 +5,7 @@ import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.ToString;
-@Schema(description = "管理后台 - crm联系人创建 Request VO")
+@Schema(description = "管理后台 - CRM 联系人创建 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -9,7 +9,7 @@ import org.springframework.format.annotation.DateTimeFormat;
 import static cn.iocoder.yudao.framework.common.util.date.DateUtils.FORMAT_YEAR_MONTH_DAY_HOUR_MINUTE_SECOND;
-@Schema(description = "管理后台 - crm联系人 Excel 导出 Request VO，参数和 ContactPageReqVO 是一致的")
+@Schema(description = "管理后台 - CRM 联系人 Excel 导出 Request VO，参数和 ContactPageReqVO 是一致的")
 @Data
 public class ContactExportReqVO {
@@ -11,7 +11,7 @@ import java.time.LocalDateTime;
 import static cn.iocoder.yudao.framework.common.util.date.DateUtils.FORMAT_YEAR_MONTH_DAY_HOUR_MINUTE_SECOND;
-@Schema(description = "管理后台 - crm联系人分页 Request VO")
+@Schema(description = "管理后台 - CRM 联系人分页 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -4,7 +4,7 @@ import io.swagger.v3.oas.annotations.media.Schema;
 import lombok.*;
 import java.time.LocalDateTime;
-@Schema(description = "管理后台 - crm联系人 Response VO")
+@Schema(description = "管理后台 - CRM 联系人 Response VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -5,7 +5,7 @@ import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.ToString;
-@Schema(description = "管理后台 - crm联系人更新 Request VO")
+@Schema(description = "管理后台 - CRM 联系人更新 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -5,7 +5,7 @@ import lombok.Data;
 import javax.validation.constraints.NotNull;
-@Schema(description = "管理后台 - 联系人转移 Request VO")
+@Schema(description = "管理后台 - CRM 联系人转移 Request VO")
 @Data
 public class CrmTransferContactReqVO {
@@ -25,7 +25,7 @@ import static cn.iocoder.yudao.framework.common.pojo.CommonResult.success;
 import static cn.iocoder.yudao.framework.operatelog.core.enums.OperateTypeEnum.EXPORT;
 import static cn.iocoder.yudao.framework.security.core.util.SecurityFrameworkUtils.getLoginUserId;
-@Tag(name = "管理后台 - 合同")
+@Tag(name = "管理后台 - CRM 合同")
 @RestController
 @RequestMapping("/crm/contract")
 @Validated
-/**
- * 合同
- */
-package cn.iocoder.yudao.module.crm.controller.admin.contract;
@@ -5,7 +5,7 @@ import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.ToString;
-@Schema(description = "管理后台 - 合同创建 Request VO")
+@Schema(description = "管理后台 - CRM 合同创建 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -6,7 +6,7 @@ import lombok.Data;
 import java.time.LocalDateTime;
 /**
- * 合同 Excel VO
+ * CRM 合同 Excel VO
  *
  * @author dhb52
  */
@@ -8,7 +8,7 @@ import java.time.LocalDateTime;
 import static cn.iocoder.yudao.framework.common.util.date.DateUtils.FORMAT_YEAR_MONTH_DAY_HOUR_MINUTE_SECOND;
-@Schema(description = "管理后台 - 合同 Excel 导出 Request VO，参数和 ContractPageReqVO 是一致的")
+@Schema(description = "管理后台 - CRM 合同 Excel 导出 Request VO，参数和 ContractPageReqVO 是一致的")
 @Data
 public class ContractExportReqVO {
@@ -11,7 +11,7 @@ import java.time.LocalDateTime;
 import static cn.iocoder.yudao.framework.common.util.date.DateUtils.FORMAT_YEAR_MONTH_DAY_HOUR_MINUTE_SECOND;
-@Schema(description = "管理后台 - 合同分页 Request VO")
+@Schema(description = "管理后台 - CRM 合同分页 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -7,7 +7,7 @@ import lombok.ToString;
 import java.time.LocalDateTime;
-@Schema(description = "管理后台 - 合同 Response VO")
+@Schema(description = "管理后台 - CRM 合同 Response VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -7,7 +7,7 @@ import lombok.ToString;
 import javax.validation.constraints.NotNull;
-@Schema(description = "管理后台 - 合同更新 Request VO")
+@Schema(description = "管理后台 - CRM 合同更新 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -5,7 +5,8 @@ import lombok.Data;
 import javax.validation.constraints.NotNull;
-@Schema(description = "管理后台 - 合同转移 Request VO")
+// TODO @puhui999：CrmContractTransferReqVO，模块名字要放前面；看看还有没其它类似的
+@Schema(description = "管理后台 - CRM 合同转移 Request VO")
 @Data
 public class CrmTransferContractReqVO {
@@ -25,7 +25,7 @@ import static cn.iocoder.yudao.framework.common.pojo.CommonResult.success;
 import static cn.iocoder.yudao.framework.operatelog.core.enums.OperateTypeEnum.EXPORT;
 import static cn.iocoder.yudao.framework.security.core.util.SecurityFrameworkUtils.getLoginUserId;
-@Tag(name = "管理后台 - 客户")
+@Tag(name = "管理后台 - CRM 客户")
 @RestController
 @RequestMapping("/crm/customer")
 @Validated
@@ -5,7 +5,7 @@ import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.ToString;
-@Schema(description = "管理后台 - 客户创建 Request VO")
+@Schema(description = "管理后台 - CRM 客户创建 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -11,7 +11,7 @@ import java.time.LocalDateTime;
 // TODO 芋艿：导出最后做，等基本确认的差不多之后；
 /**
- * 客户 Excel VO
+ * CRM 客户 Excel VO
  *
  * @author Wanwan
  */
@@ -4,7 +4,7 @@ import io.swagger.v3.oas.annotations.media.Schema;
 import lombok.Data;
 // TODO 芋艿：导出最后做，等基本确认的差不多之后；
-@Schema(description = "管理后台 - 客户 Excel 导出 Request VO，参数和 CrmCustomerPageReqVO 是一致的")
+@Schema(description = "管理后台 - CRM 客户 Excel 导出 Request VO，参数和 CrmCustomerPageReqVO 是一致的")
 @Data
 public class CrmCustomerExportReqVO {
@@ -6,7 +6,7 @@ import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.ToString;
-@Schema(description = "管理后台 - 客户分页 Request VO")
+@Schema(description = "管理后台 - CRM 客户分页 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -10,7 +10,7 @@ import java.time.LocalDateTime;
 import static cn.iocoder.yudao.framework.common.util.date.DateUtils.FORMAT_YEAR_MONTH_DAY_HOUR_MINUTE_SECOND;
-@Schema(description = "管理后台 - 客户 Response VO")
+@Schema(description = "管理后台 - CRM 客户 Response VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -7,7 +7,7 @@ import lombok.ToString;
 import javax.validation.constraints.NotNull;
-@Schema(description = "管理后台 - 客户更新 Request VO")
+@Schema(description = "管理后台 - CRM 客户更新 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -5,7 +5,7 @@ import lombok.Data;
 import javax.validation.constraints.NotNull;
-@Schema(description = "管理后台 - 客户转移 Request VO")
+@Schema(description = "管理后台 - CRM 客户转移 Request VO")
 @Data
 public class CrmTransferCustomerReqVO {
@@ -24,10 +24,7 @@ import java.util.List;
 import static cn.iocoder.yudao.framework.common.pojo.CommonResult.success;
 import static cn.iocoder.yudao.framework.operatelog.core.enums.OperateTypeEnum.EXPORT;
-/**
- * @author 赤焰
- */
-@Tag(name = "管理后台 - 回款管理")
+@Tag(name = "管理后台 - CRM 回款管理")
 @RestController
 @RequestMapping("/crm/receivable")
 @Validated
@@ -24,10 +24,7 @@ import java.util.List;
 import static cn.iocoder.yudao.framework.common.pojo.CommonResult.success;
 import static cn.iocoder.yudao.framework.operatelog.core.enums.OperateTypeEnum.EXPORT;
-/**
- * @author 赤焰
- */
-@Tag(name = "管理后台 - 回款计划")
+@Tag(name = "管理后台 - CRM 回款计划")
 @RestController
 @RequestMapping("/crm/receivable-plan")
 @Validated
@@ -5,7 +5,7 @@ import java.util.*;
 import io.swagger.v3.oas.annotations.media.Schema;
 import javax.validation.constraints.*;
-@Schema(description = "管理后台 - 回款管理创建 Request VO")
+@Schema(description = "管理后台 - CRM 回款创建 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -14,7 +14,7 @@ import cn.iocoder.yudao.framework.excel.core.convert.DictConvert;
 /**
- * 回款管理 Excel VO
+ * CRM 回款管理 Excel VO
  *
  * @author 赤焰
  */
@@ -12,7 +12,7 @@ import static cn.iocoder.yudao.framework.common.util.date.DateUtils.FORMAT_YEAR_
 /**
  * @author 赤焰
  */
-@Schema(description = "管理后台 - 回款管理 Excel 导出 Request VO，参数和 ReceivablePageReqVO 是一致的")
+@Schema(description = "管理后台 - CRM 回款 Excel 导出 Request VO，参数和 ReceivablePageReqVO 是一致的")
 @Data
 public class ReceivableExportReqVO {
@@ -12,7 +12,7 @@ import java.time.LocalDateTime;
 import static cn.iocoder.yudao.framework.common.util.date.DateUtils.FORMAT_YEAR_MONTH_DAY_HOUR_MINUTE_SECOND;
-@Schema(description = "管理后台 - 回款管理分页 Request VO")
+@Schema(description = "管理后台 - CRM 回款分页 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -5,7 +5,7 @@ import java.util.*;
 import io.swagger.v3.oas.annotations.media.Schema;
 import javax.validation.constraints.*;
-@Schema(description = "管理后台 - 回款计划创建 Request VO")
+@Schema(description = "管理后台 - CRM 回款计划创建 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
 package cn.iocoder.yudao.module.crm.controller.admin.receivable.vo;
-import io.swagger.v3.oas.annotations.media.Schema;
 import lombok.*;
-import java.util.*;
+
 import java.math.BigDecimal;
 import java.time.LocalDateTime;
-import java.time.LocalDateTime;
-import java.time.LocalDateTime;
-import java.time.LocalDateTime;
 import com.alibaba.excel.annotation.ExcelProperty;
 import cn.iocoder.yudao.framework.excel.core.annotations.DictFormat;
@@ -15,7 +11,7 @@ import cn.iocoder.yudao.framework.excel.core.convert.DictConvert;
 /**
- * 回款计划 Excel VO
+ * CRM 回款计划 Excel VO
  *
  * @author 芋道源码
  */
@@ -9,7 +9,7 @@ import org.springframework.format.annotation.DateTimeFormat;
 import static cn.iocoder.yudao.framework.common.util.date.DateUtils.FORMAT_YEAR_MONTH_DAY_HOUR_MINUTE_SECOND;
-@Schema(description = "管理后台 - 回款计划 Excel 导出 Request VO，参数和 ReceivablePlanPageReqVO 是一致的")
+@Schema(description = "管理后台 - CRM 回款计划 Excel 导出 Request VO，参数和 ReceivablePlanPageReqVO 是一致的")
 @Data
 public class ReceivablePlanExportReqVO {
@@ -11,7 +11,7 @@ import java.time.LocalDateTime;
 import static cn.iocoder.yudao.framework.common.util.date.DateUtils.FORMAT_YEAR_MONTH_DAY_HOUR_MINUTE_SECOND;
-@Schema(description = "管理后台 - 回款计划分页 Request VO")
+@Schema(description = "管理后台 - CRM 回款计划分页 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -4,7 +4,7 @@ import io.swagger.v3.oas.annotations.media.Schema;
 import lombok.*;
 import java.time.LocalDateTime;
-@Schema(description = "管理后台 - 回款计划 Response VO")
+@Schema(description = "管理后台 - CRM 回款计划 Response VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -5,7 +5,7 @@ import lombok.*;
 import java.util.*;
 import javax.validation.constraints.*;
-@Schema(description = "管理后台 - 回款计划更新 Request VO")
+@Schema(description = "管理后台 - CRM 回款计划更新 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -4,7 +4,7 @@ import io.swagger.v3.oas.annotations.media.Schema;
 import lombok.*;
 import java.time.LocalDateTime;
-@Schema(description = "管理后台 - 回款管理 Response VO")
+@Schema(description = "管理后台 - CRM 回款 Response VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -5,7 +5,7 @@ import lombok.*;
 import java.util.*;
 import javax.validation.constraints.*;
-@Schema(description = "管理后台 - 回款管理更新 Request VO")
+@Schema(description = "管理后台 - CRM 回款更新 Request VO")
 @Data
 @EqualsAndHashCode(callSuper = true)
 @ToString(callSuper = true)
@@ -12,7 +12,7 @@ import org.mapstruct.factory.Mappers;
 import java.util.List;
 /**
- * crm联系人 Convert
+ * crm 联系人 Convert
  *
  * @author 芋道源码
  */
@@ -47,19 +47,21 @@ public class CrmPermissionAspect {
         try {
             Integer crmType = crmPermission.crmType().getType();
             Integer operationType = crmPermission.operationType().getType();
+            // TODO @puhui999：不一定是 id 参数噢；例如说，ContactServiceImpl 的 updateContact
             Long id = (Long) joinPoint.getArgs()[0];// 获取操作数据的编号
-            // 1. 获取数据权限
+            // 1.1 获取数据权限
             CrmPermissionDO permission = crmPermissionService.getCrmPermissionByCrmTypeAndCrmDataId(crmType, id);
             if (permission == null) {
                 // 不存在说明数据也不存在
                 throw exception(CRM_PERMISSION_MODEL_NOT_EXISTS, crmPermission.crmType().getName());
             }
-            // 1.2. 校验是否为公海数据
+            // 1.2 校验是否为公海数据
+            // TODO @puhui999：这个判断去掉比较合适哈。这里更多是业务逻辑，不算权限判断。例如说，公海的客户，只要没负责人，就可以领取了；
             if (permission.getOwnerUserId() == null) {
                 return;
             }
-            // 1.3. 校验当前负责人是不是自己
+            // 1.3 校验当前负责人是不是自己
             if (ObjUtil.equal(permission.getOwnerUserId(), getUserId())) {
                 return;
             }
@@ -68,6 +70,7 @@ public class CrmPermissionAspect {
             // 2. 校验是否有读权限
             if (OperationTypeEnum.isRead(operationType)) {
                 // 校验该数据当前用户是否可读
+                // TODO @puhui999：直接 CollUtil.contains 就好，因为就是有某个 userId 呀
                 boolean isRead = CollUtil.contains(permission.getRoUserIds(), item -> ObjUtil.equal(item, getUserId()))
                         || CollUtil.contains(permission.getRwUserIds(), item -> ObjUtil.equal(item, getUserId()));
                 if (isRead) {
@@ -4,6 +4,7 @@ import cn.hutool.core.util.ObjUtil;
 import lombok.Getter;
 import lombok.RequiredArgsConstructor;
+// TODO @puhui999：可以改成 CrmBizTypeEnum，CRM 业务类型枚举
 /**
  * Crm 类型枚举
  *
@@ -4,6 +4,7 @@ import cn.hutool.core.util.ObjUtil;
 import lombok.Getter;
 import lombok.RequiredArgsConstructor;
+// TODO @puhui999：是不是可以和 PermissionTypeEnum 合并，就是 CrmPermissionEnum，负责人、读取、读写；目前阶段，不用做的特别细致；类似 linux 的 acl；
 /**
  * Crm 数据操作类型枚举
  *
@@ -13,6 +14,7 @@ import lombok.RequiredArgsConstructor;
 @Getter
 public enum OperationTypeEnum {
+    // TODO @puhui999：抽象上，就分三种，会更合理。一个 OWNER 负责人，一个 READ 读，一个 WRITE 写；
     DELETE(1, "删除"),
     UPDATE(2, "修改"),
     READ(3, "查询");
@@ -115,8 +115,8 @@ public class CrmCustomerServiceImpl implements CrmCustomerService {
     /**
      * 校验客户是否存在
      *
-     * @param customerId 客户id
-     * @return
+     * @param customerId 客户 id
+     * @return 客户
      */
     @Override
     public CrmCustomerDO validateCustomer(Long customerId) {
@@ -130,7 +130,7 @@ public class CrmCustomerServiceImpl implements CrmCustomerService {
     @Override
     @Transactional(rollbackFor = Exception.class)
     public void transferCustomer(CrmTransferCustomerReqVO reqVO, Long userId) {
-        // 1 校验合同是否存在
+        // 1. 校验合同是否存在
         validateCustomer(reqVO.getId());
         // 2. 数据权限转移
@@ -16,6 +16,7 @@ import javax.validation.Valid;
  */
 public interface CrmPermissionService {
+    // TODO @puhui999：方法名上，不用 Crm
     /**
      * 创建数据权限
      *
@@ -41,6 +41,7 @@ public class CrmPermissionServiceImpl implements CrmPermissionService {
     @Override
     @Transactional(rollbackFor = Exception.class)
     public Long createCrmPermission(CrmPermissionCreateBO createBO) {
+        // TODO @puhui999：createDO 改成 permission，保持统一哈；
         CrmPermissionDO createDO = CrmPermissionConvert.INSTANCE.convert(createBO);
         crmPermissionMapper.insert(createDO);
         return createDO.getId();
@@ -50,7 +51,7 @@ public class CrmPermissionServiceImpl implements CrmPermissionService {
     @Transactional(rollbackFor = Exception.class)
     public void updateCrmPermission(CrmPermissionUpdateBO updateBO) {
         validateCrmPermissionExists(updateBO.getId());
-
+        // 更新操作
         CrmPermissionDO updateDO = CrmPermissionConvert.INSTANCE.convert(updateBO);
         crmPermissionMapper.updateById(updateDO);
     }
@@ -59,7 +60,7 @@ public class CrmPermissionServiceImpl implements CrmPermissionService {
     @Transactional(rollbackFor = Exception.class)
     public void deleteCrmPermission(Long id) {
         validateCrmPermissionExists(id);
-
+        // 删除
         crmPermissionMapper.deleteById(id);
     }
@@ -74,9 +75,11 @@ public class CrmPermissionServiceImpl implements CrmPermissionService {
         return crmPermissionMapper.selectByCrmTypeAndCrmDataId(crmType, crmDataId);
     }
+    // TODO @puhui999：参数名，是不是 transferReqBO
     @Override
     public void transferCrmPermission(TransferCrmPermissionBO transferCrmPermissionBO) {
-        // 1 校验商机是否存在
+        // 1.1 校验商机是否存在
+        // TODO puhui999：这里直接调用 crmPermissionMapper 的 selectByCrmTypeAndCrmDataId 方法，会更简洁一点；
         CrmPermissionDO permission = getCrmPermissionByCrmTypeAndCrmDataId(transferCrmPermissionBO.getCrmType(),
                 transferCrmPermissionBO.getCrmDataId());
         String crmName = CrmEnum.getNameByType(transferCrmPermissionBO.getCrmType());
@@ -85,6 +88,7 @@ public class CrmPermissionServiceImpl implements CrmPermissionService {
         }
         // 1.2 校验转移对象是否已经是该负责人
         if (ObjUtil.equal(permission.getOwnerUserId(), permission.getOwnerUserId())) {
+            // TODO @puhui999：是不是这个错误码不太对。。。
             throw exception(CRM_PERMISSION_MODEL_TRANSFER_FAIL_OWNER_USER_NOT_EXISTS, crmName);
         }
         // 1.3 校验新负责人是否存在
@@ -94,11 +98,12 @@ public class CrmPermissionServiceImpl implements CrmPermissionService {
         }
         // TODO 校验是否为超级管理员 || 1.4
         // 1.4 校验是否有写权限
+        // TODO puhui999：CollUtil.contains 就够了，不用后面写个表达式；
         if (!CollUtil.contains(permission.getRwUserIds(), id -> ObjUtil.equal(id, transferCrmPermissionBO.getUserId()))) {
             throw exception(CRM_PERMISSION_DENIED, crmName);
         }
-        // 2 权限转移
+        // 2. 权限转移
         CrmPermissionDO updateCrmPermission = new CrmPermissionDO().setId(permission.getId())
                 .setOwnerUserId(transferCrmPermissionBO.getOwnerUserId());
         if (ObjUtil.equal(TransferTypeEnum.TEAM.getType(), transferCrmPermissionBO.getTransferType())) {
@@ -116,6 +121,7 @@ public class CrmPermissionServiceImpl implements CrmPermissionService {
         crmPermissionMapper.updateById(updateCrmPermission);
         // 3. TODO 记录机转移日志
+        // TODO @puhui999：是不是交给业务记录哈；
     }
 }
@@ -6,6 +6,7 @@ import lombok.Data;
 import javax.validation.constraints.NotNull;
 import java.util.Set;
+// TODO @puhui999：，一个是 Crm 前缀，一个 Req 表示入参
 /**
  * crm 数据权限 Create BO
  *
@@ -14,6 +15,8 @@ import java.util.Set;
 @Data
 public class CrmPermissionCreateBO {
+    // TODO @puhui999：如果是关联字段，换一行写它的注释；不然看着略乱哈
+
     /**
      * Crm 类型 关联 {@link CrmEnum}
      */
@@ -24,6 +27,7 @@ public class CrmPermissionCreateBO {
      */
     @NotNull(message = "Crm 数据编号不能为空")
     private Long crmDataId;
+
     /**
      * 负责人的用户编号 关联 AdminUser#id, null 则为公海数据
      */
@@ -7,6 +7,7 @@ import lombok.Data;
 import javax.validation.constraints.NotNull;
 import java.util.Set;
+// TODO @puhui999：，一个是 Crm 前缀，一个 Req 表示入参
 /**
  * crm 数据权限 Update BO
  *
@@ -15,6 +16,7 @@ import java.util.Set;
 @Data
 public class CrmPermissionUpdateBO {
+    // TODO @puhui999：id 和 crmType + crmDataId 是不是重叠了；
     /**
      * 数据权限编号 {@link CrmPermissionDO#getId()}
      */
@@ -7,6 +7,7 @@ import lombok.Data;
 import javax.validation.constraints.NotNull;
+// TODO @puhui999：CrmTransferPermissionReqBO，一个是 Crm 前缀，一个 Req 表示入参
 /**
  * 数据权限转移 BO
  *
@@ -15,26 +16,30 @@ import javax.validation.constraints.NotNull;
 @Data
 public class TransferCrmPermissionBO {
+    // TODO @puhui999：参数的注释
     @NotNull(message = "用户编号不能为空")
     private Long userId;
+    // TODO @puhui999：bizType
     /**
-     * Crm 类型 关联 {@link CrmEnum}
+     * Crm 类型 关联 {@link CrmEnum} TODO 这种不用再写关联了，直接 @InEnum 参数校验
      */
     @NotNull(message = "Crm 类型不能为空")
     private Integer crmType;
+    // TODO @puhui999：bizId
     /**
-     * 数据编号 关联 {@link CrmEnum} 对应模块 DO#getId()
+     * 数据编号
      */
     @NotNull(message = "Crm 数据编号不能为空")
     private Long crmDataId;
+    // TODO @puhui999：要不这里改成 newOwnerUserId;然后，transferType 和 permissionType，合并成 oldOwnerPermission（空就是移除）
     @NotNull(message = "新负责人的用户编号不能为空")
     private Long ownerUserId;
     /**
-     * 原负责人移除方式, 关联 {@link TransferTypeEnum}
+     * 原负责人移除方式, 关联 {@link TransferTypeEnum} TODO 这种不用再写关联了，直接 @InEnum 参数校验
      */
     @NotNull(message = "原负责人移除方式不能为空")
     private Integer transferType;