feat(reference): Add hasUnprocessedChanges_ flag to CollectionReferen…

…ce and DocumentReference
Alwatr · Dec 29, 2023 · dff3553 · dff3553
1 parent 68695a9
commit dff3553
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/packages/reference/src/collection-reference.ts b/packages/reference/src/collection-reference.ts
@@ -488,6 +488,7 @@ export class CollectionReference<TItem extends Dictionary = Dictionary> {
    */
   private async updated__(id?: string | number): Promise<void> {
     this.logger__.logMethodArgs?.('updated__', {delayed: this.updateDelayed__});
+
     this.hasUnprocessedChanges_ = true;
     this.updateMeta__(id);
 

diff --git a/packages/reference/src/document-reference.ts b/packages/reference/src/document-reference.ts
@@ -140,6 +140,11 @@ export class DocumentReference<TDoc extends Dictionary = Dictionary> {
    */
   readonly path: string;
 
+  /**
+   * Indicates whether the collection has unsaved changes.
+   */
+  hasUnprocessedChanges_ = false;
+
   /**
    * Logger instance for this document.
    */
@@ -259,6 +264,9 @@ export class DocumentReference<TDoc extends Dictionary = Dictionary> {
    */
   private async updated__(): Promise<void> {
     this.logger__.logMethodArgs?.('updated__', {delayed: this.updateDelayed__});
+
+    this.hasUnprocessedChanges_ = true;
+
     if (this.updateDelayed__ === true) return;
     // else
 
@@ -269,7 +277,10 @@ export class DocumentReference<TDoc extends Dictionary = Dictionary> {
     }
 
     this.updateMeta__();
-    this.updatedCallback__.call(null, this);
+    if (this.hasUnprocessedChanges_ === true) {
+      this.hasUnprocessedChanges_ = false;
+      this.updatedCallback__.call(null, this);
+    }
   }
 
   /**