From ef5505e22c84a63d966f35cee78afa544ece57df Mon Sep 17 00:00:00 2001
From: wzy <wzy19931122ai@163.com>
Date: Sun, 07 Aug 2022 14:33:32 +0800
Subject: [PATCH] fix

---
 src/main/java/com/xcong/farmer/cms/modules/system/service/Impl/ReleaseServiceImpl.java |  137 +++++++++++++++++++++------------------------
 1 files changed, 64 insertions(+), 73 deletions(-)

diff --git a/src/main/java/com/xcong/farmer/cms/modules/system/service/Impl/ReleaseServiceImpl.java b/src/main/java/com/xcong/farmer/cms/modules/system/service/Impl/ReleaseServiceImpl.java
index 63f984f..cc7d003 100644
--- a/src/main/java/com/xcong/farmer/cms/modules/system/service/Impl/ReleaseServiceImpl.java
+++ b/src/main/java/com/xcong/farmer/cms/modules/system/service/Impl/ReleaseServiceImpl.java
@@ -55,8 +55,6 @@
     @Autowired
     private WebSetMapper webSetMapper;
 
-    private final Executor executor = new ThreadPoolExecutor(5, 20, 600, TimeUnit.SECONDS, new LinkedBlockingQueue<Runnable>());
-
     @Override
     public void releaseArticle(Long id, Long companyId) {
         ArticleEntity article = articleMapper.selectById(id);
@@ -70,24 +68,23 @@
         data.put("id", article.getId());
         data.put("companyId", companyId);
 
-        executor.execute(() -> {
-            log.info("执行文章发布");
-            cmsCoreService.articleProcess(data, column.getArticleTemplate(), column.getPath());
-            this.releaseColumn(column.getId(), 1, companyId);
-            if (column.getParentId() != 0L) {
-                releaseColumn(column.getParentId(), 1, companyId);
-            }
+        log.info("执行文章发布");
+        cmsCoreService.articleProcess(data, column.getArticleTemplate(), column.getPath());
 
-            if (beforeColumnId != null && !article.getColumnId().equals(beforeColumnId)) {
-                ColumnEntity beforeColumn = columnMapper.selectById(beforeColumnId);
-                this.releaseColumn(beforeColumnId, 1, companyId);
-                if (beforeColumn.getParentId() != 0L) {
-                    this.releaseColumn(beforeColumn.getParentId(), 1, companyId);
-                }
-            }
+        this.releaseColumn(column.getId(), 1, companyId);
+        if (column.getParentId() != 0L) {
+            releaseColumn(column.getParentId(), 1, companyId);
+        }
 
-            this.releaseIndex(companyId, false);
-        });
+        if (beforeColumnId != null && !article.getColumnId().equals(beforeColumnId)) {
+            ColumnEntity beforeColumn = columnMapper.selectById(beforeColumnId);
+            this.releaseColumn(beforeColumnId, 1, companyId);
+            if (beforeColumn.getParentId() != 0L) {
+                this.releaseColumn(beforeColumn.getParentId(), 1, companyId);
+            }
+        }
+
+        this.releaseIndex(companyId, false);
     }
 
     @Override
@@ -95,6 +92,7 @@
         ColumnEntity columnEntity = columnMapper.selectById(id);
         Map<String, Object> map = buildColumnData(columnEntity);
 
+        // 判断栏目是否编辑了栏目编码或者修改了父级栏目
         if (columnEntity.getBeforeParentId() != -1 || !"-1".equals(columnEntity.getBeforeColumnCode())) {
             CompanyEntity company = companyMapper.selectById(companyId);
             if (!"-1".equals(columnEntity.getBeforeColumnCode())) {
@@ -117,65 +115,60 @@
             return;
         }
 
-        executor.execute(() -> {
-            List<ArticleEntity> articles = articleMapper.selectArticleByColumnId(columnEntity.getId(), companyId, type);
-            if (CollUtil.isNotEmpty(articles)) {
-                for (ArticleEntity article : articles) {
-                    if (article.getReleaseStatus() == 0) {
-                        article.setReleaseStatus(1);
-                        articleMapper.updateById(article);
-                    }
+        List<ArticleEntity> articles = articleMapper.selectArticleByColumnId(columnEntity.getId(), companyId, type);
+        if (CollUtil.isNotEmpty(articles)) {
+            for (ArticleEntity article : articles) {
+                if (article.getReleaseStatus() == 0) {
+                    article.setReleaseStatus(1);
+                    articleMapper.updateById(article);
+                }
 
-                    if (article.getType() == 1) {
-                        map.put("id", article.getId());
-                        cmsCoreService.articleProcess(map, columnEntity.getArticleTemplate(), columnEntity.getPath());
-                    }
+                if (article.getType() == 1) {
+                    map.put("id", article.getId());
+                    cmsCoreService.articleProcess(map, columnEntity.getArticleTemplate(), columnEntity.getPath());
                 }
             }
+        }
 
-            map.put("id", columnEntity.getId());
+        map.put("id", columnEntity.getId());
 
-            cmsCoreService.columnProcess(map, columnEntity.getListTemplate());
-            if (columnEntity.getParentId() != 0L) {
-                ColumnEntity parentColumn = columnMapper.selectById(columnEntity.getParentId());
-                Map<String, Object> parentMap = buildColumnData(parentColumn);
-                parentMap.put("id", parentColumn.getId());
-                cmsCoreService.columnProcess(parentMap, parentColumn.getListTemplate());
+        cmsCoreService.columnProcess(map, columnEntity.getListTemplate());
+        if (columnEntity.getParentId() != 0L) {
+            ColumnEntity parentColumn = columnMapper.selectById(columnEntity.getParentId());
+            Map<String, Object> parentMap = buildColumnData(parentColumn);
+            parentMap.put("id", parentColumn.getId());
+            cmsCoreService.columnProcess(parentMap, parentColumn.getListTemplate());
+        }
+
+        releaseIndex(companyId, false);
+
+        if (type != 1) {
+            if (columnEntity.getParentId() == 0L) {
+                List<ColumnEntity> columns = columnMapper.selectColumnByParentId(columnEntity.getId(), companyId, 2);
+                if (CollUtil.isNotEmpty(columns)) {
+                    for (ColumnEntity column : columns) {
+                        List<ArticleEntity> articleList = articleMapper.selectArticleByColumnId(column.getId(), companyId, type);
+                        if (CollUtil.isNotEmpty(articleList)) {
+                            for (ArticleEntity article : articleList) {
+                                if (article.getReleaseStatus() == 0) {
+                                    article.setReleaseStatus(1);
+                                    articleMapper.updateById(article);
+                                }
+
+                                if (article.getType() == 1) {
+                                    map.put("id", article.getId());
+                                    cmsCoreService.articleProcess(map, column.getArticleTemplate(), column.getPath());
+                                }
+                            }
+                        }
+
+                        Map<String, Object> data = buildColumnData(column);
+                        cmsCoreService.columnProcess(data, column.getListTemplate());
+                    }
+                }
             }
 
             releaseIndex(companyId, false);
-        });
-
-        if (type != 1) {
-            executor.execute(() -> {
-                if (columnEntity.getParentId() == 0L) {
-                    List<ColumnEntity> columns = columnMapper.selectColumnByParentId(columnEntity.getId(), companyId, 2);
-                    if (CollUtil.isNotEmpty(columns)) {
-                        for (ColumnEntity column : columns) {
-                            List<ArticleEntity> articles = articleMapper.selectArticleByColumnId(column.getId(), companyId, type);
-                            if (CollUtil.isNotEmpty(articles)) {
-                                for (ArticleEntity article : articles) {
-                                    if (article.getReleaseStatus() == 0) {
-                                        article.setReleaseStatus(1);
-                                        articleMapper.updateById(article);
-                                    }
-
-                                    if (article.getType() == 1) {
-                                        map.put("id", article.getId());
-                                        cmsCoreService.articleProcess(map, column.getArticleTemplate(), column.getPath());
-                                    }
-                                }
-                            }
-
-                            Map<String, Object> data = buildColumnData(column);
-                            cmsCoreService.columnProcess(data, column.getListTemplate());
-                        }
-                    }
-                }
-
-                releaseIndex(companyId, false);
-            });
-
         }
     }
 
@@ -217,10 +210,8 @@
 
         cmsCoreService.indexProcess(map, webSetting.getIndexTemplate());
         if (hasSearch) {
-            executor.execute(() -> {
-                cmsCoreService.process(map, "search", webSetting.getSearchTemplate());
-                cmsCoreService.process(map, "message", webSetting.getMsgTemplate());
-            });
+            cmsCoreService.process(map, "search", webSetting.getSearchTemplate());
+            cmsCoreService.process(map, "message", webSetting.getMsgTemplate());
         }
     }
 

--
Gitblit v1.9.1