[Bugs] #2157 IMME: reimplement sugar.presence so it doesn't use the Presence Service

Sugar Labs Bugs bugtracker-noreply at sugarlabs.org
Fri Aug 20 10:25:44 EDT 2010


#2157: reimplement sugar.presence so it doesn't use the Presence Service
------------------------------+---------------------------------------------
    Reporter:  tomeu          |          Owner:  erikos     
        Type:  enhancement    |         Status:  new        
    Priority:  Immediate      |      Milestone:  0.90       
   Component:  sugar-toolkit  |        Version:  Unspecified
    Severity:  Unspecified    |       Keywords:  r?         
Distribution:  Unspecified    |   Status_field:  Assigned   
------------------------------+---------------------------------------------
Changes (by tomeu):

  * keywords:  r! => r?


Comment:

 Replying to [comment:5 erikos]:
 > src/sugar/activity/activityhandle.py
 > - thanks for changing it to handle.invited
 >
 > src/sugar/activity/activityservice.py
 > - (please add a comment that it is deprecated)
 > def Invite(self, buddy_key):
 >         raise NotImplementedError

 Removed it, as all python activities will be using InviteContact.

 > In the shell (jarabe.view.buddymenu) you are trying to call it:
 > logging.warning('Trying deprecated Activity.Invite')
 > service.Invite(self._buddy.props.key)
 > Does this handle cases where old Sugar versions share with new ones?

 That's there only for Etoys, which hasn't moved to the new API yet. All
 python activities are going to be using the new InviteContact.

 > src/sugar/datastore/datastore.py
 > - self._object_id = None  just a pylint fix, right?

 Yup.

 > src/sugar/presence/connectionmanager.py
 > - can you add a doc what it does and mark it unstable API as it is only
 of use to the sugar-PS?
 >
 > src/sugar/presence/presenceservice.py
 > - please adjust the line: "UI class to access system-level presence
 object"
 > - please adjust the docstring: "UI-side interface to the dbus presence
 service..."

 Done

 > - remove the class OfflineInterface (not used anymore)
 > - remove the 'activity-disappeared' etc signals as the shell does not
 listen to them anymore. From the s\
 > ignals only 'activity-shared' seems to be in use.

 Done

 > Btw, all the approach looks really good!

 Thanks for the review!

-- 
Ticket URL: <http://bugs.sugarlabs.org/ticket/2157#comment:6>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system


More information about the Bugs mailing list