Merge branch 'ios-699-app-crashes-when-deleting-a-custom-api-access-m…

…ethod'
mullvad · May 29, 2024 · c58361e · c58361e
2 parents 322bff6 + c4d789a
commit c58361e
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/ios/MullvadVPN/Coordinators/Settings/APIAccess/List/ListAccessMethodViewController.swift b/ios/MullvadVPN/Coordinators/Settings/APIAccess/List/ListAccessMethodViewController.swift
@@ -158,6 +158,7 @@ class ListAccessMethodViewController: UIViewController, UITableViewDelegate {
     }
 
     private func updateDataSource(animated: Bool = true) {
+        guard let dataSource else { return }
         fetchedItems = interactor.fetch()
 
         var snapshot = NSDiffableDataSourceSnapshot<ListAccessMethodSectionIdentifier, ListAccessMethodItemIdentifier>()
@@ -168,11 +169,13 @@ class ListAccessMethodViewController: UIViewController, UITableViewDelegate {
         }
         snapshot.appendItems(itemIdentifiers, toSection: .primary)
 
-        for item in fetchedItems {
-            snapshot.reloadItems([ListAccessMethodItemIdentifier(id: item.id)])
+        if dataSource.snapshot().numberOfItems == fetchedItems.count {
+            for item in fetchedItems {
+                snapshot.reloadItems([ListAccessMethodItemIdentifier(id: item.id)])
+            }
         }
 
-        dataSource?.apply(snapshot, animatingDifferences: animated)
+        dataSource.apply(snapshot, animatingDifferences: animated)
     }
 
     private func dequeueCell(