Merge pull request #30 from 9roomMoa/feat/#26

Feat/#26: 공지 수정, 공지 삭제 api

Author: EveryPine

src/main/java/com/groommoa/aether_back_notification/domain/notices/controller/NoticeController.java

@@ -3,12 +3,14 @@
 import com.groommoa.aether_back_notification.domain.notices.dto.CreateNoticeRequestDto;
 import com.groommoa.aether_back_notification.domain.notices.dto.CreateNoticeResponseDto;
 import com.groommoa.aether_back_notification.domain.notices.dto.NoticeDto;
+import com.groommoa.aether_back_notification.domain.notices.dto.UpdateNoticeRequestDto;
 import com.groommoa.aether_back_notification.domain.notices.entity.Notice;
 import com.groommoa.aether_back_notification.domain.notices.service.NoticeService;
 import com.groommoa.aether_back_notification.global.common.constants.HttpStatus;
 import com.groommoa.aether_back_notification.global.common.response.CommonResponse;
 import com.groommoa.aether_back_notification.global.common.utils.DtoUtils;
 import io.jsonwebtoken.Claims;
+import jakarta.validation.Valid;
 import jakarta.validation.constraints.Max;
 import jakarta.validation.constraints.Min;
 import lombok.RequiredArgsConstructor;
@@ -53,4 +55,34 @@ public ResponseEntity<CommonResponse> getNotices(
         return ResponseEntity.ok(response);
     }
 
+    @PatchMapping("/{noticeId}")
+    public ResponseEntity<CommonResponse> updateNotice(
+            @AuthenticationPrincipal Claims claims,
+            @PathVariable String noticeId,
+            @Valid @RequestBody UpdateNoticeRequestDto request
+    ) {
+        String requesterId = claims.getSubject();
+        String content = request.getContent();
+        NoticeDto updatedNotice = noticeService.updateNotice(requesterId, noticeId, content);
+
+        CommonResponse response = new CommonResponse(
+                HttpStatus.OK, "공지 수정에 성공했습니다.", DtoUtils.toMap(updatedNotice));
+
+        return ResponseEntity.ok(response);
+    }
+
+    @DeleteMapping("/{noticeId}")
+    public ResponseEntity<CommonResponse> deleteNotice(
+            @AuthenticationPrincipal Claims claims,
+            @PathVariable String noticeId
+    ) {
+        String requesterId = claims.getSubject();
+        noticeService.deleteNotice(requesterId, noticeId);
+
+        CommonResponse response = new CommonResponse(
+                HttpStatus.OK, "공지 삭제에 성공했습니다.", null);
+
+        return ResponseEntity.ok(response);
+    }
+
 }

src/main/java/com/groommoa/aether_back_notification/domain/notices/dto/UpdateNoticeRequestDto.java

@@ -0,0 +1,10 @@
+package com.groommoa.aether_back_notification.domain.notices.dto;
+
+import lombok.Builder;
+import lombok.Getter;
+
+@Getter
+@Builder
+public class UpdateNoticeRequestDto {
+    private final String content;
+}

src/main/java/com/groommoa/aether_back_notification/domain/notices/exception/NoticeException.java

@@ -0,0 +1,10 @@
+package com.groommoa.aether_back_notification.domain.notices.exception;
+
+import com.groommoa.aether_back_notification.global.common.exception.CustomException;
+import com.groommoa.aether_back_notification.global.common.exception.ErrorCode;
+
+public class NoticeException extends CustomException {
+    public NoticeException(ErrorCode errorCode) {
+        super(errorCode);
+    }
+}

src/main/java/com/groommoa/aether_back_notification/domain/notices/service/NoticeService.java

@@ -3,8 +3,11 @@
 import com.groommoa.aether_back_notification.domain.notices.dto.CreateNoticeRequestDto;
 import com.groommoa.aether_back_notification.domain.notices.dto.NoticeDto;
 import com.groommoa.aether_back_notification.domain.notices.entity.Notice;
+import com.groommoa.aether_back_notification.domain.notices.exception.NoticeException;
 import com.groommoa.aether_back_notification.domain.notices.repository.NoticeRepository;
+import com.groommoa.aether_back_notification.global.common.exception.ErrorCode;
 import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
 import org.bson.types.ObjectId;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Pageable;
@@ -13,7 +16,9 @@
 
 import java.time.Instant;
 import java.util.List;
+import java.util.Optional;
 
+@Slf4j
 @RequiredArgsConstructor
 @Service
 public class NoticeService {
@@ -40,4 +45,31 @@ public List<NoticeDto> getNotices(int limit) {
                 .map(NoticeDto::from)
                 .toList();
     }
+
+    public NoticeDto updateNotice(String requesterId, String noticeId, String content) {
+        Notice notice = getValidatedNotice(requesterId, noticeId);
+
+        notice.setContent(content);
+        noticeRepository.save(notice);
+
+        return NoticeDto.from(notice);
+    }
+
+    public void deleteNotice(String requesterId, String noticeId) {
+        Notice notice = getValidatedNotice(requesterId, noticeId);
+        noticeRepository.delete(notice);
+    }
+
+    private Notice getValidatedNotice(String requesterId, String noticeId) {
+        Optional<Notice> optionalNotice = noticeRepository.findById(noticeId);
+        if (optionalNotice.isEmpty()) {
+            throw new NoticeException(ErrorCode.NOTICE_NOT_FOUND);
+        }
+        Notice notice = optionalNotice.get();
+        String createdBy = notice.getCreatedBy().toHexString();
+        if (requesterId != null && !requesterId.equals(createdBy)) {
+            throw new NoticeException(ErrorCode.NOTICE_ACCESS_DENIED);
+        }
+        return notice;
+    }
 }

src/main/java/com/groommoa/aether_back_notification/global/common/exception/ErrorCode.java

@@ -17,7 +17,11 @@ public enum ErrorCode {
 
     // notification
     NOTIFICATION_NOT_FOUNT(NOT_FOUND, "요청한 알림을 찾을 수 없습니다."),
-    NOTIFICATION_ACCESS_DENIED(FORBIDDEN, "요청한 알림에 접근할 권한이 없습니다.");
+    NOTIFICATION_ACCESS_DENIED(FORBIDDEN, "요청한 알림에 접근할 권한이 없습니다."),
+
+    // notice
+    NOTICE_NOT_FOUND(NOT_FOUND, "요청한 공지를 찾을 수 없습니다."),
+    NOTICE_ACCESS_DENIED(FORBIDDEN, "요청한 공지에 접근할 권한이 없습니다.");
 
     private final HttpStatus httpStatus;
     private final String message;

src/main/java/com/groommoa/aether_back_notification/global/common/handler/GlobalExceptionHandler.java

@@ -3,6 +3,7 @@
 import com.fasterxml.jackson.core.JsonParseException;
 import com.fasterxml.jackson.databind.exc.InvalidFormatException;
 import com.fasterxml.jackson.databind.exc.MismatchedInputException;
+import com.groommoa.aether_back_notification.domain.notices.exception.NoticeException;
 import com.groommoa.aether_back_notification.domain.notifications.exception.NotificationException;
 import com.groommoa.aether_back_notification.global.common.exception.ErrorCode;
 import com.groommoa.aether_back_notification.global.common.response.ErrorResponse;
@@ -223,6 +224,18 @@ public ResponseEntity<ErrorResponse> handleNotificationException(NotificationExc
         return ResponseEntity.status(status).body(response);
     }
 
+    @ExceptionHandler(NoticeException.class)
+    public ResponseEntity<ErrorResponse> handleNoticeException(NoticeException ex) {
+        ex.printStackTrace();
+        ErrorCode errorCode = ex.getErrorCode();
+        HttpStatus status = errorCode.getHttpStatus();
+        String message = errorCode.getMessage();
+
+        ErrorResponse response = new ErrorResponse(
+                status.value(), message, null);
+        return ResponseEntity.status(status).body(response);
+    }
+
     @ExceptionHandler(Exception.class)
     public ResponseEntity<ErrorResponse> handleException(Exception ex) {
         ex.printStackTrace();