Skip to content

Commit c9ed491

Browse files
authored
feat(MyKSuite-07): Clean data field (#2188)
2 parents 50e3328 + 2f2643b commit c9ed491

File tree

8 files changed

+5
-25
lines changed

8 files changed

+5
-25
lines changed

app/src/main/java/com/infomaniak/mail/data/cache/RealmDatabase.kt

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,6 @@ object RealmDatabase {
159159
private object RealmConfig {
160160

161161
//region Configurations versions
162-
const val APP_SETTINGS_SCHEMA_VERSION = 2L
163162
const val USER_INFO_SCHEMA_VERSION = 2L
164163
const val MAILBOX_INFO_SCHEMA_VERSION = 8L
165164
const val MAILBOX_CONTENT_SCHEMA_VERSION = 21L
@@ -209,8 +208,6 @@ object RealmDatabase {
209208
val appSettings = RealmConfiguration
210209
.Builder(appSettingsSet)
211210
.name(appSettingsDbName)
212-
.schemaVersion(APP_SETTINGS_SCHEMA_VERSION)
213-
.migration(APP_SETTINGS_MIGRATION)
214211
.build()
215212

216213
val userInfo

app/src/main/java/com/infomaniak/mail/data/cache/RealmMigrations.kt

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,6 @@ import io.realm.kotlin.dynamic.getValue
2424
import io.realm.kotlin.migration.AutomaticSchemaMigration
2525
import io.realm.kotlin.migration.AutomaticSchemaMigration.MigrationContext
2626

27-
val APP_SETTINGS_MIGRATION = AutomaticSchemaMigration { migrationContext ->
28-
SentryDebug.addMigrationBreadcrumb(migrationContext)
29-
}
30-
3127
val USER_INFO_MIGRATION = AutomaticSchemaMigration { migrationContext ->
3228
SentryDebug.addMigrationBreadcrumb(migrationContext)
3329
migrationContext.deleteRealmFromFirstMigration()

app/src/main/java/com/infomaniak/mail/data/models/AppSettings.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ class AppSettings : RealmObject {
2323

2424
var currentUserId: Int = DEFAULT_ID
2525
var currentMailboxId: Int = DEFAULT_ID
26-
var myKSuiteId: Int = DEFAULT_ID
2726

2827
companion object {
2928
const val DEFAULT_ID = -1

app/src/main/java/com/infomaniak/mail/ui/main/settings/SettingsFragment.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ class SettingsFragment : Fragment() {
9090
private fun setupMyKSuiteLayout(myKSuiteData: MyKSuiteData) = with(binding) {
9191
observeMyKSuiteMailbox()
9292

93-
myKSuiteData.kSuitePack.type?.displayNameRes?.let(myKSuiteSettingsTitle::setText)
93+
myKSuiteSettingsTitle.setText(myKSuiteData.name)
9494

9595
myKSuiteViewModel.getMyKSuiteMailbox(myKSuiteData.mail.mailboxId)
9696

app/src/main/java/com/infomaniak/mail/utils/LogoutUser.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ class LogoutUser @Inject constructor(
5252

5353
user.logoutToken()
5454
AccountUtils.removeUser(user)
55+
MyKSuiteDataUtils.deleteKSuiteData(user.id)
5556
RealmDatabase.removeUserData(appContext, user.id)
5657
mailboxController.deleteUserMailboxes(user.id)
5758
localSettings.removeRegisteredFirebaseUser(userId = user.id)

app/src/main/java/com/infomaniak/mail/utils/MyKSuiteDataUtils.kt

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,6 @@ import com.infomaniak.core.myksuite.ui.data.MyKSuiteDataManager
2222
import com.infomaniak.lib.core.networking.HttpClient
2323
import com.infomaniak.lib.core.utils.SentryLog
2424
import com.infomaniak.mail.data.api.ApiRepository
25-
import com.infomaniak.mail.data.cache.appSettings.AppSettingsController
26-
import com.infomaniak.mail.data.models.AppSettings
2725
import kotlinx.serialization.ExperimentalSerializationApi
2826
import kotlinx.serialization.MissingFieldException
2927
import kotlin.coroutines.cancellation.CancellationException
@@ -32,19 +30,9 @@ object MyKSuiteDataUtils : MyKSuiteDataManager() {
3230

3331
private val TAG = MyKSuiteDataUtils::class.simpleName.toString()
3432

35-
override val userId get() = AccountUtils.currentUserId
36-
37-
override var myKSuiteId: Int = AppSettingsController.getAppSettings().myKSuiteId
38-
set(myKSuiteId) {
39-
field = myKSuiteId
40-
AppSettingsController.updateAppSettings { appSettings -> appSettings.myKSuiteId = myKSuiteId }
41-
}
33+
override val currentUserId get() = AccountUtils.currentUserId
4234

4335
override var myKSuite: MyKSuiteData? = null
44-
set(myKSuiteData) {
45-
field = myKSuiteData
46-
myKSuiteId = myKSuiteData?.id ?: AppSettings.DEFAULT_ID
47-
}
4836

4937
suspend fun fetchMyKSuiteData(): MyKSuiteData? = runCatching {
5038
MyKSuiteDataUtils.requestKSuiteData()

app/src/main/java/com/infomaniak/mail/utils/MyKSuiteUiUtils.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ package com.infomaniak.mail.utils
2020
import android.content.Context
2121
import androidx.fragment.app.Fragment
2222
import androidx.navigation.NavController
23-
import com.infomaniak.core.myksuite.ui.components.MyKSuiteTier
2423
import com.infomaniak.core.myksuite.ui.data.MyKSuiteData
2524
import com.infomaniak.core.myksuite.ui.screens.KSuiteApp
2625
import com.infomaniak.core.myksuite.ui.screens.MyKSuiteDashboardScreenData
@@ -51,7 +50,7 @@ object MyKSuiteUiUtils {
5150

5251
fun getDashboardData(context: Context, myKSuiteData: MyKSuiteData): MyKSuiteDashboardScreenData {
5352
return MyKSuiteDashboardScreenData(
54-
myKSuiteTier = if (myKSuiteData.isMyKSuitePlus) MyKSuiteTier.Plus else MyKSuiteTier.Free,
53+
myKSuiteTier = myKSuiteData.tier,
5554
email = myKSuiteData.mail.email,
5655
avatarUri = AccountUtils.currentUser?.avatar ?: "",
5756
dailySendingLimit = myKSuiteData.mail.dailyLimitSent.toString(),

0 commit comments

Comments
 (0)