From f4165a9eea32ca265cc1e3e88cdae599bc813cdb Mon Sep 17 00:00:00 2001 From: Jan Mulder Date: Sun, 8 Oct 2017 10:21:22 +0200 Subject: mobile: only set old credential status in one place The old credential status should only be set when changing the actual credential status using the setCredentialStatus function. Setting it here is just wrong. It sets the old status to the current, and than adds the current to the prefences, obviously, resulting in old = current, which can not be right and results in a wrong flow of control in the credential state processing. Signed-off-by: Jan Mulder --- mobile-widgets/qmlmanager.cpp | 1 - 1 file changed, 1 deletion(-) (limited to 'mobile-widgets/qmlmanager.cpp') diff --git a/mobile-widgets/qmlmanager.cpp b/mobile-widgets/qmlmanager.cpp index 982861ba0..b604e7e5a 100644 --- a/mobile-widgets/qmlmanager.cpp +++ b/mobile-widgets/qmlmanager.cpp @@ -334,7 +334,6 @@ void QMLManager::saveCloudCredentials() return; } } - setOldStatus(credentialStatus()); s.beginGroup("CloudStorage"); s.setValue("email", cloudUser); s.setValue("password", cloudPwd); -- cgit v1.2.3-70-g09d2