davidedmundson requested changes to this revision.
davidedmundson added a comment.
This revision now requires changes to proceed.
Latest master features the line:
if (!entry->noDisplay() ) {
//add entry
which isn't in your diff, is this made against some old code?
KService::noDislpay already checks if
(!KAuthorized::authorizeControlModule(storageId())) {
as well as other important things.
REPOSITORY
R124 System Settings
REVISION DETAIL
https://phabricator.kde.org/D9055
To: sancelot, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, sancelot, ZrenBot, progwolff, lesliezhai,
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart