Fix bug where offline servers can be selected

mullvad · Apr 9, 2024 · 1d69764 · 1d69764
1 parent e859f00
commit 1d69764
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/ios/MullvadVPN/View controllers/SelectLocation/AllLocationDataSource.swift b/ios/MullvadVPN/View controllers/SelectLocation/AllLocationDataSource.swift
@@ -64,7 +64,7 @@ class AllLocationDataSource: LocationDataSourceProtocol {
                 name: serverLocation.country,
                 code: LocationNode.combineNodeCodes([countryCode]),
                 locations: [location],
-                isActive: relay.active
+                isActive: true // Defaults to true, updated when children are populated.
             )
 
             if !rootNode.children.contains(countryNode) {
@@ -77,7 +77,7 @@ class AllLocationDataSource: LocationDataSourceProtocol {
                 name: serverLocation.city,
                 code: LocationNode.combineNodeCodes([countryCode, cityCode]),
                 locations: [location],
-                isActive: relay.active
+                isActive: true // Defaults to true, updated when children are populated.
             )
 
             if let countryNode = rootNode.countryFor(code: countryCode),
@@ -101,6 +101,14 @@ class AllLocationDataSource: LocationDataSourceProtocol {
                 hostNode.parent = cityNode
                 cityNode.children.append(hostNode)
                 cityNode.children.sort()
+
+                cityNode.isActive = cityNode.children.contains(where: { hostNode in
+                    hostNode.isActive
+                })
+
+                countryNode.isActive = countryNode.children.contains(where: { cityNode in
+                    cityNode.isActive
+                })
             }
         }
     }

diff --git a/ios/MullvadVPN/View controllers/SelectLocation/LocationDataSource.swift b/ios/MullvadVPN/View controllers/SelectLocation/LocationDataSource.swift
@@ -236,6 +236,10 @@ extension LocationDataSource: UITableViewDelegate {
         }
     }
 
+    func tableView(_ tableView: UITableView, shouldHighlightRowAt indexPath: IndexPath) -> Bool {
+        itemIdentifier(for: indexPath)?.node.isActive ?? false
+    }
+
     func tableView(_ tableView: UITableView, indentationLevelForRowAt indexPath: IndexPath) -> Int {
         itemIdentifier(for: indexPath)?.indentationLevel ?? 0
     }