Merge pull request #1986 from Infomaniak/fix/add-sentry-for-mailbox-with-default-id

Add SentryLog when mailboxId uses the default value for configuration

Author: TommyDL-Infomaniak

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

@@ -19,6 +19,7 @@ package com.infomaniak.mail.data.cache
 
 import android.content.Context
 import com.infomaniak.mail.data.models.AppSettings
+import com.infomaniak.mail.data.models.AppSettings.Companion.DEFAULT_ID
 import com.infomaniak.mail.data.models.Attachment
 import com.infomaniak.mail.data.models.Bimi
 import com.infomaniak.mail.data.models.Folder
@@ -44,6 +45,8 @@ import com.infomaniak.mail.utils.LocalStorageUtils
 import io.realm.kotlin.Realm
 import io.realm.kotlin.RealmConfiguration
 import io.realm.kotlin.internal.platform.WeakReference
+import io.sentry.Sentry
+import io.sentry.SentryLevel
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.runBlocking
 import kotlinx.coroutines.sync.Mutex
@@ -86,7 +89,7 @@ object RealmDatabase {
     val mailboxInfo get() = Realm.open(RealmConfig.mailboxInfo)
 
     val newMailboxContentInstance get() = newMailboxContentInstance(AccountUtils.currentUserId, AccountUtils.currentMailboxId)
-    fun newMailboxContentInstance(userId: Int, mailboxId: Int) = Realm.open(RealmConfig.mailboxContent(mailboxId, userId))
+    fun newMailboxContentInstance(userId: Int, mailboxId: Int) = Realm.open(RealmConfig.mailboxContent(userId, mailboxId))
 
     class MailboxContent {
         operator fun invoke() = runBlocking(Dispatchers.IO) {
@@ -139,7 +142,7 @@ object RealmDatabase {
 
     //region Delete Realm
     fun deleteMailboxContent(mailboxId: Int, userId: Int = AccountUtils.currentUserId) {
-        Realm.deleteRealm(RealmConfig.mailboxContent(mailboxId, userId))
+        Realm.deleteRealm(RealmConfig.mailboxContent(userId, mailboxId))
     }
 
     fun removeUserData(context: Context, userId: Int) {
@@ -229,12 +232,18 @@ object RealmDatabase {
             .migration(MAILBOX_INFO_MIGRATION)
             .build()
 
-        fun mailboxContent(mailboxId: Int, userId: Int) = RealmConfiguration
-            .Builder(mailboxContentSet)
-            .name(mailboxContentDbName(userId, mailboxId))
-            .schemaVersion(MAILBOX_CONTENT_SCHEMA_VERSION)
-            .migration(MAILBOX_CONTENT_MIGRATION)
-            .build()
+        fun mailboxContent(userId: Int, mailboxId: Int): RealmConfiguration {
+
+            if (mailboxId == DEFAULT_ID) {
+                Sentry.captureMessage("RealmConfiguration problem with mailbox content, default ID is used.", SentryLevel.ERROR)
+            }
+
+            return RealmConfiguration.Builder(mailboxContentSet)
+                .name(mailboxContentDbName(userId, mailboxId))
+                .schemaVersion(MAILBOX_CONTENT_SCHEMA_VERSION)
+                .migration(MAILBOX_CONTENT_MIGRATION)
+                .build()
+        }
         //endregion
     }
 }