Merge branch 'main' into feature/spezi-account-2.0

StanfordSpezi · Aug 2, 2024 · 982d664 · 982d664
2 parents 55c9a32 + e381b7d
commit 982d664
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/Sources/SpeziFirebaseAccountStorage/FirestoreAccountStorage.swift b/Sources/SpeziFirebaseAccountStorage/FirestoreAccountStorage.swift
@@ -170,8 +170,10 @@ public actor FirestoreAccountStorage: AccountStorageProvider { // TODO: complete
         do {
             switch result {
             case let .success(data):
-                try await userDocument(for: accountId)
-                    .setData(data, merge: true)
+                if !data.isEmpty {
+                    try await userDocument(for: accountId)
+                        .setData(data, merge: true)
+                }
             case let .failure(error):
                 throw error
             }
@@ -180,8 +182,10 @@ public actor FirestoreAccountStorage: AccountStorageProvider { // TODO: complete
                 result[key.identifier] = FieldValue.delete()
             }
 
-            try await userDocument(for: accountId)
-                .updateData(removedFields)
+            if !removedFields.isEmpty {
+                try await userDocument(for: accountId)
+                    .updateData(removedFields)
+            }
         } catch {
             throw FirestoreError(error)
         }