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

@@ -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()));