Bug 160506 - "Restore volumes on login"-setting is ignored
Summary: "Restore volumes on login"-setting is ignored
Status: RESOLVED FIXED
Alias: None
Product: kmix
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Christian Esken
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-07 12:19 UTC by Tomas Åkesson
Modified: 2008-05-31 10:59 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tomas Åkesson 2008-04-07 12:19:51 UTC
Version:           3.0 (using 4.0.3 (KDE 4.0.3), Kubuntu packages)
Compiler:          gcc
OS:                Linux (i686) release 2.6.22-14-generic

The setting "Restore volumes on login" is ignored by KDE 4's kmix. Volume levels are restored to previous values when logging in even if this setting is unchecked.

I noticed that the file /usr/lib/kde4/share/autostart/restore_kmix_volumes.desktop has this entry:
  X-KDE-autostart-condition=kmixrc::startkdeRestore:true
which should prevent this command to run if the setting startkdeRestore is false (which it is in my case), right?

It looked wrong though to have an empty section (::) in that file so I tried to change it to:
  X-KDE-autostart-condition=kmixrc:Global:startkdeRestore:true
and now it seems to work.

I don't know if it is legal to have empty sections in X-KDE-autostart-condition, but if it is not, then I think this could be the cause of the bug.
Comment 1 Tomas Åkesson 2008-04-07 12:23:47 UTC
By the way, I noticed now that KDE 3's version of restore_kmix_volumes.desktop looks the same as the one I critizised. So maybe it is something that has changed in KDE 4's autostart-handling?
Comment 2 Christian Esken 2008-05-18 20:00:03 UTC
Thanks for reporting. Great work. :-)

I didn't notice this issue, as I moved the "startkdeRestore" entry from the empty group "" to "Global". I'll fix this immediately. Hold on a second ...
Comment 3 Christian Esken 2008-05-18 20:03:04 UTC
SVN commit 809311 by esken:

BUGS: 160506


 M  +1 -1      restore_kmix_volumes.desktop  


WebSVN link: http://websvn.kde.org/?view=rev&revision=809311
Comment 4 Christian Esken 2008-05-31 10:59:25 UTC
SVN commit 814737 by esken:

Backport of fix for bug 160506 ("Restore volumes on login"-setting is
ignored) to KDE4.0 branch.
CCBUGS: 160506


 M  +1 -1      restore_kmix_volumes.desktop  


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