From c0f773214f5beb0d4c84df03f382ee1231b9b0b5 Mon Sep 17 00:00:00 2001
From: Markus Unterwaditzer <markus@unterwaditzer.net>
Date: Fri, 25 Dec 2015 21:06:55 +0100
Subject: [PATCH] Bugfix: open_graphical_browser opens CLI browsers

---
 CHANGELOG.rst                | 6 ++++++
 vdirsyncer/utils/__init__.py | 4 ++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.rst b/CHANGELOG.rst
index 1305260..cce0550 100644
--- a/CHANGELOG.rst
+++ b/CHANGELOG.rst
@@ -9,6 +9,12 @@ Package maintainers and users who have to manually update their installation
 may want to subscribe to `GitHub's tag feed
 <https://github.com/untitaker/vdirsyncer/tags.atom>`_.
 
+Version 0.7.5
+=============
+
+- Fixed a bug in :storage:`remotestorage` that would try to open a CLI browser
+  for OAuth.
+
 Version 0.7.4
 =============
 
diff --git a/vdirsyncer/utils/__init__.py b/vdirsyncer/utils/__init__.py
index fce7411..a3321a1 100644
--- a/vdirsyncer/utils/__init__.py
+++ b/vdirsyncer/utils/__init__.py
@@ -209,10 +209,10 @@ def open_graphical_browser(url, new=0, autoraise=True):
                      'w3m'])
 
     for name in webbrowser._tryorder:
-        browser = webbrowser.get(name)
-        if browser in cli_names:
+        if name in cli_names:
             continue
 
+        browser = webbrowser.get(name)
         if browser.open(url, new, autoraise):
             return