Follow-up to 426a72f: make sure loggingIn goes away in the case of a …

…half-complete login method.
picaross · Nov 20, 2012 · ba76b0f · ba76b0f
1 parent 426a72f
commit ba76b0f
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/packages/accounts-base/accounts_client.js b/packages/accounts-base/accounts_client.js
@@ -119,8 +119,10 @@
       if (reconnected)
         return;
 
-      if (!options._suppressLoggingIn)
-        Accounts._setLoggingIn(false);
+      // Note that we need to call this even if _suppressLoggingIn is true,
+      // because it could be matching a _setLoggingIn(true) from a
+      // half-completed pre-reconnect login method.
+      Accounts._setLoggingIn(false);
       if (error || !result) {
         error = error || new Error(
           "No result from call to " + options.methodName);