fix: Tab index out of number of tabs range

wix · Dec 12, 2023 · 6bfcf16 · 6bfcf16
1 parent 6e7e8f6
commit 6bfcf16
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/.../main/java/com/reactnativenavigation/viewcontrollers/bottomtabs/BottomTabsController.java b/.../main/java/com/reactnativenavigation/viewcontrollers/bottomtabs/BottomTabsController.java
@@ -269,10 +269,11 @@ public void selectTab(final int newIndex) {
     }
 
     private void selectTab(int newIndex, boolean enableSelectionHistory) {
-        saveTabSelection(newIndex, enableSelectionHistory);
-        tabsAttacher.onTabSelected(tabs.get(newIndex));
+        final int nextIndex = Math.max(0, Math.min(newIndex, tabs.size() - 1));
+        saveTabSelection(nextIndex, enableSelectionHistory);
+        tabsAttacher.onTabSelected(tabs.get(nextIndex));
         getCurrentView().setVisibility(View.INVISIBLE);
-        bottomTabs.setCurrentItem(newIndex, false);
+        bottomTabs.setCurrentItem(nextIndex, false);
         getCurrentView().setVisibility(View.VISIBLE);
         getCurrentChild().onViewWillAppear();
         getCurrentChild().onViewDidAppear();