调整数据库字段,优化部分接口 #1

Merged
xiaohai2271 merged 33 commits from dev into master 2020-05-27 16:45:03 +08:00
9 changed files with 28 additions and 13 deletions
Showing only changes of commit 7d7a0fc82d - Show all commits

View File

@@ -23,7 +23,7 @@ public class PartnerSite {
private String desc; private String desc;
private boolean delete; private Boolean delete = false;
public PartnerSite() { public PartnerSite() {
} }

View File

@@ -13,7 +13,7 @@ public class TagCategory {
private String name; private String name;
private boolean category = true; private Boolean category = true;
private boolean deleted = false; private Boolean deleted = false;
} }

View File

@@ -321,7 +321,10 @@ public class ArticleServiceImpl implements ArticleService {
open.forEach(article -> { open.forEach(article -> {
ArticleModel model = ModalTrans.article(article, true); ArticleModel model = ModalTrans.article(article, true);
model.setTags(null); model.setTags(null);
setPreAndNextArticle(model); // setPreAndNextArticle(model);
model.setNextArticle(null);
model.setPreArticle(null);
modelList.add(model);
}); });
return new PageData<ArticleModel>(new PageInfo<Article>(open), modelList); return new PageData<ArticleModel>(new PageInfo<Article>(open), modelList);
} }
@@ -337,6 +340,9 @@ public class ArticleServiceImpl implements ArticleService {
List<ArticleModel> modelList = new ArrayList<>(); List<ArticleModel> modelList = new ArrayList<>();
articleByTag.forEach(articleTag -> { articleByTag.forEach(articleTag -> {
ArticleModel model = ModalTrans.article(articleTag.getArticle(), true); ArticleModel model = ModalTrans.article(articleTag.getArticle(), true);
model.setNextArticle(null);
model.setPreArticle(null);
modelList.add(model);
}); });
return new PageData<ArticleModel>(new PageInfo<ArticleTag>(articleByTag), modelList); return new PageData<ArticleModel>(new PageInfo<ArticleTag>(articleByTag), modelList);
} }

View File

@@ -97,7 +97,9 @@ public class PartnerSiteServiceImpl implements PartnerSiteService {
@Override @Override
public List<PartnerSite> findAll() { public List<PartnerSite> findAll() {
return partnerMapper.findAll(); List<PartnerSite> all = partnerMapper.findAll();
all.forEach(partnerSite -> partnerSite.setDelete(null));
return all;
} }

View File

@@ -22,6 +22,10 @@ public class ModalTrans {
articleModel.setOriginal(article.getType()); articleModel.setOriginal(article.getType());
articleModel.setCategory(article.getCategory().getName()); articleModel.setCategory(article.getCategory().getName());
articleModel.setAuthor(user(article.getUser())); articleModel.setAuthor(user(article.getUser()));
articleModel.getTags().forEach(tag -> {
tag.setCategory(null);
tag.setDeleted(null);
});
return articleModel; return articleModel;
} }
@@ -46,6 +50,8 @@ public class ModalTrans {
userModel.setAvatarImgUrl(user.getAvatarImgUrl() == null || user.getAvatarImgUrl().length() == 0 ? userModel.setAvatarImgUrl(user.getAvatarImgUrl() == null || user.getAvatarImgUrl().length() == 0 ?
null : null :
"http://cdn.celess.cn/" + user.getAvatarImgUrl()); "http://cdn.celess.cn/" + user.getAvatarImgUrl());
userModel.setRole(null);
userModel.setEmailStatus(null);
userModel.setDisplayName(user.getDisplayName() == null ? user.getEmail() : user.getDisplayName()); userModel.setDisplayName(user.getDisplayName() == null ? user.getEmail() : user.getDisplayName());
userModel.setRecentlyLandedDate(DateFormatUtil.get(user.getRecentlyLandedDate())); userModel.setRecentlyLandedDate(DateFormatUtil.get(user.getRecentlyLandedDate()));
return userModel; return userModel;

View File

@@ -37,6 +37,7 @@
<select id="findAll" resultMap="webUpdateResultMap"> <select id="findAll" resultMap="webUpdateResultMap">
select * select *
from web_update from web_update
order by wu_id desc
</select> </select>
<select id="findAllNotDeleted" resultMap="webUpdateResultMap"> <select id="findAllNotDeleted" resultMap="webUpdateResultMap">

View File

@@ -25,11 +25,11 @@ public class CategoryMapperTest extends BaseTest {
@Test @Test
public void delete() { public void delete() {
Category category = generateCategory(); Category category = generateCategory();
assertFalse(category.isDeleted()); assertFalse(category.getDeleted());
int lines = categoryMapper.delete(category.getId()); int lines = categoryMapper.delete(category.getId());
assertNotEquals(0, lines); assertNotEquals(0, lines);
Category categoryById = categoryMapper.findCategoryById(category.getId()); Category categoryById = categoryMapper.findCategoryById(category.getId());
assertTrue(categoryById.isDeleted()); assertTrue(categoryById.getDeleted());
} }
@Test @Test
@@ -76,7 +76,7 @@ public class CategoryMapperTest extends BaseTest {
public void findAll() { public void findAll() {
List<Category> all = categoryMapper.findAll(); List<Category> all = categoryMapper.findAll();
assertNotEquals(0, all); assertNotEquals(0, all);
all.forEach(category -> assertTrue(category.isCategory())); all.forEach(category -> assertTrue(category.getCategory()));
} }
@Test @Test
@@ -107,7 +107,7 @@ public class CategoryMapperTest extends BaseTest {
@Test @Test
public void count() { public void count() {
List<Category> all = categoryMapper.findAll(); List<Category> all = categoryMapper.findAll();
List<Category> collect = all.stream().filter(category -> !category.isDeleted()).collect(Collectors.toList()); List<Category> collect = all.stream().filter(category -> !category.getDeleted()).collect(Collectors.toList());
assertEquals(collect.size(), categoryMapper.count()); assertEquals(collect.size(), categoryMapper.count());
} }

View File

@@ -25,7 +25,7 @@ public class PartnerMapperTest extends BaseTest {
PartnerSite partnerSite = generatePartnerSite(); PartnerSite partnerSite = generatePartnerSite();
assertEquals(1, partnerMapper.delete(partnerSite.getId())); assertEquals(1, partnerMapper.delete(partnerSite.getId()));
partnerSite = partnerMapper.findById(partnerSite.getId()); partnerSite = partnerMapper.findById(partnerSite.getId());
assertTrue(partnerSite.isDelete()); assertTrue(partnerSite.getDelete());
} }
@Test @Test

View File

@@ -32,10 +32,10 @@ public class TagMapperTest extends BaseTest {
@Test @Test
public void delete() { public void delete() {
Tag tag = generateTag(); Tag tag = generateTag();
assertFalse(tag.isDeleted()); assertFalse(tag.getDeleted());
assertEquals(1, tagMapper.delete(tag.getId())); assertEquals(1, tagMapper.delete(tag.getId()));
Tag tagById = tagMapper.findTagById(tag.getId()); Tag tagById = tagMapper.findTagById(tag.getId());
assertTrue(tagById.isDeleted()); assertTrue(tagById.getDeleted());
} }
@Test @Test
@@ -80,7 +80,7 @@ public class TagMapperTest extends BaseTest {
public void count() { public void count() {
assertNotEquals(0, tagMapper.count()); assertNotEquals(0, tagMapper.count());
List<Tag> all = tagMapper.findAll(); List<Tag> all = tagMapper.findAll();
List<Tag> collect = all.stream().filter(tag -> !tag.isDeleted()).collect(Collectors.toList()); List<Tag> collect = all.stream().filter(tag -> !tag.getDeleted()).collect(Collectors.toList());
assertEquals(collect.size(), tagMapper.count()); assertEquals(collect.size(), tagMapper.count());
} }