Hotfix/mateboard : 게시글 좋아요, 댓글 좋아요 수정 by zelly-log · Pull Request #181 · prgrms-web-devcourse-final-project/WEB3_4_Fr-End_BE

zelly-log · 2025-04-14T15:21:32Z

Hotfix/mateboard : 게시글 좋아요, 댓글 좋아요 수정

🔘 Part

BE
Infra

🔎 PR Type

새로운 기능 추가
버그 수정
주요 코드 리팩토링(성능 최적화 등)
간단 코드 리팩토링(주석, 코드 컨벤션, 오타 수정 등)
기타 (기타 사항 기입)

🔧 작업 내용 상세 작성

[레포 이름 #이슈번호](이슈 주소)
작업 내용을 상세하게 작성해 주세요!

✔️ PR Checklist

커밋 메세지를 컨벤션에 맞게 잘 적용 하였나요?
테스트 코드 작성 / 단위 테스트 or 통합 테스트 진행 하셨나요?

github-actions · 2025-04-14T15:23:52Z

 @RequiredArgsConstructor
 @RestController
-@RequestMapping("/api/v1/mate-board/posts/{mateId}/comments")
+@RequestMapping("/api/v1/mate-board/posts/{matePostId}/comments")


API 경로에 mateId 대신 matePostId를 사용하는 것이 더 명확합니다. 변경 사항은 적절해 보입니다.

github-actions · 2025-04-14T15:23:53Z

     * @return 조회된 댓글 ID
     */
-    @GetMapping("/{id}")
+    @GetMapping("/{commentId}")


id 대신 commentId를 사용하여 경로 변수의 의미를 명확히 하는 것이 좋습니다. 변경 사항은 좋습니다.

github-actions · 2025-04-14T15:23:54Z

     * @return 수정된 댓글 ID
     */
-    @PutMapping("/{id}")
+    @PutMapping("/{commentId}")


id 대신 commentId를 사용하여 경로 변수의 의미를 명확히 하는 것이 좋습니다. 변경 사항은 좋습니다.

github-actions · 2025-04-14T15:23:55Z

     * @return
     */
-    @DeleteMapping("/{id}")
+    @DeleteMapping("/{commentId}")


id 대신 commentId를 사용하여 경로 변수의 의미를 명확히 하는 것이 좋습니다. 변경 사항은 좋습니다.

github-actions · 2025-04-14T15:23:56Z

@@ -1,7 +1,6 @@
 package com.frend.planit.domain.mateboard.comment.controller;

 import com.frend.planit.domain.mateboard.comment.service.MateCommentLikeService;


User 객체 대신 userId를 사용하도록 변경되었습니다. 이유를 확인해야 합니다. UserService 또는 다른 곳에서 User 객체를 가져오는 로직이 변경되었을 가능성이 높습니다. 상세한 이유를 추가로 설명해주세요.

github-actions · 2025-04-14T15:23:57Z

 @RestController
 @RequiredArgsConstructor
-@RequestMapping("/api/v1/mateposts/comments")
+@RequestMapping("/api/v1/mate-board/posts/{matePostId}/comments/")


경로가 /api/v1/mateposts/comments 에서 /api/v1/mate-board/posts/{matePostId}/comments/ 로 변경되었습니다. 일관성을 위해 다른 API 경로와 일치하는지 확인해야 합니다. matePostId를 사용하는 것은 좋습니다.

github-actions · 2025-04-14T15:23:57Z

    public ResponseEntity<Void> likeComment(@PathVariable Long commentId,
-            @AuthenticationPrincipal User user) {
-        mateCommentLikeService.like(commentId, user);
+            @AuthenticationPrincipal Long userId) {


AuthenticationPrincipal을 통해 User 객체 대신 userId를 직접 주입받도록 변경되었습니다. MateCommentLikeService의 변경 사항과 일관성이 있는지 확인해야 합니다. 변경의 이유와 영향을 명확히 해주세요.

github-actions · 2025-04-14T15:23:58Z

    public ResponseEntity<Void> unlikeComment(@PathVariable Long commentId,
-            @AuthenticationPrincipal User user) {
-        mateCommentLikeService.unlike(commentId, user);
+            @AuthenticationPrincipal Long userId) {


AuthenticationPrincipal을 통해 User 객체 대신 userId를 직접 주입받도록 변경되었습니다. MateCommentLikeService의 변경 사항과 일관성이 있는지 확인해야 합니다. 변경의 이유와 영향을 명확히 해주세요.

github-actions · 2025-04-14T15:23:59Z

@@ -1,9 +1,12 @@
 package com.frend.planit.domain.mateboard.comment.service;

+import static com.frend.planit.global.response.ErrorType.USER_NOT_FOUND;


USER_NOT_FOUND 에러 타입을 import 했습니다. 에러 핸들링 로직 추가를 위한 준비로 보입니다.

github-actions · 2025-04-14T15:24:00Z

 import com.frend.planit.domain.mateboard.comment.entity.MateCommentLike;
 import com.frend.planit.domain.mateboard.comment.repository.MateCommentLikeRepository;
 import com.frend.planit.domain.user.entity.User;
+import com.frend.planit.domain.user.repository.UserRepository;


UserRepository를 추가했습니다. 유저 정보를 데이터베이스에서 가져오기 위함으로 보입니다.

github-actions · 2025-04-14T15:24:01Z


    private final MateCommentService mateCommentService;
    private final MateCommentLikeRepository mateCommentLikeRepository;
+    private final UserRepository userRepository;


UserRepository 필드를 추가했습니다. Dependency Injection을 통해 UserRepository를 주입받아 사용할 것으로 예상됩니다.

github-actions · 2025-04-14T15:24:02Z

     *
     * @param commentId
-     * @param user
+     * @param userId


파라미터 user 대신 userId를 사용하도록 변경되었습니다. 이는 Controller에서 User 객체 대신 userId를 받도록 변경된 것과 일관성을 유지하기 위함입니다.

github-actions · 2025-04-14T15:24:03Z

    @Transactional
-    public void like(Long commentId, User user) {
-        // 1. 댓글이 존재하는지 확인
+    public void like(Long commentId, Long userId) {


userId를 사용하여 User를 조회하고, 존재하지 않으면 USER_NOT_FOUND 예외를 발생시키는 로직이 추가되었습니다. 좋은 변경입니다. NullPointerException을 방지하고 에러 핸들링을 개선했습니다.

github-actions · 2025-04-14T15:24:04Z

+        // 2. 댓글이 존재하는지 확인
        MateComment mateComment = mateCommentService.findMateCommentOrThrow(commentId);
-        // 2. 이미 좋아요를 눌렀는지 확인, 이미 눌렀다면 예외
+


좋아요 확인 로직의 라인 번호가 변경되었습니다. 코드 자체는 변경되지 않았지만, 이전에 추가된 userId 확인 로직 때문에 라인 번호가 변경된 것으로 보입니다. 문제는 없습니다.

github-actions · 2025-04-14T15:24:04Z

            throw new ServiceException(ErrorType.ALREADY_LIKED);
        }
-        // 3. 아니라면 저장
+        // 4. 좋아요 안 눌렀으면 저장


저장 로직의 라인 번호가 변경되었습니다. 코드 자체는 변경되지 않았지만, 이전에 추가된 userId 확인 로직 때문에 라인 번호가 변경된 것으로 보입니다. 문제는 없습니다.

github-actions · 2025-04-14T15:24:05Z

     *
     * @param commentId
-     * @param user
+     * @param userId


파라미터 user 대신 userId를 사용하도록 변경되었습니다. 이는 Controller에서 User 객체 대신 userId를 받도록 변경된 것과 일관성을 유지하기 위함입니다.

github-actions · 2025-04-14T15:24:06Z

    @Transactional
-    public void unlike(Long commentId, User user) {
-        // 댓글 존재 확인
+    public void unlike(Long commentId, Long userId) {


userId를 사용하여 User를 조회하고, 존재하지 않으면 USER_NOT_FOUND 예외를 발생시키는 로직이 추가되었습니다. 좋은 변경입니다. NullPointerException을 방지하고 에러 핸들링을 개선했습니다.

github-actions · 2025-04-14T15:24:07Z

+        // 2. 댓글 존재 확인
        MateComment mateComment = mateCommentService.findMateCommentOrThrow(commentId);
-        // 좋아요 기록이 있는지 확인
+


좋아요 기록 확인 로직의 라인 번호가 변경되었습니다. 코드 자체는 변경되지 않았지만, 이전에 추가된 userId 확인 로직 때문에 라인 번호가 변경된 것으로 보입니다. 문제는 없습니다.

github-actions · 2025-04-14T15:24:08Z

                .orElseThrow(() -> new ServiceException(ErrorType.LIKE_NOT_FOUND));

-        // 3. 존재하면 삭제
+        // 4. 존재하면 삭제


삭제 로직의 라인 번호가 변경되었습니다. 코드 자체는 변경되지 않았지만, 이전에 추가된 userId 확인 로직 때문에 라인 번호가 변경된 것으로 보입니다. 문제는 없습니다.

github-actions · 2025-04-14T15:24:09Z

     * @return 조회된 된 게시글 id
     */
-    @GetMapping("/{id}")
+    @GetMapping("/{matePostId}")


id 대신 matePostId를 사용하여 경로 변수의 의미를 명확히 하는 것이 좋습니다. 변경 사항은 좋습니다.

github-actions · 2025-04-14T15:24:09Z

     * @return 수정된 게시글 응답 DTO
     */
-    @PutMapping("/{id}")
+    @PutMapping("/{matePostId}")


id 대신 matePostId를 사용하여 경로 변수의 의미를 명확히 하는 것이 좋습니다. 변경 사항은 좋습니다.

github-actions · 2025-04-14T15:24:10Z

     * @return 삭제한 게시글 응답 DTO
     */
-    @DeleteMapping("/{id}")
+    @DeleteMapping("/{matePostId}")


id 대신 matePostId를 사용하여 경로 변수의 의미를 명확히 하는 것이 좋습니다. 변경 사항은 좋습니다.

github-actions · 2025-04-14T15:24:11Z


 import com.frend.planit.domain.mateboard.post.service.MatePostLikeService;
-import com.frend.planit.domain.user.entity.User;
+import io.swagger.v3.oas.annotations.security.SecurityRequirement;


User entity import 제거 및 SecurityRequirement 추가. User entity 대신 userId를 사용하는 것으로 보이며, Swagger에서 보안 요구사항을 명시적으로 정의합니다. 변경 사항은 좋습니다.

github-actions · 2025-04-14T15:24:12Z

 @RestController
 @RequiredArgsConstructor
-@RequestMapping("api/v1/mateposts")
+@RequestMapping("/api/v1/mate-board/posts/")


API 경로 변경. 일관성을 위해 다른 API 경로와 일치하는지 확인해야 합니다. 변경 사항은 좋습니다.

github-actions · 2025-04-14T15:24:13Z

    private final MatePostLikeService matePostLikeService;

    @PostMapping("{matePostId}/like")
+    @SecurityRequirement(name = "bearerAuth")


SecurityRequirement 추가. Bearer 토큰 인증을 필요로 함을 명시합니다. 변경 사항은 좋습니다.

github-actions · 2025-04-14T15:24:14Z

    public ResponseEntity<Void> like(@PathVariable Long matePostId,
-            @AuthenticationPrincipal User user) {
-        matePostLikeService.like(matePostId, user);
+            @AuthenticationPrincipal Long userId) {


AuthenticationPrincipal을 통해 User 객체 대신 userId를 직접 주입받도록 변경되었습니다. MatePostLikeService의 변경 사항과 일관성이 있는지 확인해야 합니다. 변경의 이유와 영향을 명확히 해주세요.

github-actions · 2025-04-14T15:24:15Z

    public ResponseEntity<Void> unlike(@PathVariable Long matePostId,
-            @AuthenticationPrincipal User user) {
-        matePostLikeService.unLike(matePostId, user);
+            @AuthenticationPrincipal Long userId) {


AuthenticationPrincipal을 통해 User 객체 대신 userId를 직접 주입받도록 변경되었습니다. MatePostLikeService의 변경 사항과 일관성이 있는지 확인해야 합니다. 변경의 이유와 영향을 명확히 해주세요.

github-actions · 2025-04-14T15:24:16Z

@@ -1,10 +1,12 @@
 package com.frend.planit.domain.mateboard.post.service;

+import static com.frend.planit.global.response.ErrorType.USER_NOT_FOUND;


USER_NOT_FOUND 에러 타입을 import 했습니다. 에러 핸들링 로직 추가를 위한 준비로 보입니다.

github-actions · 2025-04-14T15:24:16Z

 import com.frend.planit.domain.mateboard.post.repository.MatePostLikeRepository;
-import com.frend.planit.domain.mateboard.post.repository.MateRepository;
 import com.frend.planit.domain.user.entity.User;
+import com.frend.planit.domain.user.repository.UserRepository;


UserRepository를 추가했습니다. 유저 정보를 데이터베이스에서 가져오기 위함으로 보입니다.

github-actions · 2025-04-14T15:24:17Z

@@ -16,43 +18,50 @@
 public class MatePostLikeService {

    private final MatePostLikeRepository matePostLikeRepository;


MateRepository 필드가 제거되었습니다. 게시글 정보는 다른 방법으로 가져오는 것으로 변경된 것 같습니다. 변경의 이유를 명확히 해주세요.

github-actions · 2025-04-14T15:24:19Z

-    private final MateRepository mateRepository;
    private final MateService mateService;
-
+    private final UserRepository userRepository;


UserRepository 필드를 추가했습니다. Dependency Injection을 통해 UserRepository를 주입받아 사용할 것으로 예상됩니다.

github-actions · 2025-04-14T15:24:20Z

    @Transactional
-    public void like(Long postId, User user) {
-        // 1. 게시글 존재하는지 확인
+    public void like(Long postId, Long userId) {


userId를 사용하여 User를 조회하고, 존재하지 않으면 USER_NOT_FOUND 예외를 발생시키는 로직이 추가되었습니다. 좋은 변경입니다. NullPointerException을 방지하고 에러 핸들링을 개선했습니다.

github-actions · 2025-04-14T15:24:21Z

                .isPresent();

-        // 2. 이미 좋아요 눌렀는지 확인
+        // 3. 이미 좋아요 눌렀는지 확인


좋아요 확인 로직의 라인 번호가 변경되었습니다. 코드 자체는 변경되지 않았지만, 이전에 추가된 userId 확인 로직 때문에 라인 번호가 변경된 것으로 보입니다. 문제는 없습니다.

github-actions · 2025-04-14T15:24:22Z

        }

-        // 3. 좋아요 안 눌렀으면 저장
+        // 4. 좋아요 안 눌렀으면 저장


저장 로직의 라인 번호가 변경되었습니다. 코드 자체는 변경되지 않았지만, 이전에 추가된 userId 확인 로직 때문에 라인 번호가 변경된 것으로 보입니다. 문제는 없습니다.

github-actions · 2025-04-14T15:24:22Z

    @Transactional
-    public void unLike(Long postId, User user) {
-        // 1. 게시글 존재하는지 확인
+    public void unLike(Long postId, Long userId) {


userId를 사용하여 User를 조회하고, 존재하지 않으면 USER_NOT_FOUND 예외를 발생시키는 로직이 추가되었습니다. 좋은 변경입니다. NullPointerException을 방지하고 에러 핸들링을 개선했습니다.

github-actions · 2025-04-14T15:24:23Z


-        // 2. 좋아요 기록 조회
-        MatePostLike like = matePostLikeRepository.findByUserAndMatePost(user, mate)
+        // 3. 좋아요 기록 조회


좋아요 기록 조회 로직의 라인 번호가 변경되었습니다. 코드 자체는 변경되지 않았지만, 이전에 추가된 userId 확인 로직 때문에 라인 번호가 변경된 것으로 보입니다. 문제는 없습니다. 변수 이름 like에서 matePostUnlike로 변경된 부분은 확인이 필요합니다. 의도가 명확하지 않습니다.

github-actions · 2025-04-14T15:24:24Z

-        // 3. 존재하면 삭제
-        matePostLikeRepository.delete(like);
-
+        // 4. 존재하면 삭제


삭제 로직의 라인 번호가 변경되었습니다. 코드 자체는 변경되지 않았지만, 이전에 추가된 userId 확인 로직 때문에 라인 번호가 변경된 것으로 보입니다. 문제는 없습니다. 변수 이름 변경과 마찬가지로 like에서 matePostUnlike로 변경된 부분에 대한 추가 설명이 필요합니다.

zelly-log and others added 10 commits April 14, 2025 10:51

[Feat] 메이트 게시글 groupby 추가

ec79244

[Refactor] 메이트 게시글 이미지 부분 수정

9c6b30a

Merge branch 'develop' into hotfix/Mateboard

71773f0

[Refactor] bio 누락된 값 추가

04a7544

[Refactor] isApplied로 동행 신청 여부 판단 필드 추가

2063919

[Refactor] 좋아요 여부 배열 추가

4561250

Merge branch 'develop' into hotfix/Mateboard

8c14602

[Refactor] Travel 조회 오류 수정 (#180)

744251c

[Refactor] isApplied로 동행 신청 여부 판단 필드 추가

22139eb

[Refactor] 게시글 좋아요, 댓글 좋아요 반영

11b142d

zelly-log added bug 버그 관련 fix 기능 수정 labels Apr 14, 2025

zelly-log requested a review from sete3683 April 14, 2025 15:21

zelly-log self-assigned this Apr 14, 2025

Merge branch 'develop' into hotfix/Mateboard

9f0fa38

github-actions Bot reviewed Apr 14, 2025

View reviewed changes

sete3683 merged commit 4f9a9df into develop Apr 14, 2025
1 check passed

		@@ -1,7 +1,6 @@
		package com.frend.planit.domain.mateboard.comment.controller;

		import com.frend.planit.domain.mateboard.comment.service.MateCommentLikeService;

		@@ -1,9 +1,12 @@
		package com.frend.planit.domain.mateboard.comment.service;

		import static com.frend.planit.global.response.ErrorType.USER_NOT_FOUND;

		@@ -1,10 +1,12 @@
		package com.frend.planit.domain.mateboard.post.service;

		import static com.frend.planit.global.response.ErrorType.USER_NOT_FOUND;

		@@ -16,43 +18,50 @@
		public class MatePostLikeService {

		private final MatePostLikeRepository matePostLikeRepository;

Conversation

zelly-log commented Apr 14, 2025

🔘 Part

🔎 PR Type

🔧 작업 내용 상세 작성

✔️ PR Checklist

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Apr 14, 2025