Merge lp:~seb128/indicator-bluetooth/browsing-not-working into lp:indicator-bluetooth

[Sebastien Bacher]

Remove the "browse" action since there is no working backend

Remove the "browse" action since there is no working backend

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:96
http://jenkins.qa.ubuntu.com/job/indicator-bluetooth-ci/43/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-bluetooth-wily-amd64-ci/5
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-bluetooth-wily-armhf-ci/5
        deb: http://jenkins.qa.ubuntu.com/job/indicator-bluetooth-wily-armhf-ci/5/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/indicator-bluetooth-ci/43/rebuild

review: Approve (continuous-integration)

[Robert Ancell (robert-ancell)]

I don't know the details of why / if the browse functionality should work but this and the bug report makes sense to me.

review: Approve

=== modified file 'src/desktop.vala'
--- src/desktop.vala	2015-08-27 14:24:22 +0000
+++ src/desktop.vala	2016-05-12 15:45:18 +0000
@@ -136,9 +136,14 @@
         if (device.is_connectable)
           submenu.append_item (create_device_connection_menuitem (device));
 
+        /* There is no working backend that can be used there, disable
+           the action until that situation gets sorted out
+           see http://launchpad.net/bugs/1562822
+           
         if (device.supports_browsing)
           submenu.append (_("Browse files…"),
                           @"indicator.desktop-browse-files::$(device.address)");
+        */
 
         if (device.supports_file_transfer)
           submenu.append (_("Send files…"),