Bug 385616 - launcherPosition, launcherActivities are broken in libtaskmanager
Summary: launcherPosition, launcherActivities are broken in libtaskmanager
Status: RESOLVED NOT A BUG
Alias: None
Product: plasmashell
Classification: Plasma
Component: Task Manager and Icons-Only Task Manager widgets (show other bugs)
Version: 5.11.0
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Eike Hein
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-10-11 16:54 UTC by Michail Vourlakos
Modified: 2017-10-11 20:19 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michail Vourlakos 2017-10-11 16:54:23 UTC
These two functions from TasksModel are broken, what is happening is the following. Plasma possibly added new functionality in 5.11 for which a task's launcher url is changed in something more generic. For systemsettings that would be, applications:systemsettings.desktop . I found that this is happening for launchers and windows in 5.11, I dont know for startups. Because of this launcherPosition and launcherActivities return -1 and null accordingly. For cases that instead of the above the desktop url is used e.g.
file:///usr/share/applications/systemsettings.desktop

then the functions I think they worked correctly. I used the following code on the click event of a task in Latte in order to check the behavior.


console.log("POS :::: "+ tasksModel.launcherPosition(LauncherUrlWithoutIcon) + " - " + tasksModel.launcherPosition(LauncherUrl) + " - "                          + tasksModel.launcherActivities(LauncherUrl));

at all cases of launchers and windows the results where "POS :::: -1 - -1 -"
Comment 1 Nate Graham 2017-10-11 20:16:57 UTC
Would you mind adding your comment and the result of any further investigations to https://bugs.kde.org/show_bug.cgi?id=385594?

The bug will be fixed faster if we centralize information there. Thanks!
Comment 2 Michail Vourlakos 2017-10-11 20:19:54 UTC
of course, no prob