refactor: 임시저장 편지 API 수정 (WR9-120)

Author: leeys9423

src/main/java/io/crops/warmletter/domain/letter/controller/LetterController.java

@@ -81,20 +81,19 @@ public ResponseEntity<BaseResponse<Void>> evaluateLetter(@PathVariable Long lett
     /**
      * 임시저장 편지 생성
      */
-    @PostMapping("/letters/{letterId}/temporary-save")
+    @PostMapping("/letters/temporary-save")
     public ResponseEntity<BaseResponse<LetterResponse>> temporarySaveLetter(
-            @PathVariable (name="letterId")Long letterId, @Valid @RequestBody TemporarySaveLetterRequest request) {
+            @Valid @RequestBody TemporarySaveLetterRequest request) {
 
-        return ResponseEntity.ok(BaseResponse.of(letterService.temporarySaveLetter(letterId, request),"임시 저장 완료 "));
+        return ResponseEntity.ok(BaseResponse.of(letterService.temporarySaveLetter(request),"임시 저장 완료 "));
     }
 
     /**
      * 임시저장 편지 삭제
      */
     @DeleteMapping("/letters/{letterId}/temporary-save")
     public ResponseEntity<BaseResponse<Map<String,Long>>> deleteTemporarySaveLetter(@PathVariable Long letterId) {
-        return ResponseEntity.status(HttpStatus.OK)
-                .body(new BaseResponse<>(letterService.deleteTemporarySaveLetter(letterId),"임시 저장 편지 삭제 완료"));
+        return ResponseEntity.ok(BaseResponse.of(letterService.deleteTemporarySaveLetter(letterId),"임시 저장 편지 삭제 완료"));
     }
 
     /**

src/main/java/io/crops/warmletter/domain/letter/dto/request/TemporarySaveLetterRequest.java

@@ -4,18 +4,23 @@
 import io.crops.warmletter.domain.letter.enums.FontType;
 import io.crops.warmletter.domain.letter.enums.PaperType;
 import jakarta.validation.constraints.NotBlank;
+import jakarta.validation.constraints.NotNull;
 import lombok.*;
 
 @Getter
-@Setter
 @NoArgsConstructor
 @AllArgsConstructor
 public class TemporarySaveLetterRequest {
 
+    private Long letterId;
+
+    @NotNull(message = "받는 사람을 설정해주세요.")
     private Long receiverId;
 
+    @NotNull(message = "매칭 번호를 설정해주세요.")
     private Long matchingId;
 
+    @NotNull(message = "상위 편지를 설정해주세요.")
     private Long parentLetterId;
 
     @NotBlank(message = "제목을 입력해주세요.")

src/main/java/io/crops/warmletter/domain/letter/entity/Letter.java

@@ -91,15 +91,20 @@ public void updateLetterType(LetterType letterType) {
         this.letterType = letterType;
     }
 
-
-    public void updateTemporarySave(Long receiverId, Long matchingId, Long parentLetterId, Category category, String title, String content) {
-        this.receiverId = receiverId;
-        this.matchingId = matchingId;
-        this.parentLetterId = parentLetterId;
-        this.category = category;
+    public void updateTitle(String title) {
         this.title = title;
+    }
+
+    public void updateContent(String content) {
         this.content = content;
-        this.status = Status.SAVED;
+    }
+
+    public void updatePaperType(PaperType paperType) {
+        this.paperType = paperType;
+    }
+
+    public void updateFontType(FontType fontType) {
+        this.fontType = fontType;
     }
 
     public void updateMatchingId(Long matchingId) {

src/main/java/io/crops/warmletter/domain/letter/service/LetterService.java

@@ -184,21 +184,35 @@ public void evaluateLetter(Long letterId, EvaluateLetterRequest request) {
     }
 
     @Transactional
-    public LetterResponse temporarySaveLetter(Long letterId, TemporarySaveLetterRequest request) {
+    public LetterResponse temporarySaveLetter(TemporarySaveLetterRequest request) {
         Long writerId = authFacade.getCurrentUserId();
         String writerZipCode = authFacade.getZipCode();
 
-        Letter letter = letterRepository.findByIdAndWriterId(letterId, writerId)
-                .orElseThrow(LetterNotBelongException::new);
-
-        letter.updateTemporarySave(
-                request.getReceiverId(),
-                request.getMatchingId(),
-                request.getParentLetterId(),
-                request.getCategory(),
-                request.getTitle(),
-                request.getContent()
-        );
+        Letter letter;
+
+        if (request.getLetterId() == null) {
+            letter = Letter.builder()
+                    .receiverId(request.getReceiverId())
+                    .matchingId(request.getMatchingId())
+                    .parentLetterId(request.getParentLetterId())
+                    .title(request.getTitle())
+                    .content(request.getContent())
+                    .category(request.getCategory())
+                    .paperType(request.getPaperType())
+                    .fontType(request.getFontType())
+                    .status(Status.SAVED)
+                    .build();
+
+            letterRepository.save(letter);
+        } else {
+            letter = letterRepository.findByIdAndWriterId(request.getLetterId(), writerId)
+                    .orElseThrow(LetterNotBelongException::new);
+
+            letter.updateTitle(request.getTitle());
+            letter.updateContent(request.getContent());
+            letter.updatePaperType(request.getPaperType());
+            letter.updateFontType(request.getFontType());
+        }
 
         return LetterResponse.fromEntity(letter, writerZipCode);
     }