Bug 286768

Summary: Problem resetting default wallet in KDEWalletmanager if I have more wallets
Product: [Applications] kwalletmanager Reporter: Pal Körössy <korossy>
Component: generalAssignee: Valentin Rusu <valir>
Status: RESOLVED FIXED    
Severity: normal CC: adaptee, andy-debian, cfeck, macieksitarz, valir
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In: 4.11
Sentry Crash Report:
Attachments: dropdown box in Kwalletmanager settings with the wrong content
the three wallets

Description Pal Körössy 2011-11-16 10:04:25 UTC
Created attachment 65742 [details]
dropdown box in Kwalletmanager settings with the wrong content

Version:           unspecified (using KDE 4.7.3) 
OS:                Linux

I have three different wallets ('kdewallet', 'Wallet2', 'Wallet3') and I can set the default wallet for KDE apps in Settings/Configure_wallet/Default_wallet section of KWalletmanager. If I change the default wallet from 'kdewallet' to 'Wallet3', next time I cannot scwitch back to 'kdewallet', I can see 'Wallet3' twice in the dropdown box and no 'kdewallet' any more (see the attached two screenshots). I can switch back to kdewallet as default wallet by manually editing the kwalletrc file.

Reproducible: Always

Steps to Reproduce:
Create three different wallets. Set the default wallet for KDE apps in Settings/Configure_wallet/Default wallet of KWalletmanager from 'kdewallet'  to 'Wallet3'. 

Actual Results:  
In the dropdown box in Settings/Configure_wallet/Default_wallet of KWalletmanager can bee seen the foolowing three wallets 'Wallet3'  'Wallet2' 'Wallet3' ('kdewallet' cannot be chosen any more) in the dropdown box. See the attached two screenshots. 

Expected Results:  
'kdewallet' should be displayed in the dropdown box.
Comment 1 Pal Körössy 2011-11-16 10:05:16 UTC
Created attachment 65743 [details]
the three wallets
Comment 2 Maciej Sitarz 2013-01-07 10:42:30 UTC
I have the same symptoms and I can confirm this bug.

I prepared a fix:
Review Request #108237 (https://git.reviewboard.kde.org/r/108237/)
Comment 3 Andy Simpkins 2013-01-30 19:45:30 UTC
Bug also found in Debian Wheezy
699092: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=699092
Comment 4 Valentin Rusu 2013-02-08 22:42:46 UTC
Thanks for the patch you submitted with review 108237.
Do you have a developer account or should I submit it for you?
Comment 5 Maciej Sitarz 2013-02-09 08:31:14 UTC
Valentin please submit it for me, I think I don't have git write access.
Comment 6 Valentin Rusu 2013-02-09 14:28:58 UTC
Git commit b6fdd9227d11b81d00076101672b74120b479ee5 by Valentin Rusu.
Committed on 09/02/2013 at 15:26.
Pushed by vrusu into branch 'master'.

Applying patch submitted by Maciej Sitarz

REVIEW: 108237

M  +12   -2    konfigurator/konfigurator.cpp

http://commits.kde.org/kwallet/b6fdd9227d11b81d00076101672b74120b479ee5