Don't go through sendPing: when unfreezing an account

If we go through the MLXMPPManager's connectIfNecessary, we don't want
to prevent a race condition with disconnect:, like sendPing: does: we
already know that we were cleanly disconnected and always want to
connect again. But for good measure we still try to simply ping instead
of doing a real (re)connect, if we detect we are somehow still connected
or at least seem to be (the ping will bring clarity).

Author: tmolitor-stud-tu

Monal/Classes/MLXMPPManager.m

@@ -526,8 +526,16 @@ -(void) connectAccountWithDictionary:(NSDictionary*) account
         }
         DDLogInfo(@"existing account, calling unfreeze");
         [existing unfreeze];
-        DDLogInfo(@"existing account, just pinging.");
-        [existing sendPing:SHORT_PING];     //short ping timeout to quickly check if connectivity is still okay
+        if(existing.accountState<kStateReconnecting)
+        {
+            DDLogInfo(@"disconnected existing account, reconnecting.");
+            [existing connect];
+        }
+        else
+        {
+            DDLogInfo(@"connected(?) existing account, just pinging.");
+            [existing sendPing:SHORT_PING];     //short ping timeout to quickly check if connectivity is still okay
+        }
         return;
     }
     DDLogVerbose(@"connecting account %@@%@",[account objectForKey:kUsername], [account objectForKey:kDomain]);