broulik created this revision. broulik added reviewers: Plasma, sitter. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. broulik requested review of this revision.
REVISION SUMMARY This is a similar situation we have with Flatpak when the desktop entry the application thinks it owns is actually different. In Snap this environment variable is passed to the process. When no service was found, try reading it. TEST PLAN 5.16 branch My Spotify snap now identifies as Spotify and I can configure notifications for it. F6907051: Screenshot_20190618_150619.png <https://phabricator.kde.org/F6907051> - Notification without app name still resolves to process name properly - Service lookup for normal apps still works - Job view without an app name still resolves to process name properly REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D21881 AFFECTED FILES libnotificationmanager/jobsmodel_p.cpp libnotificationmanager/notification.cpp libnotificationmanager/notification.h libnotificationmanager/notification_p.h libnotificationmanager/server_p.cpp libnotificationmanager/utils.cpp libnotificationmanager/utils_p.h To: broulik, #plasma, sitter Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
