D26389: DesktopPaths KCM: Move the settings logic to a KCoreConfigSkeleton class

[Kevin Ottens]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200103/f61fefd0/attachment.html>