diff --git a/src/main/java/cn/celess/blog/enmu/CommentStatusEnum.java b/src/main/java/cn/celess/blog/enmu/CommentStatusEnum.java index 84ac6bc..be1ef88 100644 --- a/src/main/java/cn/celess/blog/enmu/CommentStatusEnum.java +++ b/src/main/java/cn/celess/blog/enmu/CommentStatusEnum.java @@ -1,7 +1,5 @@ package cn.celess.blog.enmu; -import com.sun.javaws.jnl.IconDesc; - /** * @Author: 小海 * @Date: 2020-05-25 08:58 diff --git a/src/main/java/cn/celess/blog/entity/ArticleTag.java b/src/main/java/cn/celess/blog/entity/ArticleTag.java index af8f5d3..6ba2184 100644 --- a/src/main/java/cn/celess/blog/entity/ArticleTag.java +++ b/src/main/java/cn/celess/blog/entity/ArticleTag.java @@ -1,6 +1,5 @@ package cn.celess.blog.entity; -import com.sun.xml.internal.ws.policy.EffectiveAlternativeSelector; import lombok.AllArgsConstructor; import lombok.Data; import lombok.NoArgsConstructor; diff --git a/src/main/java/cn/celess/blog/service/serviceimpl/ArticleServiceImpl.java b/src/main/java/cn/celess/blog/service/serviceimpl/ArticleServiceImpl.java index 656187b..68ae080 100644 --- a/src/main/java/cn/celess/blog/service/serviceimpl/ArticleServiceImpl.java +++ b/src/main/java/cn/celess/blog/service/serviceimpl/ArticleServiceImpl.java @@ -281,7 +281,8 @@ public class ArticleServiceImpl implements ArticleService { PageData pageData = new PageData(new PageInfo
(articleList)); List articleModelList = new ArrayList<>(); articleList.forEach(article -> { - ArticleModel articleModel = ModalTrans.article(article, true); + ArticleModel articleModel = ModalTrans.article(article); + articleModel.setMdContent(null); articleModelList.add(articleModel); }); pageData.setList(articleModelList); diff --git a/src/main/java/cn/celess/blog/util/ModalTrans.java b/src/main/java/cn/celess/blog/util/ModalTrans.java index 9b58ab7..a8c817f 100644 --- a/src/main/java/cn/celess/blog/util/ModalTrans.java +++ b/src/main/java/cn/celess/blog/util/ModalTrans.java @@ -28,7 +28,7 @@ public class ModalTrans { public static ArticleModel article(Article article, boolean noMdContent) { ArticleModel article1 = article(article); - if (!noMdContent) { + if (!noMdContent || article1 == null) { return article1; } article1.setMdContent(null); diff --git a/src/main/resources/mapper/ArticleTagMapper.xml b/src/main/resources/mapper/ArticleTagMapper.xml index c1062b3..38f4ba6 100644 --- a/src/main/resources/mapper/ArticleTagMapper.xml +++ b/src/main/resources/mapper/ArticleTagMapper.xml @@ -31,7 +31,7 @@ t_id = #{tag.id}, - where at_id = #{tag.id} + where at_id = #{id} @@ -69,7 +69,8 @@ from article_tag, article, tag_category - where article_tag.a_id = article.a_id + where article_tag.at_id = #{id} + and article_tag.a_id = article.a_id and tag_category.t_id = article_tag.t_id diff --git a/src/test/java/cn/celess/blog/mapper/ArticleMapperTest.java b/src/test/java/cn/celess/blog/mapper/ArticleMapperTest.java index 1b7d785..163eba4 100644 --- a/src/test/java/cn/celess/blog/mapper/ArticleMapperTest.java +++ b/src/test/java/cn/celess/blog/mapper/ArticleMapperTest.java @@ -164,7 +164,7 @@ public class ArticleMapperTest extends BaseTest { @Test public void findAllByCategoryIdAndOpen() { - List
allByCategoryId = articleMapper.findAllByCategoryId(1); + List
allByCategoryId = articleMapper.findAllByCategoryIdAndOpen(1); assertNotEquals(0, allByCategoryId.size()); allByCategoryId.forEach(article -> assertTrue(article.getOpen())); } diff --git a/src/test/java/cn/celess/blog/mapper/ArticleTagMapperTest.java b/src/test/java/cn/celess/blog/mapper/ArticleTagMapperTest.java index fb8715d..0f2763a 100644 --- a/src/test/java/cn/celess/blog/mapper/ArticleTagMapperTest.java +++ b/src/test/java/cn/celess/blog/mapper/ArticleTagMapperTest.java @@ -106,7 +106,7 @@ public class ArticleTagMapperTest extends BaseTest { @Test public void findArticleByTagAndOpen() { ArticleTag articleTag = generateArticle(); - List articleByTag = articleTagMapper.findArticleByTag(21L); + List articleByTag = articleTagMapper.findArticleByTagAndOpen(21L); assertNotEquals(0, articleByTag.size()); articleByTag.forEach(articleTag1 -> assertEquals(articleTag.getTag().getName(), articleTag1.getTag().getName())); articleByTag.forEach(articleTag1 -> assertTrue(articleTag1.getArticle().getOpen())); diff --git a/src/test/java/cn/celess/blog/mapper/CategoryMapperTest.java b/src/test/java/cn/celess/blog/mapper/CategoryMapperTest.java index 4ab98a9..d43d79e 100644 --- a/src/test/java/cn/celess/blog/mapper/CategoryMapperTest.java +++ b/src/test/java/cn/celess/blog/mapper/CategoryMapperTest.java @@ -7,6 +7,7 @@ import org.springframework.beans.factory.annotation.Autowired; import java.util.Comparator; import java.util.List; +import java.util.stream.Collectors; import static org.junit.Assert.*; @@ -106,7 +107,8 @@ public class CategoryMapperTest extends BaseTest { @Test public void count() { List all = categoryMapper.findAll(); - assertEquals(all.size(), categoryMapper.count()); + List collect = all.stream().filter(category -> !category.isDeleted()).collect(Collectors.toList()); + assertEquals(collect.size(), categoryMapper.count()); } private Category generateCategory() { diff --git a/src/test/java/cn/celess/blog/mapper/TagMapperTest.java b/src/test/java/cn/celess/blog/mapper/TagMapperTest.java index feafcf3..b9c1445 100644 --- a/src/test/java/cn/celess/blog/mapper/TagMapperTest.java +++ b/src/test/java/cn/celess/blog/mapper/TagMapperTest.java @@ -6,6 +6,7 @@ import org.junit.Test; import org.springframework.beans.factory.annotation.Autowired; import java.util.List; +import java.util.stream.Collectors; import static org.junit.Assert.*; @@ -79,7 +80,8 @@ public class TagMapperTest extends BaseTest { public void count() { assertNotEquals(0, tagMapper.count()); List all = tagMapper.findAll(); - assertEquals(all.size(), tagMapper.count()); + List collect = all.stream().filter(tag -> !tag.isDeleted()).collect(Collectors.toList()); + assertEquals(collect.size(), tagMapper.count()); } private Tag generateTag() {