diff --git a/src/main/java/com/idol/board/service/article/query/ReadArticleService.java b/src/main/java/com/idol/board/service/article/query/ReadArticleService.java index 3a6dd54..0db06da 100644 --- a/src/main/java/com/idol/board/service/article/query/ReadArticleService.java +++ b/src/main/java/com/idol/board/service/article/query/ReadArticleService.java @@ -33,7 +33,7 @@ @Service @RequiredArgsConstructor -@Transactional() +@Transactional public class ReadArticleService implements ReadArticleUseCase { private final ArticleRepository articleRepository; @@ -44,7 +44,7 @@ public class ReadArticleService implements ReadArticleUseCase { private final WishRepository wishRepository; @Override - @Transactional(readOnly = true) + public ArticleReadResponseDto readArticle(Long articleId, Long userId) { Article article = articleRepository.findByArticleId(articleId) .orElseThrow(() -> new NotFoundException("Article", articleId)); @@ -94,7 +94,6 @@ public ArticleReadResponseDto readArticle(Long articleId, Long userId) { @Override - @Transactional(readOnly = true) public List searchArticleList( BigCategory bigCategory, SmallCategory smallCategory, String location, Long date, String sort, boolean sortAsc, Long limit, Long page, Long memberId) { @@ -127,7 +126,9 @@ public List searchArticleList( private void validateCheckOpenStatus(Article article) { if(article.getCurrentPerson() >= article.getMinPerson()){ - article.updateOpenStatus(OpenStatus.CONFIRMED_STATUS); + if(article.getOpenStatus().equals(OpenStatus.PENDING_STATUS)) { + article.updateOpenStatus(OpenStatus.CONFIRMED_STATUS); + } } } @@ -140,7 +141,7 @@ private OpenStatus validateCheckDeadlineStatus(Long articleId, OpenStatus status if(status.equals(OpenStatus.CONFIRMED_STATUS)) { article.updateOpenStatus(OpenStatus.DEADLINE_STATUS); return OpenStatus.DEADLINE_STATUS; - }else{ + }else if(status.equals(OpenStatus.PENDING_STATUS)){ article.updateOpenStatus(OpenStatus.CANCELED_STATUS); return OpenStatus.CANCELED_STATUS; }