[PATCH] KWallet and lastfm usage bug

Ignacio Serantes ig at aynoa.net
Sat Oct 17 20:56:00 UTC 2009


Hi!

I send you a patch to solves the problem with KWallet if you don't want to use 
it to store passwords and prefer the old plain text storage.

The problem is that "ignoreWallet" configuration property is ignored with the 
current logic so, I do some changes to 
LastFmServiceConfig::LastFmServiceConfig() to avoid the anoing KWallet dialog 
anytime Amarok is launched.

The changes are:

KConfigGroup config = KGlobal::config()->group( configSectionName() );

is now the first line of the method and

if( KWallet::Wallet::isEnabled() )

was changed for

if( ( config.readEntry( "ignoreWallet", QString() ) == "no" ) && 
KWallet::Wallet::isEnabled() )

So now kwallet is created only if "ignoreWallet" equals "no".

-- 

Un saúdo

Ignacio Serantes (kde at aynoa.net)
Urb. Mesón da Auga - Oleiros - A Coruña
España
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok/attachments/20091017/9723abf8/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix_kwallet_lastfm_bug.patch
Type: text/x-patch
Size: 2299 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/amarok/attachments/20091017/9723abf8/attachment.patch>


More information about the Amarok mailing list