browse: Improve handling of browse_filesystemHEAD0.1.2trunk

Stop trying to launch the browser if we do not have a path. Redirect
stderr and stdout to /dev/null to avoid spurious messages in the
terminal.

1 files changed, 10 insertions, 1 deletions

diff --git a/beetsplug/browse.py b/beetsplug/browse.py
index bb06a91..c1d37c7 100644
--- a/beetsplug/browse.py
+++ b/beetsplug/browse.py
@@ -70,8 +70,17 @@ class BrowseCommand(Subcommand):
         webbrowser.open(MUSICBRAINZ_LOOKUP + mbid)
 
     def browse_filesystem(self, item, _):
+        path = item.get("path")
+
+        if not path:
+            return
+
         try:
-            subprocess.Popen(self.explorer.split(" ") + [item.get("path")])
+            subprocess.Popen(
+                self.explorer.split() + [path.decode()],
+                stderr=subprocess.DEVNULL,
+                stdout=subprocess.DEVNULL,
+            )
         except OSError as err:
             raise UserError(err) from err