ervin created this revision. ervin added reviewers: Plasma, crossi, bport, meven, mart, davidedmundson. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. ervin requested review of this revision.
REVISION SUMMARY Our KCoreConfigSkeleton subclass is interestingly hand written since it's too remote from what we usually do (the resulting file needs to be sourceable and we don't want the usual behavior of removing keys when they're set to defaults). The only one which was "more standard" was the autostart path, but really with the URL conversion on top and for just a single entry, I shoved it in the hand written class as well. Doesn't reduce the code much, but at least it separates that logic from the KCM and it opens the door to properly working defaults button. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D26389 AFFECTED FILES kcms/desktoppaths/CMakeLists.txt kcms/desktoppaths/desktoppathssettings.cpp kcms/desktoppaths/desktoppathssettings.h kcms/desktoppaths/desktoppathssettingsbase.kcfg kcms/desktoppaths/globalpaths.cpp kcms/desktoppaths/globalpaths.h To: ervin, #plasma, crossi, bport, meven, mart, davidedmundson Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
