Don't report credentials error after a connection being closed

If the connection already breaks while waiting for
the credentials to be fetched, it should not report
error or proceed to login.

Change-Id: I68a6f95232bdde03cc6cd52a2e0bf3cdfc989772
Reviewed-by: Pekka Vuorela <[email protected]>

1 files changed, 11 insertions, 11 deletions

diff --git a/src/plugins/messageservices/imap/imapclient.cpp b/src/plugins/messageservices/imap/imapclient.cpp
index d25ff467..638a05a2 100644
--- a/src/plugins/messageservices/imap/imapclient.cpp
+++ b/src/plugins/messageservices/imap/imapclient.cpp
@@ -1793,19 +1793,19 @@ void ImapClient::onCredentialsStatusChanged()
 
     disconnect(_credentials, &QMailCredentialsInterface::statusChanged,
                this, &ImapClient::onCredentialsStatusChanged);
-    switch (_credentials->status()) {
-    case (QMailCredentialsInterface::Ready): {
-        QMailAccountConfiguration config(_accountId);
-        _protocol.sendLogin(config, _credentials);
-        break;
-    }
-    case (QMailCredentialsInterface::Failed):
-        if (_protocol.inUse()) {
+    if (_protocol.inUse()) {
+        switch (_credentials->status()) {
+        case (QMailCredentialsInterface::Ready): {
+            QMailAccountConfiguration config(_accountId);
+            _protocol.sendLogin(config, _credentials);
+            break;
+        }
+        case (QMailCredentialsInterface::Failed):
             operationFailed(QMailServiceAction::Status::ErrLoginFailed, _credentials->lastError());
+            break;
+        default:
+            break;
         }
-        break;
-    default:
-        break;
     }
 }