버그 수정 및 코드 리팩토링

북마크 정보 수정 시 태그 수정 추가
카테고리 순서 커스텀 기능 추가
그룹 탈퇴 로직 수정(ADMIN 기준 -> 소유주 기준)

Author: son0307

src/main/java/com/sonkim/bookmarking/auth/service/AuthService.java

@@ -68,7 +68,7 @@ public User createAccount(RegisterRequestDto dto) {
         // 기본 개인 그룹 생성
         Team personalTeam = Team.builder()
                 .name(dto.getNickname() + "님의 개인 공간")
-                .user(newUser)
+                .owner(newUser)
                 .build();
         teamService.saveTeam(personalTeam);
 

src/main/java/com/sonkim/bookmarking/domain/bookmark/repository/BookmarkTagRepository.java

@@ -4,6 +4,9 @@
 import org.springframework.data.jpa.repository.JpaRepository;
 import org.springframework.stereotype.Repository;
 
+import java.util.List;
+
 @Repository
 public interface BookmarkTagRepository extends JpaRepository<BookmarkTag, Long>, BookmarkTagRepositoryCustom {
+    List<BookmarkTag> findAllByBookmarkId(Long bookmarkId);
 }

src/main/java/com/sonkim/bookmarking/domain/bookmark/service/BookmarkService.java

@@ -201,6 +201,36 @@ public void updateBookmark(Long userId, Long bookmarkId, BookmarkRequestDto dto)
             }
         }
 
+        if (dto.getTagNames() != null) {
+            // 현재 북마크에 연결된 태그들 조회
+            List<Tag> oldTags = bookmarkTagRepository.findAllByBookmarkId(bookmarkId).stream()
+                    .map(BookmarkTag::getTag)
+                    .toList();
+
+            // 요청으로 들어온 Tag 정보 조회
+            List<Tag> newTags = tagService.findOrCreateTags(userId, bookmark.getTeam().getId(), dto.getTagNames());
+
+            // 삭제할 태그를 찾아서 BookmarkTag 연결 끊기
+            List<BookmarkTag> tagsToRemove = bookmark.getBookmarkTags().stream()
+                    .filter(bookmarkTag -> !newTags.contains(bookmarkTag.getTag()))
+                    .toList();
+            bookmark.getBookmarkTags().removeAll(tagsToRemove);
+            bookmarkTagRepository.deleteAll(tagsToRemove);
+
+            // 추가할 태그를 찾아서 BookmarkTag 생성하여 연결
+            List<Tag> tagsToAdd = newTags.stream()
+                    .filter(tag -> !oldTags.contains(tag))
+                    .toList();
+
+            for (Tag tag : tagsToAdd) {
+                BookmarkTag bookmarkTag = BookmarkTag.builder()
+                        .bookmark(bookmark)
+                        .tag(tag)
+                        .build();
+                bookmarkTagRepository.save(bookmarkTag);
+            }
+        }
+
         bookmark.update(dto);
     }
 

src/main/java/com/sonkim/bookmarking/domain/category/controller/CategoryController.java

@@ -74,4 +74,16 @@ public ResponseEntity<List<CategoryDto.CategoryResponseDto>> deleteCategory(@Aut
         List<CategoryDto.CategoryResponseDto> updatedCategories = categoryService.deleteCategory(userDetails.getId(), categoryId);
         return ResponseEntity.ok(updatedCategories);
     }
+
+    @Operation(summary = "카테고리 순서 일괄 수정", description = "특정 그룹 내 카테고리들의 표시 순서를 한 번에 업데이트합니다. EDITOR 이상의 권한이 필요합니다.")
+    @ApiResponse(responseCode = "200", description = "순서 업데이트 성공")
+    @PatchMapping("/groups/{groupId}/categories/order")
+    public ResponseEntity<Void> updateCategoryPositions(
+            @AuthenticationPrincipal UserDetailsImpl userDetails,
+            @PathVariable Long groupId,
+            @RequestBody List<CategoryDto.UpdatePositionRequestDto> requests
+    ) {
+        categoryService.updateCategoryPositions(userDetails.getId(), groupId, requests);
+        return ResponseEntity.ok().build();
+    }
 }

src/main/java/com/sonkim/bookmarking/domain/category/dto/CategoryDto.java

@@ -23,4 +23,11 @@ public static class CategoryResponseDto {
         private String name;
         private Long bookmarkCount;
     }
+
+    // 카테고리 순서 업데이트 요청 DTO
+    @Data
+    public static class UpdatePositionRequestDto {
+        private Long categoryId;
+        private Integer position;
+    }
 }

src/main/java/com/sonkim/bookmarking/domain/category/entity/Category.java

@@ -25,6 +25,14 @@ public class Category {
     @Column(nullable = false, length = 20)
     private String name;
 
+    // 카테고리 순서
+    @Column(nullable = false)
+    private Integer position;
+
     public void update(CategoryDto.CategoryRequestDto request) {
         if (request.getName() != null) this.name = request.getName();}
+
+    public void updatePosition(Integer position) {
+        this.position = position;
+    }
 }

src/main/java/com/sonkim/bookmarking/domain/category/repository/CategoryRepository.java

@@ -7,4 +7,6 @@
 @Repository
 public interface CategoryRepository extends JpaRepository<Category, Long>, CategoryRepositoryCustom {
     boolean existsByNameAndTeam_Id(String name, Long teamId);
+
+    long countByTeam_Id(Long teamId);
 }

src/main/java/com/sonkim/bookmarking/domain/category/repository/CategoryRepositoryCustom.java

@@ -6,5 +6,5 @@
 
 public interface CategoryRepositoryCustom {
     // 카테고리 정보와 해당 카테고리에 속한 북마크의 개수를 카운트
-    List<CategoryDto.CategoryResponseDto> findAllWithBookmarkCountByTeam_Id(Long teamId);
+    List<CategoryDto.CategoryResponseDto> findAllWithBookmarkCountByTeamId(Long teamId);
 }

src/main/java/com/sonkim/bookmarking/domain/category/repository/CategoryRepositoryCustomImpl.java

@@ -16,7 +16,7 @@ public class CategoryRepositoryCustomImpl implements CategoryRepositoryCustom {
     private final JPAQueryFactory queryFactory;
 
     @Override
-    public List<CategoryDto.CategoryResponseDto> findAllWithBookmarkCountByTeam_Id(Long teamId) {
+    public List<CategoryDto.CategoryResponseDto> findAllWithBookmarkCountByTeamId(Long teamId) {
         return queryFactory
                 .select(Projections.constructor(CategoryDto.CategoryResponseDto.class,
                         category.id,
@@ -27,6 +27,7 @@ public List<CategoryDto.CategoryResponseDto> findAllWithBookmarkCountByTeam_Id(L
                 .leftJoin(bookmark).on(bookmark.category.id.eq(category.id))
                 .where(category.team.id.eq(teamId))
                 .groupBy(category.id, category.name)
+                .orderBy(category.position.asc())
                 .fetch();
     }
 }

src/main/java/com/sonkim/bookmarking/domain/category/service/CategoryService.java

@@ -15,6 +15,8 @@
 
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Map;
+import java.util.stream.Collectors;
 
 @Slf4j
 @RequiredArgsConstructor
@@ -42,11 +44,15 @@ public List<CategoryDto.CategoryResponseDto> createCategory(Long userId, Long te
             throw new IllegalStateException("이미 존재하는 카테고리 이름입니다.");
         }
 
+        // 현재 그룹의 카테고리 개수
+        long currentCategoryCount = categoryRepository.countByTeam_Id(teamId);
+
         // 카테고리 객체 생성 및 저장
         Team team = teamService.getTeamById(teamId);
         Category newCategory = Category.builder()
                 .name(request.getName())
                 .team(team)
+                .position((int) currentCategoryCount + 1)
                 .build();
         categoryRepository.save(newCategory);
 
@@ -57,7 +63,7 @@ public List<CategoryDto.CategoryResponseDto> createCategory(Long userId, Long te
     @Transactional(readOnly = true)
     public List<CategoryDto.CategoryResponseDto> getCategoriesByTeam(Long teamId) {
         // 각 카테고리 정보와 북마크 개수 조회
-        List<CategoryDto.CategoryResponseDto> categoriesWithCount = categoryRepository.findAllWithBookmarkCountByTeam_Id(teamId);
+        List<CategoryDto.CategoryResponseDto> categoriesWithCount = categoryRepository.findAllWithBookmarkCountByTeamId(teamId);
 
         // 그룹 전체 북마크 개수 조회
         long totalBookmarkCount = bookmarkRepository.countByTeam_Id(teamId);
@@ -126,6 +132,26 @@ public List<CategoryDto.CategoryResponseDto> deleteCategory(Long userId, Long ca
         return getCategoriesByTeam(teamId);
     }
 
+    // 카테고리 순서 수정
+    @Transactional
+    public void updateCategoryPositions(Long userId, Long teamId, List<CategoryDto.UpdatePositionRequestDto> requests) {
+        log.info("userId: {}, teamId: {} 카테고리 순서 수정 요청", userId, teamId);
+
+        // 요청자가 EDITOR 권한이 있는지 검사
+        teamMemberService.validateEditor(userId, teamId);
+
+        // 요청된 모든 카테고리 ID 조회
+        List<Long> categoryIds = requests.stream().map(CategoryDto.UpdatePositionRequestDto::getCategoryId).toList();
+        Map<Long, Category> categoryMap = categoryRepository.findAllById(categoryIds).stream()
+                .collect(Collectors.toMap(Category::getId, category -> category));
+
+        // 각 카테고리의 position 값 업데이트
+        for (CategoryDto.UpdatePositionRequestDto request : requests) {
+            Category category = categoryMap.get(request.getCategoryId());
+            category.updatePosition(request.getPosition());
+        }
+    }
+
     public Category getCategoryById(Long categoryId) {
         return categoryRepository.findById(categoryId)
                 .orElseThrow(() -> new EntityNotFoundException("해당 카테고리를 찾을 수 없습니다."));