[DDING-124] 폼지 종료일자 수정 API 구현 (#293)

Seooooo24 · web-flow · commit 546cf9033799 · 2025-03-08T15:46:51.000+09:00
diff --git a/src/main/java/ddingdong/ddingdongBE/common/exception/FormException.java b/src/main/java/ddingdong/ddingdongBE/common/exception/FormException.java
@@ -5,8 +5,9 @@
 public class FormException extends CustomException {
 
     public static final String FORM_PERIOD_ERROR_MESSAGE = "해당 폼지의 응답기간이 아닙니다";
-    private static final String INVALID_FORM_DATE_MESSAGE = "입력한 기간과 겹치는 폼이 존재합니다.";
+    private static final String OVERLAPPED_FORM_DATE_MESSAGE = "입력한 기간과 겹치는 폼이 존재합니다.";
     private static final String INVALID_FIELD_TYPE_MESSAGE = "통계를 조회할 질문 유형이 올바르지 않습니다.";
+    private static final String INVALID_FORM_DATE_MESSAGE = "폼지 종료일자는 폼지 시작일자 이전일 수 없습니다.";
 
 
     public FormException(String message, int errorCode) {
@@ -23,6 +24,13 @@ public FormPeriodException() {
     public static final class OverlapFormPeriodException extends FormException {
 
         public OverlapFormPeriodException() {
+            super(OVERLAPPED_FORM_DATE_MESSAGE, BAD_REQUEST.value());
+        }
+    }
+
+    public static final class InvalidFormEndDateException extends FormException {
+
+        public InvalidFormEndDateException() {
             super(INVALID_FORM_DATE_MESSAGE, BAD_REQUEST.value());
         }
     }
diff --git a/src/main/java/ddingdong/ddingdongBE/domain/form/api/CentralFormApi.java b/src/main/java/ddingdong/ddingdongBE/domain/form/api/CentralFormApi.java
@@ -3,6 +3,7 @@
 import ddingdong.ddingdongBE.auth.PrincipalDetails;
 import ddingdong.ddingdongBE.domain.form.controller.dto.request.CreateFormRequest;
 import ddingdong.ddingdongBE.domain.form.controller.dto.request.SendApplicationResultEmailRequest;
+import ddingdong.ddingdongBE.domain.form.controller.dto.request.UpdateFormEndDateRequest;
 import ddingdong.ddingdongBE.domain.form.controller.dto.request.UpdateFormRequest;
 import ddingdong.ddingdongBE.domain.form.controller.dto.response.FormListResponse;
 import ddingdong.ddingdongBE.domain.form.controller.dto.response.FormResponse;
@@ -22,6 +23,7 @@
 import org.springframework.security.core.annotation.AuthenticationPrincipal;
 import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
+import org.springframework.web.bind.annotation.PatchMapping;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.PutMapping;
@@ -125,4 +127,15 @@ FormStatisticsResponse getFormStatistics(
     void sendApplicationResultEmail(@PathVariable("formId") Long formId,
                                     @AuthenticationPrincipal PrincipalDetails principalDetails,
                                     @RequestBody SendApplicationResultEmailRequest request);
+
+    @Operation(summary = "동아리 폼지 종료일자 수정 API")
+    @ApiResponse(responseCode = "204", description = "동아리 폼지 지원기간 마감일자 수정 성공")
+    @ResponseStatus(HttpStatus.NO_CONTENT)
+    @SecurityRequirement(name = "AccessToken")
+    @PatchMapping("/my/forms/{formId}/deadline")
+    void updateFormEndDate(
+            @Valid @RequestBody UpdateFormEndDateRequest updateFormEndDateRequest,
+            @PathVariable("formId") Long formId,
+            @AuthenticationPrincipal PrincipalDetails principalDetails
+    );
 }
diff --git a/src/main/java/ddingdong/ddingdongBE/domain/form/controller/CentralFormController.java b/src/main/java/ddingdong/ddingdongBE/domain/form/controller/CentralFormController.java
@@ -4,6 +4,7 @@
 import ddingdong.ddingdongBE.domain.form.api.CentralFormApi;
 import ddingdong.ddingdongBE.domain.form.controller.dto.request.CreateFormRequest;
 import ddingdong.ddingdongBE.domain.form.controller.dto.request.SendApplicationResultEmailRequest;
+import ddingdong.ddingdongBE.domain.form.controller.dto.request.UpdateFormEndDateRequest;
 import ddingdong.ddingdongBE.domain.form.controller.dto.request.UpdateFormRequest;
 import ddingdong.ddingdongBE.domain.form.controller.dto.response.FormListResponse;
 import ddingdong.ddingdongBE.domain.form.controller.dto.response.FormResponse;
@@ -104,4 +105,11 @@ public void sendApplicationResultEmail(
         User user = principalDetails.getUser();
         facadeCentralFormService.sendApplicationResultEmail(request.toCommand(user.getId(), formId));
     }
+
+    @Override
+    public void updateFormEndDate(UpdateFormEndDateRequest updateFormEndDateRequest, Long formId,
+            PrincipalDetails principalDetails) {
+        User user = principalDetails.getUser();
+        facadeCentralFormService.updateFormEndDate(updateFormEndDateRequest.toCommand(user, formId));
+    }
 }
diff --git a/src/main/java/ddingdong/ddingdongBE/domain/form/controller/dto/request/UpdateFormEndDateRequest.java b/src/main/java/ddingdong/ddingdongBE/domain/form/controller/dto/request/UpdateFormEndDateRequest.java
@@ -0,0 +1,23 @@
+package ddingdong.ddingdongBE.domain.form.controller.dto.request;
+
+import ddingdong.ddingdongBE.domain.form.service.dto.command.UpdateFormEndDateCommand;
+import ddingdong.ddingdongBE.domain.user.entity.User;
+import io.swagger.v3.oas.annotations.media.Schema;
+import jakarta.validation.constraints.NotNull;
+import com.fasterxml.jackson.annotation.JsonFormat;
+import java.time.LocalDate;
+
+public record UpdateFormEndDateRequest (
+        @Schema(description = "폼지 종료일자", example = "2025-03-10")
+        @NotNull(message = "폼지 종료일자는 null이 될 수 없습니다.")
+        @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd")
+        LocalDate endDate
+){
+    public UpdateFormEndDateCommand toCommand(User user, Long formId) {
+        return UpdateFormEndDateCommand.builder()
+                .user(user)
+                .formId(formId)
+                .endDate(endDate)
+                .build();
+    }
+}
diff --git a/src/main/java/ddingdong/ddingdongBE/domain/form/entity/Form.java b/src/main/java/ddingdong/ddingdongBE/domain/form/entity/Form.java
@@ -79,4 +79,8 @@ public void update(Form updateForm) {
     public boolean isEqualsById(Long formId) {
         return this.id.equals(formId);
     }
+
+    public void updateEndDate(LocalDate endDate) {
+        this.endDate = endDate;
+    }
 }
diff --git a/src/main/java/ddingdong/ddingdongBE/domain/form/service/FacadeCentralFormService.java b/src/main/java/ddingdong/ddingdongBE/domain/form/service/FacadeCentralFormService.java
@@ -3,6 +3,7 @@
 import ddingdong.ddingdongBE.domain.form.service.dto.command.CreateFormCommand;
 import ddingdong.ddingdongBE.domain.form.service.dto.command.SendApplicationResultEmailCommand;
 import ddingdong.ddingdongBE.domain.form.service.dto.command.UpdateFormCommand;
+import ddingdong.ddingdongBE.domain.form.service.dto.command.UpdateFormEndDateCommand;
 import ddingdong.ddingdongBE.domain.form.service.dto.query.FormListQuery;
 import ddingdong.ddingdongBE.domain.form.service.dto.query.FormQuery;
 import ddingdong.ddingdongBE.domain.form.service.dto.query.FormStatisticsQuery;
@@ -32,4 +33,6 @@ public interface FacadeCentralFormService {
     SingleFieldStatisticsQuery getTextFieldStatistics(Long fieldId);
 
     void sendApplicationResultEmail(SendApplicationResultEmailCommand command);
+
+    void updateFormEndDate(UpdateFormEndDateCommand command);
 }
diff --git a/src/main/java/ddingdong/ddingdongBE/domain/form/service/FacadeCentralFormServiceImpl.java b/src/main/java/ddingdong/ddingdongBE/domain/form/service/FacadeCentralFormServiceImpl.java
@@ -4,6 +4,7 @@
 
 import ddingdong.ddingdongBE.common.exception.AuthenticationException.NonHaveAuthority;
 import ddingdong.ddingdongBE.common.exception.FormException.InvalidFieldTypeException;
+import ddingdong.ddingdongBE.common.exception.FormException.InvalidFormEndDateException;
 import ddingdong.ddingdongBE.common.exception.FormException.OverlapFormPeriodException;
 import ddingdong.ddingdongBE.domain.club.entity.Club;
 import ddingdong.ddingdongBE.domain.club.service.ClubService;
@@ -18,6 +19,7 @@
 import ddingdong.ddingdongBE.domain.form.service.dto.command.SendApplicationResultEmailCommand;
 import ddingdong.ddingdongBE.domain.form.service.dto.command.UpdateFormCommand;
 import ddingdong.ddingdongBE.domain.form.service.dto.command.UpdateFormCommand.UpdateFormFieldCommand;
+import ddingdong.ddingdongBE.domain.form.service.dto.command.UpdateFormEndDateCommand;
 import ddingdong.ddingdongBE.domain.form.service.dto.query.FormListQuery;
 import ddingdong.ddingdongBE.domain.form.service.dto.query.FormQuery;
 import ddingdong.ddingdongBE.domain.form.service.dto.query.FormStatisticsQuery;
@@ -194,6 +196,16 @@ public void sendApplicationResultEmail(SendApplicationResultEmailCommand command
         }
     }
 
+    @Transactional
+    @Override
+    public void updateFormEndDate(UpdateFormEndDateCommand command) {
+        Club club = clubService.getByUserId(command.user().getId());
+        Form form = formService.getById(command.formId());
+        validateEndDate(form.getStartDate(), command.endDate());
+        validateDuplicationDateExcludingSelf(club, form.getStartDate(), command.endDate(), command.formId());
+        form.updateEndDate(command.endDate());
+    }
+
     private FormListQuery buildFormListQuery(Form form) {
         FormStatus formStatus = FormStatus.getDescription(LocalDate.now(), form.getStartDate(),
                 form.getEndDate());
@@ -235,6 +247,10 @@ private void validateDuplicationDateExcludingSelf(
         }
     }
 
+    private void validateEndDate(LocalDate startDate, LocalDate endDate) {
+        if (endDate.isBefore(startDate)) { throw new InvalidFormEndDateException(); }
+    }
+
     private List<FormField> toUpdateFormFields(Form originform,
                                                List<UpdateFormFieldCommand> updateFormFieldCommands) {
         return updateFormFieldCommands.stream()
diff --git a/src/main/java/ddingdong/ddingdongBE/domain/form/service/dto/command/UpdateFormEndDateCommand.java b/src/main/java/ddingdong/ddingdongBE/domain/form/service/dto/command/UpdateFormEndDateCommand.java
@@ -0,0 +1,14 @@
+package ddingdong.ddingdongBE.domain.form.service.dto.command;
+
+import ddingdong.ddingdongBE.domain.user.entity.User;
+import java.time.LocalDate;
+import lombok.Builder;
+
+@Builder
+public record UpdateFormEndDateCommand (
+        Long formId,
+        LocalDate endDate,
+        User user
+){
+
+}

Original file line number	Diff line number	Diff line change
`@@ -79,4 +79,8 @@ public void update(Form updateForm) {`
`79`	`79`	`public boolean isEqualsById(Long formId) {`
`80`	`80`	`return this.id.equals(formId);`
`81`	`81`	`}`
	`82`	`+`
	`83`	`+ public void updateEndDate(LocalDate endDate) {`
	`84`	`+ this.endDate = endDate;`
	`85`	`+ }`
`82`	`86`	`}`