Sync Plasma theme settings instead of copying the plasmarc file
Needs ReviewPublic

Authored by filipf on Feb 17 2020, 3:03 PM.

Details

Summary

The first and the easiest part of finishing T12710.

Instead of copying the user's plasmarc file in order to have SDDM use their Plasma theme, we read the user value and create the SDDM user's config ourselves.

Test Plan

Clicked Sync in the kcm
the plasmarc file was created in the right place, with the right permissions, and with the right entry inside of it
sddm used the Plasma theme we just specified

Diff Detail

Repository
R123 SDDM Configuration Panel (KCM)
Branch
safer-plasmarc (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 22580
Build 22598: arc lint + arc unit
filipf created this revision.Feb 17 2020, 3:03 PM
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 17 2020, 3:03 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
filipf requested review of this revision.Feb 17 2020, 3:03 PM
filipf added inline comments.Feb 17 2020, 3:10 PM
sddmauthhelper.cpp
138

do I need to do themeConfig->sync? It seems to work as intended without it, even when just updating the entry.