Skip to content

Commit 962f1c4

Browse files
committed
[BOOK-430] refactor: resetNotificationData 함수 내부를 try-catch로 감싸서 예외 전파 막기
1 parent 61072db commit 962f1c4

2 files changed

Lines changed: 12 additions & 12 deletions

File tree

core/data/api/src/main/kotlin/com/ninecraft/booket/core/data/api/repository/UserRepository.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ interface UserRepository {
2828

2929
suspend fun setLastNotificationSyncedEnabled(isEnabled: Boolean)
3030

31-
suspend fun resetNotificationData()
32-
3331
suspend fun updateNotificationSettings(notificationEnabled: Boolean): Result<UserProfileModel>
32+
33+
suspend fun resetNotificationData()
3434
}

core/data/impl/src/main/kotlin/com/ninecraft/booket/core/data/impl/repository/DefaultUserRepository.kt

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -71,15 +71,19 @@ internal class DefaultUserRepository @Inject constructor(
7171
notificationDataSource.setLastSyncedNotificationEnabled(isEnabled)
7272
}
7373

74-
override suspend fun resetNotificationData() {
75-
deleteRemoteFcmToken()
76-
clearNotificationDataStore()
77-
}
78-
7974
override suspend fun updateNotificationSettings(notificationEnabled: Boolean) = runSuspendCatching {
8075
service.updateNotificationSettings(NotificationSettingsRequest(notificationEnabled)).toModel()
8176
}
8277

78+
override suspend fun resetNotificationData() {
79+
try {
80+
deleteRemoteFcmToken()
81+
clearNotificationDataStore()
82+
} catch (e: Exception) {
83+
Logger.e("Failed to reset notification data: ${e.message}")
84+
}
85+
}
86+
8387
private suspend fun getRemoteFcmToken(): String {
8488
return try {
8589
firebaseMessaging.token.await()
@@ -110,11 +114,7 @@ internal class DefaultUserRepository @Inject constructor(
110114
}
111115

112116
private suspend fun deleteRemoteFcmToken() {
113-
try {
114-
firebaseMessaging.deleteToken().await()
115-
} catch (e: Exception) {
116-
Logger.e("Failed to delete FCM token: ${e.message}")
117-
}
117+
firebaseMessaging.deleteToken().await()
118118
}
119119

120120
private suspend fun clearNotificationDataStore() {

0 commit comments

Comments
 (0)