Improve some code

resources/lib/app/account.py

@@ -15,6 +15,8 @@ class Account(object):
         self.plex_username = None
         self.plex_user_id = None
         self.plex_token = None
+        # Personal access token per specific user and PMS
+        # As a rule of thumb, always use this token!
         self.pms_token = None
         self.avatar = None
         self.myplexlogin = None

resources/lib/initialsetup.py

@@ -287,7 +287,6 @@ class InitialSetup(object):
         }
         or None if unsuccessful
         """
-        server = None
         # If no server is set, let user choose one
         if not app.CONN.server or not app.CONN.machine_identifier:
             showDialog = True

resources/lib/plex_functions.py

@@ -164,7 +164,7 @@ def check_connection(url, token=None, verifySSL=None):
 def discover_pms(token=None):
     """
     Optional parameter:
-        token       token for plex.tv
+        token       token for plex.tv - WARNING: for the main Plex user only!
 
     Returns a list of available PMS to connect to, one entry is the dict:
     {

resources/lib/plex_tv.py

@@ -101,15 +101,14 @@ def switch_home_user(userid, pin, token, machine_identifier):
         # Set to empty iterable list for loop
         xml = []
 
-    found = 0
     LOG.debug('Our machine_identifier is %s', machine_identifier)
     for device in xml:
         identifier = device.attrib.get('clientIdentifier')
         LOG.debug('Found the Plex clientIdentifier: %s', identifier)
         if identifier == machine_identifier:
-            found += 1
             token = device.attrib.get('accessToken')
-    if found == 0:
+            break
+    else:
         LOG.info('No tokens found for your server! Using empty string')
         token = ''
     LOG.info('Plex.tv switch HomeUser change successfull for user %s',

resources/lib/windows/direct_path_sources.py

@@ -1,7 +1,6 @@
 #!/usr/bin/env python
 # -*- coding: utf-8 -*-
 """
-:module: plexkodiconnect.userselect
 :synopsis: Prompts the user to add network paths and username passwords for
            e.g. smb paths
 """