Remove unused COLLAPSABLE_FOLDER_EXISTENCE_HAS_CHANGED payload

Infomaniak · Jul 4, 2024 · 8820798 · 8820798
1 parent 9c9dbcb
commit 8820798
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 36 deletions.
diff --git a/app/src/main/java/com/infomaniak/mail/ui/main/menuDrawer/MenuDrawerAdapter.kt b/app/src/main/java/com/infomaniak/mail/ui/main/menuDrawer/MenuDrawerAdapter.kt
@@ -215,21 +215,13 @@ class MenuDrawerAdapter @Inject constructor() : ListAdapter<Any, MenuDrawerViewH
     }
 
     override fun onBindViewHolder(holder: MenuDrawerViewHolder, position: Int, payloads: MutableList<Any>) {
-        when (payloads.firstOrNull()) {
-            NotifyType.MAILBOXES_HEADER_CLICKED -> MailboxesHeaderItem.displayWithPayload(
+        if (payloads.firstOrNull() == NotifyType.MAILBOXES_HEADER_CLICKED) {
+            MailboxesHeaderItem.displayWithPayload(
                 item = items[position],
                 binding = holder.binding,
             )
-            NotifyType.COLLAPSABLE_FOLDER_EXISTENCE_HAS_CHANGED -> FolderItem.displayWithPayload(
-                item = items[position],
-                binding = holder.binding,
-                currentFolderId = currentFolderId,
-                hasCollapsableDefaultFolder = hasCollapsableDefaultFolder,
-                hasCollapsableCustomFolder = hasCollapsableCustomFolder,
-                onFolderClicked = onFolderClicked,
-                onCollapseChildrenClicked = onCollapseChildrenClicked,
-            )
-            else -> super.onBindViewHolder(holder, position, payloads)
+        } else {
+            super.onBindViewHolder(holder, position, payloads)
         }
     }
 
@@ -290,7 +282,6 @@ class MenuDrawerAdapter @Inject constructor() : ListAdapter<Any, MenuDrawerViewH
 
     private enum class NotifyType {
         MAILBOXES_HEADER_CLICKED,
-        COLLAPSABLE_FOLDER_EXISTENCE_HAS_CHANGED, // TODO: This seems to never be notified ??
     }
 
     private class FolderDiffCallback : DiffUtil.ItemCallback<Any>() {

diff --git a/app/src/main/java/com/infomaniak/mail/ui/main/menuDrawer/items/FolderItem.kt b/app/src/main/java/com/infomaniak/mail/ui/main/menuDrawer/items/FolderItem.kt
@@ -63,29 +63,6 @@ object FolderItem {
         )
     }
 
-    fun displayWithPayload(
-        item: Any,
-        binding: ViewBinding,
-        currentFolderId: String?,
-        hasCollapsableDefaultFolder: Boolean,
-        hasCollapsableCustomFolder: Boolean,
-        onFolderClicked: (folderId: String) -> Unit,
-        onCollapseChildrenClicked: (folderId: String, shouldCollapse: Boolean) -> Unit,
-    ) {
-        item as Folder
-        binding as ItemMenuDrawerFolderBinding
-
-        Log.d("Bind", "Bind Custom folders because of collapse change = ${item.name}")
-        binding.displayFolder(
-            item,
-            currentFolderId,
-            hasCollapsableDefaultFolder,
-            hasCollapsableCustomFolder,
-            onFolderClicked,
-            onCollapseChildrenClicked,
-        )
-    }
-
     private fun ItemMenuDrawerFolderBinding.displayFolder(
         folder: Folder,
         currentFolderId: String?,