Bug 160705

Summary:	Plasma config does not get saved.
Product:	[Unmaintained] plasma4	Reporter:	Thomas Zander <zander>
Component:	general	Assignee:	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:
Sentry Crash Report:
Attachments:	Auto-saved config before I restarted plasma.

Description Thomas Zander 2008-04-11 14:33:37 UTC

Version:            (using Devel)
Installed from:    Compiled sources

After configuring my plasma desktop (from a clean config) for 15 minutes I hit a plasmoid that crashed plasma.
Restarting it gave me an annoyingly empty plasma, though.  The config should get saved *while* I'm configuring plasma.

I get the impression that continuesly working on adding and removing stuff delays the saving of the config indefinitely.

Comment 1 FiNeX 2008-04-12 01:04:24 UTC

Confirmed: the plasma settings are not saved if plasma crash.

Comment 2 Aaron J. Seigo 2008-04-12 11:19:36 UTC

i assume that by now you no longer have the plasma-appletsrc file that you had directly after restart?

do you remember what you configured?

and i assume this is from trunk? we do save within 2 minutes when applets are added, removed or when the applet says "i need my config saved" .. so i'm not sure what would be going on here, though the report here doesn't give a lot of detail to perform forensics on.

FiNex: how did you confirm this report?

Comment 3 Robert Knight 2008-04-12 12:38:39 UTC

> and i assume this is from trunk? we do save within 2 minutes
> when applets are added, removed or when the applet says "i need my
> config saved" 

In my case that 2 minute delay was too long because I set up a panel in about a minute and then added a buggy plasmoid which crashed Plasma.

Comment 4 Thomas Zander 2008-04-12 20:18:59 UTC

Further observations;
I added a qdebug in Corona::syncConfig() to ensure its getting called.  It is, 2 minutes after adding a plasmoid, its called.

It doesn't save the changes to the panel location or which applets I added, though.
It did correctly save that I removed applets, it also saved that I don't want icons on my desktop and which background pic I configured.

Comment 5 Thomas Zander 2008-04-12 20:21:26 UTC

Created attachment 24308 [details]
Auto-saved config before I restarted plasma.

I started with an empty config, moved the panel and resized it. Removed various
applets and added various others.  Waited for more than two minutes and did a
killall plasma.

Comment 6 Chris Fordham 2008-05-13 13:20:16 UTC

I confirm this.

I think its important to save the plasma configuration on a config change of any sort.

Comment 7 giuseppe 2008-06-14 12:21:30 UTC

I confirm,on kde 4.1(4.0.82)
config is not saved if plasma
crash

Comment 8 Aaron J. Seigo 2008-06-14 13:00:23 UTC

*** Bug has been marked as fixed ***.

Comment 9 Chris Fordham 2008-06-14 13:16:01 UTC

How was it fixed exactly?

Comment 10 Aaron J. Seigo 2008-06-14 13:32:15 UTC

a series of commits that now ensures plasmarc also gets saved along with plasma-appletsrc.

Comment 11 Chris Fordham 2008-06-14 13:33:39 UTC

Thanks Aaron, when and how frequently etc. please?