fix .currentUser logic - the initial event doesn't have authenticated flag

Author: Salakar

android/src/main/java/io/fullstack/firestack/FirestackAuth.java

@@ -627,8 +627,7 @@ private WritableMap getUserMap() {
       userMap.putString("uid", uid);
       userMap.putString("providerId", provider);
       userMap.putBoolean("emailVerified", verified);
-
-
+      
       if (name != null) {
         userMap.putString("name", name);
       }

lib/modules/auth.js

@@ -14,12 +14,30 @@ export default class Auth extends Base {
     this.authenticated = false;
     this._user = null;
 
-    this.getCurrentUser().then(this._onAuthStateChanged.bind(this)).catch(() => {
+    // start listening straight away
+    // generally though the initial event fired will get ignored
+    // but this is ok as we fake it with the getCurrentUser below
+    FirestackAuth.listenForAuth();
+
+    this.getCurrentUser().then((u) => {
+      const authResult = { authenticated: !!u };
+      if (u) authResult.user = u;
+      this._onAuthStateChanged(authResult);
+      this._startListening();
+    }).catch(() => {
+      // todo check if error contains user disabled message maybe and add a disabled flag?
+      this._onAuthStateChanged({ authenticated: false });
+      this._startListening();
     });
+  }
 
-    // always track auth changes internally so we can access them synchronously
+  /**
+   * Internal function begin listening for auth changes
+   * only called after getting current user.
+   * @private
+   */
+  _startListening() {
     FirestackAuthEvt.addListener('listenForAuth', this._onAuthStateChanged.bind(this));
-    FirestackAuth.listenForAuth();
   }
 
   /**