Allow deleting custom lists

mullvad · Mar 5, 2024 · 1f599ce · 1f599ce
1 parent ba74cac
commit 1f599ce
Show file tree

Hide file tree

Showing 6 changed files with 42 additions and 25 deletions.
diff --git a/ios/MullvadSettings/CustomListRepository.swift b/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: ""
             )

diff --git a/ios/MullvadVPN/Coordinators/CustomLists/AddCustomListCoordinator.swift b/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: ""

diff --git a/ios/MullvadVPN/Coordinators/CustomLists/CustomListViewController.swift b/ios/MullvadVPN/Coordinators/CustomLists/CustomListViewController.swift
@@ -158,26 +158,40 @@ class CustomListViewController: UIViewController {
     }
 
     private func onDelete() {
-        // TODO: Show error dialog.
-        delegate?.customListDidDelete()
-    }
+        let message = NSMutableAttributedString(
+            markdownString: NSLocalizedString(
+                "CUSTOM_LISTS_DELETE_PROMPT",
+                tableName: "CustomLists",
+                value: "Do you want to delete the list **\(subject.value.name)**?",
+                comment: ""
+            ),
+            options: MarkdownStylingOptions(font: .preferredFont(forTextStyle: .body))
+        )
 
-    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.",
-                comment: ""
-            ),
+            attributedMessage: message,
             buttons: [
                 AlertAction(
                     title: NSLocalizedString(
-                        "CUSTOM_LISTS_OK_BUTTON",
-                        tableName: "APIAccess",
-                        value: "Got it!",
+                        "CUSTOM_LISTS_DELETE_BUTTON",
+                        tableName: "CustomLists",
+                        value: "Delete list",
+                        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

diff --git a/ios/MullvadVPN/Coordinators/CustomLists/EditCustomListCoordinator.swift b/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]
             )
         )

diff --git a/ios/MullvadVPN/Coordinators/Settings/APIAccess/Edit/EditAccessMethodViewController.swift b/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: ""

diff --git a/ios/MullvadVPN/View controllers/Alert/AlertViewController.swift b/ios/MullvadVPN/View controllers/Alert/AlertViewController.swift
@@ -231,8 +231,8 @@ class AlertViewController: UIViewController {
         style.lineBreakMode = .byWordWrapping
 
         label.attributedText = NSAttributedString(
-            markdownString: message,
-            options: MarkdownStylingOptions(font: font, paragraphStyle: style)
+            string: message,
+            attributes: [.paragraphStyle: style]
         )
         label.font = font
         label.textColor = .white.withAlphaComponent(0.8)