Merge lp:~3v1n0/unity-lens-files/show-folder-select into lp:unity-lens-files

[Marco Trevisan (Treviño)]

Daemon: select target file when Showing in their folder.

Launch the filemanager with the target file as parameter when showing it in its folder, so that the filemanager will select it.

[Paweł Stołowski (stolowski)]

5 var path = GLib.Path.get_dirname (uri);

Could you please remove this line, it's not needed anymore.

Apart from that it looks good and works as advertised, thanks!

review: Needs Fixing

[lp:~3v1n0/unity-lens-files/show-folder-select]

248. By Marco Trevisan (Treviño) on 2013-02-20

Damon: The path variable is not needed anymore

Cleaning up...

[Marco Trevisan (Treviño) (3v1n0)]

Ops... Sorry, thanks for pointing out.

Fixed! ;)

[Paweł Stołowski (stolowski)]

Great!

review: Approve

=== modified file 'src/daemon.vala'
--- src/daemon.vala	2012-11-08 16:50:59 +0000
+++ src/daemon.vala	2013-02-20 13:21:24 +0000
@@ -1138,13 +1138,14 @@
 
     public Unity.ActivationResponse on_preview_open_folder (string uri)
     {
-      var path = GLib.Path.get_dirname (uri);
+      var file_manager = AppInfo.get_default_for_type("inode/directory", true);
+      var uris = new GLib.List<string> ();
+      uris.append (uri);
+
       try
       {
-        if (GLib.AppInfo.launch_default_for_uri (path, null))
-        {
+        if (file_manager.launch_uris (uris, null))
           return new Unity.ActivationResponse(Unity.HandledType.HIDE_DASH);
-        }
       }
       catch (GLib.Error e)
       {