Refactor blocklist functionality to use loginId instead of IDs

Updated blocklist operations to work with user tags (loginId) instead of numeric IDs for better identification. Modified service, repository, and controller layers accordingly, and introduced BlockListReq DTO for request handling. Adjusted endpoint paths for consistency.

Author: moon-kun-woong

src/main/java/org/dfbf/soundlink/domain/blocklist/controller/BlockListController.java

@@ -3,6 +3,7 @@
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.tags.Tag;
 import lombok.RequiredArgsConstructor;
+import org.dfbf.soundlink.domain.blocklist.dto.BlockListReq;
 import org.dfbf.soundlink.domain.blocklist.dto.BlockReq;
 import org.dfbf.soundlink.domain.blocklist.service.BlockListService;
 import org.dfbf.soundlink.global.exception.ResponseResult;
@@ -25,22 +26,22 @@ public ResponseResult blockUser(
             @AuthenticationPrincipal Long userId,
             @RequestBody BlockReq req
     ) {
-        return blockListService.blockUser(userId, req.blockedUserId());
+        return blockListService.blockUser(userId, req.tag());
     }
 
-    @DeleteMapping("/{blockedUserId}")
+    @DeleteMapping
     @Operation(
             summary = "유저 차단 해제",
             description = "차단한 유저를 해제합니다."
     )
     public ResponseResult unblockUser(
             @AuthenticationPrincipal Long userId,
-            @PathVariable Long blockedUserId
+            @RequestBody BlockListReq req
     ) {
-        return blockListService.unblockUser(userId, blockedUserId);
+        return blockListService.unblockUser(userId, req.blocklistId());
     }
 
-    @GetMapping
+    @GetMapping("/mypage/blackListSearch")
     @Operation(
             summary = "차단 목록 조회",
             description = "해당 유저의 차단 목록을 가져옵니다."

src/main/java/org/dfbf/soundlink/domain/blocklist/dto/BlockListReq.java

@@ -0,0 +1,6 @@
+package org.dfbf.soundlink.domain.blocklist.dto;
+
+public record BlockListReq (
+        Long blocklistId
+){
+}

src/main/java/org/dfbf/soundlink/domain/blocklist/dto/BlockReq.java

@@ -1,6 +1,6 @@
 package org.dfbf.soundlink.domain.blocklist.dto;
 
 public record BlockReq(
-        Long blockedUserId
+        String tag
 ) {
 }

src/main/java/org/dfbf/soundlink/domain/blocklist/repository/BlockListRepository.java

@@ -19,6 +19,17 @@ public interface BlockListRepository extends JpaRepository<Blocklist, Long> {
     )
     List<Blocklist> findAllByUserId(@Param("userId") Long userId);
 
+    @Query(
+            "SELECT b " +
+                    "FROM Blocklist b " +
+                    "WHERE b.user.userId = :userId " +
+                    "AND b.blockedUser.loginId = :loginId"
+    )
+    Optional<Blocklist> findByUserIdAndLoginId(
+            @Param("userId") Long userId,
+            @Param("loginId") String loginId
+            );
+
     @Query(
             "SELECT b " +
                     "FROM Blocklist b " +
@@ -28,5 +39,5 @@ public interface BlockListRepository extends JpaRepository<Blocklist, Long> {
     Optional<Blocklist> findByUserIdAndBlockedUserId(
             @Param("userId") Long userId,
             @Param("blockedUserId") Long blockedUserId
-            );
+    );
 }

src/main/java/org/dfbf/soundlink/domain/blocklist/service/BlockListService.java

@@ -24,18 +24,18 @@ public class BlockListService {
     private final UserRepository userRepository;
 
     @Transactional
-    public ResponseResult blockUser(Long userId, Long blockedUserId) {
+    public ResponseResult blockUser(Long userId, String tag) {
         try {
             User user = userRepository.findById(userId)
                     .orElseThrow(
                             BlockedUserNotFound::new
                     );
-            User blockedUser = userRepository.findById(blockedUserId)
+            User blockedUser = userRepository.findByLoginId(tag)
                     .orElseThrow(
                             BlockingUserNotFound::new
                     );
-            blockListRepository.findByUserIdAndBlockedUserId(
-                    userId, blockedUserId
+            blockListRepository.findByUserIdAndLoginId(
+                    userId, tag
             ).ifPresent(block -> {
                 throw new AlreadyBlockedUser();
             });