misc cleanup in base

Author: Salakar

lib/modules/base.js

@@ -11,7 +11,13 @@ const FirestackModuleEvt = new NativeEventEmitter(FirestackModule);
 
 const logs = {};
 
+// single event emitter for all classes extending base
+// TODO
+// const EE = new EventEmitter();
+
 type FirestackOptions = {};
+
+// TODO cleanup
 export class Base extends EventEmitter {
   constructor(firestack: Object, options: FirestackOptions = {}) {
     super();
@@ -22,12 +28,10 @@ export class Base extends EventEmitter {
     this.options = Object.assign({}, firestack.options, options);
   }
 
+
   // Logger
   get log(): Log {
-    if (!logs[this.namespace]) {
-      const debug = this.firestack._debug;
-      logs[this.namespace] = new Log(this.namespace, debug);
-    }
+    if (!logs[this.namespace]) logs[this.namespace] = new Log(this.namespace, this.firestack._debug);
     return logs[this.namespace];
   }
 
@@ -40,9 +44,9 @@ export class Base extends EventEmitter {
 
   // TODO unused - do we need this anymore?
   _addToFirestackInstance(...methods: Array<string>) {
-    methods.forEach(name => {
+    methods.forEach((name) => {
       this.firestack[name] = this[name].bind(this);
-    })
+    });
   }
 
   /**
@@ -75,7 +79,7 @@ export class Base extends EventEmitter {
       const sub = nativeModule.addListener(name, cb);
       this.eventHandlers[name] = sub;
       resolve(sub);
-    })
+    });
   }
 
   _off(name) {
@@ -84,7 +88,7 @@ export class Base extends EventEmitter {
         const subscription = this.eventHandlers[name];
         subscription.remove(); // Remove subscription
         delete this.eventHandlers[name];
-        resolve(subscription)
+        resolve(subscription);
       }
     });
   }
@@ -94,7 +98,7 @@ export class ReferenceBase extends Base {
   constructor(firestack: Object, path: Array<string> | string) {
     super(firestack);
 
-    this.path = Array.isArray(path) ?  path : (typeof path == 'string' ?  [path] : []);
+    this.path = Array.isArray(path) ? path : (typeof path === 'string' ? [path] : []);
 
     // sanitize path, just in case
     this.path = this.path.filter(str => str !== '');
@@ -106,10 +110,10 @@ export class ReferenceBase extends Base {
   }
 
   pathToString(): string {
-    let path = this.path;
+    const path = this.path;
     let pathStr = (path.length > 0 ? path.join('/') : '/');
     if (pathStr[0] != '/') {
-      pathStr = `/${pathStr}`
+      pathStr = `/${pathStr}`;
     }
     return pathStr;
   }