Merge pull request fullstackreact#99 from chaitanya0bhagvan/master

auser · web-flow · commit 583272c55a83 · 2016-10-30T19:42:00.000-07:00
Fixed setPersistence always enabled
diff --git a/lib/modules/database.js b/lib/modules/database.js
@@ -367,11 +367,11 @@ export class Database extends Base {
   }
 
   setPersistence(enable=true) {
-    this.persistenceEnabled = enable;
     let promise;
     if (this.persistenceEnabled !== enable) {
       this.log.debug(`${enable ? 'Enabling' : 'Disabling'} persistence`);
       promise = this.whenReady(promisify('enablePersistence', FirestackDatabase)(enable));
+      this.persistenceEnabled = enable;
     } else {
       promise = this.whenReady(Promise.resolve({status: "Already enabled"}))
     }

Original file line number	Diff line number	Diff line change
`@@ -367,11 +367,11 @@ export class Database extends Base {`
`367`	`367`	`}`
`368`	`368`
`369`	`369`	`setPersistence(enable=true) {`
`370`		`- this.persistenceEnabled = enable;`
`371`	`370`	`let promise;`
`372`	`371`	`if (this.persistenceEnabled !== enable) {`
`373`	`372`	this.log.debug(`${enable ? 'Enabling' : 'Disabling'} persistence`);
`374`	`373`	`promise = this.whenReady(promisify('enablePersistence', FirestackDatabase)(enable));`
	`374`	`+ this.persistenceEnabled = enable;`
`375`	`375`	`} else {`
`376`	`376`	`promise = this.whenReady(Promise.resolve({status: "Already enabled"}))`
`377`	`377`	`}`