Merge lp:~mzanetti/unity-mir/strip-app-id into lp:unity-mir

Proposed by Michael Zanetti
Status: Work in progress
Proposed branch: lp:~mzanetti/unity-mir/strip-app-id
Merge into: lp:unity-mir
Diff against target: 18 lines (+8/-1)
1 file modified
src/modules/Unity/Application/desktopfilereader.cpp (+8/-1)
To merge this branch: bzr merge lp:~mzanetti/unity-mir/strip-app-id
Reviewer Review Type Date Requested Status
Mir development team Pending
Review via email: mp+214997@code.launchpad.net
To post a comment you must log in.

Unmerged revisions

210. By Michael Zanetti

try the same workaround as for the launcher to deal with long app ids

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/modules/Unity/Application/desktopfilereader.cpp'
--- src/modules/Unity/Application/desktopfilereader.cpp 2014-02-28 16:51:31 +0000
+++ src/modules/Unity/Application/desktopfilereader.cpp 2014-04-09 17:20:32 +0000
@@ -57,7 +57,14 @@
57 DLOG("DesktopFileReader::DesktopFileReader (this=%p, appId='%s')", this, appId.toLatin1().data());57 DLOG("DesktopFileReader::DesktopFileReader (this=%p, appId='%s')", this, appId.toLatin1().data());
58 DASSERT(appId != NULL);58 DASSERT(appId != NULL);
5959
60 file_ = findDesktopFile(appId);60 QString helper = appId;
61 if (helper.startsWith("appid://")) {
62 helper.remove("appid://");
63 // Strip hook name and current-user-version in case its there
64 helper = appId.split('/').first();
65 }
66
67 file_ = findDesktopFile(helper);
61 loaded_ = loadDesktopFile(file_);68 loaded_ = loadDesktopFile(file_);
62}69}
6370

Subscribers

People subscribed via source and target branches