From 4fce510a2f734df5dc2c676cd14bb8e627c7a52c Mon Sep 17 00:00:00 2001
From: Xayah <zds1249475336@gmail.com>
Date: Sat, 7 Dec 2024 22:43:12 +0800
Subject: [PATCH] fix: Potential IndexOutOfBoundsException

* Fatal Exception: java.lang.IndexOutOfBoundsException: Index 5 out of bounds for length 0

Change-Id: I925c0c2165c856268933b3fe2124d4053662aceb
---
 .../xayah/feature/main/processing/Index.kt    | 47 ++++++++++---------
 1 file changed, 26 insertions(+), 21 deletions(-)

diff --git a/source/feature/main/processing/src/main/kotlin/com/xayah/feature/main/processing/Index.kt b/source/feature/main/processing/src/main/kotlin/com/xayah/feature/main/processing/Index.kt
index 38deb626ff..93c6e47f51 100644
--- a/source/feature/main/processing/src/main/kotlin/com/xayah/feature/main/processing/Index.kt
+++ b/source/feature/main/processing/src/main/kotlin/com/xayah/feature/main/processing/Index.kt
@@ -267,12 +267,14 @@ fun PageProcessing(
 
                                 else -> {
                                     // Processing
-                                    val item = dataItems[it - 1]
-                                    val index = item.processingIndex
-                                    title = item.title
-                                    subtitle = item.items.getOrNull(index)?.let { it.title + if (it.content.isEmpty()) "" else " (${it.content})" } ?: stringResource(id = R.string.necessary_remaining_data_processing)
-                                    segments = item.items.size
-                                    segmentProgress = item.progress
+                                    val item = dataItems.getOrNull(it - 1)
+                                    if (item != null) {
+                                        val index = item.processingIndex
+                                        title = item.title
+                                        subtitle = item.items.getOrNull(index)?.let { it.title + if (it.content.isEmpty()) "" else " (${it.content})" } ?: stringResource(id = R.string.necessary_remaining_data_processing)
+                                        segments = item.items.size
+                                        segmentProgress = item.progress
+                                    }
                                 }
                             }
 
@@ -337,22 +339,25 @@ fun PageProcessing(
                     }
                     items(count = dataItems.size) {
                         var expanded by rememberSaveable(task, it) { mutableStateOf((task?.processingIndex?.minus(1) ?: -1) == it) }
-                        ProcessingCard(
-                            modifier = Modifier
-                                .fillMaxWidth()
-                                .paddingHorizontal(SizeTokens.Level24),
-                            title = dataItems[it].title,
-                            state = dataItems[it].state,
-                            packageName = dataItems[it].key,
-                            expanded = expanded,
-                            items = dataItems[it].items,
-                            processingIndex = dataItems[it].processingIndex,
-                            onActionBarClick = {
-                                if (uiState.state == OperationState.DONE) {
-                                    expanded = expanded.not()
+                        val item = dataItems.getOrNull(it)
+                        if (item != null) {
+                            ProcessingCard(
+                                modifier = Modifier
+                                    .fillMaxWidth()
+                                    .paddingHorizontal(SizeTokens.Level24),
+                                title = item.title,
+                                state = item.state,
+                                packageName = item.key,
+                                expanded = expanded,
+                                items = item.items,
+                                processingIndex = item.processingIndex,
+                                onActionBarClick = {
+                                    if (uiState.state == OperationState.DONE) {
+                                        expanded = expanded.not()
+                                    }
                                 }
-                            }
-                        )
+                            )
+                        }
                     }
                     item {
                         Spacer(modifier = Modifier.size(SizeTokens.Level12 + it))