Add Sentry about empty Thread

Author: KevinBoulongne

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

@@ -228,8 +228,9 @@ class Thread : RealmObject {
 
     }.getOrElse { throwable ->
         Sentry.withScope { scope ->
-            scope.setExtra("thread.folder.role", "${folder.role?.name}")
+            scope.setExtra("thread.folder.role", folder.role?.name.toString())
             scope.setExtra("thread.folder.id", folder.id)
+            scope.setExtra("thread.folderId", folderId)
             scope.setExtra("thread.uid", uid)
             scope.setExtra("thread.messages.count", "${messages.count()}")
             scope.setExtra("thread.duplicates.count", "${duplicates.count()}")

app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListAdapter.kt

@@ -60,6 +60,7 @@ import com.infomaniak.mail.ui.main.folder.ThreadListAdapter.ThreadListViewHolder
 import com.infomaniak.mail.ui.main.thread.SubjectFormatter
 import com.infomaniak.mail.ui.main.thread.SubjectFormatter.TagColor
 import com.infomaniak.mail.utils.RealmChangesBinding
+import com.infomaniak.mail.utils.SentryDebug
 import com.infomaniak.mail.utils.Utils.runCatchingRealm
 import com.infomaniak.mail.utils.extensions.*
 import dagger.hilt.android.qualifiers.ActivityContext
@@ -196,6 +197,7 @@ class ThreadListAdapter @Inject constructor(
             // TODO: Find why we are sometimes displaying empty Threads, and fix it instead of just deleting them.
             //  It's possibly because we are out of sync, and the situation will resolve itself shortly? ¯\_(ツ)_/¯
             threadListAdapterCallback?.deleteThreadInRealm?.invoke(thread.uid)
+            SentryDebug.sendEmptyThread(thread, "No Message in the Thread when displaying it in ThreadList")
             return
         }
 

app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListFragment.kt

@@ -717,7 +717,7 @@ class ThreadListFragment : TwoPaneFragment(), SwipeRefreshLayout.OnRefreshListen
                     val currentFolder = mainViewModel.currentFolder.value
                     Sentry.withScope { scope ->
                         scope.setExtra("cursor", "$currentFolderCursor")
-                        scope.setExtra("folderRole", "${currentFolder?.role?.name}")
+                        scope.setExtra("folderRole", currentFolder?.role?.name.toString())
                         scope.setExtra("folderThreadsCount", "${currentFolder?.threads?.count()}")
                         Sentry.captureMessage(
                             "Should display threads is true but there are no threads to display",

app/src/main/java/com/infomaniak/mail/ui/main/thread/ThreadViewModel.kt

@@ -274,7 +274,7 @@ class ThreadViewModel @Inject constructor(
         appContext.trackUserInfo("nbMessagesInThread", nbMessages)
 
         when (nbMessages) {
-            0 -> SentryDebug.sendEmptyThread(thread)
+            0 -> SentryDebug.sendEmptyThread(thread, "No Message in the Thread when opening it")
             1 -> appContext.trackUserInfo("oneMessagesInThread")
             else -> appContext.trackUserInfo("multipleMessagesInThread", nbMessages)
         }

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

@@ -117,14 +117,21 @@ object SentryDebug {
     //region Send Sentry
     // TODO: Added the 04/09/23. It's not supposed to be possible, but we never know…
     //  If this doesn't trigger after a certain amount of time, you can remove it.
-    fun sendEmptyThread(thread: Thread) {
+    //
+    //  Also added in ThreadListAdapter the 31/05/24.
+    fun sendEmptyThread(thread: Thread, message: String) = with(thread) {
         Sentry.withScope { scope ->
-            scope.setExtra("currentUserId", "[${AccountUtils.currentUserId}]")
-            scope.setExtra("currentMailboxEmail", "[${AccountUtils.currentMailboxEmail}]")
-            scope.setExtra("folder.role", thread.folder.role?.name.toString())
-            scope.setExtra("folder.id", thread.folder.id)
-            scope.setExtra("thread.uid", "[${thread.uid}]")
-            Sentry.captureMessage("No Message in the Thread when opening it", SentryLevel.ERROR)
+            scope.setExtra("currentUserId", "${AccountUtils.currentUserId}")
+            scope.setExtra("currentMailboxEmail", AccountUtils.currentMailboxEmail.toString())
+            scope.setExtra("folderId", folderId)
+            scope.setExtra("folder.id", folder.id)
+            scope.setExtra("folder.role", folder.role?.name.toString())
+            scope.setExtra("uid", uid)
+            scope.setExtra("messages.count", "${messages.count()}")
+            scope.setExtra("duplicates.count", "${duplicates.count()}")
+            scope.setExtra("isFromSearch", "$isFromSearch")
+            scope.setExtra("hasDrafts", "$hasDrafts")
+            Sentry.captureMessage(message, SentryLevel.ERROR)
         }
     }