Merge pull request #154 from CorkCharge/fix/#153-bookmark-group

[FIX] 그룹명 기존과 같아도 색상 및 공개여부만 변경 가능하도록 로직 수정

Author: jaepar

src/main/java/konkuk/corkCharge/domain/bookmark/repository/RestaurantBookmarkGroupRepository.java

@@ -10,6 +10,7 @@
 public interface RestaurantBookmarkGroupRepository extends JpaRepository<RestaurantBookmarkGroup, Long> {
     List<RestaurantBookmarkGroup> findAllByIdIn(List<Long> ids);
     boolean existsByUser_UserIdAndName(Long userId, String name);
+    boolean existsByUser_UserIdAndNameAndIdNot(Long userId, String name, Long groupId);
     int countByUser_UserId(Long userId);
     List<RestaurantBookmarkGroup> findAllByUser_UserIdOrderByDisplayOrderAsc(Long userId);
     @Query("""

src/main/java/konkuk/corkCharge/domain/bookmark/service/BookmarkGroupService.java

@@ -95,13 +95,15 @@ public PutBookmarkGroupResponse updateGroup(
             throw new CustomException(BAD_REQUEST);
         }
 
-        // 기존 이름과 완전히 동일한 경우
-        if (group.getName().equals(request.name())) {
-            throw new CustomException(GROUP_NAME_SAME_AS_BEFORE);
+        // 기존의 내용과 전체 동일할 경우에만 예외 터뜨릴 것(name, color, visibility)
+        if (group.getName().equals(request.name())
+                && group.getColor().equals(request.color())
+                && group.getVisibility() == request.visibility()) {
+            throw new CustomException(GROUP_CONTENTS_SAME_AS_BEFORE);
         }
 
         // 다른 그룹이 이미 사용 중인 이름인 경우
-        if (groupRepository.existsByUser_UserIdAndName(userId, request.name())) {
+        if (groupRepository.existsByUser_UserIdAndNameAndIdNot(userId, request.name(), groupId)) {
             throw new CustomException(GROUP_NAME_ALREADY_EXISTS);
         }
 

src/main/java/konkuk/corkCharge/global/response/status/BaseExceptionResponseStatus.java

@@ -69,7 +69,7 @@ public enum BaseExceptionResponseStatus implements ResponseStatus {
     GROUP_NOT_FOUND(120001, "해당 북마크 그룹을 찾을 수 없습니다."),
     GROUP_FORBIDDEN(120002, "해당 북마크 그룹에 대한 권한이 없습니다."),
     BOOKMARK_ALREADY_EXISTS(120003, "이미 저장된 매장입니다."),
-    GROUP_NAME_SAME_AS_BEFORE(120004, "기존 그룹명과 동일합니다."),
+    GROUP_CONTENTS_SAME_AS_BEFORE(120004, "기존 그룹 내용과 동일합니다."),
     GROUP_NAME_ALREADY_EXISTS(120005, "이미 존재하는 그룹명입니다."),
 
     /**