Bug 393329 - kiosk: run_command=false unawareness in containment context
Summary: kiosk: run_command=false unawareness in containment context
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Containment (other bugs)
Version First Reported In: master
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Sebastian Kügler
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-04-20 10:34 UTC by Harald Sitter
Modified: 2018-04-24 01:30 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 5.12.5
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Harald Sitter 2018-04-20 10:34:56 UTC
in /etc/xdg/kdeglobals set

[KDE Action Restrictions][$i]
run_command=false

and relog.

This disables krunnering, and works as intended (can't get to krunner in any form or fashion).
The context menu of the desktop containment's context menu however still list the 'Run Command...' action. The action is defunct and doesn't do anything.

The action should not be visible when run_command is being disabled via an action restriction.
Comment 1 Kai Uwe Broulik 2018-04-20 11:12:46 UTC
Documentation says:

NOTE: To also disable desktop context menu run command action/run_command is required at [KDE Action Restrictions]

(which admittedly is a bit weird)
Comment 2 Kai Uwe Broulik 2018-04-23 13:14:13 UTC
Git commit cfd77db90cf604e2f30afa5a0a1466cddfeeb9ac by Kai Uwe Broulik.
Committed on 23/04/2018 at 13:13.
Pushed by broulik into branch 'Plasma/5.12'.

[ContextMenu Containment Action] Fix checking for KIOSK

Anyone else checks for run_command. Docs explicitly mention the fact that you have to also
disable action/run_command for the menu entry to appear but this is a bit awkward, isn't it.
Also fix logout to check for the correct place as listed in documentation and other code places.
FIXED-IN: 5.15.5

Differential Revision: https://phabricator.kde.org/D12376

M  +2    -2    containmentactions/contextmenu/menu.cpp

https://commits.kde.org/plasma-workspace/cfd77db90cf604e2f30afa5a0a1466cddfeeb9ac