You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
plasma-desktop/plasma-desktop-PackageKit.p...

38 lines
1.6 KiB

commit 7dfb92b08584c79ef18a1a709f53759899eeac87
Author: Kai Uwe Broulik <kde@privat.broulik.de>
Date: Tue Aug 2 10:15:46 2016 +0200
[Kicker] Disable PackageKit integration (aka "Uninstall app") context menu entry
Currently we cannot populate the context menu asynchronously which means we
block waiting for PackageKit which takes multiple seconds. While fixing the missing
include was correct, the result is a terrible regression in a bugfix release.
Once changes to the context menu can be notified by the Kicker backend, ie. 5.8,
this feature will return, sorry.
As discussed with Eike
diff --git a/applets/kicker/plugin/appentry.cpp b/applets/kicker/plugin/appentry.cpp
index ee2a2d3..3bab47f 100644
--- a/applets/kicker/plugin/appentry.cpp
+++ b/applets/kicker/plugin/appentry.cpp
@@ -177,7 +177,7 @@ QVariantList AppEntry::actions() const
}
#ifdef PackageKitQt5_FOUND
- QStringList files(m_service->entryPath());
+ /*QStringList files(m_service->entryPath());
if (m_service->isApplication()) {
files += QStandardPaths::findExecutable(KShell::splitArgs(m_service->exec()).first());
@@ -191,7 +191,7 @@ QVariantList AppEntry::actions() const
QVariantMap removeAction = Kicker::createActionItem(i18n("Remove '%1'...", packageName), "removeApplication", packageName);
removeAction["icon"] = "applications-other";
actionList << removeAction;
- }
+ }*/
#endif
QQmlPropertyMap *appletConfig = qobject_cast<QQmlPropertyMap *>(appletInterface->property("configuration").value<QObject *>());