Bug 347429

Summary:	Items in Startup Services revert back to checked after unchecked.
Product:	[Applications] systemsettings	Reporter:	kalapacengkir
Component:	kcm_kded	Assignee:	Daniel Molkentin <molkentin>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bhush94, jeremy9856, plasma-bugs, rthomsen6
Priority:	NOR
Version:	5.3.0
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:	http://commits.kde.org/plasma-desktop/15253a7463f472e4ce977d95d91881f21305e471	Version Fixed In:	5.4.0
Sentry Crash Report:

Description kalapacengkir 2015-05-08 13:24:13 UTC

I did unchecked some services (Bluetooth, Free Space Notifier, Print Manager and applied, but when I open System Settings again, they're reverted back to checked value.

Reproducible: Always

Steps to Reproduce:
1. Open System Settings
2. Open Startup and Shutdown
3. Open Background Services
4. Uncheck some items
5. Apply
6. Back to step 2


Actual Results:  
Items reverted to checked value.

Expected Results:  
Items unchecked.

Comment 1 jeremy9856 2015-07-28 09:53:08 UTC

I confirm this on manjaro (plasma 5.3.2 - Qt 5.5)

This bug (https://bugs.kde.org/show_bug.cgi?id=346848) is a duplicate of this one

Comment 2 jeremy9856 2015-07-31 14:11:40 UTC

I confirm this on Kubuntu 15.10 Alpha 2.

Comment 3 jeremy9856 2015-08-10 07:09:15 UTC

Any news on this ?

Comment 4 Ragnar Thomsen 2015-08-20 20:19:35 UTC

Git commit 15253a7463f472e4ce977d95d91881f21305e471 by Ragnar Thomsen.
Committed on 20/08/2015 at 20:18.
Pushed by rthomsen into branch 'Plasma/5.4'.

Fix loading/saving in kded kcm

Fix loading/saving of the autoload status of kded startup services in
the kcm.

Loading was broken due to a KPluginMetaData::pluginId() being passed to
autoloadEnabled(), instead of a KPluginMetaData.

Saving was broken due to comparing KPluginMetaData::fileName() to
treeitem->data(LibraryRole). This didn't work due to
KPluginMetaData::fileName() sometimes containing (part of) the path in
addition to the plugin/library name. This was solved by using
KPluginMetaData::pluginId() instead of KPluginMetaData::fileName().
Related: bug 346848
REVIEW: 124840

M  +3    -3    kcms/kded/kcmkded.cpp

http://commits.kde.org/plasma-desktop/15253a7463f472e4ce977d95d91881f21305e471