[SSD-143] 타임페이퍼 삭제 기능 (#84)

devyejin · web-flow · commit ab884807dec4 · 2025-03-05T20:56:37.000+09:00
* [SSD-143] chore: 전역 예외 처리

* [SSD-143] refactor: 롤링페이퍼 삭제 로직

- 롤링페이퍼 삭제 시 포스트잇 연관 관계만 해제하도록 변경

* [SSD-143] fix: 롤링페이퍼 삭제시 author 유효성 검증
diff --git a/backend/src/main/java/com/timepaper/backend/domain/postit/entity/Postit.java b/backend/src/main/java/com/timepaper/backend/domain/postit/entity/Postit.java
@@ -36,7 +36,7 @@ public class Postit extends BaseTimeEntity {
   private Long id;
 
   @ManyToOne(fetch = FetchType.LAZY)
-  @JoinColumn(name = "time_paper_id", nullable = false)
+  @JoinColumn(name = "time_paper_id", nullable = true)
   private TimePaper timePaper;
 
   @ManyToOne(fetch = FetchType.LAZY)
diff --git a/backend/src/main/java/com/timepaper/backend/domain/postit/repository/PostitRepository.java b/backend/src/main/java/com/timepaper/backend/domain/postit/repository/PostitRepository.java
@@ -7,6 +7,9 @@
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.Pageable;
 import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.data.jpa.repository.Modifying;
+import org.springframework.data.jpa.repository.Query;
+import org.springframework.data.repository.query.Param;
 
 public interface PostitRepository extends JpaRepository<Postit, Long> {
 
@@ -15,4 +18,8 @@ public interface PostitRepository extends JpaRepository<Postit, Long> {
   List<Postit> findAllByAuthor(User author);
 
   void deleteByTimePaperId(UUID timepaperId);
+
+  @Modifying
+  @Query("UPDATE Postit p SET p.timePaper = NULL WHERE p.timePaper.id = :timePaperId")
+  void unLinkPostits(@Param("timePaperId") UUID timePaperId);
 }
diff --git a/backend/src/main/java/com/timepaper/backend/domain/timepaper/controller/TimePaperController.java b/backend/src/main/java/com/timepaper/backend/domain/timepaper/controller/TimePaperController.java
@@ -40,31 +40,33 @@ public ResponseEntity<ApiResponse<TimePaperResponseDto>> createTimePaper(
         timePaperService.createTimePaper(timePaperCreateRequestDto, authentication);
 
     return ResponseEntity.status(HttpStatus.CREATED)
-               .body(ApiResponse
-                         .ok("타임페이퍼 생성 성공",
-                             "CREATED",
-                             responseDto));
+        .body(ApiResponse
+            .ok("타임페이퍼 생성 성공",
+                "CREATED",
+                responseDto));
   }
 
   @GetMapping("/{timepaperId}")
-  public ResponseEntity<ApiResponse<TimePaperResponseDto>> getTimePaperById(@PathVariable UUID timepaperId) {
+  public ResponseEntity<ApiResponse<TimePaperResponseDto>> getTimePaperById(
+      @PathVariable UUID timepaperId) {
 
     TimePaperResponseDto responseDto = timePaperService.getTimePaperById(timepaperId);
     return ResponseEntity.status(HttpStatus.OK)
-               .body(ApiResponse
-                         .ok("타임페이퍼 조회 성공",
-                             "OK",
-                             responseDto));
+        .body(ApiResponse
+            .ok("타임페이퍼 조회 성공",
+                "OK",
+                responseDto));
   }
 
   @DeleteMapping("/{timepaperId}")
-  public ResponseEntity<ApiResponse<Void>> deleteTimePaper(@PathVariable UUID timepaperId) {
-    timePaperService.deleteTimePaper(timepaperId);
+  public ResponseEntity<ApiResponse<Void>> deleteTimePaper(@PathVariable UUID timepaperId,
+      @AuthenticationPrincipal User user) {
+    timePaperService.deleteTimePaper(timepaperId, user.getId());
     return ResponseEntity.status(HttpStatus.NO_CONTENT)
-               .body(ApiResponse
-                         .ok("타임페이퍼 삭제 성공",
-                             "NO_CONTENT",
-                             null));
+        .body(ApiResponse
+            .ok("타임페이퍼 삭제 성공",
+                "NO_CONTENT",
+                null));
   }
 
   @PatchMapping("/{timePaperId}/lock")
@@ -77,7 +79,7 @@ public ResponseEntity<ApiResponse<TimePaperLockResponseDto>> lockTimePaper(
     TimePaperLockResponseDto responseDto = timePaperService.lockTimePaper(timePaperId,
         timePaperLockRequestDto, requester.getId());
     return ResponseEntity.status(HttpStatus.OK)
-               .body(ApiResponse.ok("타임페이퍼 잠금 처리 성공", "OK", responseDto));
+        .body(ApiResponse.ok("타임페이퍼 잠금 처리 성공", "OK", responseDto));
   }
 
 }
diff --git a/backend/src/main/java/com/timepaper/backend/domain/timepaper/service/TimePaperService.java b/backend/src/main/java/com/timepaper/backend/domain/timepaper/service/TimePaperService.java
@@ -14,10 +14,12 @@
 import com.timepaper.backend.global.exception.custom.common.ResourceNotFoundException;
 import java.util.UUID;
 import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.security.core.Authentication;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
+@Slf4j
 @Service
 @RequiredArgsConstructor
 @Transactional(readOnly = true)
@@ -54,15 +56,16 @@ public TimePaperResponseDto getTimePaperById(UUID timepaperId) {
   }
 
   @Transactional
-  public void deleteTimePaper(UUID timepaperId) {
-    // 삭제할 타임페이퍼 조회
+  public void deleteTimePaper(UUID timepaperId, Long userId) {
+
     TimePaper timePaper = timePaperRepository.findById(timepaperId)
         .orElseThrow(() -> new ResourceNotFoundException(ErrorCode.TIMEPAPER_NOT_FOUND));
 
-    // 1. 해당 타임페이퍼를 참조하는 모든 포스트잇 삭제
-    postitRepository.deleteByTimePaperId(timepaperId);
+    if (timePaper.getCreator().getId() != userId) {
+      throw new ForBiddenException(ErrorCode.AUTHOR_ONLY);
+    }
 
-    // 2. 타임페이퍼 삭제
+    postitRepository.unLinkPostits(timepaperId);
     timePaperRepository.delete(timePaper);
   }
 
diff --git a/backend/src/main/java/com/timepaper/backend/global/exception/handler/GlobalExceptionHandler.java b/backend/src/main/java/com/timepaper/backend/global/exception/handler/GlobalExceptionHandler.java
@@ -35,4 +35,13 @@ public ResponseEntity<ApiResponse<Object>> malUploadSizeExceededHandler(
             .error(ErrorCode.PAYLOAD_TOO_LARGE.getMessage(),
                 ErrorCode.PAYLOAD_TOO_LARGE.getCode()));
   }
+
+//  @ExceptionHandler(RuntimeException.class)
+//  public ResponseEntity<ApiResponse<Object>> exceptionHandler(RuntimeException ex) {
+//    return ResponseEntity.status(ErrorCode.SERVER_ERROR.getStatus())
+//        .body(ApiResponse
+//            .error(ErrorCode.SERVER_ERROR.getMessage(), ErrorCode.SERVER_ERROR.getCode()));
+//  }
+
+
 }
diff --git a/backend/src/main/resources/application.properties b/backend/src/main/resources/application.properties
@@ -11,7 +11,7 @@ jwt.secret=${JWT_SECRET}
 spring.data.redis.host=${REDIS_HOST}
 spring.data.redis.port=${REDIS_PORT}
 spring.data.redis.password=${REDIS_PASSWORD}
-#JavaEmailSender관련 설정
+#JavaEmailSender관련 설정l
 spring.mail.host=${MAIL_HOST}
 spring.mail.port=${MAIL_PORT}
 spring.mail.username=${MAIL_USERNAME}
diff --git a/frontend/src/pages/login/Login.jsx b/frontend/src/pages/login/Login.jsx
@@ -18,7 +18,7 @@ export default function Login() {
   const location = useLocation();
   const { isLoggedIn } = useSelector((state) => state.auth);
 
-  const handleLoginButtonClick = (event) => {
+  const handleLoginButtonClick = async (event) => {
     setIsLoginButtonEnable(false);
     event.preventDefault();
     (async () => {
diff --git a/frontend/yarn.lock b/frontend/yarn.lock

Original file line number	Diff line number	Diff line change
`@@ -35,4 +35,13 @@ public ResponseEntity<ApiResponse<Object>> malUploadSizeExceededHandler(`
`35`	`35`	`.error(ErrorCode.PAYLOAD_TOO_LARGE.getMessage(),`
`36`	`36`	`ErrorCode.PAYLOAD_TOO_LARGE.getCode()));`
`37`	`37`	`}`
	`38`	`+`
	`39`	`+// @ExceptionHandler(RuntimeException.class)`
	`40`	`+// public ResponseEntity<ApiResponse<Object>> exceptionHandler(RuntimeException ex) {`
	`41`	`+// return ResponseEntity.status(ErrorCode.SERVER_ERROR.getStatus())`
	`42`	`+// .body(ApiResponse`
	`43`	`+// .error(ErrorCode.SERVER_ERROR.getMessage(), ErrorCode.SERVER_ERROR.getCode()));`
	`44`	`+// }`
	`45`	`+`
	`46`	`+`
`38`	`47`	`}`