Merge branch 'master' of github.com:fullstackreact/react-native-firestack

auser · auser · commit 6c2230baa14f · 2016-10-26T14:55:35.000+02:00
* 'master' of github.com:fullstackreact/react-native-firestack:
  fixing NPE after Database#off
diff --git a/lib/modules/database.js b/lib/modules/database.js
@@ -215,7 +215,7 @@ class DatabaseRef extends ReferenceBase {
     const path = this.dbPath();
     return this.db.off(path, evt, origCB)
       .then(({callback, subscriptions}) => {
-        if (dbSubscriptions[path][evt].length > 0) {
+        if (dbSubscriptions[path] && dbSubscriptions[path][evt].length > 0) {
           return subscriptions;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -215,7 +215,7 @@ class DatabaseRef extends ReferenceBase {`
`215`	`215`	`const path = this.dbPath();`
`216`	`216`	`return this.db.off(path, evt, origCB)`
`217`	`217`	`.then(({callback, subscriptions}) => {`
`218`		`- if (dbSubscriptions[path][evt].length > 0) {`
	`218`	`+ if (dbSubscriptions[path] && dbSubscriptions[path][evt].length > 0) {`
`219`	`219`	`return subscriptions;`
`220`	`220`	`}`
`221`	`221`