Dash ignores NoDisplay=true if desktop file name contains a colon

This may well be a duplicate of Bug #879889, although the symptoms reported there seem different than what I've noticed.

This was created by this commit:

------------------------------------------------------------
revno: 242.1.5
committer: Michal Hruby <email address hidden>
branch nick: unity-lens-applications
timestamp: Wed 2011-10-05 12:47:44 +0200
message:
  Fix the de-duplication of Installed and Available apps
diff:
=== modified file 'src/daemon.vala'
--- src/daemon.vala 2011-10-05 10:47:11 +0000
+++ src/daemon.vala 2011-10-05 10:47:44 +0000
@@ -626,6 +626,13 @@
         string desktop_id = Path.get_basename (pkginfo.desktop_file);
         string full_path;

+ /* S-C uses "app_name:desktop_id.desktop", get rid of the prefix */
+ int colon_pos = desktop_id.index_of (":");
+ if (colon_pos > 0)
+ {
+ desktop_id = desktop_id[colon_pos+1:desktop_id.length];
+ }
+
         AppInfo? app = appmanager.lookup (desktop_id);
         full_path = appmanager.get_path (desktop_id);

This patch fixes the issue for me.

The attachment "916201.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

Thanks for the patch, but I suggest you to make a branch of unity-lens-applications and request it for merge with trunk.