From 62f821abf201b7130a10133a7e5820ecbbad3e98 Mon Sep 17 00:00:00 2001 From: wangfy Date: Thu, 28 Sep 2023 23:15:00 +0800 Subject: [PATCH 1/9] =?UTF-8?q?wangfangyuan,fix:=E4=BF=AE=E6=94=B9springdo?= =?UTF-8?q?c=E7=89=88=E6=9C=AC=EF=BC=8C=E8=A7=A3=E5=86=B3Swagger=E6=96=87?= =?UTF-8?q?=E6=A1=A3=E9=83=BD=E6=98=AFform=E6=8F=90=E4=BA=A4=E3=80=82?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- yudao-dependencies/pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/yudao-dependencies/pom.xml b/yudao-dependencies/pom.xml index 5a41bdba3..2e589fbac 100644 --- a/yudao-dependencies/pom.xml +++ b/yudao-dependencies/pom.xml @@ -19,7 +19,7 @@ 2.7.16 - 1.7.0 + 1.6.15 4.3.0 2.5 From 7d497ce09b2d9e42447ceac5c2c020cac4c894e0 Mon Sep 17 00:00:00 2001 From: kehaiyou Date: Fri, 29 Sep 2023 14:57:28 +0800 Subject: [PATCH 2/9] =?UTF-8?q?feat:=20=E3=80=90=E5=B7=A5=E4=BD=9C?= =?UTF-8?q?=E6=B5=81=E3=80=91--=E5=8A=A0=E7=AD=BE?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../common/enums/SymbolConstant.java | 48 +++ .../module/bpm/enums/ErrorCodeConstants.java | 4 +- .../bpm/enums/task/BpmCommentTypeEnum.java | 33 ++ .../task/BpmProcessInstanceResultEnum.java | 25 +- .../enums/task/BpmTaskAddSignTypeEnum.java | 40 +++ .../admin/task/BpmTaskController.java | 7 + .../task/vo/task/BpmTaskAddSignReqVO.java | 29 ++ .../bpm/convert/task/BpmTaskConvert.java | 17 + .../bpm/dal/mysql/task/BpmTaskExtMapper.java | 3 + .../bpm/service/task/BpmTaskService.java | 16 + .../bpm/service/task/BpmTaskServiceImpl.java | 311 ++++++++++++++++-- 11 files changed, 508 insertions(+), 25 deletions(-) create mode 100644 yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java create mode 100644 yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java create mode 100644 yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmTaskAddSignTypeEnum.java create mode 100644 yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java diff --git a/yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java b/yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java new file mode 100644 index 000000000..a73ff7d30 --- /dev/null +++ b/yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java @@ -0,0 +1,48 @@ +package cn.iocoder.yudao.framework.common.enums; + +/** + * 符号常量 + */ +public interface SymbolConstant { + + String D =","; + + /** + * _ + */ + String XH="_"; + /** + * - + */ + String HG="-"; + + /** + * / + */ + String XG="/"; + + /** + * 箭头 + */ + String ARROWHEAD="->"; + + /** + * 数组的开始元素 + */ + String ARRAY_START="["; + + /** + * 数组的结束元素 + */ + String ARRAY_END="]"; + + /** + * null 字符串 + */ + String NULL_STRING="null"; + + /** + * 点号 + */ + String DIAN="\\."; +} diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java index b84781ed6..55a2330c2 100644 --- a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java @@ -50,7 +50,9 @@ public interface ErrorCodeConstants { ErrorCode TASK_RETURN_FAIL_SOURCE_TARGET_ERROR = new ErrorCode(1_009_005_006, "回退任务失败，目标节点是在并行网关上或非同一路线上，不可跳转"); ErrorCode TASK_DELEGATE_FAIL_USER_REPEAT = new ErrorCode(1_009_005_007, "任务委派失败，委派人和当前审批人为同一人"); ErrorCode TASK_DELEGATE_FAIL_USER_NOT_EXISTS = new ErrorCode(1_009_005_008, "任务委派失败，被委派人不存在"); - + ErrorCode TASK_ADD_SIGN_USER_NOT_EXIST = new ErrorCode(1_009_005_009, "任务加签：选择的用户不存在"); + ErrorCode TASK_ADD_SIGN_TYPE_ERROR = new ErrorCode(1_009_005_010, "任务加签：当前任务已经{}，不能{}"); + ErrorCode TASK_ADD_SIGN_USER_REPEAT = new ErrorCode(1_009_005_011, "任务加签失败，加签人与现有审批人[{}]重复"); // ========== 流程任务分配规则 1-009-006-000 ========== ErrorCode TASK_ASSIGN_RULE_EXISTS = new ErrorCode(1_009_006_000, "流程({}) 的任务({}) 已经存在分配规则"); ErrorCode TASK_ASSIGN_RULE_NOT_EXISTS = new ErrorCode(1_009_006_001, "流程任务分配规则不存在"); diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java new file mode 100644 index 000000000..98e175e3f --- /dev/null +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java @@ -0,0 +1,33 @@ +package cn.iocoder.yudao.module.bpm.enums.task; + +import lombok.AllArgsConstructor; +import lombok.Getter; + +/** + * 流程任务 -- comment类型枚举 + */ +@Getter +@AllArgsConstructor +public enum BpmCommentTypeEnum { + + APPROVE(1, "通过"), + REJECT(2, "不通过"), + CANCEL(3, "已取消"), + + // ========== 流程任务独有的状态 ========== + + BACK(4, "退回"), // 退回 + DELEGATE(5, "委派"), + ADD_SIGN(6, "加签"), + SUB_SIGN(7,"减签"), + ; + + /** + * 结果 + */ + private final Integer result; + /** + * 描述 + */ + private final String desc; +} diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java index a19c68ba8..62b18f54a 100644 --- a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java @@ -21,11 +21,26 @@ public enum BpmProcessInstanceResultEnum { // ========== 流程任务独有的状态 ========== BACK(5, "驳回"), // 退回 - DELEGATE(6, "委派"); + DELEGATE(6, "委派"), + /** + * 源任务已经审批完成，但是它使用了后加签，后加签的任务未完成，源任务就会是这个状态 + * 例如：A审批， A 后加签了 B ，并且审批通过了任务，但是 B 还未审批，则当前任务状态为 待后加签任务完成 + */ + ADD_SIGN_AFTER(7, "待后加签任务完成"), + /** + * 源任务未审批，但是向前加签了，所以源任务状态变为 待前加签任务完成 + * 例如：A审批， A 前加签了 B ，B 还未审核 + */ + ADD_SIGN_BEFORE(8, "待前加签任务完成"), + /** + * 后加签任务被创建时的初始状态 + * 因为需要源任务先完成，才能到后加签的人来审批，所以加了一个状态区分 + */ + WAIT_BEFORE_TASK(9, "待前置任务完成"); /** * 结果 - * + *

* 如果新增时，注意 {@link #isEndResult(Integer)} 是否需要变更 */ private final Integer result; @@ -36,14 +51,16 @@ public enum BpmProcessInstanceResultEnum { /** * 判断该结果是否已经处于 End 最终结果 - * + *

* 主要用于一些结果更新的逻辑，如果已经是最终结果，就不再进行更新 * * @param result 结果 * @return 是否 */ public static boolean isEndResult(Integer result) { - return ObjectUtils.equalsAny(result, APPROVE.getResult(), REJECT.getResult(), CANCEL.getResult(), BACK.getResult()); + return ObjectUtils.equalsAny(result, APPROVE.getResult(), REJECT.getResult(), + CANCEL.getResult(), BACK.getResult(), + ADD_SIGN_AFTER.getResult()); } } diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmTaskAddSignTypeEnum.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmTaskAddSignTypeEnum.java new file mode 100644 index 000000000..3b46b6f25 --- /dev/null +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmTaskAddSignTypeEnum.java @@ -0,0 +1,40 @@ +package cn.iocoder.yudao.module.bpm.enums.task; + +import lombok.AllArgsConstructor; +import lombok.Getter; + + +/** + * 流程任务 -- 加签类型枚举类型 + */ +@Getter +@AllArgsConstructor +public enum BpmTaskAddSignTypeEnum { + + /** + * 向前加签，需要前置任务审批完成，才回到原审批人 + */ + BEFORE("before", "向前加签"), + /** + * 向后加签，需要后置任务全部审批完，才会通过原审批人节点 + */ + AFTER("after", "向后加签"), + /** + * 创建后置加签时的过度状态，用于控制向后加签生成的任务状态 + */ + AFTER_CHILDREN_TASK("afterChildrenTask", "向后加签生成的子任务"); + + private final String type; + + private final String desc; + + public static String formatDesc(String type) { + for (BpmTaskAddSignTypeEnum value : values()) { + if (value.type.equals(type)) { + return value.desc; + } + } + return null; + } +} + \ No newline at end of file diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java index e980fddcf..e442d7aad 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java @@ -100,4 +100,11 @@ public class BpmTaskController { return success(true); } + @PutMapping("/add-sign") + @Operation(summary = "加签", description = "before, after为前加签后加签") + @PreAuthorize("@ss.hasPermission('bpm:task:add-sign')") + public CommonResult addSign(@RequestBody @Valid BpmTaskAddSignReqVO reqVO) { + taskService.addSign(reqVO,getLoginUserId()); + return success(true); + } } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java new file mode 100644 index 000000000..88b123ce8 --- /dev/null +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java @@ -0,0 +1,29 @@ +package cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task; + +import io.swagger.v3.oas.annotations.media.Schema; +import lombok.Data; + +import javax.validation.constraints.NotEmpty; +import java.util.List; +import java.util.Set; + +@Schema(description = "管理后台 - 加签流程任务的 Request VO") +@Data +public class BpmTaskAddSignReqVO { + + @Schema(description = "加签的用户 ID") + @NotEmpty(message = "加签用户ID不能为空") + private Set userIdList; + + @Schema(description = "加签类型，before 向前加签，after 向后加签") + @NotEmpty(message = "加签类型不能为空") + private String type; + + @Schema(description = "加签原因") + @NotEmpty(message = "加签原因不能为空") + private String reason; + + @Schema(description = "需要加签的任务 ID") + @NotEmpty(message = "任务编号不能为空") + private String id; +} \ No newline at end of file diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java index ce693e468..ebbcf11fc 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java @@ -18,9 +18,11 @@ import org.flowable.engine.history.HistoricProcessInstance; import org.flowable.engine.runtime.ProcessInstance; import org.flowable.task.api.Task; import org.flowable.task.api.history.HistoricTaskInstance; +import org.flowable.task.service.impl.persistence.entity.TaskEntityImpl; import org.mapstruct.*; import org.mapstruct.factory.Mappers; +import java.util.Date; import java.util.List; import java.util.Map; @@ -145,4 +147,19 @@ public interface BpmTaskConvert { .setName(element.getName()) .setDefinitionKey(element.getId())); } + + default TaskEntityImpl convert(TaskEntityImpl task,TaskEntityImpl parentTask){ + task.setCategory(parentTask.getCategory()); + task.setDescription(parentTask.getDescription()); + task.setTenantId(parentTask.getTenantId()); + task.setName(parentTask.getName()); + task.setParentTaskId(parentTask.getId()); + task.setProcessDefinitionId(parentTask.getProcessDefinitionId()); + task.setProcessInstanceId(parentTask.getProcessInstanceId()); + task.setTaskDefinitionKey(parentTask.getTaskDefinitionKey()); + task.setTaskDefinitionId(parentTask.getTaskDefinitionId()); + task.setPriority(parentTask.getPriority()); + task.setCreateTime(new Date()); + return task; + } } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java index 6683b536d..7d4d18d8f 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java @@ -23,4 +23,7 @@ public interface BpmTaskExtMapper extends BaseMapperX { return selectOne(BpmTaskExtDO::getTaskId, taskId); } + default void updateBatchByTaskIdList(List taskIdList, BpmTaskExtDO entity){ + update(entity,new LambdaQueryWrapper().in(BpmTaskExtDO::getTaskId, taskIdList)); + } } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java index 99699d791..2eb222b7c 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java @@ -3,6 +3,7 @@ package cn.iocoder.yudao.module.bpm.service.task; import cn.iocoder.yudao.framework.common.pojo.PageResult; import cn.iocoder.yudao.framework.common.util.collection.CollectionUtils; import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task.*; +import cn.iocoder.yudao.module.bpm.dal.dataobject.task.BpmTaskExtDO; import org.flowable.task.api.Task; import javax.validation.Valid; @@ -62,6 +63,14 @@ public interface BpmTaskService { */ List getTaskListByProcessInstanceId(String processInstanceId); + + /** + * 通过任务 ID 集合，获取任务扩展表信息集合 + * @param taskIdList 任务 ID + * @return 任务列表 + */ + List getTaskListByTaskIdList(List taskIdList); + /** * 通过任务 * @@ -147,4 +156,11 @@ public interface BpmTaskService { */ void delegateTask(BpmTaskDelegateReqVO reqVO, Long userId); + /** + * 任务加签 + * @param reqVO 被加签的用户和任务ID，加签类型 + * @param userId 当前用户 ID + */ + void addSign(BpmTaskAddSignReqVO reqVO, Long userId); + } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java index 86f1ea4ff..6e57fcff4 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java @@ -2,7 +2,9 @@ package cn.iocoder.yudao.module.bpm.service.task; import cn.hutool.core.collection.CollUtil; import cn.hutool.core.util.ArrayUtil; +import cn.hutool.core.util.IdUtil; import cn.hutool.core.util.StrUtil; +import cn.iocoder.yudao.framework.common.enums.SymbolConstant; import cn.iocoder.yudao.framework.common.pojo.PageResult; import cn.iocoder.yudao.framework.common.util.date.DateUtils; import cn.iocoder.yudao.framework.common.util.number.NumberUtils; @@ -13,8 +15,7 @@ import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task.*; import cn.iocoder.yudao.module.bpm.convert.task.BpmTaskConvert; import cn.iocoder.yudao.module.bpm.dal.dataobject.task.BpmTaskExtDO; import cn.iocoder.yudao.module.bpm.dal.mysql.task.BpmTaskExtMapper; -import cn.iocoder.yudao.module.bpm.enums.task.BpmProcessInstanceDeleteReasonEnum; -import cn.iocoder.yudao.module.bpm.enums.task.BpmProcessInstanceResultEnum; +import cn.iocoder.yudao.module.bpm.enums.task.*; import cn.iocoder.yudao.module.bpm.service.definition.BpmModelService; import cn.iocoder.yudao.module.bpm.service.message.BpmMessageService; import cn.iocoder.yudao.module.system.api.dept.DeptApi; @@ -26,6 +27,7 @@ import org.flowable.bpmn.model.BpmnModel; import org.flowable.bpmn.model.FlowElement; import org.flowable.bpmn.model.UserTask; import org.flowable.engine.HistoryService; +import org.flowable.engine.ManagementService; import org.flowable.engine.RuntimeService; import org.flowable.engine.TaskService; import org.flowable.engine.history.HistoricProcessInstance; @@ -35,6 +37,8 @@ import org.flowable.task.api.Task; import org.flowable.task.api.TaskQuery; import org.flowable.task.api.history.HistoricTaskInstance; import org.flowable.task.api.history.HistoricTaskInstanceQuery; +import org.flowable.task.service.impl.persistence.entity.TaskEntity; +import org.flowable.task.service.impl.persistence.entity.TaskEntityImpl; import org.springframework.stereotype.Service; import org.springframework.transaction.annotation.Transactional; import org.springframework.transaction.support.TransactionSynchronization; @@ -82,6 +86,9 @@ public class BpmTaskServiceImpl implements BpmTaskService { @Resource private BpmTaskExtMapper taskExtMapper; + @Resource + private ManagementService managementService; + @Override public PageResult getTodoTaskPage(Long userId, BpmTaskTodoPageReqVO pageVO) { // 查询待办任务 @@ -186,6 +193,11 @@ public class BpmTaskServiceImpl implements BpmTaskService { return BpmTaskConvert.INSTANCE.convertList3(tasks, bpmTaskExtDOMap, processInstance, userMap, deptMap); } + @Override + public List getTaskListByTaskIdList(List taskIdList) { + return taskExtMapper.selectListByTaskIds(taskIdList); + } + @Override @Transactional(rollbackFor = Exception.class) public void approveTask(Long userId, @Valid BpmTaskApproveReqVO reqVO) { @@ -202,14 +214,151 @@ public class BpmTaskServiceImpl implements BpmTaskService { approveDelegateTask(reqVO, task); return; } - - // 情况二：自己审批的任务，调用 complete 去完成任务 + // 情况二：后加签的任务 + if(BpmTaskAddSignTypeEnum.AFTER.getType().equals(task.getScopeType())) { + //后加签处理 + handleAfterSignTask(task, reqVO); + return; + } + // 情况三：自己审批的任务，调用 complete 去完成任务 // 完成任务，审批通过 taskService.complete(task.getId(), instance.getProcessVariables()); // 更新任务拓展表为通过 taskExtMapper.updateByTaskId( new BpmTaskExtDO().setTaskId(task.getId()).setResult(BpmProcessInstanceResultEnum.APPROVE.getResult()) .setReason(reqVO.getReason())); + //处理加签任务 + this.approveTask0(task); + } + + /** + * 当前任务存在后加签任务，则不能马上完成，需要一个中间状态，并激活剩余所有子任务 + * @param task 当前任务 + * @param reqVO 前端请求参数 + */ + private void handleAfterSignTask(Task task,BpmTaskApproveReqVO reqVO){ + //1.有向后加签，则该任务状态临时设置为 等待后加签任务完成 状态 + taskExtMapper.updateByTaskId( + new BpmTaskExtDO().setTaskId(task.getId()).setResult(BpmProcessInstanceResultEnum.ADD_SIGN_AFTER.getResult()) + .setReason(reqVO.getReason()).setEndTime(LocalDateTime.now())); + //2. 激活子任务 + List childrenTaskIdList = getChildrenTaskIdList(task.getId()); + for (String childrenTaskId : childrenTaskIdList) { + taskService.resolveTask(childrenTaskId); + } + //更新任务扩展表中子任务为进行中 + taskExtMapper.updateBatchByTaskIdList(childrenTaskIdList, + new BpmTaskExtDO().setResult(BpmProcessInstanceResultEnum.PROCESS.getResult())); + } + + + /** + * 处理当前任务的父任务 + * + * @param task 当前任务 + */ + private void approveTask0(Task task) { + String parentTaskId = task.getParentTaskId(); + if (StrUtil.isNotBlank(parentTaskId)) { + //1. 判断当前任务的父任务是否还有子任务 + long subTaskCount = getSubTaskCount(parentTaskId); + if (subTaskCount == 0) { + //2. 获取父任务 + Task parentTask = getTask(parentTaskId); + String scopeType = parentTask.getScopeType(); + //3. 处理向前加签 + if (BpmTaskAddSignTypeEnum.BEFORE.getType().equals(scopeType)) { + //3.1 如果是向前加签的任务，则调用 resolveTask 指派父任务，将 owner 重新赋值给父任务的 assignee + taskService.resolveTask(parentTaskId); + //3.2 更新任务拓展表为处理中 + taskExtMapper.updateByTaskId( + new BpmTaskExtDO().setTaskId(parentTask.getId()).setResult(BpmProcessInstanceResultEnum.PROCESS.getResult())); + } + //4. 处理向后加签 + if (BpmTaskAddSignTypeEnum.AFTER.getType().equals(scopeType)) { + handleAfterSign(parentTask, parentTaskId); + } + //5. 子任务已处理完成，清空 scopeType 字段，修改 parentTask 信息，方便后续可以继续向前后向后加签 + // 再查询一次的原因是避免报错：Task was updated by another transaction concurrently + // 因为前面处理后可能会导致 parentTask rev 字段被修改，需要重新获取最新的 + parentTask = getTask(parentTaskId); + if(parentTask !=null){ + //为空的情况是：已经通过 handleAfterSign 方法将任务完成了，所以 ru_task 表会查不到数据 + clearTaskScopeTypeAndSave(parentTask); + } + + } + } + } + + /** + * 处理后加签任务 + * + * @param parentTask 当前审批任务的父任务 + * @param parentTaskId 当前审批任务的父ID + */ + private void handleAfterSign(Task parentTask, String parentTaskId) { + //4.1 更新 parentTask 的任务拓展表为通过 + BpmTaskExtDO currentTaskExt = taskExtMapper.selectByTaskId(parentTask.getId()); + BpmTaskExtDO currentTaskUpdateEntity = new BpmTaskExtDO().setTaskId(parentTask.getId()) + .setResult(BpmProcessInstanceResultEnum.APPROVE.getResult()); + if (currentTaskExt.getEndTime() == null) { + //有这个判断是因为 + //4.2 以前没设置过结束时间，才去设置 + currentTaskUpdateEntity.setEndTime(LocalDateTime.now()); + } + taskExtMapper.updateByTaskId(currentTaskUpdateEntity); + //5. 继续往上处理，父任务继续往上查询 + if (StrUtil.isEmpty(parentTask.getParentTaskId())) { + //5.1 该父任务为最顶级的祖先任务（没有父级ID）， 则直接完成父任务流转到下一步 + taskService.complete(parentTaskId); + } else { + //5.2 如果有父级，递归查询上级任务是否都已经完成 + //判断整条链路的任务是否完成 + //例如从 A 任务加签了一个 B 任务， B 任务又加签了一个 C 任务，C任务加签了 D 任务 + //此时，D 任务完成，要一直往上找到祖先任务 A ，调用 complete 方法完成 A 任务 + boolean allChildrenTaskFinish = true; + while (StrUtil.isNotBlank(parentTask.getParentTaskId())) { + parentTask = getTask(parentTask.getParentTaskId()); + BpmTaskExtDO bpmTaskExtDO = taskExtMapper.selectByTaskId(parentTask.getId()); + if (bpmTaskExtDO == null) { + break; + } + //任务已完成 + allChildrenTaskFinish = BpmProcessInstanceResultEnum.isEndResult(bpmTaskExtDO.getResult()); + if (!allChildrenTaskFinish) { + //有一个未完成，则退出循环 + break; + } + } + if (allChildrenTaskFinish) { + //完成最后的顶级祖先任务 + taskService.complete(parentTask.getId()); + } + } + } + + /** + * 清空 scopeType 字段，用于任务没有子任务时使用该方法，方便任务可以再次被不同的方式加签 + * + * @param task 需要被清空的任务 + */ + private void clearTaskScopeTypeAndSave(Task task) { + TaskEntityImpl taskImpl = (TaskEntityImpl) task; + taskImpl.setScopeType(null); + taskService.saveTask(task); + } + + /** + * 获取子任务个数 + * + * @param parentTaskId 父类ID + * @return 剩余子任务个数 + */ + private Long getSubTaskCount(String parentTaskId) { + String tableName = managementService.getTableName(TaskEntity.class); + String sql = "select count(1) from " + tableName + " where PARENT_TASK_ID_=#{parentTaskId}"; + return taskService.createNativeTaskQuery().sql(sql).parameter("parentTaskId", parentTaskId).count(); } /** @@ -226,7 +375,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { String comment = StrUtil.format("[{}]完成委派任务，任务重新回到[{}]手中，审批意见为:{}", currentUser.getNickname(), sourceApproveUser.getNickname(), reqVO.getReason()); taskService.addComment(reqVO.getId(), task.getProcessInstanceId(), - BpmProcessInstanceResultEnum.DELEGATE.getResult().toString(), comment); + BpmCommentTypeEnum.DELEGATE.getDesc(), comment); // 2.1 调用 resolveTask 完成任务。 // 底层调用 TaskHelper.changeTaskAssignee(task, task.getOwner())：将 owner 设置为 assignee @@ -290,6 +439,10 @@ public class BpmTaskServiceImpl implements BpmTaskService { public void createTaskExt(Task task) { BpmTaskExtDO taskExtDO = BpmTaskConvert.INSTANCE.convert2TaskExt(task).setResult(BpmProcessInstanceResultEnum.PROCESS.getResult()); + //向后加签生成的任务，状态不能为进行中，需要等前面父任务完成 + if(BpmTaskAddSignTypeEnum.AFTER_CHILDREN_TASK.getType().equals(task.getScopeType())){ + taskExtDO.setResult(BpmProcessInstanceResultEnum.WAIT_BEFORE_TASK.getResult()); + } taskExtMapper.insert(taskExtDO); } @@ -343,21 +496,19 @@ public class BpmTaskServiceImpl implements BpmTaskService { TransactionSynchronizationManager.registerSynchronization(new TransactionSynchronization() { @Override public void afterCommit() { - ProcessInstance processInstance = - processInstanceService.getProcessInstance(task.getProcessInstanceId()); - AdminUserRespDTO startUser = adminUserApi.getUser(Long.valueOf(processInstance.getStartUserId())); - messageService.sendMessageWhenTaskAssigned( - BpmTaskConvert.INSTANCE.convert(processInstance, startUser, task)); + if(StrUtil.isNotEmpty(task.getAssignee())){ + ProcessInstance processInstance = + processInstanceService.getProcessInstance(task.getProcessInstanceId()); + AdminUserRespDTO startUser = adminUserApi.getUser(Long.valueOf(processInstance.getStartUserId())); + messageService.sendMessageWhenTaskAssigned( + BpmTaskConvert.INSTANCE.convert(processInstance, startUser, task)); + } } }); } private Task getTask(String id) { - Task task = taskService.createTaskQuery().taskId(id).singleResult(); - if (null == task) { - throw exception(TASK_NOT_EXISTS); - } - return task; + return taskService.createTaskQuery().taskId(id).singleResult(); } private HistoricTaskInstance getHistoricTask(String id) { @@ -437,7 +588,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { /** * 执行回退逻辑 * - * @param currentTask 当前回退的任务 + * @param currentTask 当前回退的任务 * @param targetElement 需要回退到的目标任务 * @param reqVO 前端参数封装 */ @@ -458,7 +609,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { return; } taskService.addComment(task.getId(), currentTask.getProcessInstanceId(), - BpmProcessInstanceResultEnum.BACK.getResult().toString(), reqVO.getReason()); + BpmCommentTypeEnum.BACK.getDesc(), reqVO.getReason()); }); // 3. 执行驳回 @@ -485,9 +636,8 @@ public class BpmTaskServiceImpl implements BpmTaskService { String comment = StrUtil.format("[{}]将任务委派给[{}]，委派理由为:{}", currentUser.getNickname(), delegateUser.getNickname(), reqVO.getReason()); String taskId = reqVO.getId(); - // TODO 海：后面改；感觉 comment 应该 type 做个枚举；不和 result 耦合在一起； taskService.addComment(taskId, task.getProcessInstanceId(), - BpmProcessInstanceResultEnum.DELEGATE.getResult().toString(), comment); + BpmCommentTypeEnum.DELEGATE.getDesc(), comment); // 3.1 设置任务所有人 (owner) 为原任务的处理人 (assignee) taskService.setOwner(taskId, task.getAssignee()); @@ -503,7 +653,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { * 校验任务委派参数 * * @param userId 用户编号 - * @param reqVO 任务编号，接收人ID + * @param reqVO 任务编号，接收人ID * @return 当前任务信息 */ private Task validateTaskCanDelegate(Long userId, BpmTaskDelegateReqVO reqVO) { @@ -516,4 +666,125 @@ public class BpmTaskServiceImpl implements BpmTaskService { return task; } + @Override + @Transactional(rollbackFor = Exception.class) + public void addSign(BpmTaskAddSignReqVO reqVO, Long userId) { + //1.获取和校验任务 + TaskEntityImpl taskEntity = validateAddSign(userId, reqVO); + AdminUserRespDTO currentUser = adminUserApi.getUser(userId); + List userList = adminUserApi.getUserList(reqVO.getUserIdList()); + if (CollUtil.isEmpty(userList)) { + throw exception(TASK_ADD_SIGN_USER_NOT_EXIST); + } + //2.处理当前任务 + //2.1 开启计数功能 + taskEntity.setCountEnabled(true); + boolean addSignToBefore = reqVO.getType().equals(BpmTaskAddSignTypeEnum.BEFORE.getType()); + if (addSignToBefore) { + //2.2 向前加签，设置 owner ，置空 assign ,等子任务都完成后，再调用 resolveTask 重新将 owner 设置为 assign + //原因是：不能和向前加签的子任务一起审批，需要等前面的子任务都完成才能审批 + taskEntity.setOwner(taskEntity.getAssignee()); + taskEntity.setAssignee(null); + //2.3 更新扩展表状态 + taskExtMapper.updateByTaskId( + new BpmTaskExtDO().setTaskId(taskEntity.getId()) + .setResult(BpmProcessInstanceResultEnum.ADD_SIGN_BEFORE.getResult()) + .setReason(reqVO.getReason())); + } + //2.4 记录加签方式，完成任务时需要用到判断 + taskEntity.setScopeType(reqVO.getType()); + //2.5 保存当前任务修改后的值 + taskService.saveTask(taskEntity); + + //3. 处理加签任务 + this.createSignSubTasks(convertList(reqVO.getUserIdList(), String::valueOf), taskEntity); + //4. 记录加签 comment，拼接结果为： [当前用户]向前加签/向后加签给了[多个用户],理由为：reason + String comment = StrUtil.format("[{}]{}给了[{}],理由为：{}", currentUser.getNickname(), BpmTaskAddSignTypeEnum.formatDesc(reqVO.getType()), + String.join(SymbolConstant.D, convertList(userList, AdminUserRespDTO::getNickname)), reqVO.getReason()); + taskService.addComment(reqVO.getId(), taskEntity.getProcessInstanceId(), + BpmCommentTypeEnum.ADD_SIGN.getDesc(), comment); + } + + + /** + * 校验任务存在和 存在 向前加签 的任务，则不能 向后加签，存在 向后加签 则不能 向前加签 + * + * @param userId 当前用户 ID + * @param reqVO 请求参数，包含任务 ID 和加签类型 + * @return 当前任务 + */ + private TaskEntityImpl validateAddSign(Long userId, BpmTaskAddSignReqVO reqVO) { + TaskEntityImpl taskEntity = (TaskEntityImpl) validateTask(userId, reqVO.getId()); + //向前加签和向后加签不能同时存在 + if (StrUtil.isNotBlank(taskEntity.getScopeType()) + && !BpmTaskAddSignTypeEnum.AFTER_CHILDREN_TASK.getDesc().equals(taskEntity.getScopeType())) { + if (!taskEntity.getScopeType().equals(reqVO.getType())) { + throw exception(TASK_ADD_SIGN_TYPE_ERROR, + BpmTaskAddSignTypeEnum.formatDesc(taskEntity.getScopeType()), BpmTaskAddSignTypeEnum.formatDesc(reqVO.getType())); + } + } + //同一个 key 的任务，审批人不重复 + List taskList = taskService.createTaskQuery().processInstanceId(taskEntity.getProcessInstanceId()) + .taskDefinitionKey(taskEntity.getTaskDefinitionKey()).list(); + List currentAssigneeList = convertList(taskList, Task::getAssignee); + List frontAssigneeList = convertList(reqVO.getUserIdList(), String::valueOf); + currentAssigneeList.retainAll(frontAssigneeList); + if (CollUtil.isNotEmpty(currentAssigneeList)) { + List userList = adminUserApi.getUserList(convertList(currentAssigneeList, NumberUtils::parseLong)); + throw exception(TASK_ADD_SIGN_USER_REPEAT, String.join(SymbolConstant.D, convertList(userList, AdminUserRespDTO::getNickname))); + } + return taskEntity; + } + + /** + * 创建加签子任务 + * + * @param addSingUserIdList 被加签的用户ID + * @param taskEntity 被加签的任务 + */ + private void createSignSubTasks(List addSingUserIdList, TaskEntityImpl taskEntity) { + if (CollUtil.isNotEmpty(addSingUserIdList)) { + // 创建加签人的新任务,全部基于 taskEntity 为父任务来创建 + addSingUserIdList.forEach(addSignId -> { + if (StrUtil.isNotBlank(addSignId)) { + this.createSubTask(taskEntity, addSignId); + } + }); + } + } + + /** + * 创建子任务 + * + * @param parentTask 父任务 + * @param assignee 子任务的执行人 + * @return + */ + private void createSubTask(TaskEntityImpl parentTask, String assignee) { + //1.生成子任务 + TaskEntityImpl task = (TaskEntityImpl) taskService.newTask(IdUtil.fastSimpleUUID()); + task = BpmTaskConvert.INSTANCE.convert(task,parentTask); + if (BpmTaskAddSignTypeEnum.BEFORE.getType().equals(parentTask.getScopeType())) { + // 2. 前加签，才设置审批人，否则设置 owner + task.setAssignee(assignee); + } else { + //2.2 设置 owner 不设置 assignee 是因为不能同时审批，需要等父任务完成 + task.setOwner(assignee); + //2.3 设置向后加签任务的 scopeType 为 afterChildrenTask，用于设置任务扩展表的状态 + task.setScopeType(BpmTaskAddSignTypeEnum.AFTER_CHILDREN_TASK.getType()); + } + taskService.saveTask(task); + } + + /** + * 获取指定父级任务的所有子任务 ID 集合 + * @param parentTaskId 父任务 ID + * @return 所有子任务的 ID 集合 + */ + private List getChildrenTaskIdList(String parentTaskId){ + String tableName = managementService.getTableName(TaskEntity.class); + String sql = "select ID_ from " + tableName + " where PARENT_TASK_ID_=#{parentTaskId}"; + List childrenTaskList = taskService.createNativeTaskQuery().sql(sql).parameter("parentTaskId", parentTaskId).list(); + return convertList(childrenTaskList, Task::getId); + } } From 80d532b0f159fb57d8be2c883aa1afd419a08acb Mon Sep 17 00:00:00 2001 From: kehaiyou Date: Fri, 29 Sep 2023 15:51:00 +0800 Subject: [PATCH 3/9] =?UTF-8?q?feat:=20=E3=80=90=E5=B7=A5=E4=BD=9C?= =?UTF-8?q?=E6=B5=81=E3=80=91--=E5=87=8F=E7=AD=BE?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../module/bpm/enums/ErrorCodeConstants.java | 1 + .../admin/task/BpmTaskController.java | 17 ++- .../admin/task/vo/task/BpmTaskRespVO.java | 5 + .../task/vo/task/BpmTaskSubSignReqVO.java | 19 +++ .../task/vo/task/BpmTaskSubSignRespVO.java | 15 +++ .../bpm/convert/task/BpmTaskConvert.java | 18 ++- .../bpm/service/task/BpmTaskService.java | 13 ++ .../bpm/service/task/BpmTaskServiceImpl.java | 123 ++++++++++++++++-- 8 files changed, 192 insertions(+), 19 deletions(-) create mode 100644 yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskSubSignReqVO.java create mode 100644 yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskSubSignRespVO.java diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java index 55a2330c2..76c91a255 100644 --- a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java @@ -53,6 +53,7 @@ public interface ErrorCodeConstants { ErrorCode TASK_ADD_SIGN_USER_NOT_EXIST = new ErrorCode(1_009_005_009, "任务加签：选择的用户不存在"); ErrorCode TASK_ADD_SIGN_TYPE_ERROR = new ErrorCode(1_009_005_010, "任务加签：当前任务已经{}，不能{}"); ErrorCode TASK_ADD_SIGN_USER_REPEAT = new ErrorCode(1_009_005_011, "任务加签失败，加签人与现有审批人[{}]重复"); + ErrorCode TASK_SUB_SIGN_NO_PARENT = new ErrorCode(1_009_005_011, "任务减签失败，被减签的任务必须是通过加签生成的任务"); // ========== 流程任务分配规则 1-009-006-000 ========== ErrorCode TASK_ASSIGN_RULE_EXISTS = new ErrorCode(1_009_006_000, "流程({}) 的任务({}) 已经存在分配规则"); ErrorCode TASK_ASSIGN_RULE_NOT_EXISTS = new ErrorCode(1_009_006_001, "流程任务分配规则不存在"); diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java index e442d7aad..2f0e2db2a 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java @@ -103,8 +103,23 @@ public class BpmTaskController { @PutMapping("/add-sign") @Operation(summary = "加签", description = "before, after为前加签后加签") @PreAuthorize("@ss.hasPermission('bpm:task:add-sign')") - public CommonResult addSign(@RequestBody @Valid BpmTaskAddSignReqVO reqVO) { + public CommonResult addSign(@Valid @RequestBody BpmTaskAddSignReqVO reqVO) { taskService.addSign(reqVO,getLoginUserId()); return success(true); } + + @PutMapping("/sub-sign") + @Operation(summary = "减签", description = "") + @PreAuthorize("@ss.hasPermission('bpm:task:sub-sign')") + public CommonResult subSign(@Valid @RequestBody BpmTaskSubSignReqVO bpmTaskSubSignReqVO) { + taskService.subSign(bpmTaskSubSignReqVO,getLoginUserId()); + return success(true); + } + @GetMapping("/get-sub-sign") + @Operation(summary = "获取能被减签的任务", description = "") + @PreAuthorize("@ss.hasPermission('bpm:task:sub-sign')") + public CommonResult> getChildrenTaskList(@RequestParam("taskId") String taskId) { + return success(taskService.getChildrenTaskList(taskId)); + } + } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskRespVO.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskRespVO.java index 3c404678f..198727dd6 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskRespVO.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskRespVO.java @@ -19,6 +19,11 @@ public class BpmTaskRespVO extends BpmTaskDonePageItemRespVO { */ private User assigneeUser; + /** + * 父任务ID + */ + private String parentTaskId; + @Schema(description = "用户信息") @Data public static class User { diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskSubSignReqVO.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskSubSignReqVO.java new file mode 100644 index 000000000..665a9aec8 --- /dev/null +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskSubSignReqVO.java @@ -0,0 +1,19 @@ +package cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task; + +import io.swagger.v3.oas.annotations.media.Schema; +import lombok.Data; + +import javax.validation.constraints.NotEmpty; + +@Schema(description = "管理后台 - 减签流程任务的 Request VO") +@Data +public class BpmTaskSubSignReqVO { + + @Schema(description = "被减签的任务 ID") + @NotEmpty(message = "任务编号不能为空") + private String id; + + @Schema(description = "加签原因") + @NotEmpty(message = "加签原因不能为空") + private String reason; +} diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskSubSignRespVO.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskSubSignRespVO.java new file mode 100644 index 000000000..efecbf582 --- /dev/null +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskSubSignRespVO.java @@ -0,0 +1,15 @@ +package cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task; + +import io.swagger.v3.oas.annotations.media.Schema; +import lombok.Data; + +@Schema(description = "管理后台 - 减签流程任务的 Response VO") +@Data +public class BpmTaskSubSignRespVO { + @Schema(description = "审核的用户信息", requiredMode = Schema.RequiredMode.REQUIRED, example = "小李") + private BpmTaskRespVO.User assigneeUser; + @Schema(description = "任务 ID", requiredMode = Schema.RequiredMode.REQUIRED, example = "12312") + private String id; + @Schema(description = "任务名称", requiredMode = Schema.RequiredMode.REQUIRED, example = "经理审批") + private String name; +} diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java index ebbcf11fc..e61061c61 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java @@ -4,10 +4,7 @@ import cn.hutool.core.date.LocalDateTimeUtil; import cn.iocoder.yudao.framework.common.util.collection.CollectionUtils; import cn.iocoder.yudao.framework.common.util.date.DateUtils; import cn.iocoder.yudao.framework.common.util.number.NumberUtils; -import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task.BpmTaskDonePageItemRespVO; -import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task.BpmTaskRespVO; -import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task.BpmTaskSimpleRespVO; -import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task.BpmTaskTodoPageItemRespVO; +import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task.*; import cn.iocoder.yudao.module.bpm.dal.dataobject.task.BpmTaskExtDO; import cn.iocoder.yudao.module.bpm.service.message.dto.BpmMessageSendWhenTaskCreatedReqDTO; import cn.iocoder.yudao.module.system.api.dept.dto.DeptRespDTO; @@ -162,4 +159,17 @@ public interface BpmTaskConvert { task.setCreateTime(new Date()); return task; } + + default List convertList(List bpmTaskExtDOList, Map userMap){ + return CollectionUtils.convertList(bpmTaskExtDOList, task->{ + BpmTaskSubSignRespVO bpmTaskSubSignRespVO = new BpmTaskSubSignRespVO(); + bpmTaskSubSignRespVO.setName(task.getName()); + bpmTaskSubSignRespVO.setId(task.getTaskId()); + AdminUserRespDTO assignUser = userMap.get(task.getAssigneeUserId()); + if (assignUser != null) { + bpmTaskSubSignRespVO.setAssigneeUser(convert3(assignUser)); + } + return bpmTaskSubSignRespVO; + }); + } } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java index 2eb222b7c..3a7f84c07 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java @@ -163,4 +163,17 @@ public interface BpmTaskService { */ void addSign(BpmTaskAddSignReqVO reqVO, Long userId); + /** + * 减签 + * @param bpmTaskSubSignReqVO 被减签的任务ID，理由 + * @param loginUserId 当前用户ID + */ + void subSign(BpmTaskSubSignReqVO bpmTaskSubSignReqVO, Long loginUserId); + + /** + * 获取指定任务的子任务和审批人信息 + * @param taskId 指定任务ID + * @return 子任务列表 + */ + List getChildrenTaskList(String taskId); } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java index 6e57fcff4..3dd806697 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java @@ -15,6 +15,7 @@ import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task.*; import cn.iocoder.yudao.module.bpm.convert.task.BpmTaskConvert; import cn.iocoder.yudao.module.bpm.dal.dataobject.task.BpmTaskExtDO; import cn.iocoder.yudao.module.bpm.dal.mysql.task.BpmTaskExtMapper; +import cn.iocoder.yudao.module.bpm.enums.ErrorCodeConstants; import cn.iocoder.yudao.module.bpm.enums.task.*; import cn.iocoder.yudao.module.bpm.service.definition.BpmModelService; import cn.iocoder.yudao.module.bpm.service.message.BpmMessageService; @@ -264,7 +265,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { long subTaskCount = getSubTaskCount(parentTaskId); if (subTaskCount == 0) { //2. 获取父任务 - Task parentTask = getTask(parentTaskId); + Task parentTask = validateTaskExist(parentTaskId); String scopeType = parentTask.getScopeType(); //3. 处理向前加签 if (BpmTaskAddSignTypeEnum.BEFORE.getType().equals(scopeType)) { @@ -319,7 +320,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { //此时，D 任务完成，要一直往上找到祖先任务 A ，调用 complete 方法完成 A 任务 boolean allChildrenTaskFinish = true; while (StrUtil.isNotBlank(parentTask.getParentTaskId())) { - parentTask = getTask(parentTask.getParentTaskId()); + parentTask = validateTaskExist(parentTask.getParentTaskId()); BpmTaskExtDO bpmTaskExtDO = taskExtMapper.selectByTaskId(parentTask.getId()); if (bpmTaskExtDO == null) { break; @@ -375,7 +376,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { String comment = StrUtil.format("[{}]完成委派任务，任务重新回到[{}]手中，审批意见为:{}", currentUser.getNickname(), sourceApproveUser.getNickname(), reqVO.getReason()); taskService.addComment(reqVO.getId(), task.getProcessInstanceId(), - BpmCommentTypeEnum.DELEGATE.getDesc(), comment); + BpmCommentTypeEnum.DELEGATE.getResult().toString(), comment); // 2.1 调用 resolveTask 完成任务。 // 底层调用 TaskHelper.changeTaskAssignee(task, task.getOwner())：将 owner 设置为 assignee @@ -425,10 +426,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { * @param taskId task id */ private Task validateTask(Long userId, String taskId) { - Task task = getTask(taskId); - if (task == null) { - throw exception(TASK_NOT_EXISTS); - } + Task task = validateTaskExist(taskId); if (!Objects.equals(userId, NumberUtils.parseLong(task.getAssignee()))) { throw exception(TASK_OPERATE_FAIL_ASSIGN_NOT_SELF); } @@ -507,6 +505,14 @@ public class BpmTaskServiceImpl implements BpmTaskService { }); } + private Task validateTaskExist(String id){ + Task task = taskService.createTaskQuery().taskId(id).singleResult(); + if(task == null){ + throw exception(TASK_NOT_EXISTS); + } + return task; + } + private Task getTask(String id) { return taskService.createTaskQuery().taskId(id).singleResult(); } @@ -518,10 +524,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { @Override public List getReturnTaskList(String taskId) { // 1. 校验当前任务 task 存在 - Task task = getTask(taskId); - if (task == null) { - throw exception(TASK_NOT_EXISTS); - } + Task task = validateTaskExist(taskId); // 根据流程定义获取流程模型信息 BpmnModel bpmnModel = bpmModelService.getBpmnModelByDefinitionId(task.getProcessDefinitionId()); FlowElement source = BpmnModelUtils.getFlowElementById(bpmnModel, task.getTaskDefinitionKey()); @@ -609,7 +612,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { return; } taskService.addComment(task.getId(), currentTask.getProcessInstanceId(), - BpmCommentTypeEnum.BACK.getDesc(), reqVO.getReason()); + BpmCommentTypeEnum.BACK.getResult().toString(), reqVO.getReason()); }); // 3. 执行驳回 @@ -637,7 +640,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { delegateUser.getNickname(), reqVO.getReason()); String taskId = reqVO.getId(); taskService.addComment(taskId, task.getProcessInstanceId(), - BpmCommentTypeEnum.DELEGATE.getDesc(), comment); + BpmCommentTypeEnum.DELEGATE.getResult().toString(), comment); // 3.1 设置任务所有人 (owner) 为原任务的处理人 (assignee) taskService.setOwner(taskId, task.getAssignee()); @@ -702,7 +705,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { String comment = StrUtil.format("[{}]{}给了[{}],理由为：{}", currentUser.getNickname(), BpmTaskAddSignTypeEnum.formatDesc(reqVO.getType()), String.join(SymbolConstant.D, convertList(userList, AdminUserRespDTO::getNickname)), reqVO.getReason()); taskService.addComment(reqVO.getId(), taskEntity.getProcessInstanceId(), - BpmCommentTypeEnum.ADD_SIGN.getDesc(), comment); + BpmCommentTypeEnum.ADD_SIGN.getResult().toString(), comment); } @@ -776,6 +779,86 @@ public class BpmTaskServiceImpl implements BpmTaskService { taskService.saveTask(task); } + @Override + @Transactional(rollbackFor = Exception.class) + public void subSign(BpmTaskSubSignReqVO reqVO,Long userId) { + Task task = validateSubSign(reqVO.getId()); + AdminUserRespDTO user = adminUserApi.getUser(userId); + AdminUserRespDTO cancelUser = null; + if(StrUtil.isNotBlank(task.getAssignee())){ + cancelUser = adminUserApi.getUser(NumberUtils.parseLong(task.getAssignee())); + } + if(cancelUser == null && StrUtil.isNotBlank(task.getOwner())){ + cancelUser = adminUserApi.getUser(NumberUtils.parseLong(task.getOwner())); + } + Assert.notNull(cancelUser,"任务中没有所有者和审批人，数据错误"); + //1. 获取所有需要删除的任务 ID ，包含当前任务和所有子任务 + List allTaskIdList = getAllChildTaskIds(task.getId()); + //2. 删除任务和所有子任务 + taskService.deleteTasks(allTaskIdList); + //3. 修改扩展表状态为取消 + taskExtMapper.updateBatchByTaskIdList(allTaskIdList,new BpmTaskExtDO().setResult(BpmProcessInstanceResultEnum.CANCEL.getResult()) + .setReason(StrUtil.format("由于{}操作[减签]，任务被取消",user.getNickname()))); + //4. 判断当前任务的父任务是否还有子任务 + Task parentTask = validateTaskExist(task.getParentTaskId()); + Long subTaskCount = getSubTaskCount(task.getParentTaskId()); + if(subTaskCount == 0){ + if(BpmTaskAddSignTypeEnum.BEFORE.getType().equals(parentTask.getScopeType())){ + //4.1 父任务是往前加签的，则进入判断,将当前任务的状态设置为进行中，并将 owner 设置回 assignee + taskExtMapper.updateByTaskId(new BpmTaskExtDO().setResult(BpmProcessInstanceResultEnum.PROCESS.getResult()).setTaskId(task.getParentTaskId())); + parentTask.setAssignee(parentTask.getOwner()); + parentTask.setOwner(null); + } + //4.2 清空 scopeType 字段，修改task + clearTaskScopeTypeAndSave(parentTask); + } + //5.记录日志到父任务中 + String comment = StrUtil.format("{}操作了【减签】,审批人{}的任务被取消",user.getNickname(),cancelUser.getNickname()); + taskService.addComment(parentTask.getId(),parentTask.getProcessInstanceId(), + BpmCommentTypeEnum.SUB_SIGN.getResult().toString(),comment); + } + + /** + * 校验任务是否能被减签 + * @param id 任务ID + * @return 任务信息 + */ + private Task validateSubSign(String id) { + Task task = validateTaskExist(id); + //必须有parentId + if(StrUtil.isEmpty(task.getParentTaskId())){ + throw exception(TASK_SUB_SIGN_NO_PARENT); + } + return task; + } + + /** + * 获取所有要被取消的删除的任务 ID 集合 + * @param parentTaskId 父级任务ID + * @return 所有任务ID + */ + public List getAllChildTaskIds(String parentTaskId) { + List allChildTaskIds = new ArrayList<>(); + //1. 先将自己放入 + allChildTaskIds.add(parentTaskId); + //2. 递归获取子级 + recursiveGetChildTaskIds(parentTaskId, allChildTaskIds); + return allChildTaskIds; + } + + /** + * 递归处理子级任务 + * @param taskId 当前任务ID + * @param taskIds 结果 + */ + private void recursiveGetChildTaskIds(String taskId, List taskIds) { + List childrenTaskIdList = getChildrenTaskIdList(taskId); + for (String childTaskId : childrenTaskIdList) { + taskIds.add(childTaskId); // 将子任务的ID添加到集合中 + recursiveGetChildTaskIds(childTaskId, taskIds); // 递归获取子任务的子任务 + } + } + /** * 获取指定父级任务的所有子任务 ID 集合 * @param parentTaskId 父任务 ID @@ -787,4 +870,16 @@ public class BpmTaskServiceImpl implements BpmTaskService { List childrenTaskList = taskService.createNativeTaskQuery().sql(sql).parameter("parentTaskId", parentTaskId).list(); return convertList(childrenTaskList, Task::getId); } + + @Override + public List getChildrenTaskList(String taskId){ + List childrenTaskIdList = getChildrenTaskIdList(taskId); + if(CollUtil.isEmpty(childrenTaskIdList)){ + return Collections.emptyList(); + } + List bpmTaskExtDOList = taskExtMapper.selectListByTaskIds(childrenTaskIdList); + Set assigneeUserIdSet = convertSet(bpmTaskExtDOList, BpmTaskExtDO::getAssigneeUserId); + Map userMap = adminUserApi.getUserMap(assigneeUserIdSet); + return BpmTaskConvert.INSTANCE.convertList(bpmTaskExtDOList,userMap); + } } From cd678e997ad152a8aa5657877dbe877b210ccf61 Mon Sep 17 00:00:00 2001 From: kehaiyou Date: Sat, 30 Sep 2023 14:56:16 +0800 Subject: [PATCH 4/9] =?UTF-8?q?feat:=20=E3=80=90=E5=B7=A5=E4=BD=9C?= =?UTF-8?q?=E6=B5=81=E3=80=91--=E5=87=8F=E7=AD=BE=E5=92=8C=E5=8A=A0?= =?UTF-8?q?=E7=AD=BE=E5=85=B1=E7=94=A8=E5=A4=84=E7=90=86=E7=88=B6=E4=BB=BB?= =?UTF-8?q?=E5=8A=A1=E9=80=BB=E8=BE=91?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../bpm/service/task/BpmTaskServiceImpl.java | 33 ++++++------------- 1 file changed, 10 insertions(+), 23 deletions(-) diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java index 3dd806697..c5021ba08 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java @@ -15,7 +15,6 @@ import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task.*; import cn.iocoder.yudao.module.bpm.convert.task.BpmTaskConvert; import cn.iocoder.yudao.module.bpm.dal.dataobject.task.BpmTaskExtDO; import cn.iocoder.yudao.module.bpm.dal.mysql.task.BpmTaskExtMapper; -import cn.iocoder.yudao.module.bpm.enums.ErrorCodeConstants; import cn.iocoder.yudao.module.bpm.enums.task.*; import cn.iocoder.yudao.module.bpm.service.definition.BpmModelService; import cn.iocoder.yudao.module.bpm.service.message.BpmMessageService; @@ -229,7 +228,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { new BpmTaskExtDO().setTaskId(task.getId()).setResult(BpmProcessInstanceResultEnum.APPROVE.getResult()) .setReason(reqVO.getReason())); //处理加签任务 - this.approveTask0(task); + this.handleParentTask(task); } /** @@ -258,7 +257,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { * * @param task 当前任务 */ - private void approveTask0(Task task) { + private void handleParentTask(Task task) { String parentTaskId = task.getParentTaskId(); if (StrUtil.isNotBlank(parentTaskId)) { //1. 判断当前任务的父任务是否还有子任务 @@ -274,10 +273,9 @@ public class BpmTaskServiceImpl implements BpmTaskService { //3.2 更新任务拓展表为处理中 taskExtMapper.updateByTaskId( new BpmTaskExtDO().setTaskId(parentTask.getId()).setResult(BpmProcessInstanceResultEnum.PROCESS.getResult())); - } - //4. 处理向后加签 - if (BpmTaskAddSignTypeEnum.AFTER.getType().equals(scopeType)) { - handleAfterSign(parentTask, parentTaskId); + }else if (BpmTaskAddSignTypeEnum.AFTER.getType().equals(scopeType)) { + //4. 处理向后加签 + handleAfterSign(parentTask); } //5. 子任务已处理完成，清空 scopeType 字段，修改 parentTask 信息，方便后续可以继续向前后向后加签 // 再查询一次的原因是避免报错：Task was updated by another transaction concurrently @@ -296,9 +294,9 @@ public class BpmTaskServiceImpl implements BpmTaskService { * 处理后加签任务 * * @param parentTask 当前审批任务的父任务 - * @param parentTaskId 当前审批任务的父ID */ - private void handleAfterSign(Task parentTask, String parentTaskId) { + private void handleAfterSign(Task parentTask) { + String parentTaskId = parentTask.getId(); //4.1 更新 parentTask 的任务拓展表为通过 BpmTaskExtDO currentTaskExt = taskExtMapper.selectByTaskId(parentTask.getId()); BpmTaskExtDO currentTaskUpdateEntity = new BpmTaskExtDO().setTaskId(parentTask.getId()) @@ -799,22 +797,11 @@ public class BpmTaskServiceImpl implements BpmTaskService { //3. 修改扩展表状态为取消 taskExtMapper.updateBatchByTaskIdList(allTaskIdList,new BpmTaskExtDO().setResult(BpmProcessInstanceResultEnum.CANCEL.getResult()) .setReason(StrUtil.format("由于{}操作[减签]，任务被取消",user.getNickname()))); - //4. 判断当前任务的父任务是否还有子任务 - Task parentTask = validateTaskExist(task.getParentTaskId()); - Long subTaskCount = getSubTaskCount(task.getParentTaskId()); - if(subTaskCount == 0){ - if(BpmTaskAddSignTypeEnum.BEFORE.getType().equals(parentTask.getScopeType())){ - //4.1 父任务是往前加签的，则进入判断,将当前任务的状态设置为进行中，并将 owner 设置回 assignee - taskExtMapper.updateByTaskId(new BpmTaskExtDO().setResult(BpmProcessInstanceResultEnum.PROCESS.getResult()).setTaskId(task.getParentTaskId())); - parentTask.setAssignee(parentTask.getOwner()); - parentTask.setOwner(null); - } - //4.2 清空 scopeType 字段，修改task - clearTaskScopeTypeAndSave(parentTask); - } + //4. 处理当前任务的父任务 + this.handleParentTask(task); //5.记录日志到父任务中 String comment = StrUtil.format("{}操作了【减签】,审批人{}的任务被取消",user.getNickname(),cancelUser.getNickname()); - taskService.addComment(parentTask.getId(),parentTask.getProcessInstanceId(), + taskService.addComment(task.getParentTaskId(),task.getProcessInstanceId(), BpmCommentTypeEnum.SUB_SIGN.getResult().toString(),comment); } From d7ae94443824afc1f2476b2b021fdcbda5b30188 Mon Sep 17 00:00:00 2001 From: kehaiyou Date: Sat, 30 Sep 2023 15:35:14 +0800 Subject: [PATCH 5/9] =?UTF-8?q?feat:=20=E3=80=90=E5=B7=A5=E4=BD=9C?= =?UTF-8?q?=E6=B5=81=E3=80=91--=E5=87=8F=E7=AD=BE=E9=80=BB=E8=BE=91?= =?UTF-8?q?=E4=BC=98=E5=8C=96?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../task/BpmProcessInstanceResultEnum.java | 8 +++++ .../bpm/convert/task/BpmTaskConvert.java | 10 +++++-- .../bpm/dal/mysql/task/BpmTaskExtMapper.java | 9 ++++++ .../bpm/service/task/BpmTaskServiceImpl.java | 29 ++++++++++++++----- 4 files changed, 47 insertions(+), 9 deletions(-) diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java index 62b18f54a..0fbb5d58f 100644 --- a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java @@ -4,6 +4,9 @@ import cn.iocoder.yudao.framework.common.util.object.ObjectUtils; import lombok.AllArgsConstructor; import lombok.Getter; +import java.util.Arrays; +import java.util.List; + /** * 流程实例的结果 * @@ -63,4 +66,9 @@ public enum BpmProcessInstanceResultEnum { ADD_SIGN_AFTER.getResult()); } + /** + * 能被减签的状态 + */ + public static final List CAN_SUB_SIGN_STATUS = Arrays.asList(PROCESS.result, WAIT_BEFORE_TASK.result); + } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java index e61061c61..eff2d0a25 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java @@ -1,6 +1,7 @@ package cn.iocoder.yudao.module.bpm.convert.task; import cn.hutool.core.date.LocalDateTimeUtil; +import cn.hutool.core.util.StrUtil; import cn.iocoder.yudao.framework.common.util.collection.CollectionUtils; import cn.iocoder.yudao.framework.common.util.date.DateUtils; import cn.iocoder.yudao.framework.common.util.number.NumberUtils; @@ -160,12 +161,17 @@ public interface BpmTaskConvert { return task; } - default List convertList(List bpmTaskExtDOList, Map userMap){ + default List convertList(List bpmTaskExtDOList, + Map userMap, + Map idTaskMap){ return CollectionUtils.convertList(bpmTaskExtDOList, task->{ BpmTaskSubSignRespVO bpmTaskSubSignRespVO = new BpmTaskSubSignRespVO(); bpmTaskSubSignRespVO.setName(task.getName()); bpmTaskSubSignRespVO.setId(task.getTaskId()); - AdminUserRespDTO assignUser = userMap.get(task.getAssigneeUserId()); + Task sourceTask = idTaskMap.get(task.getTaskId()); + // 后加签任务不会直接设置 assignee ,所以不存在 assignee 的情况，则去取 owner + String assignee = StrUtil.isNotEmpty(sourceTask.getAssignee()) ? sourceTask.getAssignee() : sourceTask.getOwner(); + AdminUserRespDTO assignUser = userMap.get(NumberUtils.parseLong(assignee)); if (assignUser != null) { bpmTaskSubSignRespVO.setAssigneeUser(convert3(assignUser)); } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java index 7d4d18d8f..9b0fe8bb1 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java @@ -1,7 +1,9 @@ package cn.iocoder.yudao.module.bpm.dal.mysql.task; import cn.iocoder.yudao.framework.mybatis.core.mapper.BaseMapperX; +import cn.iocoder.yudao.framework.mybatis.core.query.LambdaQueryWrapperX; import cn.iocoder.yudao.module.bpm.dal.dataobject.task.BpmTaskExtDO; +import cn.iocoder.yudao.module.bpm.enums.task.BpmProcessInstanceResultEnum; import com.baomidou.mybatisplus.core.conditions.query.LambdaQueryWrapper; import org.apache.ibatis.annotations.Mapper; @@ -19,6 +21,13 @@ public interface BpmTaskExtMapper extends BaseMapperX { return selectList(BpmTaskExtDO::getTaskId, taskIds); } + default List selectProcessListByTaskIds(Collection taskIds) { + return selectList(new LambdaQueryWrapperX() + .in(BpmTaskExtDO::getTaskId, taskIds) + .in(BpmTaskExtDO::getResult, BpmProcessInstanceResultEnum.CAN_SUB_SIGN_STATUS)); + } + + default BpmTaskExtDO selectByTaskId(String taskId) { return selectOne(BpmTaskExtDO::getTaskId, taskId); } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java index c5021ba08..ecad585ea 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java @@ -34,6 +34,7 @@ import org.flowable.engine.history.HistoricProcessInstance; import org.flowable.engine.runtime.ProcessInstance; import org.flowable.task.api.DelegationState; import org.flowable.task.api.Task; +import org.flowable.task.api.TaskInfo; import org.flowable.task.api.TaskQuery; import org.flowable.task.api.history.HistoricTaskInstance; import org.flowable.task.api.history.HistoricTaskInstanceQuery; @@ -49,6 +50,8 @@ import javax.annotation.Resource; import javax.validation.Valid; import java.time.LocalDateTime; import java.util.*; +import java.util.stream.Collectors; +import java.util.stream.Stream; import static cn.iocoder.yudao.framework.common.exception.util.ServiceExceptionUtil.exception; import static cn.iocoder.yudao.framework.common.util.collection.CollectionUtils.*; @@ -797,12 +800,13 @@ public class BpmTaskServiceImpl implements BpmTaskService { //3. 修改扩展表状态为取消 taskExtMapper.updateBatchByTaskIdList(allTaskIdList,new BpmTaskExtDO().setResult(BpmProcessInstanceResultEnum.CANCEL.getResult()) .setReason(StrUtil.format("由于{}操作[减签]，任务被取消",user.getNickname()))); - //4. 处理当前任务的父任务 - this.handleParentTask(task); - //5.记录日志到父任务中 + //4.记录日志到父任务中 先记录日志是因为，通过 handleParentTask 方法之后，任务可能被完成了，并且不存在了，会报异常，所以先记录 String comment = StrUtil.format("{}操作了【减签】,审批人{}的任务被取消",user.getNickname(),cancelUser.getNickname()); taskService.addComment(task.getParentTaskId(),task.getProcessInstanceId(), BpmCommentTypeEnum.SUB_SIGN.getResult().toString(),comment); + //5. 处理当前任务的父任务 + this.handleParentTask(task); + } /** @@ -864,9 +868,20 @@ public class BpmTaskServiceImpl implements BpmTaskService { if(CollUtil.isEmpty(childrenTaskIdList)){ return Collections.emptyList(); } - List bpmTaskExtDOList = taskExtMapper.selectListByTaskIds(childrenTaskIdList); - Set assigneeUserIdSet = convertSet(bpmTaskExtDOList, BpmTaskExtDO::getAssigneeUserId); - Map userMap = adminUserApi.getUserMap(assigneeUserIdSet); - return BpmTaskConvert.INSTANCE.convertList(bpmTaskExtDOList,userMap); + //1. 只查询进行中的任务 + List bpmTaskExtDOList = taskExtMapper.selectProcessListByTaskIds(childrenTaskIdList); + //2. 后加签的任务，可能不存在 assignee,所以还需要查询 owner + List taskList = taskService.createTaskQuery().taskIds(childrenTaskIdList).list(); + Map idTaskMap = convertMap(taskList, TaskInfo::getId); + //3. 将 owner 和 assignee 统一到一个集合中 + List userIds = taskList.stream() + .flatMap(control -> + Stream.of(control.getAssignee(), control.getOwner()) + .filter(Objects::nonNull)) + .distinct() + .map(NumberUtils::parseLong) + .collect(Collectors.toList()); + Map userMap = adminUserApi.getUserMap(userIds); + return BpmTaskConvert.INSTANCE.convertList(bpmTaskExtDOList,userMap,idTaskMap); } } From 10af05066c14b82c02ed75d82d27a36709850906 Mon Sep 17 00:00:00 2001 From: kehaiyou Date: Sun, 8 Oct 2023 22:35:02 +0800 Subject: [PATCH 6/9] =?UTF-8?q?feat:=20=E3=80=90=E5=B7=A5=E4=BD=9C?= =?UTF-8?q?=E6=B5=81=E3=80=91--=E4=BB=BB=E5=8A=A1=E5=88=97=E8=A1=A8?= =?UTF-8?q?=E5=8F=AA=E8=BF=94=E5=9B=9E=E9=A1=B6=E7=BA=A7=E4=BB=BB=E5=8A=A1?= =?UTF-8?q?=EF=BC=8C=E6=8B=BC=E8=A3=85=E6=88=90=E7=88=B6=E5=AD=90=E5=85=B3?= =?UTF-8?q?=E7=B3=BB=E5=B9=B6=E8=BF=94=E5=9B=9E?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../admin/task/vo/task/BpmTaskRespVO.java | 5 +++++ .../module/bpm/convert/task/BpmTaskConvert.java | 15 +++++++++++++++ .../bpm/service/task/BpmTaskServiceImpl.java | 3 ++- 3 files changed, 22 insertions(+), 1 deletion(-) diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskRespVO.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskRespVO.java index 198727dd6..1af839e26 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskRespVO.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskRespVO.java @@ -5,6 +5,8 @@ import lombok.Data; import lombok.EqualsAndHashCode; import lombok.ToString; +import java.util.List; + @Schema(description = "管理后台 - 流程任务的 Response VO") @Data @EqualsAndHashCode(callSuper = true) @@ -24,6 +26,9 @@ public class BpmTaskRespVO extends BpmTaskDonePageItemRespVO { */ private String parentTaskId; + @Schema(description = "子任务（由加签生成）", requiredMode = Schema.RequiredMode.REQUIRED, example = "childrenTask") + private List children; + @Schema(description = "用户信息") @Data public static class User { diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java index eff2d0a25..05fcb1df4 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java @@ -1,5 +1,6 @@ package cn.iocoder.yudao.module.bpm.convert.task; +import cn.hutool.core.collection.CollUtil; import cn.hutool.core.date.LocalDateTimeUtil; import cn.hutool.core.util.StrUtil; import cn.iocoder.yudao.framework.common.util.collection.CollectionUtils; @@ -178,4 +179,18 @@ public interface BpmTaskConvert { return bpmTaskSubSignRespVO; }); } + + /** + * 转换任务为父子级 + * @param result + * @return + */ + default List convertChildrenList(List result){ + List childrenTaskList = CollectionUtils.filterList(result, r -> StrUtil.isNotEmpty(r.getParentTaskId())); + Map> parentChildrenTaskListMap = CollectionUtils.convertMultiMap(childrenTaskList, BpmTaskRespVO::getParentTaskId); + for (BpmTaskRespVO bpmTaskRespVO : result) { + bpmTaskRespVO.setChildren(parentChildrenTaskListMap.get(bpmTaskRespVO.getId())); + } + return CollectionUtils.filterList(result, r -> StrUtil.isEmpty(r.getParentTaskId())); + } } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java index ecad585ea..b6a2155dc 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java @@ -193,7 +193,8 @@ public class BpmTaskServiceImpl implements BpmTaskService { Map deptMap = deptApi.getDeptMap(convertSet(userMap.values(), AdminUserRespDTO::getDeptId)); // 拼接数据 - return BpmTaskConvert.INSTANCE.convertList3(tasks, bpmTaskExtDOMap, processInstance, userMap, deptMap); + List result = BpmTaskConvert.INSTANCE.convertList3(tasks, bpmTaskExtDOMap, processInstance, userMap, deptMap); + return BpmTaskConvert.INSTANCE.convertChildrenList(result); } @Override From 5d6faa802351cc41b11b3ec60ac91b957ffad399 Mon Sep 17 00:00:00 2001 From: youkehai Date: Mon, 9 Oct 2023 16:40:31 +0800 Subject: [PATCH 7/9] =?UTF-8?q?fix:=20=E3=80=90=E5=B7=A5=E4=BD=9C=E6=B5=81?= =?UTF-8?q?=E3=80=91-=E5=8A=A0=E5=87=8F=E7=AD=BE=E7=88=B6=E4=BB=BB?= =?UTF-8?q?=E5=8A=A1=E5=A4=84=E7=90=86=E9=80=BB=E8=BE=91=E5=AE=8C=E5=96=84?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../bpm/service/task/BpmTaskServiceImpl.java | 37 ++++++++++++++----- 1 file changed, 27 insertions(+), 10 deletions(-) diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java index b6a2155dc..b99b0c73d 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java @@ -312,10 +312,9 @@ public class BpmTaskServiceImpl implements BpmTaskService { } taskExtMapper.updateByTaskId(currentTaskUpdateEntity); //5. 继续往上处理，父任务继续往上查询 - if (StrUtil.isEmpty(parentTask.getParentTaskId())) { - //5.1 该父任务为最顶级的祖先任务（没有父级ID）， 则直接完成父任务流转到下一步 - taskService.complete(parentTaskId); - } else { + //5.1 先完成自己 + taskService.complete(parentTaskId); + if (StrUtil.isNotEmpty(parentTask.getParentTaskId())) { //5.2 如果有父级，递归查询上级任务是否都已经完成 //判断整条链路的任务是否完成 //例如从 A 任务加签了一个 B 任务， B 任务又加签了一个 C 任务，C任务加签了 D 任务 @@ -327,15 +326,33 @@ public class BpmTaskServiceImpl implements BpmTaskService { if (bpmTaskExtDO == null) { break; } - //任务已完成 - allChildrenTaskFinish = BpmProcessInstanceResultEnum.isEndResult(bpmTaskExtDO.getResult()); - if (!allChildrenTaskFinish) { - //有一个未完成，则退出循环 - break; + boolean currentTaskFinish = BpmProcessInstanceResultEnum.isEndResult(bpmTaskExtDO.getResult()); + //5.3 如果 allChildrenTaskFinish 已经被赋值为 false ,则不会再赋值为 true，因为整个链路没有完成 + if (allChildrenTaskFinish) { + allChildrenTaskFinish = currentTaskFinish; + } + if (!currentTaskFinish) { + //6 处理非完成状态的任务 + //6.1 判断当前任务的父任务是否还有子任务 + Long subTaskCount = getSubTaskCount(bpmTaskExtDO.getTaskId()); + if (subTaskCount == 0) { + //6.2 没有子任务，判断当前任务状态是否为 ADD_SIGN_BEFORE 待前加签任务完成 + if (BpmProcessInstanceResultEnum.ADD_SIGN_BEFORE.getResult().equals(bpmTaskExtDO.getResult())) { + //6.3 需要修改该任务状态为处理中 + taskService.resolveTask(bpmTaskExtDO.getTaskId()); + bpmTaskExtDO.setResult(BpmProcessInstanceResultEnum.PROCESS.getResult()); + taskExtMapper.updateByTaskId(bpmTaskExtDO); + } + //6.4 清空 scopeType 字段，用于任务没有子任务时使用该方法，方便任务可以再次被不同的方式加签 + parentTask = getTask(bpmTaskExtDO.getTaskId()); + if (parentTask != null) { + clearTaskScopeTypeAndSave(parentTask); + } + } } } if (allChildrenTaskFinish) { - //完成最后的顶级祖先任务 + //7. 完成最后的顶级祖先任务 taskService.complete(parentTask.getId()); } } From 6388950ab2b6db8d25a209789064d270da1e0846 Mon Sep 17 00:00:00 2001 From: "zhijiantianya@gmail.com" Date: Mon, 9 Oct 2023 20:57:23 +0800 Subject: [PATCH 8/9] =?UTF-8?q?code=20review=EF=BC=9A=E3=80=90=E5=B7=A5?= =?UTF-8?q?=E4=BD=9C=E6=B5=81=E3=80=91=E2=80=94=E2=80=94=20=E5=8A=A0?= =?UTF-8?q?=E7=AD=BE=E7=9A=84=E5=AE=9E=E7=8E=B0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../common/enums/SymbolConstant.java | 1 + .../module/bpm/enums/ErrorCodeConstants.java | 1 + .../bpm/enums/task/BpmCommentTypeEnum.java | 2 + .../task/BpmProcessInstanceResultEnum.java | 17 +- .../enums/task/BpmTaskAddSignTypeEnum.java | 1 + .../admin/task/BpmTaskController.java | 12 +- .../task/vo/task/BpmTaskAddSignReqVO.java | 4 +- .../bpm/convert/task/BpmTaskConvert.java | 1 + .../bpm/dal/mysql/task/BpmTaskExtMapper.java | 5 +- .../bpm/service/task/BpmTaskService.java | 14 +- .../bpm/service/task/BpmTaskServiceImpl.java | 157 +++++++++++------- 11 files changed, 133 insertions(+), 82 deletions(-) diff --git a/yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java b/yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java index a73ff7d30..48e380219 100644 --- a/yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java +++ b/yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java @@ -1,5 +1,6 @@ package cn.iocoder.yudao.framework.common.enums; +// TODO 这种简单的，暂时不用枚举哈，直接代码里写死就好啦； /** * 符号常量 */ diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java index 76c91a255..51eebda23 100644 --- a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/ErrorCodeConstants.java @@ -54,6 +54,7 @@ public interface ErrorCodeConstants { ErrorCode TASK_ADD_SIGN_TYPE_ERROR = new ErrorCode(1_009_005_010, "任务加签：当前任务已经{}，不能{}"); ErrorCode TASK_ADD_SIGN_USER_REPEAT = new ErrorCode(1_009_005_011, "任务加签失败，加签人与现有审批人[{}]重复"); ErrorCode TASK_SUB_SIGN_NO_PARENT = new ErrorCode(1_009_005_011, "任务减签失败，被减签的任务必须是通过加签生成的任务"); + // ========== 流程任务分配规则 1-009-006-000 ========== ErrorCode TASK_ASSIGN_RULE_EXISTS = new ErrorCode(1_009_006_000, "流程({}) 的任务({}) 已经存在分配规则"); ErrorCode TASK_ASSIGN_RULE_NOT_EXISTS = new ErrorCode(1_009_006_001, "流程任务分配规则不存在"); diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java index 98e175e3f..b4809b83f 100644 --- a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java @@ -14,6 +14,7 @@ public enum BpmCommentTypeEnum { REJECT(2, "不通过"), CANCEL(3, "已取消"), + // TODO @海：18 行可以去掉哈；这个是之前为了 status 隔离用的； // ========== 流程任务独有的状态 ========== BACK(4, "退回"), // 退回 @@ -22,6 +23,7 @@ public enum BpmCommentTypeEnum { SUB_SIGN(7,"减签"), ; + // TODO @海：字段叫 type 更合适噢 /** * 结果 */ diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java index 0fbb5d58f..aca732ba9 100644 --- a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java @@ -26,20 +26,20 @@ public enum BpmProcessInstanceResultEnum { BACK(5, "驳回"), // 退回 DELEGATE(6, "委派"), /** - * 源任务已经审批完成，但是它使用了后加签，后加签的任务未完成，源任务就会是这个状态 - * 例如：A审批， A 后加签了 B ，并且审批通过了任务，但是 B 还未审批，则当前任务状态为 待后加签任务完成 + * 【加签】源任务已经审批完成，但是它使用了后加签，后加签的任务未完成，源任务就会是这个状态 + * 例如：A审批， A 后加签了 B，并且审批通过了任务，但是 B 还未审批，则当前任务状态为“待后加签任务完成” */ - ADD_SIGN_AFTER(7, "待后加签任务完成"), + ADD_SIGN_AFTER(7, "待后加签任务完成"), // TODO @海：这个定义，是不是 通过（待后加签任务完成），相当于是 APPROVE 的特殊状态 /** - * 源任务未审批，但是向前加签了，所以源任务状态变为 待前加签任务完成 - * 例如：A审批， A 前加签了 B ，B 还未审核 + * 【加签】源任务未审批，但是向前加签了，所以源任务状态变为“待前加签任务完成” + * 例如：A 审批， A 前加签了 B，B 还未审核 */ - ADD_SIGN_BEFORE(8, "待前加签任务完成"), + ADD_SIGN_BEFORE(8, "待前加签任务完成"), // TODO @海：这个定义，是不是 处理中（待前加签任务审批），相当于是 PROCESS 的特殊状态 /** - * 后加签任务被创建时的初始状态 + * 【加签】后加签任务被创建时的初始状态 * 因为需要源任务先完成，才能到后加签的人来审批，所以加了一个状态区分 */ - WAIT_BEFORE_TASK(9, "待前置任务完成"); + WAIT_BEFORE_TASK(9, "待前置任务完成"); // TODO @海：这个定义，是不是 处理中（待前置任务审批），相当于是 PROCESS 的特殊状态 /** * 结果 @@ -66,6 +66,7 @@ public enum BpmProcessInstanceResultEnum { ADD_SIGN_AFTER.getResult()); } + // TODO @海：静态变量，需要放到成员变量前面；另外，如果是复数，可以加 S(ES) 或者 LIST /** * 能被减签的状态 */ diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmTaskAddSignTypeEnum.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmTaskAddSignTypeEnum.java index 3b46b6f25..3bf41a153 100644 --- a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmTaskAddSignTypeEnum.java +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmTaskAddSignTypeEnum.java @@ -36,5 +36,6 @@ public enum BpmTaskAddSignTypeEnum { } return null; } + } \ No newline at end of file diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java index 2f0e2db2a..694b35da4 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java @@ -100,23 +100,29 @@ public class BpmTaskController { return success(true); } + // TODO @海：权限统一使用 bpm:task:update；是否可以加减签，可以交给后续的权限配置实现； @PutMapping("/add-sign") - @Operation(summary = "加签", description = "before, after为前加签后加签") + @Operation(summary = "加签", description = "before 前加签，after 后加签") @PreAuthorize("@ss.hasPermission('bpm:task:add-sign')") public CommonResult addSign(@Valid @RequestBody BpmTaskAddSignReqVO reqVO) { + // TODO @海：userId 建议作为第一个参数；一般是，谁做了什么操作；另外，addSignTask，保持风格统一哈； taskService.addSign(reqVO,getLoginUserId()); return success(true); } + // TODO @海：权限统一使用 bpm:task:update；是否可以加减签，可以交给后续的权限配置实现； @PutMapping("/sub-sign") - @Operation(summary = "减签", description = "") + @Operation(summary = "减签") @PreAuthorize("@ss.hasPermission('bpm:task:sub-sign')") + // TODO @海： @RequestBody BpmTaskSubSignReqVO 应该是一个空格；然后参数名可以简写成 reqVO； public CommonResult subSign(@Valid @RequestBody BpmTaskSubSignReqVO bpmTaskSubSignReqVO) { taskService.subSign(bpmTaskSubSignReqVO,getLoginUserId()); return success(true); } + + // TODO @海：是不是 url 和方法名，叫 get-child-task-list，更抽象和复用一些？ @GetMapping("/get-sub-sign") - @Operation(summary = "获取能被减签的任务", description = "") + @Operation(summary = "获取能被减签的任务") @PreAuthorize("@ss.hasPermission('bpm:task:sub-sign')") public CommonResult> getChildrenTaskList(@RequestParam("taskId") String taskId) { return success(taskService.getChildrenTaskList(taskId)); diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java index 88b123ce8..f4b9ac0ac 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java @@ -12,7 +12,7 @@ import java.util.Set; public class BpmTaskAddSignReqVO { @Schema(description = "加签的用户 ID") - @NotEmpty(message = "加签用户ID不能为空") + @NotEmpty(message = "加签用户 ID 不能为空") private Set userIdList; @Schema(description = "加签类型，before 向前加签，after 向后加签") @@ -23,7 +23,9 @@ public class BpmTaskAddSignReqVO { @NotEmpty(message = "加签原因不能为空") private String reason; + // TODO @海：重要参数，可以放到最前面哈； @Schema(description = "需要加签的任务 ID") @NotEmpty(message = "任务编号不能为空") private String id; + } \ No newline at end of file diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java index 05fcb1df4..9a0006e8d 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java @@ -147,6 +147,7 @@ public interface BpmTaskConvert { .setDefinitionKey(element.getId())); } + // TODO @海：可以使用 mapstruct 映射么？ default TaskEntityImpl convert(TaskEntityImpl task,TaskEntityImpl parentTask){ task.setCategory(parentTask.getCategory()); task.setDescription(parentTask.getDescription()); diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java index 9b0fe8bb1..a9031b3b6 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java @@ -32,7 +32,8 @@ public interface BpmTaskExtMapper extends BaseMapperX { return selectOne(BpmTaskExtDO::getTaskId, taskId); } - default void updateBatchByTaskIdList(List taskIdList, BpmTaskExtDO entity){ - update(entity,new LambdaQueryWrapper().in(BpmTaskExtDO::getTaskId, taskIdList)); + default void updateBatchByTaskIdList(List taskIdList, BpmTaskExtDO updateObj) { + update(updateObj, new LambdaQueryWrapper().in(BpmTaskExtDO::getTaskId, taskIdList)); } + } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java index 3a7f84c07..229e68d23 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java @@ -65,8 +65,8 @@ public interface BpmTaskService { /** - * 通过任务 ID 集合，获取任务扩展表信息集合 - * @param taskIdList 任务 ID + * 通过任务 ID 集合，获取任务扩展表信息集合 // TODO @海洋：方法注释，和下面的参数，需要空一行 + * @param taskIdList 任务 ID 集合 * @return 任务列表 */ List getTaskListByTaskIdList(List taskIdList); @@ -158,22 +158,26 @@ public interface BpmTaskService { /** * 任务加签 - * @param reqVO 被加签的用户和任务ID，加签类型 + * + * @param reqVO 被加签的用户和任务 ID，加签类型 * @param userId 当前用户 ID */ void addSign(BpmTaskAddSignReqVO reqVO, Long userId); /** - * 减签 - * @param bpmTaskSubSignReqVO 被减签的任务ID，理由 + * 任务减签名 + * + * @param bpmTaskSubSignReqVO 被减签的任务 ID，理由 * @param loginUserId 当前用户ID */ void subSign(BpmTaskSubSignReqVO bpmTaskSubSignReqVO, Long loginUserId); /** * 获取指定任务的子任务和审批人信息 + * * @param taskId 指定任务ID * @return 子任务列表 */ List getChildrenTaskList(String taskId); + } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java index b99b0c73d..371c30d84 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java @@ -218,12 +218,15 @@ public class BpmTaskServiceImpl implements BpmTaskService { approveDelegateTask(reqVO, task); return; } + // 情况二：后加签的任务 - if(BpmTaskAddSignTypeEnum.AFTER.getType().equals(task.getScopeType())) { - //后加签处理 + if (BpmTaskAddSignTypeEnum.AFTER.getType().equals(task.getScopeType())) { + // 后加签处理 + // TODO @海洋：这个是不是 approveAfterSignTask handleAfterSignTask(task, reqVO); return; } + // 情况三：自己审批的任务，调用 complete 去完成任务 // 完成任务，审批通过 taskService.complete(task.getId(), instance.getProcessVariables()); @@ -231,31 +234,34 @@ public class BpmTaskServiceImpl implements BpmTaskService { taskExtMapper.updateByTaskId( new BpmTaskExtDO().setTaskId(task.getId()).setResult(BpmProcessInstanceResultEnum.APPROVE.getResult()) .setReason(reqVO.getReason())); - //处理加签任务 - this.handleParentTask(task); + // 处理加签任务 + handleParentTask(task); } /** - * 当前任务存在后加签任务，则不能马上完成，需要一个中间状态，并激活剩余所有子任务 + * 审批通过存在“后加签”的任务。 + * + * 注意：该任务不能马上完成，需要一个中间状态（ADD_SIGN_AFTER），并激活剩余所有子任务（PROCESS）为可审批处理 + * * @param task 当前任务 * @param reqVO 前端请求参数 */ private void handleAfterSignTask(Task task,BpmTaskApproveReqVO reqVO){ - //1.有向后加签，则该任务状态临时设置为 等待后加签任务完成 状态 + // 1. 有向后加签，则该任务状态临时设置为 ADD_SIGN_AFTER 状态 taskExtMapper.updateByTaskId( new BpmTaskExtDO().setTaskId(task.getId()).setResult(BpmProcessInstanceResultEnum.ADD_SIGN_AFTER.getResult()) .setReason(reqVO.getReason()).setEndTime(LocalDateTime.now())); - //2. 激活子任务 + + // 2. 激活子任务 List childrenTaskIdList = getChildrenTaskIdList(task.getId()); for (String childrenTaskId : childrenTaskIdList) { taskService.resolveTask(childrenTaskId); } - //更新任务扩展表中子任务为进行中 + // 更新任务扩展表中子任务为进行中 taskExtMapper.updateBatchByTaskIdList(childrenTaskIdList, new BpmTaskExtDO().setResult(BpmProcessInstanceResultEnum.PROCESS.getResult())); } - /** * 处理当前任务的父任务 * @@ -263,62 +269,70 @@ public class BpmTaskServiceImpl implements BpmTaskService { */ private void handleParentTask(Task task) { String parentTaskId = task.getParentTaskId(); + // TODO @ 海：if return 原则 if (StrUtil.isNotBlank(parentTaskId)) { - //1. 判断当前任务的父任务是否还有子任务 + // 1. 判断当前任务的父任务是否还有子任务 long subTaskCount = getSubTaskCount(parentTaskId); + // TODO @ 海：if >= 0 return；这样括号又可以少一层； if (subTaskCount == 0) { - //2. 获取父任务 + // 2. 获取父任务 Task parentTask = validateTaskExist(parentTaskId); + + // 3. 情况一：处理向前加签 String scopeType = parentTask.getScopeType(); - //3. 处理向前加签 if (BpmTaskAddSignTypeEnum.BEFORE.getType().equals(scopeType)) { - //3.1 如果是向前加签的任务，则调用 resolveTask 指派父任务，将 owner 重新赋值给父任务的 assignee + // 3.1 如果是向前加签的任务，则调用 resolveTask 指派父任务，将 owner 重新赋值给父任务的 assignee taskService.resolveTask(parentTaskId); - //3.2 更新任务拓展表为处理中 + // 3.2 更新任务拓展表为处理中 taskExtMapper.updateByTaskId( new BpmTaskExtDO().setTaskId(parentTask.getId()).setResult(BpmProcessInstanceResultEnum.PROCESS.getResult())); - }else if (BpmTaskAddSignTypeEnum.AFTER.getType().equals(scopeType)) { - //4. 处理向后加签 + } else if (BpmTaskAddSignTypeEnum.AFTER.getType().equals(scopeType)) { + // 3. 情况二：处理向后加签 handleAfterSign(parentTask); } - //5. 子任务已处理完成，清空 scopeType 字段，修改 parentTask 信息，方便后续可以继续向前后向后加签 + + // 4. 子任务已处理完成，清空 scopeType 字段，修改 parentTask 信息，方便后续可以继续向前后向后加签 // 再查询一次的原因是避免报错：Task was updated by another transaction concurrently // 因为前面处理后可能会导致 parentTask rev 字段被修改，需要重新获取最新的 parentTask = getTask(parentTaskId); - if(parentTask !=null){ - //为空的情况是：已经通过 handleAfterSign 方法将任务完成了，所以 ru_task 表会查不到数据 + // TODO @ 海：if return 原则； + if (parentTask != null) { + // 为空的情况是：已经通过 handleAfterSign 方法将任务完成了，所以 ru_task 表会查不到数据 clearTaskScopeTypeAndSave(parentTask); } - } } } + // TODO @海：这个方法的 4.1 从 1 开始计数哈；另外；看看能不能用 if return 进一步简化代码的层级； /** * 处理后加签任务 * * @param parentTask 当前审批任务的父任务 */ private void handleAfterSign(Task parentTask) { + // TODO @海：这个方法在注释下。感觉整体是，先完成自己；然后递归处理父节点；但是完成机子被放到了 5.1，就感觉上有点割裂； String parentTaskId = parentTask.getId(); //4.1 更新 parentTask 的任务拓展表为通过 BpmTaskExtDO currentTaskExt = taskExtMapper.selectByTaskId(parentTask.getId()); BpmTaskExtDO currentTaskUpdateEntity = new BpmTaskExtDO().setTaskId(parentTask.getId()) .setResult(BpmProcessInstanceResultEnum.APPROVE.getResult()); if (currentTaskExt.getEndTime() == null) { - //有这个判断是因为 - //4.2 以前没设置过结束时间，才去设置 + // 有这个判断是因为 + // 4.2 以前没设置过结束时间，才去设置 currentTaskUpdateEntity.setEndTime(LocalDateTime.now()); } taskExtMapper.updateByTaskId(currentTaskUpdateEntity); - //5. 继续往上处理，父任务继续往上查询 - //5.1 先完成自己 + + // 5. 继续往上处理，父任务继续往上查询 + // 5.1 先完成自己 taskService.complete(parentTaskId); + // 5.2 如果有父级，递归查询上级任务是否都已经完成 + // TODO @海：这块待讨论，脑子略乱；感觉 handleAfterSign 的后半段，和 handleParentTask 有点重叠？？？ if (StrUtil.isNotEmpty(parentTask.getParentTaskId())) { - //5.2 如果有父级，递归查询上级任务是否都已经完成 - //判断整条链路的任务是否完成 - //例如从 A 任务加签了一个 B 任务， B 任务又加签了一个 C 任务，C任务加签了 D 任务 - //此时，D 任务完成，要一直往上找到祖先任务 A ，调用 complete 方法完成 A 任务 + // 判断整条链路的任务是否完成 + // 例如从 A 任务加签了一个 B 任务，B 任务又加签了一个 C 任务，C 任务加签了 D 任务 + // 此时，D 任务完成，要一直往上找到祖先任务 A调用 complete 方法完成 A 任务 boolean allChildrenTaskFinish = true; while (StrUtil.isNotBlank(parentTask.getParentTaskId())) { parentTask = validateTaskExist(parentTask.getParentTaskId()); @@ -327,23 +341,23 @@ public class BpmTaskServiceImpl implements BpmTaskService { break; } boolean currentTaskFinish = BpmProcessInstanceResultEnum.isEndResult(bpmTaskExtDO.getResult()); - //5.3 如果 allChildrenTaskFinish 已经被赋值为 false ,则不会再赋值为 true，因为整个链路没有完成 + // 5.3 如果 allChildrenTaskFinish 已经被赋值为 false ,则不会再赋值为 true，因为整个链路没有完成 if (allChildrenTaskFinish) { allChildrenTaskFinish = currentTaskFinish; } if (!currentTaskFinish) { - //6 处理非完成状态的任务 - //6.1 判断当前任务的父任务是否还有子任务 + // 6 处理非完成状态的任务 + // 6.1 判断当前任务的父任务是否还有子任务 Long subTaskCount = getSubTaskCount(bpmTaskExtDO.getTaskId()); if (subTaskCount == 0) { - //6.2 没有子任务，判断当前任务状态是否为 ADD_SIGN_BEFORE 待前加签任务完成 + // 6.2 没有子任务，判断当前任务状态是否为 ADD_SIGN_BEFORE 待前加签任务完成 if (BpmProcessInstanceResultEnum.ADD_SIGN_BEFORE.getResult().equals(bpmTaskExtDO.getResult())) { - //6.3 需要修改该任务状态为处理中 + // 6.3 需要修改该任务状态为处理中 taskService.resolveTask(bpmTaskExtDO.getTaskId()); bpmTaskExtDO.setResult(BpmProcessInstanceResultEnum.PROCESS.getResult()); taskExtMapper.updateByTaskId(bpmTaskExtDO); } - //6.4 清空 scopeType 字段，用于任务没有子任务时使用该方法，方便任务可以再次被不同的方式加签 + // 6.4 清空 scopeType 字段，用于任务没有子任务时使用该方法，方便任务可以再次被不同的方式加签 parentTask = getTask(bpmTaskExtDO.getTaskId()); if (parentTask != null) { clearTaskScopeTypeAndSave(parentTask); @@ -352,7 +366,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { } } if (allChildrenTaskFinish) { - //7. 完成最后的顶级祖先任务 + // 7. 完成最后的顶级祖先任务 taskService.complete(parentTask.getId()); } } @@ -369,15 +383,16 @@ public class BpmTaskServiceImpl implements BpmTaskService { taskService.saveTask(task); } + // TODO @海：Sub 还有 Child 感觉整体用词不是很统一；是不是要统一下；然后 Sub 还可以指的减签；看看是不是名词确实得一致哈； /** * 获取子任务个数 * - * @param parentTaskId 父类ID + * @param parentTaskId 父任务 ID * @return 剩余子任务个数 */ private Long getSubTaskCount(String parentTaskId) { String tableName = managementService.getTableName(TaskEntity.class); - String sql = "select count(1) from " + tableName + " where PARENT_TASK_ID_=#{parentTaskId}"; + String sql = "SELECT COUNT(1) from " + tableName + " WHERE PARENT_TASK_ID_=#{parentTaskId}"; return taskService.createNativeTaskQuery().sql(sql).parameter("parentTaskId", parentTaskId).count(); } @@ -454,10 +469,10 @@ public class BpmTaskServiceImpl implements BpmTaskService { @Override public void createTaskExt(Task task) { - BpmTaskExtDO taskExtDO = - BpmTaskConvert.INSTANCE.convert2TaskExt(task).setResult(BpmProcessInstanceResultEnum.PROCESS.getResult()); - //向后加签生成的任务，状态不能为进行中，需要等前面父任务完成 - if(BpmTaskAddSignTypeEnum.AFTER_CHILDREN_TASK.getType().equals(task.getScopeType())){ + BpmTaskExtDO taskExtDO = BpmTaskConvert.INSTANCE.convert2TaskExt(task) + .setResult(BpmProcessInstanceResultEnum.PROCESS.getResult()); + // 向后加签生成的任务，状态不能为进行中，需要等前面父任务完成 + if (BpmTaskAddSignTypeEnum.AFTER_CHILDREN_TASK.getType().equals(task.getScopeType())) { taskExtDO.setResult(BpmProcessInstanceResultEnum.WAIT_BEFORE_TASK.getResult()); } taskExtMapper.insert(taskExtDO); @@ -526,7 +541,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { private Task validateTaskExist(String id){ Task task = taskService.createTaskQuery().taskId(id).singleResult(); - if(task == null){ + if (task == null) { throw exception(TASK_NOT_EXISTS); } return task; @@ -691,37 +706,43 @@ public class BpmTaskServiceImpl implements BpmTaskService { @Override @Transactional(rollbackFor = Exception.class) public void addSign(BpmTaskAddSignReqVO reqVO, Long userId) { + // TODO @海：// 后面要有个空格；中英文之间，也要有空格；例如说：// 1. 获取和校验任务 //1.获取和校验任务 TaskEntityImpl taskEntity = validateAddSign(userId, reqVO); + // TODO @海：每个变量，以及相关逻辑，一定要和自己的逻辑块呆在一起。例如说这里的 currentUser 获取，应该在 4. 那块 AdminUserRespDTO currentUser = adminUserApi.getUser(userId); List userList = adminUserApi.getUserList(reqVO.getUserIdList()); if (CollUtil.isEmpty(userList)) { throw exception(TASK_ADD_SIGN_USER_NOT_EXIST); } - //2.处理当前任务 - //2.1 开启计数功能 + // TODO @海：大的逻辑块之间，最好有一个空格。这样的目的，是避免逻辑堆砌在一起，方便流量； + // 2.处理当前任务 + // 2.1 开启计数功能 // TODO @海：这个目的可以写下； taskEntity.setCountEnabled(true); + // TODO @海：可以直接 if，不搞一个变量哈 boolean addSignToBefore = reqVO.getType().equals(BpmTaskAddSignTypeEnum.BEFORE.getType()); if (addSignToBefore) { - //2.2 向前加签，设置 owner ，置空 assign ,等子任务都完成后，再调用 resolveTask 重新将 owner 设置为 assign - //原因是：不能和向前加签的子任务一起审批，需要等前面的子任务都完成才能审批 + // 2.2 向前加签，设置 owner，置空 assign。等子任务都完成后，再调用 resolveTask 重新将 owner 设置为 assign + // 原因是：不能和向前加签的子任务一起审批，需要等前面的子任务都完成才能审批 taskEntity.setOwner(taskEntity.getAssignee()); taskEntity.setAssignee(null); - //2.3 更新扩展表状态 + // 2.3 更新扩展表状态 taskExtMapper.updateByTaskId( new BpmTaskExtDO().setTaskId(taskEntity.getId()) .setResult(BpmProcessInstanceResultEnum.ADD_SIGN_BEFORE.getResult()) .setReason(reqVO.getReason())); } - //2.4 记录加签方式，完成任务时需要用到判断 + // 2.4 记录加签方式，完成任务时需要用到判断 taskEntity.setScopeType(reqVO.getType()); - //2.5 保存当前任务修改后的值 + // 2.5 保存当前任务修改后的值 taskService.saveTask(taskEntity); - //3. 处理加签任务 - this.createSignSubTasks(convertList(reqVO.getUserIdList(), String::valueOf), taskEntity); - //4. 记录加签 comment，拼接结果为： [当前用户]向前加签/向后加签给了[多个用户],理由为：reason - String comment = StrUtil.format("[{}]{}给了[{}],理由为：{}", currentUser.getNickname(), BpmTaskAddSignTypeEnum.formatDesc(reqVO.getType()), + // 3. 创建加签任务 + createSignSubTasks(convertList(reqVO.getUserIdList(), String::valueOf), taskEntity); + + // 4. 记录加签 comment，拼接结果为： [当前用户]向前加签/向后加签给了[多个用户]，理由为：reason + // TODO @海：BpmCommentTypeEnum 可以加一个 comment 字段，作为评论模版，统一管理； + String comment = StrUtil.format("[{}]{}给了[{}]，理由为：{}", currentUser.getNickname(), BpmTaskAddSignTypeEnum.formatDesc(reqVO.getType()), String.join(SymbolConstant.D, convertList(userList, AdminUserRespDTO::getNickname)), reqVO.getReason()); taskService.addComment(reqVO.getId(), taskEntity.getProcessInstanceId(), BpmCommentTypeEnum.ADD_SIGN.getResult().toString(), comment); @@ -729,7 +750,10 @@ public class BpmTaskServiceImpl implements BpmTaskService { /** - * 校验任务存在和 存在 向前加签 的任务，则不能 向后加签，存在 向后加签 则不能 向前加签 + * 校验任务的加签是否一致 + * + * 1. 如果存在“向前加签”的任务，则不能“向后加签” + * 2. 如果存在“向后加签”的任务，则不能“向前加签” * * @param userId 当前用户 ID * @param reqVO 请求参数，包含任务 ID 和加签类型 @@ -737,18 +761,22 @@ public class BpmTaskServiceImpl implements BpmTaskService { */ private TaskEntityImpl validateAddSign(Long userId, BpmTaskAddSignReqVO reqVO) { TaskEntityImpl taskEntity = (TaskEntityImpl) validateTask(userId, reqVO.getId()); - //向前加签和向后加签不能同时存在 + // 向前加签和向后加签不能同时存在 if (StrUtil.isNotBlank(taskEntity.getScopeType()) && !BpmTaskAddSignTypeEnum.AFTER_CHILDREN_TASK.getDesc().equals(taskEntity.getScopeType())) { + // TODO @海：下面这个判断，是不是可以写上面这个判断后面？ + // TODO @海：一个小技巧，如果写不等于的时候，一般可以用 ObjectUtil.notEquals，这样少一层取反，理解起来简单点； if (!taskEntity.getScopeType().equals(reqVO.getType())) { throw exception(TASK_ADD_SIGN_TYPE_ERROR, BpmTaskAddSignTypeEnum.formatDesc(taskEntity.getScopeType()), BpmTaskAddSignTypeEnum.formatDesc(reqVO.getType())); } } - //同一个 key 的任务，审批人不重复 + // 同一个 key 的任务，审批人不重复 List taskList = taskService.createTaskQuery().processInstanceId(taskEntity.getProcessInstanceId()) .taskDefinitionKey(taskEntity.getTaskDefinitionKey()).list(); + // TODO @海：这里是不是 Task::getAssignee 解析成 List 下面会更简洁一点？ List currentAssigneeList = convertList(taskList, Task::getAssignee); + // TODO @海：frontAssigneeList 改成 addAssigneeList，新增的；避免和 front 前端这样界面耦合的名词哈； List frontAssigneeList = convertList(reqVO.getUserIdList(), String::valueOf); currentAssigneeList.retainAll(frontAssigneeList); if (CollUtil.isNotEmpty(currentAssigneeList)) { @@ -761,20 +789,22 @@ public class BpmTaskServiceImpl implements BpmTaskService { /** * 创建加签子任务 * - * @param addSingUserIdList 被加签的用户ID + * @param addSingUserIdList 被加签的用户 ID * @param taskEntity 被加签的任务 */ private void createSignSubTasks(List addSingUserIdList, TaskEntityImpl taskEntity) { + // TODO @海：可以 if return；这样括号层级少一点；下面的 if (StrUtil.isNotBlank(addSignId)) { 也是类似； if (CollUtil.isNotEmpty(addSingUserIdList)) { - // 创建加签人的新任务,全部基于 taskEntity 为父任务来创建 + // 创建加签人的新任务，全部基于 taskEntity 为父任务来创建 addSingUserIdList.forEach(addSignId -> { if (StrUtil.isNotBlank(addSignId)) { - this.createSubTask(taskEntity, addSignId); + createSubTask(taskEntity, addSignId); } }); } } + // TODO @海：这个是不是命名上，要和 createSignSubTasks 保持一致？ /** * 创建子任务 * @@ -783,18 +813,19 @@ public class BpmTaskServiceImpl implements BpmTaskService { * @return */ private void createSubTask(TaskEntityImpl parentTask, String assignee) { - //1.生成子任务 + // 1. 生成子任务 TaskEntityImpl task = (TaskEntityImpl) taskService.newTask(IdUtil.fastSimpleUUID()); task = BpmTaskConvert.INSTANCE.convert(task,parentTask); if (BpmTaskAddSignTypeEnum.BEFORE.getType().equals(parentTask.getScopeType())) { - // 2. 前加签，才设置审批人，否则设置 owner + // 2.1 前加签，才设置审批人，否则设置 owner task.setAssignee(assignee); } else { - //2.2 设置 owner 不设置 assignee 是因为不能同时审批，需要等父任务完成 + // 2.2.1 设置 owner 不设置 assignee 是因为不能同时审批，需要等父任务完成 task.setOwner(assignee); - //2.3 设置向后加签任务的 scopeType 为 afterChildrenTask，用于设置任务扩展表的状态 + // 2.2.2 设置向后加签任务的 scopeType 为 afterChildrenTask，用于设置任务扩展表的状态 task.setScopeType(BpmTaskAddSignTypeEnum.AFTER_CHILDREN_TASK.getType()); } + // 2. 保存子任务 taskService.saveTask(task); } From bc781d947368535d0775d6d22823b255ceb62847 Mon Sep 17 00:00:00 2001 From: youkehai Date: Tue, 10 Oct 2023 10:46:16 +0800 Subject: [PATCH 9/9] =?UTF-8?q?fix:=20=E3=80=90=E5=B7=A5=E4=BD=9C=E6=B5=81?= =?UTF-8?q?=E3=80=91=E6=8C=89=20review=20=E4=BF=AE=E6=94=B9=E4=BB=A3?= =?UTF-8?q?=E7=A0=81?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../common/enums/SymbolConstant.java | 49 --- .../bpm/enums/task/BpmCommentTypeEnum.java | 26 +- .../task/BpmProcessInstanceResultEnum.java | 20 +- .../admin/task/BpmTaskController.java | 29 +- .../task/vo/task/BpmTaskAddSignReqVO.java | 10 +- .../bpm/convert/task/BpmTaskConvert.java | 15 +- .../bpm/dal/mysql/task/BpmTaskExtMapper.java | 2 +- .../bpm/service/task/BpmTaskService.java | 23 +- .../bpm/service/task/BpmTaskServiceImpl.java | 302 +++++++++--------- 9 files changed, 210 insertions(+), 266 deletions(-) delete mode 100644 yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java diff --git a/yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java b/yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java deleted file mode 100644 index 48e380219..000000000 --- a/yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/enums/SymbolConstant.java +++ /dev/null @@ -1,49 +0,0 @@ -package cn.iocoder.yudao.framework.common.enums; - -// TODO 这种简单的，暂时不用枚举哈，直接代码里写死就好啦； -/** - * 符号常量 - */ -public interface SymbolConstant { - - String D =","; - - /** - * _ - */ - String XH="_"; - /** - * - - */ - String HG="-"; - - /** - * / - */ - String XG="/"; - - /** - * 箭头 - */ - String ARROWHEAD="->"; - - /** - * 数组的开始元素 - */ - String ARRAY_START="["; - - /** - * 数组的结束元素 - */ - String ARRAY_END="]"; - - /** - * null 字符串 - */ - String NULL_STRING="null"; - - /** - * 点号 - */ - String DIAN="\\."; -} diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java index b4809b83f..67a4a222e 100644 --- a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmCommentTypeEnum.java @@ -10,26 +10,26 @@ import lombok.Getter; @AllArgsConstructor public enum BpmCommentTypeEnum { - APPROVE(1, "通过"), - REJECT(2, "不通过"), - CANCEL(3, "已取消"), - - // TODO @海：18 行可以去掉哈；这个是之前为了 status 隔离用的； - // ========== 流程任务独有的状态 ========== - - BACK(4, "退回"), // 退回 - DELEGATE(5, "委派"), - ADD_SIGN(6, "加签"), - SUB_SIGN(7,"减签"), + APPROVE(1, "通过", ""), + REJECT(2, "不通过", ""), + CANCEL(3, "已取消", ""), + BACK(4, "退回", ""), + DELEGATE(5, "委派", ""), + ADD_SIGN(6, "加签", "[{}]{}给了[{}]，理由为：{}"), + SUB_SIGN(7, "减签", "[{}]操作了【减签】,审批人[{}]的任务被取消"), ; - // TODO @海：字段叫 type 更合适噢 /** * 结果 */ - private final Integer result; + private final Integer type; /** * 描述 */ private final String desc; + /** + * 模板信息 + */ + private final String templateComment; + } diff --git a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java index aca732ba9..519c7c68c 100644 --- a/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java +++ b/yudao-module-bpm/yudao-module-bpm-api/src/main/java/cn/iocoder/yudao/module/bpm/enums/task/BpmProcessInstanceResultEnum.java @@ -27,19 +27,27 @@ public enum BpmProcessInstanceResultEnum { DELEGATE(6, "委派"), /** * 【加签】源任务已经审批完成，但是它使用了后加签，后加签的任务未完成，源任务就会是这个状态 + * 相当于是 通过 APPROVE 的特殊状态 * 例如：A审批， A 后加签了 B，并且审批通过了任务，但是 B 还未审批，则当前任务状态为“待后加签任务完成” */ - ADD_SIGN_AFTER(7, "待后加签任务完成"), // TODO @海：这个定义，是不是 通过（待后加签任务完成），相当于是 APPROVE 的特殊状态 + ADD_SIGN_AFTER(7, "待后加签任务完成"), /** * 【加签】源任务未审批，但是向前加签了，所以源任务状态变为“待前加签任务完成” + * 相当于是 处理中 PROCESS 的特殊状态 * 例如：A 审批， A 前加签了 B，B 还未审核 */ - ADD_SIGN_BEFORE(8, "待前加签任务完成"), // TODO @海：这个定义，是不是 处理中（待前加签任务审批），相当于是 PROCESS 的特殊状态 + ADD_SIGN_BEFORE(8, "待前加签任务完成"), /** * 【加签】后加签任务被创建时的初始状态 + * 相当于是 处理中 PROCESS 的特殊状态 * 因为需要源任务先完成，才能到后加签的人来审批，所以加了一个状态区分 */ - WAIT_BEFORE_TASK(9, "待前置任务完成"); // TODO @海：这个定义，是不是 处理中（待前置任务审批），相当于是 PROCESS 的特殊状态 + WAIT_BEFORE_TASK(9, "待前置任务完成"); + + /** + * 能被减签的状态 + */ + public static final List CAN_SUB_SIGN_STATUS_LIST = Arrays.asList(PROCESS.result, WAIT_BEFORE_TASK.result); /** * 结果 @@ -66,10 +74,4 @@ public enum BpmProcessInstanceResultEnum { ADD_SIGN_AFTER.getResult()); } - // TODO @海：静态变量，需要放到成员变量前面；另外，如果是复数，可以加 S(ES) 或者 LIST - /** - * 能被减签的状态 - */ - public static final List CAN_SUB_SIGN_STATUS = Arrays.asList(PROCESS.result, WAIT_BEFORE_TASK.result); - } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java index 694b35da4..a67af2eeb 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/BpmTaskController.java @@ -4,16 +4,15 @@ import cn.iocoder.yudao.framework.common.pojo.CommonResult; import cn.iocoder.yudao.framework.common.pojo.PageResult; import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task.*; import cn.iocoder.yudao.module.bpm.service.task.BpmTaskService; -import io.swagger.v3.oas.annotations.tags.Tag; -import io.swagger.v3.oas.annotations.Parameter; import io.swagger.v3.oas.annotations.Operation; +import io.swagger.v3.oas.annotations.Parameter; +import io.swagger.v3.oas.annotations.tags.Tag; import org.springframework.security.access.prepost.PreAuthorize; import org.springframework.validation.annotation.Validated; import org.springframework.web.bind.annotation.*; import javax.annotation.Resource; import javax.validation.Valid; - import java.util.List; import static cn.iocoder.yudao.framework.common.pojo.CommonResult.success; @@ -95,35 +94,29 @@ public class BpmTaskController { @Operation(summary = "委派任务", description = "用于【流程详情】的【委派】按钮。和向前【加签】有点像，唯一区别是【委托】没有单独创立任务") @PreAuthorize("@ss.hasPermission('bpm:task:update')") public CommonResult delegateTask(@Valid @RequestBody BpmTaskDelegateReqVO reqVO) { - // TODO @海：, 后面要有空格 - taskService.delegateTask(reqVO,getLoginUserId()); + taskService.delegateTask(getLoginUserId(), reqVO); return success(true); } - // TODO @海：权限统一使用 bpm:task:update；是否可以加减签，可以交给后续的权限配置实现； @PutMapping("/add-sign") @Operation(summary = "加签", description = "before 前加签，after 后加签") - @PreAuthorize("@ss.hasPermission('bpm:task:add-sign')") - public CommonResult addSign(@Valid @RequestBody BpmTaskAddSignReqVO reqVO) { - // TODO @海：userId 建议作为第一个参数；一般是，谁做了什么操作；另外，addSignTask，保持风格统一哈； - taskService.addSign(reqVO,getLoginUserId()); + @PreAuthorize("@ss.hasPermission('bpm:task:update')") + public CommonResult addSignTask(@Valid @RequestBody BpmTaskAddSignReqVO reqVO) { + taskService.addSignTask(getLoginUserId(), reqVO); return success(true); } - // TODO @海：权限统一使用 bpm:task:update；是否可以加减签，可以交给后续的权限配置实现； @PutMapping("/sub-sign") @Operation(summary = "减签") - @PreAuthorize("@ss.hasPermission('bpm:task:sub-sign')") - // TODO @海： @RequestBody BpmTaskSubSignReqVO 应该是一个空格；然后参数名可以简写成 reqVO； - public CommonResult subSign(@Valid @RequestBody BpmTaskSubSignReqVO bpmTaskSubSignReqVO) { - taskService.subSign(bpmTaskSubSignReqVO,getLoginUserId()); + @PreAuthorize("@ss.hasPermission('bpm:task:update')") + public CommonResult subSignTask(@Valid @RequestBody BpmTaskSubSignReqVO reqVO) { + taskService.subSignTask(getLoginUserId(), reqVO); return success(true); } - // TODO @海：是不是 url 和方法名，叫 get-child-task-list，更抽象和复用一些？ - @GetMapping("/get-sub-sign") + @GetMapping("/get-children-task-list") @Operation(summary = "获取能被减签的任务") - @PreAuthorize("@ss.hasPermission('bpm:task:sub-sign')") + @PreAuthorize("@ss.hasPermission('bpm:task:update')") public CommonResult> getChildrenTaskList(@RequestParam("taskId") String taskId) { return success(taskService.getChildrenTaskList(taskId)); } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java index f4b9ac0ac..93b95d3fd 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/controller/admin/task/vo/task/BpmTaskAddSignReqVO.java @@ -4,13 +4,16 @@ import io.swagger.v3.oas.annotations.media.Schema; import lombok.Data; import javax.validation.constraints.NotEmpty; -import java.util.List; import java.util.Set; @Schema(description = "管理后台 - 加签流程任务的 Request VO") @Data public class BpmTaskAddSignReqVO { + @Schema(description = "需要加签的任务 ID") + @NotEmpty(message = "任务编号不能为空") + private String id; + @Schema(description = "加签的用户 ID") @NotEmpty(message = "加签用户 ID 不能为空") private Set userIdList; @@ -23,9 +26,4 @@ public class BpmTaskAddSignReqVO { @NotEmpty(message = "加签原因不能为空") private String reason; - // TODO @海：重要参数，可以放到最前面哈； - @Schema(description = "需要加签的任务 ID") - @NotEmpty(message = "任务编号不能为空") - private String id; - } \ No newline at end of file diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java index 9a0006e8d..0d55a82eb 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/convert/task/BpmTaskConvert.java @@ -147,7 +147,8 @@ public interface BpmTaskConvert { .setDefinitionKey(element.getId())); } - // TODO @海：可以使用 mapstruct 映射么？ + //此处不用 mapstruct 映射，因为 TaskEntityImpl 还有很多其他属性，这里我们只设置我们需要的 + //使用 mapstruct 会将里面嵌套的各个属性值都设置进去，会出现意想不到的问题 default TaskEntityImpl convert(TaskEntityImpl task,TaskEntityImpl parentTask){ task.setCategory(parentTask.getCategory()); task.setDescription(parentTask.getDescription()); @@ -183,15 +184,15 @@ public interface BpmTaskConvert { /** * 转换任务为父子级 - * @param result - * @return + * @param sourceList 原始数据 + * @return 转换后的父子级数组 */ - default List convertChildrenList(List result){ - List childrenTaskList = CollectionUtils.filterList(result, r -> StrUtil.isNotEmpty(r.getParentTaskId())); + default List convertChildrenList(List sourceList){ + List childrenTaskList = CollectionUtils.filterList(sourceList, r -> StrUtil.isNotEmpty(r.getParentTaskId())); Map> parentChildrenTaskListMap = CollectionUtils.convertMultiMap(childrenTaskList, BpmTaskRespVO::getParentTaskId); - for (BpmTaskRespVO bpmTaskRespVO : result) { + for (BpmTaskRespVO bpmTaskRespVO : sourceList) { bpmTaskRespVO.setChildren(parentChildrenTaskListMap.get(bpmTaskRespVO.getId())); } - return CollectionUtils.filterList(result, r -> StrUtil.isEmpty(r.getParentTaskId())); + return CollectionUtils.filterList(sourceList, r -> StrUtil.isEmpty(r.getParentTaskId())); } } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java index a9031b3b6..852ccb2e0 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmTaskExtMapper.java @@ -24,7 +24,7 @@ public interface BpmTaskExtMapper extends BaseMapperX { default List selectProcessListByTaskIds(Collection taskIds) { return selectList(new LambdaQueryWrapperX() .in(BpmTaskExtDO::getTaskId, taskIds) - .in(BpmTaskExtDO::getResult, BpmProcessInstanceResultEnum.CAN_SUB_SIGN_STATUS)); + .in(BpmTaskExtDO::getResult, BpmProcessInstanceResultEnum.CAN_SUB_SIGN_STATUS_LIST)); } diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java index 229e68d23..772425b66 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskService.java @@ -65,7 +65,8 @@ public interface BpmTaskService { /** - * 通过任务 ID 集合，获取任务扩展表信息集合 // TODO @海洋：方法注释，和下面的参数，需要空一行 + * 通过任务 ID 集合，获取任务扩展表信息集合 + * * @param taskIdList 任务 ID 集合 * @return 任务列表 */ @@ -143,34 +144,34 @@ public interface BpmTaskService { * 将任务回退到指定的 targetDefinitionKey 位置 * * @param userId 用户编号 - * @param reqVO 回退的任务key和当前所在的任务ID + * @param reqVO 回退的任务key和当前所在的任务ID */ void returnTask(Long userId, BpmTaskReturnReqVO reqVO); - // TODO @海：userId 放前面 + /** * 将指定任务委派给其他人处理，等接收人处理后再回到原审批人手中审批 * - * @param reqVO 被委派人和被委派的任务编号理由参数 * @param userId 用户编号 + * @param reqVO 被委派人和被委派的任务编号理由参数 */ - void delegateTask(BpmTaskDelegateReqVO reqVO, Long userId); + void delegateTask(Long userId, BpmTaskDelegateReqVO reqVO); /** * 任务加签 * - * @param reqVO 被加签的用户和任务 ID，加签类型 - * @param userId 当前用户 ID + * @param userId 被加签的用户和任务 ID，加签类型 + * @param reqVO 当前用户 ID */ - void addSign(BpmTaskAddSignReqVO reqVO, Long userId); + void addSignTask(Long userId, BpmTaskAddSignReqVO reqVO); /** * 任务减签名 * - * @param bpmTaskSubSignReqVO 被减签的任务 ID，理由 - * @param loginUserId 当前用户ID + * @param userId 当前用户ID + * @param reqVO 被减签的任务 ID，理由 */ - void subSign(BpmTaskSubSignReqVO bpmTaskSubSignReqVO, Long loginUserId); + void subSignTask(Long userId, BpmTaskSubSignReqVO reqVO); /** * 获取指定任务的子任务和审批人信息 diff --git a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java index 371c30d84..6bf2fb5b8 100644 --- a/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java +++ b/yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmTaskServiceImpl.java @@ -3,8 +3,8 @@ package cn.iocoder.yudao.module.bpm.service.task; import cn.hutool.core.collection.CollUtil; import cn.hutool.core.util.ArrayUtil; import cn.hutool.core.util.IdUtil; +import cn.hutool.core.util.ObjectUtil; import cn.hutool.core.util.StrUtil; -import cn.iocoder.yudao.framework.common.enums.SymbolConstant; import cn.iocoder.yudao.framework.common.pojo.PageResult; import cn.iocoder.yudao.framework.common.util.date.DateUtils; import cn.iocoder.yudao.framework.common.util.number.NumberUtils; @@ -15,7 +15,10 @@ import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.task.*; import cn.iocoder.yudao.module.bpm.convert.task.BpmTaskConvert; import cn.iocoder.yudao.module.bpm.dal.dataobject.task.BpmTaskExtDO; import cn.iocoder.yudao.module.bpm.dal.mysql.task.BpmTaskExtMapper; -import cn.iocoder.yudao.module.bpm.enums.task.*; +import cn.iocoder.yudao.module.bpm.enums.task.BpmCommentTypeEnum; +import cn.iocoder.yudao.module.bpm.enums.task.BpmProcessInstanceDeleteReasonEnum; +import cn.iocoder.yudao.module.bpm.enums.task.BpmProcessInstanceResultEnum; +import cn.iocoder.yudao.module.bpm.enums.task.BpmTaskAddSignTypeEnum; import cn.iocoder.yudao.module.bpm.service.definition.BpmModelService; import cn.iocoder.yudao.module.bpm.service.message.BpmMessageService; import cn.iocoder.yudao.module.system.api.dept.DeptApi; @@ -222,8 +225,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { // 情况二：后加签的任务 if (BpmTaskAddSignTypeEnum.AFTER.getType().equals(task.getScopeType())) { // 后加签处理 - // TODO @海洋：这个是不是 approveAfterSignTask - handleAfterSignTask(task, reqVO); + approveAfterSignTask(task, reqVO); return; } @@ -240,13 +242,13 @@ public class BpmTaskServiceImpl implements BpmTaskService { /** * 审批通过存在“后加签”的任务。 - * + *

* 注意：该任务不能马上完成，需要一个中间状态（ADD_SIGN_AFTER），并激活剩余所有子任务（PROCESS）为可审批处理 * - * @param task 当前任务 + * @param task 当前任务 * @param reqVO 前端请求参数 */ - private void handleAfterSignTask(Task task,BpmTaskApproveReqVO reqVO){ + private void approveAfterSignTask(Task task, BpmTaskApproveReqVO reqVO) { // 1. 有向后加签，则该任务状态临时设置为 ADD_SIGN_AFTER 状态 taskExtMapper.updateByTaskId( new BpmTaskExtDO().setTaskId(task.getId()).setResult(BpmProcessInstanceResultEnum.ADD_SIGN_AFTER.getResult()) @@ -257,7 +259,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { for (String childrenTaskId : childrenTaskIdList) { taskService.resolveTask(childrenTaskId); } - // 更新任务扩展表中子任务为进行中 + // 2.1 更新任务扩展表中子任务为进行中 taskExtMapper.updateBatchByTaskIdList(childrenTaskIdList, new BpmTaskExtDO().setResult(BpmProcessInstanceResultEnum.PROCESS.getResult())); } @@ -269,106 +271,108 @@ public class BpmTaskServiceImpl implements BpmTaskService { */ private void handleParentTask(Task task) { String parentTaskId = task.getParentTaskId(); - // TODO @ 海：if return 原则 + if (StrUtil.isBlank(parentTaskId)) { + return; + } if (StrUtil.isNotBlank(parentTaskId)) { // 1. 判断当前任务的父任务是否还有子任务 - long subTaskCount = getSubTaskCount(parentTaskId); - // TODO @ 海：if >= 0 return；这样括号又可以少一层； - if (subTaskCount == 0) { - // 2. 获取父任务 - Task parentTask = validateTaskExist(parentTaskId); - - // 3. 情况一：处理向前加签 - String scopeType = parentTask.getScopeType(); - if (BpmTaskAddSignTypeEnum.BEFORE.getType().equals(scopeType)) { - // 3.1 如果是向前加签的任务，则调用 resolveTask 指派父任务，将 owner 重新赋值给父任务的 assignee - taskService.resolveTask(parentTaskId); - // 3.2 更新任务拓展表为处理中 - taskExtMapper.updateByTaskId( - new BpmTaskExtDO().setTaskId(parentTask.getId()).setResult(BpmProcessInstanceResultEnum.PROCESS.getResult())); - } else if (BpmTaskAddSignTypeEnum.AFTER.getType().equals(scopeType)) { - // 3. 情况二：处理向后加签 - handleAfterSign(parentTask); - } - - // 4. 子任务已处理完成，清空 scopeType 字段，修改 parentTask 信息，方便后续可以继续向前后向后加签 - // 再查询一次的原因是避免报错：Task was updated by another transaction concurrently - // 因为前面处理后可能会导致 parentTask rev 字段被修改，需要重新获取最新的 - parentTask = getTask(parentTaskId); - // TODO @ 海：if return 原则； - if (parentTask != null) { - // 为空的情况是：已经通过 handleAfterSign 方法将任务完成了，所以 ru_task 表会查不到数据 - clearTaskScopeTypeAndSave(parentTask); - } + Long childrenTaskCount = getChildrenTaskCount(parentTaskId); + if (childrenTaskCount > 0) { + return; } + // 2. 获取父任务 + Task parentTask = validateTaskExist(parentTaskId); + + // 3. 情况一：处理向前加签 + String scopeType = parentTask.getScopeType(); + if (BpmTaskAddSignTypeEnum.BEFORE.getType().equals(scopeType)) { + // 3.1 如果是向前加签的任务，则调用 resolveTask 指派父任务，将 owner 重新赋值给父任务的 assignee + taskService.resolveTask(parentTaskId); + // 3.2 更新任务拓展表为处理中 + taskExtMapper.updateByTaskId( + new BpmTaskExtDO().setTaskId(parentTask.getId()).setResult(BpmProcessInstanceResultEnum.PROCESS.getResult())); + } else if (BpmTaskAddSignTypeEnum.AFTER.getType().equals(scopeType)) { + // 3. 情况二：处理向后加签 + handleAfterSign(parentTask); + } + + // 4. 子任务已处理完成，清空 scopeType 字段，修改 parentTask 信息，方便后续可以继续向前后向后加签 + // 再查询一次的原因是避免报错：Task was updated by another transaction concurrently + // 因为前面处理后可能会导致 parentTask rev 字段被修改，需要重新获取最新的 + parentTask = getTask(parentTaskId); + if (parentTask == null) { + // 为空的情况是：已经通过 handleAfterSign 方法将任务完成了，所以 ru_task 表会查不到数据 + return; + } + clearTaskScopeTypeAndSave(parentTask); } } - // TODO @海：这个方法的 4.1 从 1 开始计数哈；另外；看看能不能用 if return 进一步简化代码的层级； /** * 处理后加签任务 * - * @param parentTask 当前审批任务的父任务 + * @param parentTask 当前审批任务的父任务 */ private void handleAfterSign(Task parentTask) { - // TODO @海：这个方法在注释下。感觉整体是，先完成自己；然后递归处理父节点；但是完成机子被放到了 5.1，就感觉上有点割裂； String parentTaskId = parentTask.getId(); - //4.1 更新 parentTask 的任务拓展表为通过 + //1. 更新 parentTask 的任务拓展表为通过，并调用 complete 完成自己 BpmTaskExtDO currentTaskExt = taskExtMapper.selectByTaskId(parentTask.getId()); BpmTaskExtDO currentTaskUpdateEntity = new BpmTaskExtDO().setTaskId(parentTask.getId()) .setResult(BpmProcessInstanceResultEnum.APPROVE.getResult()); if (currentTaskExt.getEndTime() == null) { - // 有这个判断是因为 - // 4.2 以前没设置过结束时间，才去设置 + // 1.1 有这个判断是因为,以前没设置过结束时间，才去设置 currentTaskUpdateEntity.setEndTime(LocalDateTime.now()); } + // 1.2 完成自己 taskExtMapper.updateByTaskId(currentTaskUpdateEntity); - - // 5. 继续往上处理，父任务继续往上查询 - // 5.1 先完成自己 taskService.complete(parentTaskId); - // 5.2 如果有父级，递归查询上级任务是否都已经完成 + + // 2. 如果有父级，递归查询上级任务是否都已经完成 + if (StrUtil.isEmpty(parentTask.getParentTaskId())) { + return; + } // TODO @海：这块待讨论，脑子略乱；感觉 handleAfterSign 的后半段，和 handleParentTask 有点重叠？？？ - if (StrUtil.isNotEmpty(parentTask.getParentTaskId())) { - // 判断整条链路的任务是否完成 - // 例如从 A 任务加签了一个 B 任务，B 任务又加签了一个 C 任务，C 任务加签了 D 任务 - // 此时，D 任务完成，要一直往上找到祖先任务 A调用 complete 方法完成 A 任务 - boolean allChildrenTaskFinish = true; - while (StrUtil.isNotBlank(parentTask.getParentTaskId())) { - parentTask = validateTaskExist(parentTask.getParentTaskId()); - BpmTaskExtDO bpmTaskExtDO = taskExtMapper.selectByTaskId(parentTask.getId()); - if (bpmTaskExtDO == null) { - break; - } - boolean currentTaskFinish = BpmProcessInstanceResultEnum.isEndResult(bpmTaskExtDO.getResult()); - // 5.3 如果 allChildrenTaskFinish 已经被赋值为 false ,则不会再赋值为 true，因为整个链路没有完成 - if (allChildrenTaskFinish) { - allChildrenTaskFinish = currentTaskFinish; - } - if (!currentTaskFinish) { - // 6 处理非完成状态的任务 - // 6.1 判断当前任务的父任务是否还有子任务 - Long subTaskCount = getSubTaskCount(bpmTaskExtDO.getTaskId()); - if (subTaskCount == 0) { - // 6.2 没有子任务，判断当前任务状态是否为 ADD_SIGN_BEFORE 待前加签任务完成 - if (BpmProcessInstanceResultEnum.ADD_SIGN_BEFORE.getResult().equals(bpmTaskExtDO.getResult())) { - // 6.3 需要修改该任务状态为处理中 - taskService.resolveTask(bpmTaskExtDO.getTaskId()); - bpmTaskExtDO.setResult(BpmProcessInstanceResultEnum.PROCESS.getResult()); - taskExtMapper.updateByTaskId(bpmTaskExtDO); - } - // 6.4 清空 scopeType 字段，用于任务没有子任务时使用该方法，方便任务可以再次被不同的方式加签 - parentTask = getTask(bpmTaskExtDO.getTaskId()); - if (parentTask != null) { - clearTaskScopeTypeAndSave(parentTask); - } - } - } + // 2.1 判断整条链路的任务是否完成 + // 例如从 A 任务加签了一个 B 任务，B 任务又加签了一个 C 任务，C 任务加签了 D 任务 + // 此时，D 任务完成，要一直往上找到祖先任务 A调用 complete 方法完成 A 任务 + boolean allChildrenTaskFinish = true; + while (StrUtil.isNotBlank(parentTask.getParentTaskId())) { + parentTask = validateTaskExist(parentTask.getParentTaskId()); + BpmTaskExtDO bpmTaskExtDO = taskExtMapper.selectByTaskId(parentTask.getId()); + if (bpmTaskExtDO == null) { + break; } + boolean currentTaskFinish = BpmProcessInstanceResultEnum.isEndResult(bpmTaskExtDO.getResult()); + // 2.2 如果 allChildrenTaskFinish 已经被赋值为 false ,则不会再赋值为 true，因为整个链路没有完成 if (allChildrenTaskFinish) { - // 7. 完成最后的顶级祖先任务 - taskService.complete(parentTask.getId()); + allChildrenTaskFinish = currentTaskFinish; } + // 2.3 任务已完成则不处理 + if (currentTaskFinish) { + continue; + } + + // 3 处理非完成状态的任务 + // 3.1 判断当前任务的父任务是否还有子任务 + Long childrenTaskCount = getChildrenTaskCount(bpmTaskExtDO.getTaskId()); + if (childrenTaskCount > 0) { + continue; + } + // 3.2 没有子任务，判断当前任务状态是否为 ADD_SIGN_BEFORE 待前加签任务完成 + if (BpmProcessInstanceResultEnum.ADD_SIGN_BEFORE.getResult().equals(bpmTaskExtDO.getResult())) { + // 3.3 需要修改该任务状态为处理中 + taskService.resolveTask(bpmTaskExtDO.getTaskId()); + bpmTaskExtDO.setResult(BpmProcessInstanceResultEnum.PROCESS.getResult()); + taskExtMapper.updateByTaskId(bpmTaskExtDO); + } + // 3.4 清空 scopeType 字段，用于任务没有子任务时使用该方法，方便任务可以再次被不同的方式加签 + parentTask = validateTaskExist(bpmTaskExtDO.getTaskId()); + clearTaskScopeTypeAndSave(parentTask); + } + + // 4. 完成最后的顶级祖先任务 + if (allChildrenTaskFinish) { + taskService.complete(parentTask.getId()); } } @@ -383,14 +387,14 @@ public class BpmTaskServiceImpl implements BpmTaskService { taskService.saveTask(task); } - // TODO @海：Sub 还有 Child 感觉整体用词不是很统一；是不是要统一下；然后 Sub 还可以指的减签；看看是不是名词确实得一致哈； + /** * 获取子任务个数 * * @param parentTaskId 父任务 ID * @return 剩余子任务个数 */ - private Long getSubTaskCount(String parentTaskId) { + private Long getChildrenTaskCount(String parentTaskId) { String tableName = managementService.getTableName(TaskEntity.class); String sql = "SELECT COUNT(1) from " + tableName + " WHERE PARENT_TASK_ID_=#{parentTaskId}"; return taskService.createNativeTaskQuery().sql(sql).parameter("parentTaskId", parentTaskId).count(); @@ -410,7 +414,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { String comment = StrUtil.format("[{}]完成委派任务，任务重新回到[{}]手中，审批意见为:{}", currentUser.getNickname(), sourceApproveUser.getNickname(), reqVO.getReason()); taskService.addComment(reqVO.getId(), task.getProcessInstanceId(), - BpmCommentTypeEnum.DELEGATE.getResult().toString(), comment); + BpmCommentTypeEnum.DELEGATE.getType().toString(), comment); // 2.1 调用 resolveTask 完成任务。 // 底层调用 TaskHelper.changeTaskAssignee(task, task.getOwner())：将 owner 设置为 assignee @@ -528,7 +532,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { TransactionSynchronizationManager.registerSynchronization(new TransactionSynchronization() { @Override public void afterCommit() { - if(StrUtil.isNotEmpty(task.getAssignee())){ + if (StrUtil.isNotEmpty(task.getAssignee())) { ProcessInstance processInstance = processInstanceService.getProcessInstance(task.getProcessInstanceId()); AdminUserRespDTO startUser = adminUserApi.getUser(Long.valueOf(processInstance.getStartUserId())); @@ -539,8 +543,8 @@ public class BpmTaskServiceImpl implements BpmTaskService { }); } - private Task validateTaskExist(String id){ - Task task = taskService.createTaskQuery().taskId(id).singleResult(); + private Task validateTaskExist(String id) { + Task task = getTask(id); if (task == null) { throw exception(TASK_NOT_EXISTS); } @@ -646,7 +650,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { return; } taskService.addComment(task.getId(), currentTask.getProcessInstanceId(), - BpmCommentTypeEnum.BACK.getResult().toString(), reqVO.getReason()); + BpmCommentTypeEnum.BACK.getType().toString(), reqVO.getReason()); }); // 3. 执行驳回 @@ -659,7 +663,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { @Override @Transactional(rollbackFor = Exception.class) - public void delegateTask(BpmTaskDelegateReqVO reqVO, Long userId) { + public void delegateTask(Long userId, BpmTaskDelegateReqVO reqVO) { // 1.1 校验任务 Task task = validateTaskCanDelegate(userId, reqVO); // 1.2 校验目标用户存在 @@ -674,7 +678,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { delegateUser.getNickname(), reqVO.getReason()); String taskId = reqVO.getId(); taskService.addComment(taskId, task.getProcessInstanceId(), - BpmCommentTypeEnum.DELEGATE.getResult().toString(), comment); + BpmCommentTypeEnum.DELEGATE.getType().toString(), comment); // 3.1 设置任务所有人 (owner) 为原任务的处理人 (assignee) taskService.setOwner(taskId, task.getAssignee()); @@ -705,23 +709,18 @@ public class BpmTaskServiceImpl implements BpmTaskService { @Override @Transactional(rollbackFor = Exception.class) - public void addSign(BpmTaskAddSignReqVO reqVO, Long userId) { - // TODO @海：// 后面要有个空格；中英文之间，也要有空格；例如说：// 1. 获取和校验任务 - //1.获取和校验任务 + public void addSignTask(Long userId, BpmTaskAddSignReqVO reqVO) { + // 1. 获取和校验任务 TaskEntityImpl taskEntity = validateAddSign(userId, reqVO); - // TODO @海：每个变量，以及相关逻辑，一定要和自己的逻辑块呆在一起。例如说这里的 currentUser 获取，应该在 4. 那块 - AdminUserRespDTO currentUser = adminUserApi.getUser(userId); List userList = adminUserApi.getUserList(reqVO.getUserIdList()); if (CollUtil.isEmpty(userList)) { throw exception(TASK_ADD_SIGN_USER_NOT_EXIST); } - // TODO @海：大的逻辑块之间，最好有一个空格。这样的目的，是避免逻辑堆砌在一起，方便流量； - // 2.处理当前任务 - // 2.1 开启计数功能 // TODO @海：这个目的可以写下； + + // 2. 处理当前任务 + // 2.1 开启计数功能，主要用于为了让表 ACT_RU_TASK 中的 SUB_TASK_COUNT_ 字段记录下总共有多少子任务，后续可能有用 taskEntity.setCountEnabled(true); - // TODO @海：可以直接 if，不搞一个变量哈 - boolean addSignToBefore = reqVO.getType().equals(BpmTaskAddSignTypeEnum.BEFORE.getType()); - if (addSignToBefore) { + if (reqVO.getType().equals(BpmTaskAddSignTypeEnum.BEFORE.getType())) { // 2.2 向前加签，设置 owner，置空 assign。等子任务都完成后，再调用 resolveTask 重新将 owner 设置为 assign // 原因是：不能和向前加签的子任务一起审批，需要等前面的子任务都完成才能审批 taskEntity.setOwner(taskEntity.getAssignee()); @@ -738,20 +737,20 @@ public class BpmTaskServiceImpl implements BpmTaskService { taskService.saveTask(taskEntity); // 3. 创建加签任务 - createSignSubTasks(convertList(reqVO.getUserIdList(), String::valueOf), taskEntity); + createAddSignChildrenTasks(convertList(reqVO.getUserIdList(), String::valueOf), taskEntity); // 4. 记录加签 comment，拼接结果为： [当前用户]向前加签/向后加签给了[多个用户]，理由为：reason - // TODO @海：BpmCommentTypeEnum 可以加一个 comment 字段，作为评论模版，统一管理； - String comment = StrUtil.format("[{}]{}给了[{}]，理由为：{}", currentUser.getNickname(), BpmTaskAddSignTypeEnum.formatDesc(reqVO.getType()), - String.join(SymbolConstant.D, convertList(userList, AdminUserRespDTO::getNickname)), reqVO.getReason()); + AdminUserRespDTO currentUser = adminUserApi.getUser(userId); + String comment = StrUtil.format(BpmCommentTypeEnum.ADD_SIGN.getTemplateComment(), currentUser.getNickname(), BpmTaskAddSignTypeEnum.formatDesc(reqVO.getType()), + String.join(",", convertList(userList, AdminUserRespDTO::getNickname)), reqVO.getReason()); taskService.addComment(reqVO.getId(), taskEntity.getProcessInstanceId(), - BpmCommentTypeEnum.ADD_SIGN.getResult().toString(), comment); + BpmCommentTypeEnum.ADD_SIGN.getType().toString(), comment); } /** * 校验任务的加签是否一致 - * + *

* 1. 如果存在“向前加签”的任务，则不能“向后加签” * 2. 如果存在“向后加签”的任务，则不能“向前加签” * @@ -763,25 +762,20 @@ public class BpmTaskServiceImpl implements BpmTaskService { TaskEntityImpl taskEntity = (TaskEntityImpl) validateTask(userId, reqVO.getId()); // 向前加签和向后加签不能同时存在 if (StrUtil.isNotBlank(taskEntity.getScopeType()) - && !BpmTaskAddSignTypeEnum.AFTER_CHILDREN_TASK.getDesc().equals(taskEntity.getScopeType())) { - // TODO @海：下面这个判断，是不是可以写上面这个判断后面？ - // TODO @海：一个小技巧，如果写不等于的时候，一般可以用 ObjectUtil.notEquals，这样少一层取反，理解起来简单点； - if (!taskEntity.getScopeType().equals(reqVO.getType())) { - throw exception(TASK_ADD_SIGN_TYPE_ERROR, - BpmTaskAddSignTypeEnum.formatDesc(taskEntity.getScopeType()), BpmTaskAddSignTypeEnum.formatDesc(reqVO.getType())); - } + && ObjectUtil.notEqual(BpmTaskAddSignTypeEnum.AFTER_CHILDREN_TASK.getType(), taskEntity.getScopeType()) + && ObjectUtil.notEqual(taskEntity.getScopeType(), reqVO.getType())) { + throw exception(TASK_ADD_SIGN_TYPE_ERROR, + BpmTaskAddSignTypeEnum.formatDesc(taskEntity.getScopeType()), BpmTaskAddSignTypeEnum.formatDesc(reqVO.getType())); } // 同一个 key 的任务，审批人不重复 List taskList = taskService.createTaskQuery().processInstanceId(taskEntity.getProcessInstanceId()) .taskDefinitionKey(taskEntity.getTaskDefinitionKey()).list(); - // TODO @海：这里是不是 Task::getAssignee 解析成 List 下面会更简洁一点？ - List currentAssigneeList = convertList(taskList, Task::getAssignee); - // TODO @海：frontAssigneeList 改成 addAssigneeList，新增的；避免和 front 前端这样界面耦合的名词哈； - List frontAssigneeList = convertList(reqVO.getUserIdList(), String::valueOf); - currentAssigneeList.retainAll(frontAssigneeList); + List currentAssigneeList = convertList(taskList, task -> NumberUtils.parseLong(task.getAssignee())); + // 保留交集在 currentAssigneeList 中 + currentAssigneeList.retainAll(reqVO.getUserIdList()); if (CollUtil.isNotEmpty(currentAssigneeList)) { - List userList = adminUserApi.getUserList(convertList(currentAssigneeList, NumberUtils::parseLong)); - throw exception(TASK_ADD_SIGN_USER_REPEAT, String.join(SymbolConstant.D, convertList(userList, AdminUserRespDTO::getNickname))); + List userList = adminUserApi.getUserList(currentAssigneeList); + throw exception(TASK_ADD_SIGN_USER_REPEAT, String.join(",", convertList(userList, AdminUserRespDTO::getNickname))); } return taskEntity; } @@ -792,19 +786,19 @@ public class BpmTaskServiceImpl implements BpmTaskService { * @param addSingUserIdList 被加签的用户 ID * @param taskEntity 被加签的任务 */ - private void createSignSubTasks(List addSingUserIdList, TaskEntityImpl taskEntity) { - // TODO @海：可以 if return；这样括号层级少一点；下面的 if (StrUtil.isNotBlank(addSignId)) { 也是类似； - if (CollUtil.isNotEmpty(addSingUserIdList)) { - // 创建加签人的新任务，全部基于 taskEntity 为父任务来创建 - addSingUserIdList.forEach(addSignId -> { - if (StrUtil.isNotBlank(addSignId)) { - createSubTask(taskEntity, addSignId); - } - }); + private void createAddSignChildrenTasks(List addSingUserIdList, TaskEntityImpl taskEntity) { + if (CollUtil.isEmpty(addSingUserIdList)) { + return; + } + // 创建加签人的新任务，全部基于 taskEntity 为父任务来创建 + for (String addSignId : addSingUserIdList) { + if (StrUtil.isBlank(addSignId)) { + continue; + } + createChildrenTask(taskEntity, addSignId); } } - // TODO @海：这个是不是命名上，要和 createSignSubTasks 保持一致？ /** * 创建子任务 * @@ -812,10 +806,10 @@ public class BpmTaskServiceImpl implements BpmTaskService { * @param assignee 子任务的执行人 * @return */ - private void createSubTask(TaskEntityImpl parentTask, String assignee) { + private void createChildrenTask(TaskEntityImpl parentTask, String assignee) { // 1. 生成子任务 TaskEntityImpl task = (TaskEntityImpl) taskService.newTask(IdUtil.fastSimpleUUID()); - task = BpmTaskConvert.INSTANCE.convert(task,parentTask); + task = BpmTaskConvert.INSTANCE.convert(task, parentTask); if (BpmTaskAddSignTypeEnum.BEFORE.getType().equals(parentTask.getScopeType())) { // 2.1 前加签，才设置审批人，否则设置 owner task.setAssignee(assignee); @@ -831,28 +825,28 @@ public class BpmTaskServiceImpl implements BpmTaskService { @Override @Transactional(rollbackFor = Exception.class) - public void subSign(BpmTaskSubSignReqVO reqVO,Long userId) { + public void subSignTask(Long userId, BpmTaskSubSignReqVO reqVO) { Task task = validateSubSign(reqVO.getId()); AdminUserRespDTO user = adminUserApi.getUser(userId); AdminUserRespDTO cancelUser = null; - if(StrUtil.isNotBlank(task.getAssignee())){ + if (StrUtil.isNotBlank(task.getAssignee())) { cancelUser = adminUserApi.getUser(NumberUtils.parseLong(task.getAssignee())); } - if(cancelUser == null && StrUtil.isNotBlank(task.getOwner())){ + if (cancelUser == null && StrUtil.isNotBlank(task.getOwner())) { cancelUser = adminUserApi.getUser(NumberUtils.parseLong(task.getOwner())); } - Assert.notNull(cancelUser,"任务中没有所有者和审批人，数据错误"); + Assert.notNull(cancelUser, "任务中没有所有者和审批人，数据错误"); //1. 获取所有需要删除的任务 ID ，包含当前任务和所有子任务 List allTaskIdList = getAllChildTaskIds(task.getId()); //2. 删除任务和所有子任务 taskService.deleteTasks(allTaskIdList); //3. 修改扩展表状态为取消 - taskExtMapper.updateBatchByTaskIdList(allTaskIdList,new BpmTaskExtDO().setResult(BpmProcessInstanceResultEnum.CANCEL.getResult()) - .setReason(StrUtil.format("由于{}操作[减签]，任务被取消",user.getNickname()))); + taskExtMapper.updateBatchByTaskIdList(allTaskIdList, new BpmTaskExtDO().setResult(BpmProcessInstanceResultEnum.CANCEL.getResult()) + .setReason(StrUtil.format("由于{}操作[减签]，任务被取消", user.getNickname()))); //4.记录日志到父任务中 先记录日志是因为，通过 handleParentTask 方法之后，任务可能被完成了，并且不存在了，会报异常，所以先记录 - String comment = StrUtil.format("{}操作了【减签】,审批人{}的任务被取消",user.getNickname(),cancelUser.getNickname()); - taskService.addComment(task.getParentTaskId(),task.getProcessInstanceId(), - BpmCommentTypeEnum.SUB_SIGN.getResult().toString(),comment); + String comment = StrUtil.format(BpmCommentTypeEnum.SUB_SIGN.getTemplateComment(), user.getNickname(), cancelUser.getNickname()); + taskService.addComment(task.getParentTaskId(), task.getProcessInstanceId(), + BpmCommentTypeEnum.SUB_SIGN.getType().toString(), comment); //5. 处理当前任务的父任务 this.handleParentTask(task); @@ -860,13 +854,14 @@ public class BpmTaskServiceImpl implements BpmTaskService { /** * 校验任务是否能被减签 + * * @param id 任务ID * @return 任务信息 */ private Task validateSubSign(String id) { Task task = validateTaskExist(id); //必须有parentId - if(StrUtil.isEmpty(task.getParentTaskId())){ + if (StrUtil.isEmpty(task.getParentTaskId())) { throw exception(TASK_SUB_SIGN_NO_PARENT); } return task; @@ -874,6 +869,7 @@ public class BpmTaskServiceImpl implements BpmTaskService { /** * 获取所有要被取消的删除的任务 ID 集合 + * * @param parentTaskId 父级任务ID * @return 所有任务ID */ @@ -888,7 +884,8 @@ public class BpmTaskServiceImpl implements BpmTaskService { /** * 递归处理子级任务 - * @param taskId 当前任务ID + * + * @param taskId 当前任务ID * @param taskIds 结果 */ private void recursiveGetChildTaskIds(String taskId, List taskIds) { @@ -901,10 +898,11 @@ public class BpmTaskServiceImpl implements BpmTaskService { /** * 获取指定父级任务的所有子任务 ID 集合 + * * @param parentTaskId 父任务 ID * @return 所有子任务的 ID 集合 */ - private List getChildrenTaskIdList(String parentTaskId){ + private List getChildrenTaskIdList(String parentTaskId) { String tableName = managementService.getTableName(TaskEntity.class); String sql = "select ID_ from " + tableName + " where PARENT_TASK_ID_=#{parentTaskId}"; List childrenTaskList = taskService.createNativeTaskQuery().sql(sql).parameter("parentTaskId", parentTaskId).list(); @@ -912,9 +910,9 @@ public class BpmTaskServiceImpl implements BpmTaskService { } @Override - public List getChildrenTaskList(String taskId){ + public List getChildrenTaskList(String taskId) { List childrenTaskIdList = getChildrenTaskIdList(taskId); - if(CollUtil.isEmpty(childrenTaskIdList)){ + if (CollUtil.isEmpty(childrenTaskIdList)) { return Collections.emptyList(); } //1. 只查询进行中的任务 @@ -926,11 +924,11 @@ public class BpmTaskServiceImpl implements BpmTaskService { List userIds = taskList.stream() .flatMap(control -> Stream.of(control.getAssignee(), control.getOwner()) - .filter(Objects::nonNull)) + .filter(Objects::nonNull)) .distinct() .map(NumberUtils::parseLong) .collect(Collectors.toList()); Map userMap = adminUserApi.getUserMap(userIds); - return BpmTaskConvert.INSTANCE.convertList(bpmTaskExtDOList,userMap,idTaskMap); + return BpmTaskConvert.INSTANCE.convertList(bpmTaskExtDOList, userMap, idTaskMap); } }