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

Reply via email to