Bug 165496 - Plasma should save its settings more
Summary: Plasma should save its settings more
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-07-01 16:53 UTC by Bráulio Barros de Oliveira
Modified: 2009-02-12 11:41 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Bráulio Barros de Oliveira 2008-07-01 16:53:10 UTC
Version:           20080630 (using Devel)
Installed from:    Compiled sources
OS:                Linux

After using plasma for some time I realized it only save settings on session close (logout, shutdtown, restart). This can lead to several problems and incovinients, for example:
 
- clashes on plasma/X/system leads to configuration loss
- user must save it session to save plasma settings, keeping recorded all the apps it is running, which may be underiseble. In my case, as I am lazzy to close the apps and keep the session the way I want it to be the next time I login (few apps loaded), I just restart X or run halt/restart directly (most of times). 

IMHO, at every operation on desktop, like add, remove, move, rotate, etc, the plasma settings should be saved, or at least on a timeout of a timer.
Comment 1 Sebastian Sauer 2008-07-01 17:31:54 UTC
dup of bug #165495
Comment 2 Bráulio Barros de Oliveira 2008-07-01 17:33:00 UTC
actually the bug 165495 is the invalid one
Comment 3 Christophe Marin 2008-07-01 17:49:21 UTC
reopen this one
Comment 4 Michael Rudolph 2008-07-01 19:39:19 UTC
Hi Bráulio,

the behaviour you described as a bug is actually intentional. Instantaneous saving  of every operation would cause lots of disc access and would also be bad for battery life on mobile devices.

I'm not a developer, so you might search the mailinglist archives (https://mail.kde.org/mailman/listinfo/panel-devel) for a better explanation of why the current behaviour is desirable. This issue has been discussed before.

michael
Comment 5 Bráulio Barros de Oliveira 2008-07-01 19:52:28 UTC
Hello Michael,

I agree that it shouldn't be saved every you change something for power effiency reasons as you said, but I think it should be saved only on session exit.
I don't know, maybe a delayed save would do it.

Couldn't find anything about it on panel-devel archives...

bráulio
Comment 6 Aaron J. Seigo 2008-07-01 20:48:58 UTC
> I don't know, maybe a delayed save would do it

that's what plasma does.

> like add, remove, move, rotate, etc,

we request a sync add and remove, i don't think we do so for move and rotate, however.

if there are any problems with this, it would be specific settings either not requesting a sync or specific settings not getting written out to the config prior to sync.

the general issue is addressed, however, and keeping a general report open to hunt specific issues makes it impossible to close this one ever (because other specific issues may arise).
Comment 7 Iuri Fiedoruk 2008-12-10 00:23:47 UTC
I do not see plasma saving layout/config in KDE 4.2 beta1. I have a crash several hours after adjusting/configuring Plasma and loose everything :-P
Also, loose the settings in logout. Using ubuntu packages.

Is this an expected behavior for beta?
Comment 8 M-A Loyer 2009-02-11 23:00:19 UTC
I have the same bug in KDE 4.2, Plasma ONLY save on logout.
If plasma crash before I logout, even if the modification have been made a few days ago, the move/resize/added/removed plasmoids are lost.

Having a save at each 2 min or just after a delay of 60 second after the changes to make sure multiple moves or resizes are saved in one shot to save disk IO.

A temporary workaround would be to have a command to force plasma to save it's configs. Is there such a command ?
Comment 9 Iuri Fiedoruk 2009-02-12 11:41:34 UTC
dbus-send --dest=org.kde.ksmserver /KSMServer org.kde.KSMServerInterface.saveCurrentSession

Source: http://hack.mypclinuxos.com/index.php?action=profile;u=14;sa=showPosts