fix: crash current must not be NaN during bulk download (#63) (#412)

- Add validation to ensure that the progress value is not NaN. fixes: LEARNER-10248 Co-authored-by: Farhan Arshad <43750646+farhan-arshad-dev@users.noreply.github.com>
openedx · Dec 11, 2024 · 46c32a2 · 46c32a2
1 parent 4566b1a
commit 46c32a2
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/course/src/main/java/org/openedx/course/presentation/ui/CourseVideosUI.kt b/course/src/main/java/org/openedx/course/presentation/ui/CourseVideosUI.kt
@@ -543,7 +543,12 @@ private fun AllVideosDownloadItem(
         )
     }
     if (isDownloadingAllVideos) {
-        val progress = 1 - downloadModelsSize.remainingSize.toFloat() / downloadModelsSize.allSize
+        val progress =
+            if (downloadModelsSize.allSize == 0L) {
+                0f
+            } else {
+                1 - downloadModelsSize.remainingSize.toFloat() / downloadModelsSize.allSize
+            }
 
         val animatedProgress by animateFloatAsState(
             targetValue = progress,