Bug 474228

Summary: Pinning app to task manager from Kicker/Kickoff doesn't work
Product: [Plasma] plasmashell Reporter: Nicolas Fella <nicolas.fella>
Component: Application Launcher (Kickoff)Assignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: major CC: bugseforuns, mikel5764, nate, noahadvs, notmart
Priority: NOR Keywords: qt6
Version: master   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=473922
Latest Commit: https://invent.kde.org/plasma/plasma-workspace/-/commit/93e609707a5c7fb98209d43759c8751cc8bce603 Version Fixed In:
Sentry Crash Report:

Description Nicolas Fella 2023-09-06 14:45:50 UTC 
STEPS TO REPRODUCE
1. Open Kicker/Kickoff 
2. Right click on an app
3. Pin to Task Manager

OBSERVED RESULT
Nothing

EXPECTED RESULT
The app is pinned to the task manager

SOFTWARE/OS VERSIONS
KDE Plasma Version: master
KDE Frameworks Version: master
Qt Version: 6.5.2

ADDITIONAL INFORMATION
Relevant log line:

QMetaObject::invokeMethod: No such method QQuickMouseArea::addLauncher(QVariant)
 
from https://invent.kde.org/plasma/plasma-workspace/-/blob/master/applets/kicker/plugin/containmentinterface.cpp#L162
Comment 1 Nicolas Fella 2023-09-06 14:47:33 UTC 
"Add to Desktop" is broken the same way
Comment 2 Nate Graham 2023-09-06 17:18:01 UTC 
Can confirm. Console logging looks relevant:

Sep 06 11:16:26 Liberator plasmashell[2072]: QMetaObject::invokeMethod: No such method QQuickMouseArea::hasLauncher(QVariant)
Sep 06 11:16:26 Liberator plasmashell[2072]: QMetaObject::invokeMethod: No such method QQuickMouseArea::addLauncher(QVariant)

Could be a similar issue to Bug 473922 where we're looking for a function by its method signature and that changed in Qt 6.
Comment 3 Bug Janitor Service 2023-09-16 19:37:13 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3293
Comment 4 Kai Uwe Broulik 2023-09-17 18:14:08 UTC 
Git commit 93e609707a5c7fb98209d43759c8751cc8bce603 by Kai Uwe Broulik.
Committed on 17/09/2023 at 20:05.
Pushed by broulik into branch 'master'.

applets/kicker: Invoke addLauncher/hasLauncher on the PlasmoidItem

In Plasma 6, we have a well-defined root item PlasmoidItem which contains those
functions, no need to go hunting for a random object in the tree anymore, which
is incorrect now.

M  +11   -35   applets/kicker/plugin/containmentinterface.cpp

https://invent.kde.org/plasma/plasma-workspace/-/commit/93e609707a5c7fb98209d43759c8751cc8bce603