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