Correct minor utf8, PEP8 and docstrings issues

Author: cmlccie

ciscosparkapi/api/people.py

@@ -91,14 +91,15 @@ def list(self, email=None, displayName=None, max=None):
 
         An email address or displayName must be provided.
 
-        This method supports Cisco Spark's implmentation of RFC5988 Web Linking
-        to provide pagination support.  It returns a generator container that
-        incrementally yield all people returned by the query.  The generator
-        will automatically request additional 'pages' of responses from Spark
-        as needed until all responses have been returned.  The container makes
-        the generator safe for reuse.  A new API call will be made, using the
-        same parameters that were specified when the generator was created,
-        every time a new iterator is requested from the container.
+        This method supports Cisco Spark's implementation of RFC5988 Web
+        Linking to provide pagination support.  It returns a generator
+        container that incrementally yield all people returned by the
+        query.  The generator will automatically request additional 'pages' of
+        responses from Spark as needed until all responses have been returned.
+        The container makes the generator safe for reuse.  A new API call will
+        be made, using the same parameters that were specified when the
+        generator was created, every time a new iterator is requested from the
+        container.
 
         Args:
             email(unicode, str): The e-mail address of the person to be found.
@@ -123,9 +124,9 @@ def list(self, email=None, displayName=None, max=None):
         assert max is None or isinstance(max, int)
         params = {}
         if email:
-            params[u'email'] = email
+            params[u'email'] = utf8(email)
         elif displayName:
-            params[u'displayName'] = displayName
+            params[u'displayName'] = utf8(displayName)
         else:
             error_message = "An email or displayName argument must be " \
                             "specified."

ciscosparkapi/api/rooms.py

@@ -102,14 +102,15 @@ def list(self, max=None, **query_params):
 
         By default, lists rooms to which the authenticated user belongs.
 
-        This method supports Cisco Spark's implmentation of RFC5988 Web Linking
-        to provide pagination support.  It returns a generator container that
-        incrementally yield all rooms returned by the query.  The generator
-        will automatically request additional 'pages' of responses from Spark
-        as needed until all responses have been returned.  The container makes
-        the generator safe for reuse.  A new API call will be made, using the
-        same parameters that were specified when the generator was created,
-        every time a new iterator is requested from the container.
+        This method supports Cisco Spark's implementation of RFC5988 Web
+        Linking to provide pagination support.  It returns a generator
+        container that incrementally yield all rooms returned by the
+        query.  The generator will automatically request additional 'pages' of
+        responses from Spark as needed until all responses have been returned.
+        The container makes the generator safe for reuse.  A new API call will
+        be made, using the same parameters that were specified when the
+        generator was created, every time a new iterator is requested from the
+        container.
 
         Args:
             max(int): Limits the maximum number of rooms returned from the
@@ -132,11 +133,13 @@ def list(self, max=None, **query_params):
         # Process args
         assert max is None or isinstance(max, int)
         params = {}
-        if max: params[u'max'] = max
+        if max:
+            params[u'max'] = max
         # Process query_param keyword arguments
         if query_params:
             for param, value in query_params.items():
-                if isinstance(value, basestring): value = utf8(value)
+                if isinstance(value, basestring):
+                    value = utf8(value)
                 params[utf8(param)] = value
         # API request - get items
         items = self.session.get_items('rooms', params=params)
@@ -164,10 +167,11 @@ def create(self, title, teamId=None):
         assert teamId is None or isinstance(teamId, basestring)
         post_data = {}
         post_data[u'title'] = utf8(title)
-        if teamId: post_data[u'teamId'] = utf8(teamId)
+        if teamId:
+            post_data[u'teamId'] = utf8(teamId)
         # API request
         json_obj = self.session.post('rooms', json=post_data)
-        # Return a Person object created from the response JSON data
+        # Return a Room object created from the response JSON data
         return Room(json_obj)
 
     def get(self, roomId):
@@ -185,7 +189,7 @@ def get(self, roomId):
         assert isinstance(roomId, basestring)
         # API request
         json_obj = self.session.get('rooms/'+roomId)
-        # Return a Person object created from the response JSON data
+        # Return a Room object created from the response JSON data
         return Room(json_obj)
 
     def update(self, roomId, **update_attributes):
@@ -215,11 +219,12 @@ def update(self, roomId, **update_attributes):
             raise ciscosparkapiException(error_message)
         put_data = {}
         for param, value in update_attributes.items():
-            if isinstance(value, basestring): value = utf8(value)
+            if isinstance(value, basestring):
+                value = utf8(value)
             put_data[utf8(param)] = value
         # API request
         json_obj = self.session.post('rooms', json=put_data)
-        # Return a Person object created from the response JSON data
+        # Return a Room object created from the response JSON data
         return Room(json_obj)
 
     def delete(self, roomId):