@live-change/dao 0.9.28 → 0.9.30

package/lib/Observable.js

@@ -27,7 +27,7 @@ class Observable {
     for(const observer of observersToFire) {
       handled = this.fireObserver(observer, signal, ...args) || handled
     }
-    if(signal == 'error') {
+    if(signal === 'error') {
       let error = args[0]
       handled = this.handleError(error) || handled
       if(!handled) {
@@ -57,7 +57,7 @@ class Observable {
 
   uncatch(errorObserver) {
     let id = this.errorObservers.indexOf(errorObserver)
-    if(id == -1) throw new Error("error observer not found")
+    if(id === -1) throw new Error("error observer not found")
     this.errorObservers.splice(id, 1)
   }
 
@@ -67,13 +67,13 @@ class Observable {
   }
   unobserve(observer) {
     let id = this.observers.indexOf(observer)
-    if(id == -1) throw new Error("observer not found")
+    if(id === -1) throw new Error("observer not found")
     this.observers.splice(id, 1)
     if(this.isUseless()) this.dispose()
   }
 
   isUseless() {
-    return this.observers.length == 0
+    return this.observers.length === 0
   }
 
   isDisposed() {
@@ -102,16 +102,16 @@ class Observable {
     let errorObserver
 
     const waitPromise = new Promise((resolve, reject) => {
-      errorObserver = (error) => {
-        console.log("ERROR SIGNAL", error)
-        console.trace("WEER")
+      errorObserver = (signal, error) => {
+        console.log("ERROR SIGNAL", error ?? signal)
+        console.trace("ERROR SIGNAL")
         if(resultObserver) this.unobserve(resultObserver)
         resultObserver = undefined
         if(errorObserver) this.uncatch(errorObserver)
         errorObserver = undefined
         if(finished) return
         finished = true
-        reject(error)
+        reject(error ?? signal)
       }
       if(!finished) this.catch(errorObserver)
       resultObserver = (signal) => {

package/lib/ObservableList.js

@@ -21,7 +21,7 @@ class ObservableList extends Observable {
     this.handleError(null)
     if(list === this.list) return
     try {
-      if (JSON.stringify(list) == JSON.stringify(this.list)) return
+      if (JSON.stringify(list) === JSON.stringify(this.list)) return
     } catch(e) {}
     this.list = this.valueActivator ? this.valueActivator(list) : list
     this.fireObservers('set', list)
@@ -167,7 +167,7 @@ class ObservableList extends Observable {
   unbindProperty(object, property) {
     for(let i = 0; i < this.properties.length; i++) {
       const prop = this.properties[i]
-      if(prop[0] == object && prop[1] == property) {
+      if(prop[0] === object && prop[1] === property) {
         this.properties.splice(i, 1)
         i--
         if(this.isUseless()) this.dispose()
@@ -185,7 +185,7 @@ class ObservableList extends Observable {
   unbindErrorProperty(object, property) {
     for(let i = 0; i < this.errorProperties.length; i++) {
       const prop = this.errorProperties[i]
-      if(prop[0] == object && prop[1] == property) {
+      if(prop[0] === object && prop[1] === property) {
         this.errorProperties.splice(i, 1)
         i--
         if(this.isUseless()) this.dispose()
@@ -206,8 +206,8 @@ class ObservableList extends Observable {
   }
 
   isUseless() {
-    return (this.observers.length == 0) && (this.properties.length == 0)
-        && (this.errorProperties.length == 0)
+    return (this.observers.length === 0) && (this.properties.length === 0)
+        && (this.errorProperties.length === 0)
   }
 
   save() {
@@ -298,6 +298,13 @@ class ObservableList extends Observable {
   getError() {
     return this.savedError
   }
+
+  dispose() {
+    this.savedError = null
+    this.list = undefined
+    super.dispose()
+  }
+
 }
 
 export default ObservableList

package/lib/ObservableProxy.js

@@ -61,8 +61,8 @@ class ObservableProxy extends Observable {
   }
 
   isUseless() {
-    return (this.observers.length == 0) && (this.properties.length == 0)
-        && (this.errorProperties.length == 0)
+    return (this.observers.length === 0) && (this.properties.length === 0)
+        && (this.errorProperties.length === 0)
   }
 
   catch(...args) {
@@ -121,7 +121,7 @@ class ObservableProxy extends Observable {
   unbindErrorProperty(object, property) {
     for(var i = 0; i < this.errorProperties.length; i++) {
       var prop = this.errorProperties[i]
-      if(prop[0] == object && prop[1] == property) {
+      if(prop[0] === object && prop[1] === property) {
         this.errorProperties.splice(i,1)
         if(this.observable) this.observable.unbindErrorProperty(object, property)
         if(this.isUseless()) this.dispose()

package/lib/ObservableValue.js

@@ -20,7 +20,7 @@ class ObservableValue extends Observable {
     this.handleError(null)
     if(value === this.value) return;
     try {
-      if (JSON.stringify(value) == JSON.stringify(this.value)) return;
+      if (JSON.stringify(value) === JSON.stringify(this.value)) return;
     } catch(e) {}
     this.value = value
     this.fireObservers('set', value)
@@ -37,7 +37,7 @@ class ObservableValue extends Observable {
   unbindProperty(object, property) {
     for(let i = 0; i < this.properties.length; i++) {
       let prop = this.properties[i]
-      if(prop[0] == object && prop[1] == property) {
+      if(prop[0] === object && prop[1] === property) {
         this.properties.splice(i,1)
         if(this.isUseless()) this.dispose()
         return;
@@ -54,7 +54,7 @@ class ObservableValue extends Observable {
   unbindErrorProperty(object, property) {
     for(let i = 0; i < this.errorProperties.length; i++) {
       let prop = this.errorProperties[i]
-      if(prop[0] == object && prop[1] == property) {
+      if(prop[0] === object && prop[1] === property) {
         this.errorProperties.splice(i,1)
         if(this.isUseless()) this.dispose()
         return;
@@ -74,8 +74,8 @@ class ObservableValue extends Observable {
   }
 
   isUseless() {
-    return (this.observers.length == 0) && (this.properties.length == 0)
-        && (this.errorProperties.length == 0)
+    return (this.observers.length === 0) && (this.properties.length === 0)
+        && (this.errorProperties.length === 0)
   }
 
   save() {
@@ -165,6 +165,12 @@ class ObservableValue extends Observable {
   getError() {
     return this.savedError
   }
+
+  dispose() {
+    this.savedError = null
+    this.value = undefined
+    super.dispose()
+  }
 }
 
 export default ObservableValue

package/package.json

@@ -35,6 +35,6 @@
   "scripts": {
     "test": "NODE_ENV=test tape tests/*"
   },
-  "version": "0.9.28",
-  "gitHead": "f308e368e678fa38ddef6a6d4999ad730b18e8ce"
+  "version": "0.9.30",
+  "gitHead": "dc063de0998d29acb7c7d5e2032ed63be8895a8e"
 }