Add explanatory comment and remove TODO message

dexie · Oct 14, 2020 · 1f2cad1 · 1f2cad1
1 parent a9f030a
commit 1f2cad1
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/addons/Dexie.Observable/src/Dexie.Observable.js b/addons/Dexie.Observable/src/Dexie.Observable.js
@@ -256,9 +256,7 @@ function Observable(db) {
                         }
                     }
 
-                    // TODO: Why is this safety check required?
-                    // Without it, this test fails: https://github.com/dfahlander/Dexie.js/blob/77f0b08c58784bfaaf9e2f5e26dd4e1d7b7d3094/test/tests-exception-handling.js#L411-L494
-                    // ... due to an unhandled global TypeError: Cannot read property 'id' of null
+                    // The local node reference may be unassigned at any point by a database close() operation
                     if (!mySyncNode.node) return;
 
                     // Assign the local node state