Skip to content

Commit db98094

Browse files
committed
♻️ :: 일정 삭제 오류메시지 미전송 에러 수정
1 parent cca3642 commit db98094

3 files changed

Lines changed: 11 additions & 30 deletions

File tree

src/main/kotlin/app/xqaure/schedule/application/schedule/ScheduleUsecase.kt

Lines changed: 6 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ package app.xqaure.schedule.application.schedule
33
import app.xqaure.schedule.application.schedule.exceptions.ScheduleNotFoundException
44
import app.xqaure.schedule.domain.schedule.Schedule
55
import app.xqaure.schedule.domain.schedule.ScheduleRepository
6-
import app.xqaure.schedule.global.error.ErrorCode
76
import app.xqaure.schedule.presentation.dto.BasicResponse
87
import app.xqaure.schedule.presentation.dto.ResponseCreator
98
import kotlinx.coroutines.reactor.awaitSingleOrNull
@@ -54,18 +53,12 @@ class ScheduleUsecase(
5453

5554
@Transactional(isolation = Isolation.READ_COMMITTED)
5655
suspend fun deleteSchedule(uuid: String, userId: String): BasicResponse {
57-
runCatching {
58-
scheduleRepository.findScheduleByIdAndUserId(uuid, userId)
59-
.flatMap {
60-
scheduleRepository.deleteById(uuid)
61-
}.awaitSingleOrNull() ?: throw ScheduleNotFoundException()
62-
}.onFailure {
63-
return responseCreator.onSuccess(
64-
code = DELETE_SCHEDULE_CODE,
65-
propertyName = DELETE_SCHEDULE_CODE,
66-
uuid
67-
)
68-
}
56+
val schedule = scheduleRepository.findScheduleByIdAndUserId(uuid, userId)
57+
.awaitSingleOrNull() ?: throw ScheduleNotFoundException()
58+
59+
scheduleRepository.delete(schedule)
60+
.awaitSingleOrNull()
61+
6962
return responseCreator.onSuccess(
7063
code = DELETE_SCHEDULE_CODE,
7164
propertyName = DELETE_SCHEDULE_CODE,

src/main/kotlin/app/xqaure/schedule/application/school/SchoolScheduleUsecase.kt

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import app.xqaure.schedule.presentation.dto.ResponseCreator
1010
import app.xqaure.schedule.presentation.schedule.dto.QueryScheduleListResponse
1111
import app.xqaure.schedule.presentation.schedule.dto.ScheduleElement
1212
import kotlinx.coroutines.Dispatchers
13-
import kotlinx.coroutines.reactive.awaitSingle
1413
import kotlinx.coroutines.reactor.awaitSingleOrNull
1514
import kotlinx.coroutines.withContext
1615
import org.springframework.stereotype.Service
@@ -63,10 +62,11 @@ class SchoolScheduleUsecase(
6362

6463
@Transactional(isolation = Isolation.READ_COMMITTED)
6564
suspend fun deleteSchoolSchedule(uuid: String): BasicResponse {
66-
schoolScheduleRepository.findById(uuid)
67-
.flatMap {
68-
schoolScheduleRepository.deleteById(uuid)
69-
}.awaitSingle() ?: throw ScheduleNotFoundException()
65+
val schoolSchedule = schoolScheduleRepository.findById(uuid)
66+
.awaitSingleOrNull() ?: throw ScheduleNotFoundException()
67+
68+
schoolScheduleRepository.delete(schoolSchedule)
69+
.awaitSingleOrNull()
7070

7171
return responseCreator.onSuccess(
7272
code = DELETE_SCHEDULE_CODE,

src/main/kotlin/app/xqaure/schedule/presentation/dto/Response.kt

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package app.xqaure.schedule.presentation.dto
22

3-
import app.xqaure.schedule.global.error.ErrorResponse
43
import org.springframework.context.MessageSource
54
import org.springframework.context.i18n.LocaleContextHolder
65
import org.springframework.stereotype.Component
@@ -19,17 +18,6 @@ class ResponseCreator(
1918
message = messageSource.getMessage(propertyName, args, LocaleContextHolder.getLocale())
2019
)
2120
}
22-
23-
fun onError(
24-
code: Int,
25-
propertyName: String,
26-
vararg args: Any?
27-
): ErrorResponse {
28-
return ErrorResponse(
29-
responseStatus = code,
30-
errorMessage = messageSource.getMessage(propertyName, args, LocaleContextHolder.getLocale())
31-
)
32-
}
3321
}
3422

3523
data class BasicResponse(

0 commit comments

Comments
 (0)