fix: handle NoCachedDataException

Author: PavloNetrebchuk

app/src/main/java/org/openedx/app/di/ScreenModule.kt

@@ -494,6 +494,7 @@ val screenModule = module {
             get(),
             get(),
             get(),
+            get()
         )
     }
     viewModel { (courseId: String) ->

course/src/main/java/org/openedx/course/presentation/offline/CourseOfflineViewModel.kt

@@ -25,6 +25,8 @@ import org.openedx.core.presentation.CoreAnalytics
 import org.openedx.core.presentation.dialog.downloaddialog.DownloadDialogItem
 import org.openedx.core.presentation.dialog.downloaddialog.DownloadDialogManager
 import org.openedx.core.system.connection.NetworkConnection
+import org.openedx.core.system.notifier.CourseNotifier
+import org.openedx.core.system.notifier.CourseStructureGot
 import org.openedx.course.domain.interactor.CourseInteractor
 import org.openedx.foundation.extension.toFileSize
 import org.openedx.foundation.utils.FileUtil
@@ -37,6 +39,7 @@ class CourseOfflineViewModel(
     private val downloadDialogManager: DownloadDialogManager,
     private val fileUtil: FileUtil,
     private val networkConnection: NetworkConnection,
+    private val courseNotifier: CourseNotifier,
     coreAnalytics: CoreAnalytics,
     downloadDao: DownloadDao,
     workerController: DownloadWorkerController,
@@ -71,38 +74,36 @@ class CourseOfflineViewModel(
                 _uiState.update { it.copy(isDownloading = isDownloading) }
             }
         }
-
-        viewModelScope.launch {
-            async { initDownloadFragment() }.await()
-            getOfflineData()
-        }
+        collectCourseNotifier()
     }
 
     fun downloadAllBlocks(fragmentManager: FragmentManager) {
         viewModelScope.launch {
-            val courseStructure = courseInteractor.getCourseStructureFromCache(courseId)
-            val downloadModels = courseInteractor.getAllDownloadModels()
-            val subSectionsBlocks = allBlocks.values.filter { it.type == BlockType.SEQUENTIAL }
-            val notDownloadedSubSectionBlocks = subSectionsBlocks.mapNotNull { subSection ->
-                val verticalBlocks = allBlocks.values.filter { it.id in subSection.descendants }
-                val notDownloadedBlocks = courseStructure.blockData.filter { block ->
-                    block.id in verticalBlocks.flatMap { it.descendants } &&
-                            block.isDownloadable &&
-                            downloadModels.none { it.id == block.id }
+            runCatching {
+                val courseStructure = courseInteractor.getCourseStructureFromCache(courseId)
+                val downloadModels = courseInteractor.getAllDownloadModels()
+                val subSectionsBlocks = allBlocks.values.filter { it.type == BlockType.SEQUENTIAL }
+                val notDownloadedSubSectionBlocks = subSectionsBlocks.mapNotNull { subSection ->
+                    val verticalBlocks = allBlocks.values.filter { it.id in subSection.descendants }
+                    val notDownloadedBlocks = courseStructure.blockData.filter { block ->
+                        block.id in verticalBlocks.flatMap { it.descendants } &&
+                                block.isDownloadable &&
+                                downloadModels.none { it.id == block.id }
+                    }
+                    if (notDownloadedBlocks.isNotEmpty()) subSection else null
                 }
-                if (notDownloadedBlocks.isNotEmpty()) subSection else null
-            }
 
-            downloadDialogManager.showPopup(
-                subSectionsBlocks = notDownloadedSubSectionBlocks,
-                courseId = courseId,
-                isBlocksDownloaded = false,
-                fragmentManager = fragmentManager,
-                removeDownloadModels = ::removeDownloadModels,
-                saveDownloadModels = { blockId ->
-                    saveDownloadModels(fileUtil.getExternalAppDir().path, courseId, blockId)
-                }
-            )
+                downloadDialogManager.showPopup(
+                    subSectionsBlocks = notDownloadedSubSectionBlocks,
+                    courseId = courseId,
+                    isBlocksDownloaded = false,
+                    fragmentManager = fragmentManager,
+                    removeDownloadModels = ::removeDownloadModels,
+                    saveDownloadModels = { blockId ->
+                        saveDownloadModels(fileUtil.getExternalAppDir().path, courseId, blockId)
+                    }
+                )
+            }
         }
     }
 
@@ -224,4 +225,17 @@ class CourseOfflineViewModel(
             }
         }
     }
+
+    private fun collectCourseNotifier() {
+        viewModelScope.launch {
+            courseNotifier.notifier.collect { event ->
+                when (event) {
+                    is CourseStructureGot -> {
+                        async { initDownloadFragment() }.await()
+                        getOfflineData()
+                    }
+                }
+            }
+        }
+    }
 }