HOTIFX: 게시글 업데이트 로직 수정

Leets-Official · Aug 27, 2024 · 80c9f10 · 80c9f10
1 parent f9babec
commit 80c9f10
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 11 deletions.
diff --git a/src/main/java/leets/weeth/domain/board/application/usecase/NoticeUsecaseImpl.java b/src/main/java/leets/weeth/domain/board/application/usecase/NoticeUsecaseImpl.java
@@ -77,8 +77,11 @@ public List<NoticeDTO.ResponseAll> findNotices(Long noticeId, Integer count) {
     public void update(Long noticeId, NoticeDTO.Update dto, List<MultipartFile> files, Long userId) throws UserNotMatchException {
         Notice notice = validateOwner(noticeId, userId);
 
-        List<String> fileUrls;
-        fileUrls = fileSaveService.uploadFiles(files);
+        List<String> fileUrls = notice.getFileUrls();
+        List<String> uploadedFileUrls = fileSaveService.uploadFiles(files);
+
+        fileUrls.addAll(uploadedFileUrls);
+
         noticeUpdateService.update(notice, dto, fileUrls);
     }
 

diff --git a/src/main/java/leets/weeth/domain/board/application/usecase/PostUseCaseImpl.java b/src/main/java/leets/weeth/domain/board/application/usecase/PostUseCaseImpl.java
@@ -77,8 +77,10 @@ public List<PostDTO.ResponseAll> findPosts(Long postId, Integer count) {
     public void update(Long postId, PostDTO.Update dto, List<MultipartFile> files, Long userId) throws UserNotMatchException {
         Post post = validateOwner(postId, userId);
 
-        List<String> fileUrls;
-        fileUrls = fileSaveService.uploadFiles(files);
+        List<String> fileUrls = post.getFileUrls();
+        List<String> uploadedFileUrls = fileSaveService.uploadFiles(files);
+
+        fileUrls.addAll(uploadedFileUrls);
 
         postUpdateService.update(post, dto, fileUrls);
     }

diff --git a/src/main/java/leets/weeth/domain/board/domain/entity/Board.java b/src/main/java/leets/weeth/domain/board/domain/entity/Board.java
@@ -53,7 +53,7 @@ public void increaseCommentCount() {
     }
 
     public void decreaseCommentCount() {
-        if(commentCount > 0) {
+        if (commentCount > 0) {
             commentCount--;
         }
     }
@@ -67,17 +67,13 @@ public void updateCommentCount(List<Comment> comments) {
     public void updateUpperClass(NoticeDTO.Update dto, List<String> fileUrls) {
         this.title = dto.title();
         this.content = dto.content();
-        if (!fileUrls.isEmpty()) {
-            this.fileUrls = fileUrls;
-        }
+        this.fileUrls = fileUrls;
     }
 
     public void updateUpperClass(PostDTO.Update dto, List<String> fileUrls) {
         this.title = dto.title();
         this.content = dto.content();
-        if (!fileUrls.isEmpty()) {
-            this.fileUrls = fileUrls;
-        }
+        this.fileUrls = fileUrls;
     }
 
 }