From 3e54282420d6203951b858d370297a914509e6de Mon Sep 17 00:00:00 2001 From: Alexander Lohnau Date: Thu, 10 Feb 2022 19:28:59 +0000 Subject: [PATCH] Desktop: Still show services with nodisplay=true set When adding KCMs to the desktop, we add the entry with NoDisplay=true so that it does not show up in a search. However before this commit, that would cause the entry to not show up on the desktop as well. Because of the filtering in the services runner, we already make sure that only relevant entries can get added to the desktop in the first place. BUG: 449243 FIXED-IN: 5.24.1 --- kioslave/desktop/kio_desktop.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kioslave/desktop/kio_desktop.cpp b/kioslave/desktop/kio_desktop.cpp index e6a7c1dd3..7ab099ef1 100644 --- a/kioslave/desktop/kio_desktop.cpp +++ b/kioslave/desktop/kio_desktop.cpp @@ -161,7 +161,7 @@ void DesktopProtocol::prepareUDSEntry(KIO::UDSEntry &entry, bool listing) const if (!name.isEmpty()) entry.replace(KIO::UDSEntry::UDS_DISPLAY_NAME, name); - if (file.noDisplay() || !file.tryExec()) + if (!file.tryExec()) entry.replace(KIO::UDSEntry::UDS_HIDDEN, 1); }