fix: handle studentViewData for video and discussion blocks

- Add checks to show appropriate view in case studentViewData is NullOrEmpty
openedx · Dec 11, 2023 · 044da26 · 044da26
1 parent 1393a28
commit 044da26
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/...rc/main/java/org/openedx/course/presentation/unit/container/CourseUnitContainerAdapter.kt b/...rc/main/java/org/openedx/course/presentation/unit/container/CourseUnitContainerAdapter.kt
@@ -23,8 +23,8 @@ class CourseUnitContainerAdapter(
 
     private fun unitBlockFragment(block: Block): Fragment {
         return when {
-            block.isVideoBlock -> {
-                val encodedVideos = block.studentViewData!!.encodedVideos!!
+            (block.isVideoBlock && block.studentViewData?.encodedVideos != null) -> {
+                val encodedVideos = block.studentViewData?.encodedVideos!!
                 val transcripts = block.studentViewData!!.transcripts
                 with(encodedVideos) {
                     var isDownloaded = false
@@ -65,7 +65,7 @@ class CourseUnitContainerAdapter(
                 }
             }
 
-            block.isDiscussionBlock -> {
+            (block.isDiscussionBlock && block.studentViewData?.topicId.isNullOrEmpty().not()) -> {
                 DiscussionThreadsFragment.newInstance(
                     DiscussionTopicsFragment.TOPIC,
                     viewModel.courseId,