Commit 846d0605d84ed42c2b50829c5c3b32a131162f6d
1 parent
281d3d1d
【代码评审】IoT:清理流程的实例们
Showing
3 changed files
with
5 additions
and
11 deletions
yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/dal/mysql/task/BpmProcessInstanceCopyMapper.java
| @@ -7,8 +7,6 @@ import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.instance.BpmProcessI | @@ -7,8 +7,6 @@ import cn.iocoder.yudao.module.bpm.controller.admin.task.vo.instance.BpmProcessI | ||
| 7 | import cn.iocoder.yudao.module.bpm.dal.dataobject.task.BpmProcessInstanceCopyDO; | 7 | import cn.iocoder.yudao.module.bpm.dal.dataobject.task.BpmProcessInstanceCopyDO; |
| 8 | import org.apache.ibatis.annotations.Mapper; | 8 | import org.apache.ibatis.annotations.Mapper; |
| 9 | 9 | ||
| 10 | -import java.util.List; | ||
| 11 | - | ||
| 12 | @Mapper | 10 | @Mapper |
| 13 | public interface BpmProcessInstanceCopyMapper extends BaseMapperX<BpmProcessInstanceCopyDO> { | 11 | public interface BpmProcessInstanceCopyMapper extends BaseMapperX<BpmProcessInstanceCopyDO> { |
| 14 | 12 | ||
| @@ -20,12 +18,8 @@ public interface BpmProcessInstanceCopyMapper extends BaseMapperX<BpmProcessInst | @@ -20,12 +18,8 @@ public interface BpmProcessInstanceCopyMapper extends BaseMapperX<BpmProcessInst | ||
| 20 | .orderByDesc(BpmProcessInstanceCopyDO::getId)); | 18 | .orderByDesc(BpmProcessInstanceCopyDO::getId)); |
| 21 | } | 19 | } |
| 22 | 20 | ||
| 23 | - default List<BpmProcessInstanceCopyDO> selectListByProcessInstanceIdAndActivityId(String processInstanceId, String activityId) { | ||
| 24 | - return selectList(BpmProcessInstanceCopyDO::getProcessInstanceId, processInstanceId, | ||
| 25 | - BpmProcessInstanceCopyDO::getActivityId, activityId); | ||
| 26 | - } | ||
| 27 | - | ||
| 28 | default void deleteByProcessInstanceId(String processInstanceId) { | 21 | default void deleteByProcessInstanceId(String processInstanceId) { |
| 29 | delete(BpmProcessInstanceCopyDO::getProcessInstanceId, processInstanceId); | 22 | delete(BpmProcessInstanceCopyDO::getProcessInstanceId, processInstanceId); |
| 30 | } | 23 | } |
| 24 | + | ||
| 31 | } | 25 | } |
yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/definition/BpmModelServiceImpl.java
| @@ -288,9 +288,8 @@ public class BpmModelServiceImpl implements BpmModelService { | @@ -288,9 +288,8 @@ public class BpmModelServiceImpl implements BpmModelService { | ||
| 288 | // 2.3 清理所有 Task | 288 | // 2.3 清理所有 Task |
| 289 | List<Task> tasks = taskService.createTaskQuery() | 289 | List<Task> tasks = taskService.createTaskQuery() |
| 290 | .processDefinitionKey(model.getKey()).list(); | 290 | .processDefinitionKey(model.getKey()).list(); |
| 291 | - tasks.forEach(task -> { | ||
| 292 | - taskService.deleteTask(task.getId()); | ||
| 293 | - }); | 291 | + // TODO @lesan:貌似传递一个 reason 会好点! |
| 292 | + tasks.forEach(task -> taskService.deleteTask(task.getId())); | ||
| 294 | } | 293 | } |
| 295 | 294 | ||
| 296 | @Override | 295 | @Override |
yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/service/task/BpmProcessInstanceCopyService.java
| @@ -53,7 +53,8 @@ public interface BpmProcessInstanceCopyService { | @@ -53,7 +53,8 @@ public interface BpmProcessInstanceCopyService { | ||
| 53 | /** | 53 | /** |
| 54 | * 删除抄送流程 | 54 | * 删除抄送流程 |
| 55 | * | 55 | * |
| 56 | - * @param processInstanceId 流程实例id | 56 | + * @param processInstanceId 流程实例 ID |
| 57 | */ | 57 | */ |
| 58 | void deleteProcessInstanceCopy(String processInstanceId); | 58 | void deleteProcessInstanceCopy(String processInstanceId); |
| 59 | + | ||
| 59 | } | 60 | } |