Bug 163546 - Changed plasma configuration is not stored on logout
Summary: Changed plasma configuration is not stored on logout
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-06-08 19:15 UTC by erik_hahn
Modified: 2008-06-14 03:33 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
My plasma applets configuration (4.62 KB, text/plain)
2008-06-13 15:48 UTC, Walter de Donato
Details

Note You need to log in before you can comment on or make changes to this bug.
Description erik_hahn 2008-06-08 19:15:34 UTC
Version:            (using Devel)
Installed from:    Compiled sources
Compiler:          gcc 4.2.3 
OS:                Linux

After changing some plasma settings I logged out normally. When logging on the next time, everything was like before my changes. 

kdebase revision 818297, kdelibs revision 818130. This is a regression, the last time I updated it worked.
Comment 1 Anne-Marie Mahfouf 2008-06-08 20:08:03 UTC
What did you change that was not stored? Did you look if it was written in the config file (plasma-appletsrc or plasmarc)
Please indicate what you changed and attach the 2 above files to this bug report atefr you logged out, thanks!
Comment 2 Walter de Donato 2008-06-13 15:48:32 UTC
Created attachment 25306 [details]
My plasma applets configuration

I have 3 panels at the bottom of the screen (the first on the left, the second
centered, and the third on the right). Every time I login the third panel
appears on the left (overlapped to the first). I can't figure out which
parameter in the plasma-appletsrc refers to panel alignment, so I can't check
if it is correctly stored this information.
Comment 3 Walter de Donato 2008-06-13 15:51:13 UTC
A GENERAL NOTE: changes to plasma settings should be written to plasma-appletsrc as soon as possible. If, for some reasons, the user can't logout normally (eg X/Plasma crash/freeze) every setting modified during the current session will be lost!
Comment 4 Aaron J. Seigo 2008-06-13 17:13:27 UTC
> I can't figure out which
> parameter in the plasma-appletsrc refers to panel alignment

it's not in plasma-appletsrc. that file is for the scene (or "data model"). the view related configuration is in plasmarc.

> changes to plasma settings should be written to plasma-appletsrc as soon as 
> possible.

we could write after every single key value change. i'm sure your harddisk and battery would have us for that, however.

there is a small timeout between a change happening and it being flushed to disk. this allows us to aggregate several changes (these changes tend to happen in bursts) before forcing the OS kernel to flush the changes out.

> If, for some reasons, the user can't logout normally (eg X/Plasma 
> crash/freeze) every setting modified during the current session will be lost! 

this is simply not true. right now the worst case scenario is that settings modified in the 2 minutes previous will be lost.

in any case, please stick to reporting the issues, answering questions the devs ask in return and save the coding suggestions. unless you're familiar with the code in question, chances are your insights will not be overly insightful. =)

in this case, the issue is that we aren't sync'ing plasmarc at the same time we do plasma-appletsrc. this is on my TOFIX list for 4.1.
Comment 5 Aaron J. Seigo 2008-06-14 03:33:15 UTC
SVN commit 820384 by aseigo:

* save the app config when the corona config is saved
* request a corona save when panelcontroller is All Done
BUG:163546


 M  +6 -5      panelcontroller.cpp  
 M  +0 -1      panelview.cpp  
 M  +11 -5     plasmaapp.cpp  
 M  +1 -0      plasmaapp.h  


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