Do not accept listeners on read-only transactions (#1706)

project-chip · Feb 13, 2025 · 1187f81 · 1187f81
1 parent fd72520
commit 1187f81
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/packages/general/src/transaction/Tx.ts b/packages/general/src/transaction/Tx.ts
@@ -189,6 +189,9 @@ class Tx implements Transaction {
     }
 
     onShared(listener: () => void, once?: boolean) {
+        if (this.status === Status.ReadOnly) {
+            return;
+        }
         if (this.#shared === undefined) {
             this.#shared = Observable();
         }
@@ -197,6 +200,9 @@ class Tx implements Transaction {
     }
 
     onClose(listener: () => void) {
+        if (this.status === Status.ReadOnly) {
+            return;
+        }
         if (this.status === Status.Destroyed) {
             listener();
         }