Allow deleting custom lists

Author: Jon Petersson

ios/MullvadSettings/CustomListRepository.swift

@@ -18,8 +18,8 @@ public enum CustomRelayListError: LocalizedError, Equatable {
         switch self {
         case .duplicateName:
             NSLocalizedString(
-                "DUPLICATE_CUSTOM_LIST_ERROR",
-                tableName: "CustomListRepository",
+                "DUPLICATE_CUSTOM_LISTS_ERROR",
+                tableName: "CustomLists",
                 value: "Name is already taken.",
                 comment: ""
             )

ios/MullvadVPN/Coordinators/CustomLists/AddCustomListCoordinator.swift

@@ -42,14 +42,14 @@ class AddCustomListCoordinator: Coordinator, Presentable, Presenting {
         controller.delegate = self
 
         controller.navigationItem.title = NSLocalizedString(
-            "CUSTOM_LIST_NAVIGATION_EDIT_TITLE",
+            "CUSTOM_LISTS_NAVIGATION_EDIT_TITLE",
             tableName: "CustomLists",
             value: "New custom list",
             comment: ""
         )
 
         controller.saveBarButton.title = NSLocalizedString(
-            "CUSTOM_LIST_NAVIGATION_CREATE_BUTTON",
+            "CUSTOM_LISTS_NAVIGATION_CREATE_BUTTON",
             tableName: "CustomLists",
             value: "Create",
             comment: ""

ios/MullvadVPN/Coordinators/CustomLists/CustomListViewController.swift

@@ -158,26 +158,35 @@ class CustomListViewController: UIViewController {
     }
 
     private func onDelete() {
-        // TODO: Show error dialog.
-        delegate?.customListDidDelete()
-    }
-
-    private func showSaveErrorAlert() {
         let presentation = AlertPresentation(
-            id: "api-custom-lists-save-list-alert",
+            id: "api-custom-lists-delete-list-alert",
             icon: .alert,
             message: NSLocalizedString(
-                "CUSTOM_LISTS_SAVE_ERROR_PROMPT",
-                tableName: "APIAccess",
-                value: "List name is already taken.",
+                "CUSTOM_LISTS_DELETE_PROMPT",
+                tableName: "CustomLists",
+                value: "Delete custom list \(subject.value.name)?",
                 comment: ""
             ),
             buttons: [
                 AlertAction(
                     title: NSLocalizedString(
-                        "CUSTOM_LISTS_OK_BUTTON",
-                        tableName: "APIAccess",
-                        value: "Got it!",
+                        "CUSTOM_LISTS_DELETE_BUTTON",
+                        tableName: "CustomLists",
+                        value: "Delete",
+                        comment: ""
+                    ),
+                    style: .destructive,
+                    handler: {
+                        self.interactor.deleteCustomList(id: self.subject.value.id)
+                        self.dismiss(animated: true)
+                        self.delegate?.customListDidDelete()
+                    }
+                ),
+                AlertAction(
+                    title: NSLocalizedString(
+                        "CUSTOM_LISTS_CANCEL_BUTTON",
+                        tableName: "CustomLists",
+                        value: "Cancel",
                         comment: ""
                     ),
                     style: .default

ios/MullvadVPN/Coordinators/CustomLists/EditCustomListCoordinator.swift

@@ -14,6 +14,7 @@ import UIKit
 class EditCustomListCoordinator: Coordinator, Presentable, Presenting {
     let navigationController: UINavigationController
     let customListInteractor: CustomListInteractorProtocol
+    let customList: CustomList
 
     var presentedViewController: UIViewController {
         navigationController
@@ -23,18 +24,20 @@ class EditCustomListCoordinator: Coordinator, Presentable, Presenting {
 
     init(
         navigationController: UINavigationController,
-        customListInteractor: CustomListInteractorProtocol
+        customListInteractor: CustomListInteractorProtocol,
+        customList: CustomList
     ) {
         self.navigationController = navigationController
         self.customListInteractor = customListInteractor
+        self.customList = customList
     }
 
     func start() {
         let subject = CurrentValueSubject<CustomListViewModel, Never>(
             CustomListViewModel(
-                id: UUID(),
-                name: "A list",
-                locations: [],
+                id: customList.id,
+                name: customList.name,
+                locations: customList.locations,
                 tableSections: [.name, .editLocations, .deleteList]
             )
         )

ios/MullvadVPN/Coordinators/SelectLocationCoordinator.swift

@@ -11,8 +11,6 @@ import MullvadTypes
 import Routing
 import UIKit
 
-import MullvadSettings
-
 class SelectLocationCoordinator: Coordinator, Presentable, Presenting, RelayCacheTrackerObserver {
     private let tunnelManager: TunnelManager
     private let relayCacheTracker: RelayCacheTracker

ios/MullvadVPN/Coordinators/Settings/APIAccess/Edit/EditAccessMethodViewController.swift

@@ -333,7 +333,7 @@ class EditAccessMethodViewController: UITableViewController {
             id: "api-access-methods-delete-method-alert",
             icon: .alert,
             message: NSLocalizedString(
-                "METHOD_SETTINGS_SAVE_PROMPT",
+                "METHOD_SETTINGS_DELETE_PROMPT",
                 tableName: "APIAccess",
                 value: "Delete \(methodName)?",
                 comment: ""