Commit b133acd8cdd741ded5bce27225ba12fefc3f2da2

Authored by YunaiV
1 parent 07f3d09e

code review:crm 合同流程

yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contract/ContractController.java
... ... @@ -19,7 +19,6 @@ import javax.annotation.Resource;
19 19 import javax.servlet.http.HttpServletResponse;
20 20 import javax.validation.Valid;
21 21 import java.io.IOException;
22   -import java.util.Collection;
23 22 import java.util.List;
24 23  
25 24 import static cn.iocoder.yudao.framework.common.pojo.CommonResult.success;
... ... @@ -67,15 +66,6 @@ public class ContractController {
67 66 return success(ContractConvert.INSTANCE.convert(contract));
68 67 }
69 68  
70   - @GetMapping("/list")
71   - @Operation(summary = "获得合同列表")
72   - @Parameter(name = "ids", description = "编号列表", required = true, example = "1024,2048")
73   - @PreAuthorize("@ss.hasPermission('crm:contract:query')")
74   - public CommonResult<List<ContractRespVO>> getContractList(@RequestParam("ids") Collection<Long> ids) {
75   - List<ContractDO> list = contractService.getContractList(ids);
76   - return success(ContractConvert.INSTANCE.convertList(list));
77   - }
78   -
79 69 @GetMapping("/page")
80 70 @Operation(summary = "获得合同分页")
81 71 @PreAuthorize("@ss.hasPermission('crm:contract:query')")
... ...
yudao-module-crm/yudao-module-crm-biz/src/main/java/cn/iocoder/yudao/module/crm/controller/admin/contract/vo/ContractBaseVO.java
... ... @@ -9,6 +9,7 @@ import java.time.LocalDateTime;
9 9  
10 10 import static cn.iocoder.yudao.framework.common.util.date.DateUtils.FORMAT_YEAR_MONTH_DAY_HOUR_MINUTE_SECOND;
11 11  
  12 +// TODO @dhb52:所有类,带下 Crm 前缀,避免和别的模块重复
12 13 /**
13 14 * 合同 Base VO,提供给添加、修改、详细的子 VO 使用
14 15 * 如果子 VO 存在差异的字段,请不要添加到这里,影响 Swagger 文档生成
... ... @@ -16,10 +17,13 @@ import static cn.iocoder.yudao.framework.common.util.date.DateUtils.FORMAT_YEAR_
16 17 @Data
17 18 public class ContractBaseVO {
18 19  
  20 + // TODO @dhb52:类似 no 字段的 example 要写xia 哈;
  21 +
19 22 @Schema(description = "合同名称", requiredMode = Schema.RequiredMode.REQUIRED, example = "王五")
20 23 @NotNull(message = "合同名称不能为空")
21 24 private String name;
22 25  
  26 + // TODO @dhb52:这个必须传递
23 27 @Schema(description = "客户编号", example = "18336")
24 28 private Long customerId;
25 29  
... ... @@ -29,13 +33,17 @@ public class ContractBaseVO {
29 33 @Schema(description = "工作流编号", example = "1043")
30 34 private Long processInstanceId;
31 35  
  36 + // TODO @dhb52:这个必须传递
32 37 @Schema(description = "下单日期")
33 38 @DateTimeFormat(pattern = FORMAT_YEAR_MONTH_DAY_HOUR_MINUTE_SECOND)
34 39 private LocalDateTime orderDate;
35 40  
  41 + // TODO @dhb52:这个必须传递
36 42 @Schema(description = "负责人的用户编号", example = "17144")
37 43 private Long ownerUserId;
38 44  
  45 + // TODO @芋艿:未来应该支持自动生成;
  46 + // TODO @dhb52:这个必须传递;
39 47 @Schema(description = "合同编号")
40 48 private String no;
41 49  
... ... @@ -69,4 +77,6 @@ public class ContractBaseVO {
69 77 @Schema(description = "备注", example = "你猜")
70 78 private String remark;
71 79  
  80 + // TODO @dhb52:增加一个 status 字段:具体有哪些值,你来枚举下;主要页面上有个【草稿】【提交审核】的流程,可以看看。然后要对接工作流,这块也可以看看,不确定的地方问我。
  81 +
72 82 }
... ...