- Added the option to merge objects when updating (default is false)

Author: kaevinio

Sources/FirestoreManager/CollectionRepository.swift

@@ -54,8 +54,8 @@ open class CollectionRepository<T: Codable>: ObservableObject {
         }
     }
 
-    open func update(_ object: T, id: String) {
-        FirestoreManager.updateDocument(object, id: id, reference: ref) { result in
+    open func update(_ object: T, id: String, merge: Bool = false) {
+        FirestoreManager.updateDocument(object, id: id, reference: ref, merge: merge) { result in
             switch result {
             case .success:
                 print("Object updated \(self.getPath(id: id))!")

Sources/FirestoreManager/FirestoreCRUD/Update.swift

@@ -20,10 +20,11 @@ extension FirestoreManager {
     public static func updateDocument<T>(_ object: T,
                                          id: String,
                                          reference: ReferenceProtocol,
+                                         merge: Bool = false,
                                          completion: @escaping(Result<String, FirestoreError>) -> Void) where T: Encodable {
 
         do {
-            try reference.reference().document(id).setData(from: object, merge: false)
+            try reference.reference().document(id).setData(from: object, merge: merge)
             completion(.success(id))
         } catch {
             completion(.failure(.fail(error: error, action: .update, reference: reference, id: id)))
@@ -38,7 +39,10 @@ extension FirestoreManager {
      - Parameter reference: The collection name
      - Parameter newValues: Array of keys and values for all values to be updated
      */
-    public static func updateData(id: String, reference: ReferenceProtocol, newValues: [String: Any], completion: @escaping(Result<String, FirestoreError>) -> Void) {
+    public static func updateData(id: String,
+                                  reference: ReferenceProtocol,
+                                  newValues: [String: Any],
+                                  completion: @escaping(Result<String, FirestoreError>) -> Void) {
 
         do {
             try reference.reference().document(id).setData(newValues, merge: true) { error in