Bug 155454

Summary: Plasma doesn't remember desktop icons setting between restarts
Product: [Plasma] plasma4 Reporter: Alex Merry <alex.merry>
Component: generalAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:

Description Alex Merry 2008-01-11 12:32:28 UTC
Version:           SVN r759781 (using KDE Devel)
Installed from:    Compiled sources
OS:                Linux

When I deselect the option "Show Desktop Icons" in the "Configure Desktop" dialog, the icons disappear as expected.  After restarting plasma (either logout and login, or kquitapp plasma; plasma) they are back, and the option is selected again.

After changing the setting, then running kquitapp plasma, neither plasmarc nor plasma-appletrc have a "DesktopIcons" group or a "showIcons" key.

I put this:

kDebug() << "Writing showIcons=" << (m_showIcons->checkState() == Qt::Checked ? true : false)
             << "to config group" << iconConfig.name()
             << "in config file" << iconConfig.config()->name();

At the end of BackgroundDialog::saveConfig(), and I get this line:

plasma(11484) BackgroundDialog::saveConfig: Writing showIcons= false to config group "DesktopIcons" in config file "plasmarc"

Placing iconConfig().config()->sync() at the end of BackgroundDialog::saveConfig() fixes the problem (the setting is saved and plasmarc gets a [AppletGlobals][plasma_containment_desktop][DesktopIcons] group with the correct keys), but this shouldn't be necessary, surely?
Comment 1 Aaron J. Seigo 2008-01-11 19:22:41 UTC
SVN commit 760003 by aseigo:

make globalConfig() work properly even for containments
BUG:155454


 M  +20 -5     applet.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=760003