Consolidate branches and use json dump rather than str().

hellotrei · Dec 28, 2015 · b42b7b9 · b42b7b9
1 parent c8343f0
commit b42b7b9
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/unifi/controller.py b/unifi/controller.py
@@ -143,17 +143,14 @@ def _construct_api_path(self, version):
     def _login(self, version):
         log.debug('login() as %s', self.username)
 
+        params = {'username': self.username, 'password': self.password}
         login_url = self.url
+
         if version is 'v4':
             login_url += 'api/login'
+            params = json.dumps(params)
         else:
             login_url += 'login'
-
-        params = {'username': self.username, 'password': self.password}
-
-        if version is 'v4':
-            params = str(params)
-        else:
             params.update({'login': 'login'})
             if PYTHON_VERSION is 2:
                 params = urllib.urlencode(params)