Merge lp:~mvo/weblive/new-pep8-client into lp:weblive

[Michael Vogt]

This fixes some pep8 issues that the quantal version of pep8 complains about. It builds on top of lp:~elachuni/weblive/pep8-client

Unmerged revisions

160. By Michael Vogt on 2012-08-07

use isinstance instead of comparing types as suggested by the quantal pep8 checker

=== modified file 'client/weblive.py'
--- client/weblive.py	2012-08-07 13:38:45 +0000
+++ client/weblive.py	2012-08-07 13:38:45 +0000
@@ -118,7 +118,7 @@
         query['locale'] = locale
         reply = self.do_query(query)
 
-        if type(reply['message']) != type([]):
+        if not isinstance(reply['message'], []):
             if reply['message'] == 1:
                 raise WebLiveError("Reached user limit, return false.")
             elif reply['message'] == 2:
@@ -145,7 +145,7 @@
         query['action'] = 'list_everything'
         reply = self.do_query(query)
 
-        if type(reply['message']) != type({}):
+        if not isinstance(reply['message'], {}):
             raise WebLiveError("Invalid value, expected '%s' and got '%s'."
                 % (type({}), type(reply['message'])))
 
@@ -173,7 +173,7 @@
         query['serverid'] = serverid
         reply = self.do_query(query)
 
-        if type(reply['message']) != type([]):
+        if not isinstance(reply['message'], []):
             raise WebLiveError("Invalid value, expected '%s' and got '%s'."
                 % (type({}), type(reply['message'])))
 
@@ -187,7 +187,7 @@
         query['action'] = 'list_package_blacklist'
         reply = self.do_query(query)
 
-        if type(reply['message']) != type([]):
+        if not isinstance(reply['message'], []):
             raise WebLiveError("Invalid value, expected '%s' and got '%s'."
                 % (type({}), type(reply['message'])))
 
@@ -202,7 +202,7 @@
         query['serverid'] = serverid
         reply = self.do_query(query)
 
-        if type(reply['message']) != type([]):
+        if not isinstance(reply['message'], []):
             raise WebLiveError("Invalid value, expected '%s' and got '%s'."
                 % (type({}), type(reply['message'])))
 
@@ -216,7 +216,7 @@
         query['action'] = 'list_servers'
         reply = self.do_query(query)
 
-        if type(reply['message']) != type({}):
+        if isinstance(reply['message'], {}):
             raise WebLiveError("Invalid value, expected '%s' and got '%s'."
                 % (type({}), type(reply['message'])))