[Battery/Clock] Offer KCM access only if allowed
ClosedPublic

Authored by broulik on May 18 2016, 6:00 PM.

Details

Summary

Remove the context menus that would do nothing if access to those KCMs is restricted.

Test Plan

Added the respective KCM desktop file names to [KDE Control Module Restrictions]:

  • In Battery it is shown if one of the three KCMs is allowed, works
  • In clock the two entries are treated separately, works

Confirmed that KCMShell refused to open and System Settings didn't list; still works when no restriction is set

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik updated this revision to Diff 3861.May 18 2016, 6:00 PM
broulik retitled this revision from to [Battery/Clock] Offer KCM access only if allowed.
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R120 Plasma Workspace.
Restricted Application added a project: Plasma. · View Herald TranscriptMay 18 2016, 6:00 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart accepted this revision.May 19 2016, 2:28 PM
mart added a reviewer: mart.
This revision is now accepted and ready to land.May 19 2016, 2:28 PM
This revision was automatically updated to reflect the committed changes.