[#151] Fix: PinOrderUpdate 반환값 LexoRank로 변경

Nexters · Mar 1, 2024 · 87e692b · 87e692b
1 parent 4d9100a
commit 87e692b
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 7 deletions.
diff --git a/src/main/java/com/pcb/audy/domain/pin/controller/PinController.java b/src/main/java/com/pcb/audy/domain/pin/controller/PinController.java
@@ -35,7 +35,7 @@ public BasicResponse<PinSaveRes> savePin(
     @SendTo("/sub/{courseId}/pin/modification/sequence")
     public BasicResponse<PinOrderUpdateRes> updateOrder(
             @DestinationVariable Long courseId, @RequestBody PinOrderUpdateReq pinOrderUpdateReq) {
-        return BasicResponse.success(pinService.updatePinOrder(courseId, pinOrderUpdateReq));
+        return BasicResponse.success(pinService.updatePinSequence(courseId, pinOrderUpdateReq));
     }
 
     @MessageMapping("/{courseId}/pin/modification/name")

diff --git a/src/main/java/com/pcb/audy/domain/pin/controller/PinHttpController.java b/src/main/java/com/pcb/audy/domain/pin/controller/PinHttpController.java
@@ -28,7 +28,7 @@ public BasicResponse<PinSaveRes> savePin(
     @PatchMapping("/{courseId}/order")
     public BasicResponse<PinOrderUpdateRes> updatePinOrder(
             @PathVariable("courseId") Long courseId, @RequestBody PinOrderUpdateReq pinOrderUpdateReq) {
-        return BasicResponse.success(pinService.updatePinOrder(courseId, pinOrderUpdateReq));
+        return BasicResponse.success(pinService.updatePinSequence(courseId, pinOrderUpdateReq));
     }
 
     @PatchMapping("/{courseId}/name")

diff --git a/src/main/java/com/pcb/audy/domain/pin/dto/response/PinOrderUpdateRes.java b/src/main/java/com/pcb/audy/domain/pin/dto/response/PinOrderUpdateRes.java
@@ -10,11 +10,11 @@
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 public class PinOrderUpdateRes {
     private UUID pinId;
-    private int order;
+    private String sequence;
 
     @Builder
-    private PinOrderUpdateRes(UUID pinId, int order) {
+    private PinOrderUpdateRes(UUID pinId, String sequence) {
         this.pinId = pinId;
-        this.order = order;
+        this.sequence = sequence;
     }
 }
diff --git a/src/main/java/com/pcb/audy/domain/pin/service/PinService.java b/src/main/java/com/pcb/audy/domain/pin/service/PinService.java
@@ -39,7 +39,7 @@ public PinSaveRes savePin(Long courseId, PinSaveReq pinSaveReq) {
         return PinServiceMapper.INSTANCE.toPinSaveRes(pinRedisRes);
     }
 
-    public PinOrderUpdateRes updatePinOrder(Long courseId, PinOrderUpdateReq pinOrderUpdateReq) {
+    public PinOrderUpdateRes updatePinSequence(Long courseId, PinOrderUpdateReq pinOrderUpdateReq) {
         String key = getKey(courseId, pinOrderUpdateReq.getPinId());
         String sequence = lexoRankUtil.getLexoRank(courseId, pinOrderUpdateReq.getOrder());
 
@@ -57,7 +57,10 @@ public PinOrderUpdateRes updatePinOrder(Long courseId, PinOrderUpdateReq pinOrde
                         .build();
 
         redisProvider.setPin(key, updatedPinRedisRes, PIN_EXPIRE_TIME);
-        return PinServiceMapper.INSTANCE.toPinOrderUpdateRes(pinOrderUpdateReq);
+        return PinOrderUpdateRes.builder()
+                .pinId(updatedPinRedisRes.getPinId())
+                .sequence(updatedPinRedisRes.getSequence())
+                .build();
     }
 
     public PinNameUpdateRes updatePinName(Long courseId, PinNameUpdateReq pinNameUpdateReq) {