Version: 1.0.0 (using 4.4.1 (KDE 4.4.1), 4.4.1-1.fc12 Fedora) Compiler: gcc OS: Linux (x86_64) release 2.6.32.9-67.fc12.x86_64 Steps to reproduce: 1. Set up a WLM account and store the password in kwallet. 2. Close Kopete and re-open it. 3. Wait for the kwallet password prompt to appear. 4. Ignore it. 5. After some time the kwallet password prompt will time out and Kopete's password prompt for the WLM account will appear. 6. Click cancel on Kopete's password prompt. 7. Close Kopete. 8. Re-open Kopete. 9. Wait for the kwallet password prompt to appear. 10. Enter your kwallet password. 11. Notice that the WLM account fails to connect with an invalid password error. This only happens for WLM accounts. Other types of account remember their password if you cancel the password prompt.
I've not heard anything re: this bug, but I now know the cause. WlmAccount::connectWithPassword does this: if (pass.isEmpty ()) { password ().setWrong (true); return; } but pass may be empty if the password prompt dialog was cancelled. In that case KopetePasswordGetRequestPrompt::slotCancelPressed emits finished with QString::null as the password. So calling setWrong is, err, wrong. Other protocols either call disconnect (e.g. JabberAccount::connectWithPassword ) or set the status to offline (YahooAccount::connectWithPassword).
Created attachment 43263 [details] Patch to not call setWrong if the password prompt was cancelled by the user.
Related to bug 226270 but I didn't dup this one as it has a protocol specific patch.
SVN commit 1223106 by lvsouza: Do not call setWrong if the password prompt has been cancelled by the user. Thanks Christopher January for the patch. BUG: 230181 M +1 -1 wlmaccount.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1223106
SVN commit 1223107 by lvsouza: Backport r1223106 by lvsouza from trunk to the 4.6 branch: Do not call setWrong if the password prompt has been cancelled by the user. Thanks Christopher January for the patch. CCBUG: 230181 M +1 -1 wlmaccount.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1223107