fix: [ANDROAPP-6519] Crash when accessing cell (#3924)

dhis2 · Dec 13, 2024 · fa5d302 · fa5d302
1 parent f8dcbfd
commit fa5d302
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/compose-table/src/main/java/org/dhis2/composetable/model/TableModel.kt b/compose-table/src/main/java/org/dhis2/composetable/model/TableModel.kt
@@ -35,14 +35,17 @@ data class TableModel(
     ): Pair<TableCell, TableSelection.CellSelection>? = when {
         !successValidation ->
             cellSelection
+
         cellSelection.columnIndex < tableHeaderModel.tableMaxColumns() - 1 ->
             cellSelection.copy(columnIndex = cellSelection.columnIndex + 1)
+
         cellSelection.rowIndex < tableRows.size - 1 ->
             cellSelection.copy(
                 columnIndex = 0,
                 rowIndex = cellSelection.rowIndex + 1,
                 globalIndex = cellSelection.globalIndex + 1,
             )
+
         else -> null
     }?.let { nextCell ->
         val tableCell = tableRows[nextCell.rowIndex].values[nextCell.columnIndex]
@@ -57,10 +60,14 @@ data class TableModel(
             tableRows.size == 1 && tableRows.size == cell.rowIndex -> {
                 tableRows[0].values[cell.columnIndex]?.takeIf { it.error != null }
             }
+
             tableRows.size == cell.rowIndex -> {
                 tableRows[cell.rowIndex - 1].values[cell.columnIndex]?.takeIf { it.error != null }
             }
-            else -> tableRows[cell.rowIndex].values[cell.columnIndex]?.takeIf { it.error != null }
+
+            else -> tableRows.getOrNull(cell.rowIndex)
+                ?.values?.get(cell.columnIndex)
+                ?.takeIf { it.error != null }
         }
     }