Avoid Setting Data when no data is provided

StanfordSpezi · Jul 23, 2024 · 488fba3 · 488fba3
1 parent 67981d0
commit 488fba3
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/Sources/SpeziFirebaseAccountStorage/FirestoreAccountStorage.swift b/Sources/SpeziFirebaseAccountStorage/FirestoreAccountStorage.swift
@@ -90,6 +90,10 @@ public actor FirestoreAccountStorage: Module, AccountStorageConstraint {
         do {
             switch result {
             case let .success(data):
+                guard !data.isEmpty else {
+                    return
+                }
+
                 try await userDocument(for: identifier.accountId)
                     .setData(data, merge: true)
             case let .failure(error):
@@ -137,7 +141,7 @@ public actor FirestoreAccountStorage: Module, AccountStorageConstraint {
             switch result {
             case let .success(data):
                 try await userDocument(for: identifier.accountId)
-                    .setData(data, merge: true)
+                    .updateData(data)
             case let .failure(error):
                 throw error
             }