Merge pull request #321 from prgrms-web-devcourse-final-project/chore#320

seungwookc97 · web-flow · commit b12e545beee7 · 2025-10-15T10:06:09.000+09:00
[chore] 브랜치 병합 후 테스트
diff --git a/src/main/java/com/back/domain/cocktail/comment/service/CocktailCommentService.java b/src/main/java/com/back/domain/cocktail/comment/service/CocktailCommentService.java
@@ -47,7 +47,7 @@ public CocktailCommentResponseDto createCocktailComment(Long cocktailId, Cocktai
                 .cocktail(cocktail)
                 .user(user)
                 .content(reqBody.content())
-                .status(reqBody.status())
+                .status(reqBody.status() != null ? reqBody.status() : CommentStatus.PUBLIC)
                 .build();
 
         return new CocktailCommentResponseDto(cocktailCommentRepository.save(cocktailComment));
diff --git a/src/main/java/com/back/domain/post/post/controller/PostController.java b/src/main/java/com/back/domain/post/post/controller/PostController.java
@@ -120,4 +120,17 @@ public RsData<PostLikeResponseDto> toggleLike(
   ) {
     return RsData.successOf(postService.toggleLike(postId)); // code=200, message="success"
   }
+
+  /**
+   * 사용자가 해당 게시글을 추천(좋아요)했는지 여부 확인 API
+   * @param postId 확인할 게시글 ID
+   * @return 사용자의 추천 여부 (true/false)
+   */
+  @GetMapping("/{postId}/like")
+  @Operation(summary = "사용자가 해당 게시글 추천 여부 확인")
+  public RsData<Boolean> getLike(
+      @PathVariable Long postId
+  ) {
+    return RsData.successOf(postService.getLike(postId)); // code=200, message="success"
+  }
 }
diff --git a/src/main/java/com/back/domain/post/post/dto/request/PostUpdateRequestDto.java b/src/main/java/com/back/domain/post/post/dto/request/PostUpdateRequestDto.java
@@ -8,8 +8,8 @@ public record PostUpdateRequestDto(
     PostStatus status,
     String title,
     String content,
-    // 기존 이미지 중 유지할 이미지 ID 목록
-    List<Long> keepImageIds,
+    // 기존 이미지 중 유지할 이미지 URL 목록
+    List<String> keepImageUrls,
     String videoUrl,
     List<String> tags
 ) {
diff --git a/src/main/java/com/back/domain/post/post/repository/PostLikeRepository.java b/src/main/java/com/back/domain/post/post/repository/PostLikeRepository.java
@@ -7,5 +7,7 @@
 import org.springframework.data.jpa.repository.JpaRepository;
 
 public interface PostLikeRepository extends JpaRepository<PostLike, Long> {
+  Boolean existsByPostAndUser(Post post, User user);
+
   Optional<PostLike> findByPostAndUser(Post post, User user);
 }
diff --git a/src/main/java/com/back/domain/post/post/service/PostService.java b/src/main/java/com/back/domain/post/post/service/PostService.java
@@ -24,17 +24,18 @@
 import com.back.global.file.dto.UploadedFileDto;
 import com.back.global.file.service.FileService;
 import com.back.global.rq.Rq;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.NoSuchElementException;
+import java.util.Optional;
+import java.util.stream.Collectors;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 import org.springframework.transaction.support.TransactionSynchronization;
 import org.springframework.transaction.support.TransactionSynchronizationManager;
 import org.springframework.web.multipart.MultipartFile;
 
-import java.util.*;
-import java.util.function.Function;
-import java.util.stream.Collectors;
-
 @Service
 @RequiredArgsConstructor
 public class PostService {
@@ -146,63 +147,73 @@ public PostResponseDto updatePost(Long postId, PostUpdateRequestDto reqBody, Lis
     if (reqBody.content() != null && !reqBody.content().isBlank()) {
       post.updateContent(reqBody.content());
     }
+
+    List<String> addedImgUrls = List.of();
+    List<String> uploadedFileNames = List.of();
+
     if (images != null && !images.isEmpty()) {
       // 새 이미지 업로드
       List<UploadedFileDto> uploaded = fileService.uploadFiles(images);
-      List<String> uploadedFileNames = uploaded.stream().map(UploadedFileDto::fileName).toList();
+      addedImgUrls = uploaded.stream().map(UploadedFileDto::url).toList();
+      uploadedFileNames = uploaded.stream().map(UploadedFileDto::fileName).toList();
+    }
+
+      // 요청 DTO에서 "유지할 이미지 URL 목록" 꺼내기
+      List<String> keepImageUrls = new ArrayList<>(
+          Optional.ofNullable(reqBody.keepImageUrls()).orElse(List.of()));
+    keepImageUrls.addAll(addedImgUrls);
 
-      // 요청 DTO에서 "유지할 이미지 ID 목록" 꺼내기
-      List<Long> keepIds = Optional.ofNullable(reqBody.keepImageIds()).orElse(List.of());
+    // 🔹 현재 게시글의 모든 이미지 가져오기
+    List<PostImage> existingImages = new ArrayList<>(post.getImages());
 
-      // 현재 게시글의 이미지들을 (id -> 객체) 매핑으로 변환
-      Map<Long, PostImage> existingById = post.getImages().stream()
-          .collect(Collectors.toMap(PostImage::getId, Function.identity()));
+      // 삭제될 이미지 (DB + S3)
+      List<PostImage> toRemove = post.getImages().stream()
+          .filter(img -> !keepImageUrls.contains(img.getUrl()))
+          .toList();
 
-      // 삭제할 이미지 찾기
-      List<PostImage> toDelete = post.getImages().stream()
-          .filter(img -> !keepIds.contains(img.getId()))
+      List<String> deleteKeysAfterCommit = toRemove.stream()
+          .map(PostImage::getFileName)
           .toList();
 
-      // 최종 이미지 리스트 구성
-      List<PostImage> finalImages = new ArrayList<>();
+      toRemove.forEach(img -> img.updatePost(null)); // 관계 해제
+      post.getImages().removeAll(toRemove); // orphanRemoval 트리거
+
+      // 유지할 이미지 정렬
       int order = 0;
-      for (Long keepId : keepIds) {
-        PostImage img = existingById.get(keepId);
-        if (img != null) {
-          img.updateSortOrder(order++);
-          finalImages.add(img);
+      for (String url : keepImageUrls) {
+        // 기존 이미지인지 확인
+        PostImage existing = existingImages.stream()
+            .filter(img -> img.getUrl().equals(url))
+            .findFirst()
+            .orElse(null);
+
+        if (existing != null) {
+          existing.updateSortOrder(order++);
+        } else {
+          // 새로 추가된 이미지
+          post.getImages().add(PostImage.builder()
+              .post(post)
+              .fileName(extractFileNameFromUrl(url)) // URL에서 파일명 추출 함수 아래 참고
+              .url(url)
+              .sortOrder(order++)
+              .build());
         }
       }
-      for (UploadedFileDto u : uploaded) {
-        finalImages.add(PostImage.builder()
-            .post(post)
-            .fileName(u.fileName())
-            .url(u.url())
-            .sortOrder(order++)
-            .build()
-        );
-      }
-
-      // 삭제 예정 key 모음
-      List<String> deleteKeysAfterCommit = toDelete.stream()
-          .map(PostImage::getFileName)
-          .toList();
 
-      // DB에 반영
-      post.updateImages(finalImages);
+    List<String> uploadedNames = new ArrayList<>(uploadedFileNames);
 
       // 트랜잭션 완료 후 처리
       TransactionSynchronizationManager.registerSynchronization(new TransactionSynchronization() {
         @Override
         public void afterCompletion(int status) {
           if (status == STATUS_ROLLED_BACK) {
-            uploadedFileNames.forEach(fileService::deleteFile);
+            uploadedNames.forEach(fileService::deleteFile);
           } else if (status == STATUS_COMMITTED) {
             deleteKeysAfterCommit.forEach(fileService::deleteFile);
           }
         }
       });
-    }
+
     if (reqBody.videoUrl() != null && !reqBody.videoUrl().isBlank()) {
       post.updateVideo(reqBody.videoUrl());
     }
@@ -214,6 +225,13 @@ public void afterCompletion(int status) {
     return new PostResponseDto(post);
   }
 
+  private String extractFileNameFromUrl(String url) {
+    if (url == null) return null;
+    int lastSlash = url.lastIndexOf('/');
+    return (lastSlash != -1) ? url.substring(lastSlash + 1) : url;
+  }
+
+
   // 게시글 삭제 로직
   @Transactional
   public void deletePost(Long postId) {
@@ -272,6 +290,17 @@ public PostLikeResponseDto toggleLike(Long postId) {
     }
   }
 
+  // 사용자가 해당 게시글 여부 확인 로직
+  @Transactional(readOnly = true)
+  public Boolean getLike(Long postId) {
+    User user = rq.getActor();
+
+    Post post = postRepository.findById(postId)
+        .orElseThrow(() -> new NoSuchElementException("해당 게시글을 찾을 수 없습니다. ID: " + postId));
+
+    return postLikeRepository.existsByPostAndUser(post, user);
+  }
+
   // 태그 추가 메서드
   private void addTag(List<String> tagNames, Post post) {
     for (String tagName : tagNames) {
diff --git a/src/main/java/com/back/domain/user/controller/UserAuthController.java b/src/main/java/com/back/domain/user/controller/UserAuthController.java
@@ -55,9 +55,7 @@ public RsData<Void> logout(HttpServletRequest request, HttpServletResponse respo
 
     @Operation(summary = "현재 로그인한 유저 정보 조회", description = "세션 유효성 검증 및 사용자 정보 반환")
     @ApiResponses(value = {
-            @ApiResponse(responseCode = "200", description = "인증된 유저 정보 반환 성공"),
-            @ApiResponse(responseCode = "401", description = "인증되지 않은 사용자"),
-            @ApiResponse(responseCode = "500", description = "서버 내부 오류")
+            @ApiResponse(responseCode = "200", description = "사용자 정보 조회"),
     })
 
     @GetMapping("/me")
diff --git a/src/main/java/com/back/domain/user/service/UserAuthService.java b/src/main/java/com/back/domain/user/service/UserAuthService.java
@@ -234,21 +234,28 @@ public void setFirstLoginFalse(Long id) {
     }
 
     // 현재 로그인한 사용자 정보 조회 (세션 검증용)
+    // 변경: 항상 200 응답, 비로그인 시 user: null 반환
     public UserMeResDto getCurrentUser() {
         try {
             User actor = rq.getActor();
 
+            // 비로그인 상태: user null 반환
             if (actor == null) {
-                log.debug("인증되지 않은 사용자");
-                throw new ServiceException(401, "인증되지 않은 사용자");
+                log.debug("인증되지 않은 사용자 - user: null 반환");
+                return UserMeResDto.builder()
+                        .user(null)
+                        .build();
             }
 
             Optional<User> userOpt = userRepository.findById(actor.getId());
             if (userOpt.isEmpty()) {
                 log.warn("사용자 ID {}를 DB에서 찾을 수 없음 (토큰은 유효하나 사용자 삭제됨)", actor.getId());
-                throw new ServiceException(401, "인증되지 않은 사용자");
+                return UserMeResDto.builder()
+                        .user(null)
+                        .build();
             }
 
+            // 로그인 상태: user 정보 반환
             User user = userOpt.get();
             String provider = extractProvider(user.getOauthId());
 
@@ -263,11 +270,12 @@ public UserMeResDto getCurrentUser() {
                             .build())
                     .build();
 
-        } catch (ServiceException e) {
-            throw e;
         } catch (Exception e) {
             log.error("사용자 정보 조회 중 서버 오류 발생: {}", e.getMessage(), e);
-            throw new ServiceException(500, "서버 내부 오류");
+            // 예외 발생 시에도 user: null 반환
+            return UserMeResDto.builder()
+                    .user(null)
+                    .build();
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -7,5 +7,7 @@`
`7`	`7`	`import org.springframework.data.jpa.repository.JpaRepository;`
`8`	`8`
`9`	`9`	`public interface PostLikeRepository extends JpaRepository<PostLike, Long> {`
	`10`	`+ Boolean existsByPostAndUser(Post post, User user);`
	`11`	`+`
`10`	`12`	`Optional<PostLike> findByPostAndUser(Post post, User user);`
`11`	`13`	`}`