[DPMBE-106] user profile 수정 시 사용하지 않는 이미지는 삭제한다 (#161)

* feat: user profile 수정 시 사용하지 않는 이미지는 삭제한다

* remove: imageFileExtension 사용하지 않아서 제거

* refactor: UserProfileImageUpdatedEvent 발행 Domain으로 이동하고 Kotlin takeIf 문법 사용

* style: spotless

* style: 주석해제 & s3Service로 이름 변경

Whatnow-Api/src/main/kotlin/com/depromeet/whatnow/api/image/usecase/GetPresignedUrlUseCase.kt

@@ -8,14 +8,14 @@ import com.depromeet.whatnow.config.security.SecurityUtils
 
 @UseCase
 class GetPresignedUrlUseCase(
-    val presignedUrlService: S3Service,
+    val s3Service: S3Service,
 ) {
     fun forPromise(promiseId: Long, fileExtension: ImageFileExtension): ImageUrlResponse {
-        return ImageUrlResponse.from(presignedUrlService.getPresignedUrlForPromise(promiseId, fileExtension))
+        return ImageUrlResponse.from(s3Service.getPresignedUrlForPromise(promiseId, fileExtension))
     }
 
     fun forUser(fileExtension: ImageFileExtension): ImageUrlResponse {
         val currentUserId = SecurityUtils.currentUserId
-        return ImageUrlResponse.from(presignedUrlService.getPresignedUrlForUser(currentUserId, fileExtension))
+        return ImageUrlResponse.from(s3Service.getPresignedUrlForUser(currentUserId, fileExtension))
     }
 }

Whatnow-Api/src/main/kotlin/com/depromeet/whatnow/api/user/dto/request/UpdateProfileRequest.kt

@@ -1,6 +1,8 @@
 package com.depromeet.whatnow.api.user.dto.request
 
 data class UpdateProfileRequest(
-    val profileImage: String,
     val username: String,
+    val profileImage: String,
+    val isDefaultImg: Boolean,
+    val imageKey: String,
 )

Whatnow-Api/src/main/kotlin/com/depromeet/whatnow/api/user/usecase/UpdateUserUseCase.kt

@@ -23,7 +23,13 @@ class UpdateUserUseCase(
 
     fun updateProfile(updateProfileRequest: UpdateProfileRequest): UserDetailVo {
         val currentUserId: Long = SecurityUtils.currentUserId
-        return userDomainService.updateProfile(currentUserId, updateProfileRequest.profileImage, updateProfileRequest.username)
+        return userDomainService.updateProfile(
+            currentUserId,
+            updateProfileRequest.profileImage,
+            updateProfileRequest.username,
+            updateProfileRequest.isDefaultImg,
+            updateProfileRequest.imageKey,
+        )
             .toUserDetailVo()
     }
 }

Whatnow-Common/src/main/kotlin/com/depromeet/whatnow/consts/WhatNowStatic.kt

@@ -29,6 +29,7 @@ const val RADIUS_WAIT_CONFIRM = 200
 
 const val IMAGE_DOMAIN = "https://image.whatnow.kr"
 const val ASSERT_IMAGE_DOMAIN = "https://image.whatnow.kr/assert"
+const val USER_DEFAULT_PROFILE_IMAGE = "https://image.whatnow.kr/assert/users/default.svg"
 
 const val REDIS_EXPIRE_EVENT_PATTERN = "__keyevent@*__:expired"
 

Whatnow-Domain/src/main/kotlin/com/depromeet/whatnow/domains/image/adapter/UserImageAdapter.kt

@@ -13,6 +13,10 @@ class UserImageAdapter(
         return userImageRepository.save(userImage)
     }
 
+    fun findAllByUserId(userId: Long): List<UserImage> {
+        return userImageRepository.findAllByUserId(userId)
+    }
+
     fun findByUserIdAndImageKey(userId: Long, imageKey: String): UserImage {
         return userImageRepository.findByUserIdAndImageKey(userId, imageKey) ?: run { throw UserImageNotFoundException.EXCEPTION }
     }

Whatnow-Domain/src/main/kotlin/com/depromeet/whatnow/domains/image/repository/UserImageRepository.kt

@@ -6,6 +6,7 @@ import org.springframework.stereotype.Repository
 
 @Repository
 interface UserImageRepository : JpaRepository<UserImage, Long> {
+    fun findAllByUserId(userId: Long): List<UserImage>
     fun findByUserIdAndImageKey(userId: Long, imageKey: String): UserImage?
     fun deleteByImageKeyAndUserId(imageKey: String, userId: Long)
 }

Whatnow-Domain/src/main/kotlin/com/depromeet/whatnow/domains/user/domain/User.kt

@@ -4,8 +4,10 @@ import com.depromeet.whatnow.common.BaseTimeEntity
 import com.depromeet.whatnow.common.aop.event.Events
 import com.depromeet.whatnow.common.vo.UserDetailVo
 import com.depromeet.whatnow.common.vo.UserInfoVo
+import com.depromeet.whatnow.consts.USER_DEFAULT_PROFILE_IMAGE
 import com.depromeet.whatnow.domains.user.exception.AlreadyDeletedUserException
 import com.depromeet.whatnow.domains.user.exception.ForbiddenUserException
+import com.depromeet.whatnow.events.domainEvent.UserProfileImageUpdatedEvent
 import com.depromeet.whatnow.events.domainEvent.UserSignUpEvent
 import java.time.LocalDateTime
 import javax.persistence.Column
@@ -91,12 +93,14 @@ class User(
         fcmNotification = FcmNotificationVo.updateToken(fcmNotification, fcmToken)
     }
 
-    fun updateProfile(profileImage: String, username: String) {
-        if (profileImage != profileImg) {
-            isDefaultImg = false
+    fun updateProfile(profileImage: String, username: String, isDefaultImage: Boolean, imageKey: String) {
+        this.nickname = username
+        this.isDefaultImg = isDefaultImage
+        this.profileImg = profileImage.takeIf { !isDefaultImage } ?: USER_DEFAULT_PROFILE_IMAGE
+
+        if (!isDefaultImage) {
+            Events.raise(UserProfileImageUpdatedEvent(this.id!!, imageKey))
         }
-        profileImg = profileImage
-        nickname = username
     }
 
     fun toUserInfoVo(): UserInfoVo {

Whatnow-Domain/src/main/kotlin/com/depromeet/whatnow/domains/user/service/UserDomainService.kt

@@ -103,9 +103,9 @@ class UserDomainService(
     }
 
     @Transactional
-    fun updateProfile(currentUserId: Long, profileImage: String, username: String): User {
+    fun updateProfile(currentUserId: Long, profileImage: String, username: String, isDefaultImage: Boolean, imageKey: String): User {
         val user = userAdapter.queryUser(currentUserId)
-        user.updateProfile(profileImage, username)
+        user.updateProfile(profileImage, username, isDefaultImage, imageKey)
         return user
     }
 }

Whatnow-Domain/src/main/kotlin/com/depromeet/whatnow/events/domainEvent/UserProfileImageUpdatedEvent.kt

@@ -0,0 +1,8 @@
+package com.depromeet.whatnow.events.domainEvent
+
+import com.depromeet.whatnow.common.aop.event.DomainEvent
+
+class UserProfileImageUpdatedEvent(
+    val userId: Long,
+    val imageKey: String,
+) : DomainEvent()

Whatnow-Domain/src/main/kotlin/com/depromeet/whatnow/events/handler/UserProfileImageUpdatedEventHandler.kt

@@ -0,0 +1,28 @@
+package com.depromeet.whatnow.events.handler
+
+import com.depromeet.whatnow.annotation.Handler
+import com.depromeet.whatnow.config.s3.S3Service
+import com.depromeet.whatnow.domains.image.adapter.UserImageAdapter
+import com.depromeet.whatnow.events.domainEvent.UserProfileImageUpdatedEvent
+import org.springframework.scheduling.annotation.Async
+import org.springframework.transaction.event.TransactionPhase
+import org.springframework.transaction.event.TransactionalEventListener
+
+@Handler
+class UserProfileImageUpdatedEventHandler(
+    val s3Service: S3Service,
+    val userImageAdapter: UserImageAdapter,
+) {
+    @Async
+    @TransactionalEventListener(classes = [UserProfileImageUpdatedEvent::class], phase = TransactionPhase.AFTER_COMMIT)
+    fun handleRegisterUserEvent(userProfileImageUpdatedEvent: UserProfileImageUpdatedEvent) {
+        val userId = userProfileImageUpdatedEvent.userId
+        val imageKey = userProfileImageUpdatedEvent.imageKey
+
+        userImageAdapter.findAllByUserId(userId)
+            .filter { it.imageKey != imageKey }
+            .map {
+                s3Service.deleteForUser(userId, it.imageKey, it.fileExtension)
+            }
+    }
+}