From 8192bd3aec27a77bb1ab903bb928ffe0a2f4c517 Mon Sep 17 00:00:00 2001
From: Fabian Vogt <fabian@ritter-vogt.de>
Date: Tue, 19 Feb 2019 14:21:56 +0100
Subject: [PATCH] Don't show entries with NoDisplay=true with the applauncher
 containmentaction

Summary: Currently NoDisplay services are shown, which doesn't make much sense.

Test Plan: Installed and looked through the menu, no excessive or duplicate items anymore.

Reviewers: #plasma, hein

Reviewed By: #plasma, hein

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D19157
---
 containmentactions/applauncher/launch.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/containmentactions/applauncher/launch.cpp b/containmentactions/applauncher/launch.cpp
index 4f70337ad..a06eaa9ae 100644
--- a/containmentactions/applauncher/launch.cpp
+++ b/containmentactions/applauncher/launch.cpp
@@ -49,7 +49,7 @@ QList<QAction*> AppLauncher::contextualActions()
 
 void AppLauncher::makeMenu(QMenu *menu, const KServiceGroup::Ptr group)
 {
-    foreach (KSycocaEntry::Ptr p, group->entries(true, false, true)) {
+    foreach (KSycocaEntry::Ptr p, group->entries(true, true, true)) {
         if (p->isType(KST_KService)) {
             const KService::Ptr service(static_cast<KService*>(p.data()));