Several pylint fixes.

src/sugar/presence/activity.py

@@ -281,8 +281,8 @@ class Activity(gobject.GObject):
 
         def tubes_ready():
             if self.telepathy_text_chan is None or \
-                self.telepathy_tubes_chan is None:
-                    return
+               self.telepathy_tubes_chan is None:
+                return
 
             _logger.debug('%r: finished setting up tubes', self)
             reply_handler()
@@ -303,7 +303,7 @@ class Activity(gobject.GObject):
             found_text_channel = False
             found_tubes_channel = False
 
-            for chan_path, chan_iface, handle_type, handle in chans:
+            for chan_path, chan_iface, handle_type, handle_ in chans:
                 if handle_type != telepathy.HANDLE_TYPE_ROOM:
                     return
 

src/sugar/presence/presenceservice.py

@@ -178,7 +178,7 @@ class PresenceService(gobject.GObject):
                         self._del_object, object_path)
                 try:
                     # Pre-fill the activity's ID
-                    foo = obj.props.id
+                    activity_id = obj.props.id
                 except dbus.exceptions.DBusException:
                     logging.debug('Cannot get the activity ID')
             else:
@@ -239,9 +239,11 @@ class PresenceService(gobject.GObject):
         gobject.idle_add(self._emit_activity_invitation_signal, activity_path,
                          buddy_path, message)
 
-    def _emit_private_invitation_signal(self, bus_name, connection, channel, chan_type):
+    def _emit_private_invitation_signal(self, bus_name, connection,
+                                        channel, chan_type):
         """Emit GObject event with bus_name, connection and channel"""
-        self.emit('private-invitation', bus_name, connection, channel, chan_type)
+        self.emit('private-invitation', bus_name, connection,
+                  channel, chan_type)
         return False
 
     def _private_invitation_cb(self, bus_name, connection, channel, chan_type):
@@ -469,7 +471,7 @@ class PresenceService(gobject.GObject):
                       (activity.get_id(), err))
         self.emit("activity-shared", False, None, err)
 
-    def share_activity(self, activity, properties={}, private=True):
+    def share_activity(self, activity, properties=None, private=True):
         """Ask presence service to ask the activity to share itself publicly.
         
         Uses the AdvertiseActivity method on the service to ask for the 
@@ -484,6 +486,9 @@ class PresenceService(gobject.GObject):
         """
         actid = activity.get_id()
 
+        if properties is None:
+            properties = {}
+
         # Ensure the activity is not already shared/joined
         for obj in self._objcache.values():
             if not isinstance(object, Activity):
@@ -584,7 +589,7 @@ class _MockPresenceService(gobject.GObject):
     def get_owner(self):
         return None
 
-    def share_activity(self, activity, properties={}):
+    def share_activity(self, activity, properties=None):
         return None
 
 _ps = None

src/sugar/presence/sugartubeconn.py

@@ -19,8 +19,8 @@
 from telepathy.constants import (
     CHANNEL_GROUP_FLAG_CHANNEL_SPECIFIC_HANDLES)
 
-from tubeconn import TubeConnection
-import presenceservice
+from sugar.presence.tubeconn import TubeConnection
+from sugar.presence import presenceservice
 
 
 class SugarTubeConnection(TubeConnection):