fix: Potential crash in carousel (IndexOutOfBounds)

Lastaapps · Feb 28, 2025 · 6da38a5 · 6da38a5
1 parent 078fb46
commit 6da38a5
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/app/src/main/kotlin/cz/lastaapps/menza/features/today/ui/widget/TodayDishCarousel.kt b/app/src/main/kotlin/cz/lastaapps/menza/features/today/ui/widget/TodayDishCarousel.kt
@@ -241,7 +241,12 @@ private fun DishContent(
                                 .fillMaxWidth(),
 //                            .animateItem(),
                     ) { index ->
-                        val dish = category.dishList[index]
+                        // As the carousel component is broken in almost every way,
+                        // higher index from previous dish list is provided to new items
+                        // therefore we need to check if the index is valid
+                        val dish =
+                            category.dishList.getOrNull(index)
+                                ?: return@HorizontalMultiBrowseCarousel
 
                         // outside so the computation is run one only
                         // even if progress is called multiple times