diff --git a/backend/src/main/java/io/dataease/job/sechedule/strategy/impl/EmailTaskHandler.java b/backend/src/main/java/io/dataease/job/sechedule/strategy/impl/EmailTaskHandler.java index 916c3a5457..32cac1edc0 100644 --- a/backend/src/main/java/io/dataease/job/sechedule/strategy/impl/EmailTaskHandler.java +++ b/backend/src/main/java/io/dataease/job/sechedule/strategy/impl/EmailTaskHandler.java @@ -168,12 +168,15 @@ public class EmailTaskHandler extends TaskHandler implements Job { contentStr = new String(content, "UTF-8"); } + List files = null; String viewIds = emailTemplateDTO.getViewIds(); - List viewIdList = Arrays.asList(viewIds.split(",")).stream().map(s -> (s.trim())).collect(Collectors.toList()); - - PermissionProxy proxy = new PermissionProxy(); - proxy.setUserId(user.getUserId()); - List files = viewExportExcel.export(panelId, viewIdList, proxy); + if (StringUtils.isNotBlank(viewIds)) { + List viewIdList = Arrays.asList(viewIds.split(",")).stream().filter(StringUtils::isNotBlank).map(s -> (s.trim())).collect(Collectors.toList()); + PermissionProxy proxy = new PermissionProxy(); + proxy.setUserId(user.getUserId()); + files = viewExportExcel.export(panelId, viewIdList, proxy); + } + emailService.sendWithImageAndFiles(recipients, emailTemplateDTO.getTitle(), contentStr, bytes, files); success(taskInstance); diff --git a/backend/src/main/java/io/dataease/service/chart/ViewExportExcel.java b/backend/src/main/java/io/dataease/service/chart/ViewExportExcel.java index ee7cd97324..2cd29212e7 100644 --- a/backend/src/main/java/io/dataease/service/chart/ViewExportExcel.java +++ b/backend/src/main/java/io/dataease/service/chart/ViewExportExcel.java @@ -1,5 +1,6 @@ package io.dataease.service.chart; +import org.apache.commons.collections4.CollectionUtils; import org.apache.commons.lang3.ObjectUtils; import org.apache.commons.lang3.StringUtils; import org.springframework.stereotype.Service; @@ -39,6 +40,9 @@ public class ViewExportExcel { @DePermissionProxy(paramIndex = 2) public List export(String panelId, List viewIds, PermissionProxy proxy) throws Exception { + if (CollectionUtils.isEmpty(viewIds)) { + return null; + } PanelGroupService panelGroupService = SpringContextUtil.getBean(PanelGroupService.class); PanelGroupDTO panelDto = panelGroupService.findOne(panelId);