Apply suggestions from code review

KevinBoulongne · FabianDevel · KevinBoulongne · commit 3f3d671fa96a · 2024-09-11T08:02:38.000Z
Co-authored-by: Fabian Devel &lt;fabian.devel@infomaniak.com&gt;
diff --git a/app/src/main/java/com/infomaniak/mail/ui/main/move/MoveAdapter.kt b/app/src/main/java/com/infomaniak/mail/ui/main/move/MoveAdapter.kt
@@ -60,8 +60,7 @@ class MoveAdapter @Inject constructor() : ListAdapter<Any, MoveFolderViewHolder>
 
         selectedFolderId = newSelectedFolderId
 
-        var shouldForceNotify = false
-        if (newShouldDisplayIndent != shouldDisplayIndent) shouldForceNotify = true
+        val shouldForceNotify = newShouldDisplayIndent != shouldDisplayIndent
         shouldDisplayIndent = newShouldDisplayIndent
 
         submitList(newFolders)
diff --git a/app/src/main/java/com/infomaniak/mail/ui/main/move/MoveViewModel.kt b/app/src/main/java/com/infomaniak/mail/ui/main/move/MoveViewModel.kt
@@ -69,7 +69,7 @@ class MoveViewModel @Inject constructor(
             allFolders = folderController.getMoveFolders()
                 .flattenFolderChildrenAndRemoveMessages()
                 .addDividerBeforeFirstCustomFolder(dividerType = Unit)
-                .also { filterResults.postValue(it to true) }
+                .also { folders -> filterResults.postValue(folders to true) }
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ class MoveViewModel @Inject constructor(`
`69`	`69`	`allFolders = folderController.getMoveFolders()`
`70`	`70`	`.flattenFolderChildrenAndRemoveMessages()`
`71`	`71`	`.addDividerBeforeFirstCustomFolder(dividerType = Unit)`
`72`		`- .also { filterResults.postValue(it to true) }`
	`72`	`+ .also { folders -> filterResults.postValue(folders to true) }`
`73`	`73`	`}`
`74`	`74`	`}`
`75`	`75`