Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: 상태 변경 AOP 처리 #55

Open
210-reverof opened this issue Aug 2, 2024 · 0 comments
Open

refactor: 상태 변경 AOP 처리 #55

210-reverof opened this issue Aug 2, 2024 · 0 comments
Labels

Comments

@210-reverof
Copy link
Member

기능 설명

지금은 application Service 비지니스 로직으로 책임이 무관한 상태 변경을 호출함

@Transactional
    fun uploadImages(request: UploadImageServiceRequest): UploadImageResponse {
        eventService.addImageOptions(
            userId = request.userId,
            eventId = request.eventId,
            imageUrls = request.imageUrls
        )

        // here
        if (eventService.hasEveryoneUploadedImages(request.eventId)) {
            eventService.endEventUploading(request.eventId)
        }

        return UploadImageResponse(request.eventId)
    }

AOP로 관심사 분리 예정

추가 정보

No response

@210-reverof 210-reverof added enhancement New feature or request refactor and removed enhancement New feature or request labels Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant