Merge branch 'custom-list-crashes-when-naming-a-list-with-the-name-of…

…-an-ios-599'
mullvad · Apr 15, 2024 · 6973de9 · 6973de9
2 parents 8f8bd1d + c3d575e
commit 6973de9
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/ios/MullvadSettings/CustomListRepository.swift b/ios/MullvadSettings/CustomListRepository.swift
@@ -39,11 +39,12 @@ public struct CustomListRepository: CustomListRepositoryProtocol {
     public func save(list: CustomList) throws {
         var lists = fetchAll()
 
-        if let index = lists.firstIndex(where: { $0.id == list.id }) {
+        if let listWithSameName = lists.first(where: { $0.name.caseInsensitiveCompare(list.name) == .orderedSame }),
+           listWithSameName.id != list.id {
+            throw CustomRelayListError.duplicateName
+        } else if let index = lists.firstIndex(where: { $0.id == list.id }) {
             lists[index] = list
             try write(lists)
-        } else if lists.contains(where: { $0.name == list.name }) {
-            throw CustomRelayListError.duplicateName
         } else {
             lists.append(list)
             try write(lists)