chore: Remove unnecessary nullable return

Infomaniak · Nov 28, 2024 · ca19518 · ca19518
1 parent 7d5fc95
commit ca19518
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/screen/newtransfer/ImportFilesViewModel.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/screen/newtransfer/ImportFilesViewModel.kt
@@ -104,7 +104,7 @@ class ImportFilesViewModel @Inject constructor(
         }
     }
 
-    fun getFiles(folderUuid: String?): Flow<List<FileUi>?> {
+    fun getFiles(folderUuid: String?): Flow<List<FileUi>> {
         return if (folderUuid == null) {
             importationFilesManager.importedFiles
         } else {

diff --git a/...java/com/infomaniak/swisstransfer/ui/screen/newtransfer/importfiles/FilesDetailsScreen.kt b/...java/com/infomaniak/swisstransfer/ui/screen/newtransfer/importfiles/FilesDetailsScreen.kt
@@ -49,18 +49,16 @@ fun FilesDetailsScreen(
     navigateBack: (() -> Unit),
 ) {
     // If we don't have a folderUuid, it means we have to load files from importedFiles in ImportFilesViewModel
-    val files by importFilesViewModel.getFiles(folderUuid).collectAsStateWithLifecycle(null)
+    val files by importFilesViewModel.getFiles(folderUuid).collectAsStateWithLifecycle(emptyList())
 
-    if (files?.isEmpty() == true) navigateBack()
+    if (files.isEmpty() == true) navigateBack()
 
     FilesDetailsScreen(
-        files = files ?: emptyList(),
+        files = files,
         navigateToDetails = navigateToDetails,
         withFileSize = withFileSize,
         withSpaceLeft = withSpaceLeft,
-        onFileRemoved = if (withFileDelete) {
-            { importFilesViewModel.removeFileByUid(it) }
-        } else null,
+        onFileRemoved = getOnFileRemoveCallback(withFileDelete, importFilesViewModel),
         onCloseClicked = onCloseClicked,
         navigateBack = navigateBack,
     )