diff --git a/server/odc-service/src/main/java/com/oceanbase/odc/service/databasechange/DatabaseChangeChangingOrderTemplateService.java b/server/odc-service/src/main/java/com/oceanbase/odc/service/databasechange/DatabaseChangeChangingOrderTemplateService.java index 5e096593da..6778c51e4b 100644 --- a/server/odc-service/src/main/java/com/oceanbase/odc/service/databasechange/DatabaseChangeChangingOrderTemplateService.java +++ b/server/odc-service/src/main/java/com/oceanbase/odc/service/databasechange/DatabaseChangeChangingOrderTemplateService.java @@ -191,13 +191,13 @@ public DatabaseChangeChangingOrderTemplateResp update(Long id, () -> projectRepository.existsById(req.getProjectId())); projectPermissionValidator.checkProjectRole(req.getProjectId(), ResourceRoleName.all()); DatabaseChangeChangingOrderTemplateEntity originEntity = - templateRepository.findById(id).orElseThrow( + templateRepository.findByIdAndProjectId(id, req.getProjectId()).orElseThrow( () -> new NotFoundException(ResourceType.ODC_DATABASE_CHANGE_ORDER_TEMPLATE, "id", id)); - Optional byNameAndProjectId - = templateRepository.findByNameAndProjectId(req.getName(), req.getProjectId()); - if(byNameAndProjectId.isPresent()){ + Optional byNameAndProjectId = + templateRepository.findByNameAndProjectId(req.getName(), req.getProjectId()); + if (byNameAndProjectId.isPresent()) { PreConditions.validNoDuplicated(ResourceType.ODC_DATABASE_CHANGE_ORDER_TEMPLATE, "name", req.getName(), - () -> !Objects.equals(byNameAndProjectId.get().getId(), originEntity.getId())); + () -> !Objects.equals(byNameAndProjectId.get().getId(), originEntity.getId())); } List> orders = req.getOrders(); List databaseIds = orders.stream().flatMap(List::stream).collect(Collectors.toList());