From fb3279b4c4ce0d02208c19c3a5d7fa3bc6deaaa5 Mon Sep 17 00:00:00 2001
From: TommyDL-Infomaniak <tommy.diluna@infomaniak.com>
Date: Tue, 18 Feb 2025 08:57:59 +0100
Subject: [PATCH] fix: Do not open scheduled draft on long press

---
 .../mail/ui/main/folder/ThreadListAdapter.kt         | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListAdapter.kt b/app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListAdapter.kt
index fc67b9ef6c..a7b01abe6d 100644
--- a/app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListAdapter.kt
+++ b/app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListAdapter.kt
@@ -259,15 +259,15 @@ class ThreadListAdapter @Inject constructor(
 
         multiSelection?.let { listener ->
             selectionCardView.setOnLongClickListener {
-                onThreadClickWithAbilityToOpenMultiSelection(thread, position, listener, TrackerAction.LONG_PRESS)
+                onThreadClickWithAbilityToOpenMultiSelection(thread, listener, TrackerAction.LONG_PRESS)
                 true
             }
             expeditorAvatar.apply {
                 setOnClickListener {
-                    onThreadClickWithAbilityToOpenMultiSelection(thread, position, listener, TrackerAction.CLICK)
+                    onThreadClickWithAbilityToOpenMultiSelection(thread, listener, TrackerAction.CLICK)
                 }
                 setOnLongClickListener {
-                    onThreadClickWithAbilityToOpenMultiSelection(thread, position, listener, TrackerAction.LONG_PRESS)
+                    onThreadClickWithAbilityToOpenMultiSelection(thread, listener, TrackerAction.LONG_PRESS)
                     true
                 }
             }
@@ -319,14 +319,10 @@ class ThreadListAdapter @Inject constructor(
 
     private fun CardviewThreadItemBinding.onThreadClickWithAbilityToOpenMultiSelection(
         thread: Thread,
-        position: Int,
         listener: MultiSelectionListener<Thread>,
         action: TrackerAction,
     ) {
-        if (isMultiselectDisabledInThisFolder) {
-            onThreadClicked(thread, position)
-            return
-        }
+        if (isMultiselectDisabledInThisFolder) return
 
         val hasOpened = openMultiSelectionIfClosed(listener, action)
         toggleMultiSelectedThread(thread, shouldUpdateSelectedUi = !hasOpened)