Differential D1600

Check user is allowed to run krunner when forwarding key events
ClosedPublic
Actions

Authored by davidedmundson on May 12 2016, 9:59 PM.

Details

Reviewers
broulik
Group Reviewers
Plasma
Commits
R871:35a637471f82: Check user is allowed to run krunner when forwarding key events
R120:35a637471f82: Check user is allowed to run krunner when forwarding key events

Diff Detail

Repository
R120 Plasma Workspace
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
davidedmundson updated this revision to Diff 3791.May 12 2016, 9:59 PM
davidedmundson retitled this revision from to Check user is allowed to run krunner when forwarding key events.
davidedmundson updated this object.
davidedmundson edited the test plan for this revision. (Show Details)
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma. · View Herald TranscriptMay 12 2016, 9:59 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson updated this revision to Diff 3792.May 12 2016, 10:01 PM
  • Don't run krunner if the user isn't allowed to use it
broulik accepted this revision.May 12 2016, 10:02 PM
broulik added a reviewer: broulik.
This revision is now accepted and ready to land.May 12 2016, 10:02 PM
broulik added a comment.May 12 2016, 10:03 PM

That accept was for the first revision of the patch, please split the KRunner stuff into a separate commit

krunner/main.cpp
59

Maybe print something on console

Closed by commit R120:35a637471f82: Check user is allowed to run krunner when forwarding key events (authored by davidedmundson). · Explain WhyMay 17 2016, 10:50 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mkrunner/main.cpp (4 lines)
Mkrunner/view.cpp (28 lines)
Mshell/desktopview.cpp (4 lines)

Diff 3792

View Options

krunner/main.cpp

Loading...
View Options

krunner/view.cpp

Loading...
View Options

shell/desktopview.cpp

Loading...