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

Proposed by Andrea Azzarone
Status: Merged
Approved by: Marco Trevisan (Treviño)
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) Approve
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.
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/daemon.vala'
--- src/daemon.vala 2013-09-27 12:37:24 +0000
+++ src/daemon.vala 2016-05-20 00:22:37 +0000
@@ -1306,8 +1306,16 @@
1306 case Category.INSTALLED:1306 case Category.INSTALLED:
1307 case Category.APPLICATIONS:1307 case Category.APPLICATIONS:
1308 installed_uris.add (uri);1308 installed_uris.add (uri);
1309 res.title = app.get_display_name ();1309 if (app != null)
1310 res.comment = sanitize_binary_name (app.get_executable ());1310 {
1311 res.title = app.get_display_name ();
1312 res.comment = sanitize_binary_name (app.get_executable ());
1313 }
1314 else
1315 {
1316 res.title = pkginfo.application_name;
1317 }
1318
1311 break;1319 break;
1312 case Category.AVAILABLE:1320 case Category.AVAILABLE:
1313 available_uris.add (uri);1321 available_uris.add (uri);

Subscribers

People subscribed via source and target branches