Merge lp:~azzar1/unity-lens-applications/lp-1582433 into lp:unity-lens-applications

Proposed by Andrea Azzarone on 2016-05-20
Status: Merged
Approved by: Marco Trevisan (Treviño) on 2016-05-30
Approved revision: 383
Merged at revision: 383
Proposed branch: lp:~azzar1/unity-lens-applications/lp-1582433
Merge into: lp:unity-lens-applications
Diff against target: 22 lines (+10/-2)
1 file modified
src/daemon.vala (+10/-2)
To merge this branch: bzr merge lp:~azzar1/unity-lens-applications/lp-1582433
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) 2016-05-20 Approve on 2016-05-30
Review via email: mp+295272@code.launchpad.net

Commit message

Fallback from pkginfo.application_name in case app is NULL.

Description of the change

hyphen in desktop applications are not truly supported to be honest considering the way g_key_file_load_from_data_dirs works. But this simple change make it works.

To post a comment you must log in.
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/daemon.vala'
2--- src/daemon.vala 2013-09-27 12:37:24 +0000
3+++ src/daemon.vala 2016-05-20 00:22:37 +0000
4@@ -1306,8 +1306,16 @@
5 case Category.INSTALLED:
6 case Category.APPLICATIONS:
7 installed_uris.add (uri);
8- res.title = app.get_display_name ();
9- res.comment = sanitize_binary_name (app.get_executable ());
10+ if (app != null)
11+ {
12+ res.title = app.get_display_name ();
13+ res.comment = sanitize_binary_name (app.get_executable ());
14+ }
15+ else
16+ {
17+ res.title = pkginfo.application_name;
18+ }
19+
20 break;
21 case Category.AVAILABLE:
22 available_uris.add (uri);

Subscribers

People subscribed via source and target branches