fix(proxy-state-tree): fix cleanup bug

christianalfoni · christianalfoni · commit e2a74d0ff314 · 2019-05-08T14:13:56.000+02:00
diff --git a/packages/node_modules/proxy-state-tree/src/index.ts b/packages/node_modules/proxy-state-tree/src/index.ts
@@ -83,10 +83,10 @@ export class ProxyStateTree<T extends object> implements IProxyStateTree<T> {
     )
   }
   onRemoveProxy(cb: IRemoveProxyCallback) {
-    const index = this.removeProxyCallbacks.push(cb) - 1
+    this.removeProxyCallbacks.push(cb)
 
     return () => {
-      this.removeProxyCallbacks.splice(index, 1)
+      this.removeProxyCallbacks.splice(this.removeProxyCallbacks.indexOf(cb), 1)
     }
   }
   removeProxy(path: string) {

Original file line number	Diff line number	Diff line change
`@@ -83,10 +83,10 @@ export class ProxyStateTree<T extends object> implements IProxyStateTree<T> {`
`83`	`83`	`)`
`84`	`84`	`}`
`85`	`85`	`onRemoveProxy(cb: IRemoveProxyCallback) {`
`86`		`- const index = this.removeProxyCallbacks.push(cb) - 1`
	`86`	`+ this.removeProxyCallbacks.push(cb)`
`87`	`87`
`88`	`88`	`return () => {`
`89`		`- this.removeProxyCallbacks.splice(index, 1)`
	`89`	`+ this.removeProxyCallbacks.splice(this.removeProxyCallbacks.indexOf(cb), 1)`
`90`	`90`	`}`
`91`	`91`	`}`
`92`	`92`	`removeProxy(path: string) {`