✨ [STMT-200] 스터디 수정 api 구현

src/docs/asciidoc/index.adoc

@@ -233,8 +233,16 @@ include::{snippets}/get-study-detail/fail/not-found/response-fields.adoc[]
 
 ===== 요청
 include::{snippets}/create-study/success/http-request.adoc[]
+
+===== 헤더
 include::{snippets}/create-study/success/request-headers.adoc[]
 
+===== 파트
+include::{snippets}/create-study/success/request-parts.adoc[]
+
+===== 매개변수
+include::{snippets}/create-study/success/query-parameters.adoc[]
+
 ===== 응답 성공 (200)
 
 include::{snippets}/create-study/success/response-body.adoc[]
@@ -253,6 +261,42 @@ include::{snippets}/create-study/fail/invalid-study-meeting-schedule/response-bo
 include::{snippets}/create-study/fail/invalid-study-meeting-schedule/response-fields.adoc[]
 
 
+=== 스터디 수정
+
+스터디 정보를 수정하는 API 입니다.
+
+==== PATCH /api/v1/studies/{studyId}
+
+===== 요청
+include::{snippets}/update-study/success/http-request.adoc[]
+
+===== 헤더
+include::{snippets}/update-study/success/request-headers.adoc[]
+
+===== 파트
+include::{snippets}/update-study/success/request-parts.adoc[]
+
+===== 매개변수
+include::{snippets}/update-study/success/query-parameters.adoc[]
+
+===== 응답 성공 (200)
+
+include::{snippets}/update-study/success/response-body.adoc[]
+include::{snippets}/update-study/success/response-fields.adoc[]
+
+===== 응답 실패 (404)
+
+.존재하지 않는 스터디 분야 ID를 요청한 경우
+
+include::{snippets}/update-study/fail/study-field-not-found/response-body.adoc[]
+include::{snippets}/update-study/fail/study-field-not-found/response-fields.adoc[]
+
+.일정 반복 유형 DAILY일 때를 제외하고 반복일을 null 값으로 요청한 경우
+
+include::{snippets}/update-study/fail/invalid-study-meeting-schedule/response-body.adoc[]
+include::{snippets}/update-study/fail/invalid-study-meeting-schedule/response-fields.adoc[]
+
+
 == 스터디 멤버 관리
 
 === 스터디 멤버 조회

src/main/java/com/stumeet/server/common/response/SuccessCode.java

@@ -13,6 +13,7 @@ public enum SuccessCode {
      * 200 - OK
      */
     GET_SUCCESS(HttpStatus.OK, "조회에 성공했습니다."),
+    UPDATE_SUCCESS(HttpStatus.OK, "수정에 성공했습니다."),
     STUDY_LEAVE_SUCCESS(HttpStatus.OK, "스터디 탈퇴에 성공했습니다."),
     STUDY_KICK_SUCCESS(HttpStatus.OK, "스터디원 강퇴에 성공했습니다."),
 

src/main/java/com/stumeet/server/study/adapter/in/web/StudyCreateApi.java

@@ -28,7 +28,7 @@ public ResponseEntity<ApiResponse<Void>> create(
 			@AuthenticationPrincipal LoginMember member,
 			@Valid StudyCreateCommand request
 	) {
-		studyCreateUseCase.create(request, member.getMember());
+		studyCreateUseCase.create(request, member.getMember().getId());
 
 		return new ResponseEntity<>(
 				ApiResponse.success(SuccessCode.STUDY_CREATE_SUCCESS),

src/main/java/com/stumeet/server/study/adapter/in/web/StudyUpdateApi.java

@@ -0,0 +1,39 @@
+package com.stumeet.server.study.adapter.in.web;
+
+import org.springframework.http.HttpStatus;
+import org.springframework.http.ResponseEntity;
+import org.springframework.security.core.annotation.AuthenticationPrincipal;
+import org.springframework.web.bind.annotation.PatchMapping;
+import org.springframework.web.bind.annotation.PathVariable;
+import org.springframework.web.bind.annotation.RequestMapping;
+
+import com.stumeet.server.common.annotation.WebAdapter;
+import com.stumeet.server.common.auth.model.LoginMember;
+import com.stumeet.server.common.model.ApiResponse;
+import com.stumeet.server.common.response.SuccessCode;
+import com.stumeet.server.study.application.port.in.StudyUpdateUseCase;
+import com.stumeet.server.study.application.port.in.command.StudyUpdateCommand;
+
+import jakarta.validation.Valid;
+import lombok.RequiredArgsConstructor;
+
+@WebAdapter
+@RequiredArgsConstructor
+@RequestMapping("/api/v1/studies")
+public class StudyUpdateApi {
+
+	private final StudyUpdateUseCase studyUpdateUseCase;
+
+	@PatchMapping("/{studyId}")
+	public ResponseEntity<ApiResponse<Void>> update(
+		@AuthenticationPrincipal LoginMember member,
+		@PathVariable Long studyId,
+		@Valid StudyUpdateCommand request
+	) {
+		studyUpdateUseCase.update(studyId, member.getMember().getId(), request);
+
+		return new ResponseEntity<>(
+			ApiResponse.success(SuccessCode.UPDATE_SUCCESS),
+			HttpStatus.OK);
+	}
+}

src/main/java/com/stumeet/server/study/adapter/out/persistance/JpaStudyTagRepository.java

@@ -5,4 +5,6 @@
 import com.stumeet.server.study.adapter.out.persistance.entity.StudyTagJpaEntity;
 
 public interface JpaStudyTagRepository extends JpaRepository<StudyTagJpaEntity, Long> {
+
+	void deleteAllByStudyId(Long studyId);
 }

src/main/java/com/stumeet/server/study/adapter/out/persistance/StudyTagPersistenceAdapter.java

@@ -17,10 +17,21 @@ public class StudyTagPersistenceAdapter implements StudyTagCommandPort {
 	private final StudyTagPersistenceMapper studyTagPersistenceMapper;
 
 	@Override
-	public List<String> saveAll(List<String> studyTags, Long studyId) {
+	public List<String> saveAllStudyTags(List<String> studyTags, Long studyId) {
 		List<StudyTagJpaEntity> entities =
 				studyTagRepository.saveAll(studyTagPersistenceMapper.toEntities(studyTags, studyId));
 
 		return studyTagPersistenceMapper.toDomains(entities);
 	}
+
+	@Override
+	public void clearStudyTags(Long studyId) {
+		studyTagRepository.deleteAllByStudyId(studyId);
+	}
+
+	@Override
+	public void replaceStudyTags(List<String> newStudyTags, Long studyId) {
+		clearStudyTags(studyId);
+		saveAllStudyTags(newStudyTags, studyId);
+	}
 }

src/main/java/com/stumeet/server/study/adapter/out/persistance/entity/StudyJpaEntity.java

@@ -1,14 +1,18 @@
 package com.stumeet.server.study.adapter.out.persistance.entity;
 
 import com.stumeet.server.study.domain.StudyField;
+
 import jakarta.persistence.EnumType;
 import jakarta.persistence.Enumerated;
+import jakarta.persistence.FetchType;
 import jakarta.persistence.OneToMany;
+
 import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.time.LocalTime;
 
 import java.util.List;
+
 import org.hibernate.annotations.ColumnDefault;
 import org.hibernate.annotations.Comment;
 
@@ -19,7 +23,6 @@
 import jakarta.persistence.GeneratedValue;
 import jakarta.persistence.GenerationType;
 import jakarta.persistence.Id;
-import jakarta.persistence.JoinColumn;
 import jakarta.persistence.Table;
 import lombok.AccessLevel;
 import lombok.AllArgsConstructor;
@@ -40,8 +43,7 @@ public class StudyJpaEntity extends BaseTimeEntity {
 	@Comment("스터디 그룹 아이디")
 	private Long id;
 
-	@OneToMany(orphanRemoval = true)
-	@JoinColumn(name = "study_id")
+	@OneToMany(mappedBy = "studyId", fetch = FetchType.EAGER)
 	private List<StudyTagJpaEntity> studyTags;
 
 	@Enumerated(EnumType.STRING)
@@ -111,4 +113,4 @@ public Boolean getIsFinished() {
 	public Boolean getIsDeleted() {
 		return this.isDeleted;
 	}
-}
+}

src/main/java/com/stumeet/server/study/adapter/out/persistance/entity/StudyTagJpaEntity.java

@@ -7,6 +7,7 @@
 import jakarta.persistence.GeneratedValue;
 import jakarta.persistence.GenerationType;
 import jakarta.persistence.Id;
+import jakarta.persistence.JoinColumn;
 import jakarta.persistence.Table;
 import lombok.AccessLevel;
 import lombok.AllArgsConstructor;
@@ -27,7 +28,7 @@ public class StudyTagJpaEntity {
 	@Comment("스터디 태그 id")
 	private Long id;
 
-	@Column(name = "study_id", nullable = false)
+	@JoinColumn(name = "study_id", nullable = false)
 	@Comment("스터디 id")
 	private Long studyId;
 

src/main/java/com/stumeet/server/study/adapter/out/persistance/mapper/MeetingRepetitionPersistenceMapper.java

@@ -4,24 +4,23 @@
 
 import org.springframework.stereotype.Component;
 
+import com.stumeet.server.study.domain.Repetition;
 import com.stumeet.server.study.domain.RepetitionType;
-import com.stumeet.server.study.domain.StudyMeetingSchedule;
 
 @Component
 public class MeetingRepetitionPersistenceMapper {
 
 	private static final String REPEAT_DELIMITER = ";";
 
-	public StudyMeetingSchedule.Repetition toDomain(String meetingRepetition) {
+	public Repetition toDomain(String meetingRepetition) {
 		List<String> repetitionElements = List.of(meetingRepetition.split(REPEAT_DELIMITER));
 
-		return StudyMeetingSchedule.Repetition.builder()
-				.type(RepetitionType.valueOf(repetitionElements.getFirst()))
-				.dates(repetitionElements.subList(1, repetitionElements.size()))
-				.build();
+		return Repetition.of(
+			RepetitionType.valueOf(repetitionElements.getFirst()),
+			repetitionElements.subList(1, repetitionElements.size()));
 	}
 
-	public String toColumn(StudyMeetingSchedule.Repetition repetition) {
+	public String toColumn(Repetition repetition) {
 		String dates = repetition.getDates() != null
 				? String.join(REPEAT_DELIMITER, repetition.getDates())
 				: null;

src/main/java/com/stumeet/server/study/adapter/out/persistance/mapper/StudyDomainPersistenceMapper.java

@@ -2,20 +2,18 @@
 
 import com.stumeet.server.study.adapter.out.persistance.entity.StudyJpaEntity;
 import com.stumeet.server.study.domain.StudyDomain;
+
 import lombok.RequiredArgsConstructor;
+
 import org.springframework.stereotype.Component;
 
 @Component
 @RequiredArgsConstructor
 public class StudyDomainPersistenceMapper {
 
-    private final StudyTagPersistenceMapper studyTagPersistenceMapper;
-
-    public StudyDomain toDomain(StudyJpaEntity entity) {
-        return StudyDomain.builder()
-                .studyField(entity.getStudyField())
-                .studyTags(studyTagPersistenceMapper.toDomains(entity.getStudyTags()))
-                .build();
+	private final StudyTagPersistenceMapper studyTagPersistenceMapper;
 
-    }
+	public StudyDomain toDomain(StudyJpaEntity entity) {
+		return StudyDomain.of(entity.getStudyField(), studyTagPersistenceMapper.toDomains(entity.getStudyTags()));
+	}
 }

src/main/java/com/stumeet/server/study/adapter/out/persistance/mapper/StudyMeetingSchedulePersistenceMapper.java

@@ -0,0 +1,20 @@
+package com.stumeet.server.study.adapter.out.persistance.mapper;
+
+import java.time.LocalTime;
+
+import org.springframework.stereotype.Component;
+
+import com.stumeet.server.study.domain.StudyMeetingSchedule;
+
+import lombok.RequiredArgsConstructor;
+
+@Component
+@RequiredArgsConstructor
+public class StudyMeetingSchedulePersistenceMapper {
+
+	private final MeetingRepetitionPersistenceMapper meetingRepetitionPersistenceMapper;
+
+	public StudyMeetingSchedule toDomain(LocalTime meetingTime, String meetingRepetition) {
+		return StudyMeetingSchedule.of(meetingTime, meetingRepetitionPersistenceMapper.toDomain(meetingRepetition));
+	}
+}

src/main/java/com/stumeet/server/study/adapter/out/persistance/mapper/StudyPersistenceMapper.java

@@ -4,7 +4,6 @@
 
 import com.stumeet.server.study.adapter.out.persistance.entity.StudyJpaEntity;
 import com.stumeet.server.study.domain.Study;
-import com.stumeet.server.study.domain.StudyMeetingSchedule;
 import com.stumeet.server.study.domain.StudyPeriod;
 
 import lombok.RequiredArgsConstructor;
@@ -14,8 +13,9 @@
 public class StudyPersistenceMapper {
 
 	private final StudyDomainPersistenceMapper studyDomainPersistenceMapper;
-	private final MeetingRepetitionPersistenceMapper meetingRepetitionPersistenceMapper;
 	private final StudyTagPersistenceMapper studyTagPersistenceMapper;
+	private final StudyMeetingSchedulePersistenceMapper studyMeetingSchedulePersistenceMapper;
+	private final MeetingRepetitionPersistenceMapper meetingRepetitionPersistenceMapper;
 
 	public Study toDomain(StudyJpaEntity entity) {
 		return Study.builder()
@@ -25,15 +25,9 @@ public Study toDomain(StudyJpaEntity entity) {
 				.region(entity.getRegion())
 				.intro(entity.getIntro())
 				.rule(entity.getRule())
-				.period(StudyPeriod.builder()
-						.startDate(entity.getStartDate())
-						.endDate(entity.getEndDate())
-						.build())
+				.period(StudyPeriod.of(entity.getStartDate(), entity.getEndDate()))
 				.imageUrl(entity.getImage())
-				.meetingSchedule(StudyMeetingSchedule.builder()
-						.time(entity.getMeetingTime())
-						.repetition(meetingRepetitionPersistenceMapper.toDomain(entity.getMeetingRepetition()))
-						.build())
+				.meetingSchedule(studyMeetingSchedulePersistenceMapper.toDomain(entity.getMeetingTime(), entity.getMeetingRepetition()))
 				.isFinished(entity.getIsFinished())
 				.isDeleted(entity.getIsDeleted())
 				.build();
@@ -52,8 +46,7 @@ public StudyJpaEntity toEntity(Study domain) {
 				.endDate(domain.getEndDate())
 				.image(domain.getImageUrl())
 				.meetingTime(domain.getMeetingSchedule().getTime())
-				.meetingRepetition(
-						meetingRepetitionPersistenceMapper.toColumn(domain.getMeetingSchedule().getRepetition()))
+				.meetingRepetition(meetingRepetitionPersistenceMapper.toColumn(domain.getMeetingSchedule().getRepetition()))
 				.isFinished(domain.isFinished())
 				.isDeleted(domain.isDeleted())
 				.build();

src/main/java/com/stumeet/server/study/adapter/out/persistance/mapper/StudyTagPersistenceMapper.java

@@ -11,20 +11,18 @@ public class StudyTagPersistenceMapper {
 
 	public List<String> toDomains(List<StudyTagJpaEntity> entities) {
 		return entities != null
-				? entities.stream()
-				.map(StudyTagJpaEntity::getName)
-				.toList()
-				: List.of();
+			? entities.stream()
+			.map(StudyTagJpaEntity::getName)
+			.toList()
+			: List.of();
 	}
 
 	public List<StudyTagJpaEntity> toEntities(List<String> domains, Long studyId) {
-		return !domains.isEmpty()
-				? domains.stream()
-				.map(tag -> StudyTagJpaEntity.builder()
-						.studyId(studyId)
-						.name(tag)
-						.build())
-				.toList()
-				: List.of();
+		return domains.stream()
+			.map(domain -> StudyTagJpaEntity.builder()
+				.studyId(studyId)
+				.name(domain)
+				.build())
+			.toList();
 	}
 }

src/main/java/com/stumeet/server/study/application/port/in/StudyCreateUseCase.java

@@ -1,9 +1,8 @@
 package com.stumeet.server.study.application.port.in;
 
-import com.stumeet.server.member.domain.Member;
 import com.stumeet.server.study.application.port.in.command.StudyCreateCommand;
 
 public interface StudyCreateUseCase {
 
-	Long create(StudyCreateCommand command, Member member);
+	Long create(StudyCreateCommand command, Long memberId);
 }

src/main/java/com/stumeet/server/study/application/port/in/StudyUpdateUseCase.java

@@ -0,0 +1,8 @@
+package com.stumeet.server.study.application.port.in;
+
+import com.stumeet.server.study.application.port.in.command.StudyUpdateCommand;
+
+public interface StudyUpdateUseCase {
+
+	void update(Long studyId, Long memberId, StudyUpdateCommand command);
+}