[sugar] [PATCH sugar] services/presence: Use ExportedGObject from dbus-python rather than reimplementing it
Dan Williams
dcbw at redhat.com
Fri May 11 10:30:18 EDT 2007
On Fri, 2007-05-11 at 12:39 +0100, Simon McVittie wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> dbus-python now has a working implementation of ExportedGObject, so
> there's no need for the presence service to have its own internal
> implementation(s).
Looks good.
> - ---
> services/presence/activity.py | 14 ++++++--------
> services/presence/buddy.py | 16 +++++++---------
> services/presence/presenceservice.py | 16 ++++++++--------
> 3 files changed, 21 insertions(+), 25 deletions(-)
>
> diff --git a/services/presence/activity.py b/services/presence/activity.py
> index c856f54..d955c71 100644
> - --- a/services/presence/activity.py
> +++ b/services/presence/activity.py
> @@ -16,7 +16,9 @@
> # Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
>
> import gobject
> - -import dbus, dbus.service
> +import dbus
> +import dbus.service
> +from dbus.gobject_service import ExportedGObject
> from sugar import util
> import logging
>
> @@ -25,10 +27,6 @@ from telepathy.interfaces import (CHANNEL_INTERFACE)
> _ACTIVITY_PATH = "/org/laptop/Sugar/Presence/Activities/"
> _ACTIVITY_INTERFACE = "org.laptop.Sugar.Presence.Activity"
>
> - -class DBusGObjectMetaclass(dbus.service.InterfaceType, gobject.GObjectMeta): pass
> - -class DBusGObject(dbus.service.Object, gobject.GObject): __metaclass__ = DBusGObjectMetaclass
> - -
> - -
> _PROP_ID = "id"
> _PROP_NAME = "name"
> _PROP_COLOR = "color"
> @@ -38,7 +36,7 @@ _PROP_LOCAL = "local"
> _PROP_JOINED = "joined"
> _PROP_CUSTOM_PROPS = "custom-props"
>
> - -class Activity(DBusGObject):
> +class Activity(ExportedGObject):
> """Represents a potentially shareable activity on the network.
> """
>
> @@ -84,7 +82,6 @@ class Activity(DBusGObject):
>
> self._object_id = object_id
> self._object_path = _ACTIVITY_PATH + str(self._object_id)
> - - dbus.service.Object.__init__(self, bus_name, self._object_path)
>
> self._buddies = []
> self._joined = False
> @@ -111,7 +108,8 @@ class Activity(DBusGObject):
> if not util.validate_activity_id(kwargs[_PROP_ID]):
> raise ValueError("Invalid activity id '%s'" % kwargs[_PROP_ID])
>
> - - gobject.GObject.__init__(self, **kwargs)
> + ExportedGObject.__init__(self, bus_name, self._object_path,
> + gobject_properties=kwargs)
> if self.props.local and not self.props.valid:
> raise RuntimeError("local activities require color, type, and name")
>
> diff --git a/services/presence/buddy.py b/services/presence/buddy.py
> index fcc655b..f302b8c 100644
> - --- a/services/presence/buddy.py
> +++ b/services/presence/buddy.py
> @@ -18,7 +18,9 @@
>
> import os
> import gobject
> - -import dbus, dbus.service
> +import dbus
> +import dbus.service
> +from dbus.gobject_service import ExportedGObject
> from ConfigParser import ConfigParser, NoOptionError
>
> from sugar import env, profile, util
> @@ -35,10 +37,6 @@ class NotFoundError(dbus.DBusException):
> dbus.DBusException.__init__(self)
> self._dbus_error_name = _PRESENCE_INTERFACE + '.NotFound'
>
> - -class DBusGObjectMetaclass(dbus.service.InterfaceType, gobject.GObjectMeta): pass
> - -class DBusGObject(dbus.service.Object, gobject.GObject): __metaclass__ = DBusGObjectMetaclass
> - -
> - -
> _PROP_NICK = "nick"
> _PROP_KEY = "key"
> _PROP_ICON = "icon"
> @@ -50,7 +48,7 @@ _PROP_VALID = "valid"
> # Will go away soon
> _PROP_IP4_ADDRESS = "ip4-address"
>
> - -class Buddy(DBusGObject):
> +class Buddy(ExportedGObject):
> """Person on the network (tracks properties and shared activites)
>
> The Buddy is a collection of metadata describing a particular
> @@ -111,7 +109,6 @@ class Buddy(DBusGObject):
> self._bus_name = bus_name
> self._object_id = object_id
> self._object_path = _BUDDY_PATH + str(self._object_id)
> - - dbus.service.Object.__init__(self, self._bus_name, self._object_path)
>
> self._activities = {} # Activity ID -> Activity
> self._activity_sigids = {}
> @@ -134,8 +131,9 @@ class Buddy(DBusGObject):
> if key not in _ALLOWED_INIT_PROPS:
> logging.debug("Invalid init property '%s'; ignoring..." % key)
> del kwargs[key]
> - -
> - - gobject.GObject.__init__(self, **kwargs)
> +
> + ExportedGObject.__init__(self, bus_name, self._object_path,
> + gobject_properties=kwargs)
>
> def do_get_property(self, pspec):
> """Retrieve current value for the given property specifier
> diff --git a/services/presence/presenceservice.py b/services/presence/presenceservice.py
> index 2598942..e3e217c 100644
> - --- a/services/presence/presenceservice.py
> +++ b/services/presence/presenceservice.py
> @@ -15,9 +15,14 @@
> # Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
>
> import gobject
> - -import dbus, dbus.service, dbus.glib
> +import dbus
> +import dbus.service
> +from dbus.gobject_service import ExportedGObject
> import logging
>
> +# Note that this import has side effects!
> +import dbus.glib
> +
> from telepathy.client import ManagerRegistry, Connection
> from telepathy.interfaces import (CONN_MGR_INTERFACE, CONN_INTERFACE)
> from telepathy.constants import (CONNECTION_STATUS_CONNECTING, CONNECTION_STATUS_CONNECTED,
> @@ -40,10 +45,7 @@ class NotFoundError(dbus.DBusException):
> dbus.DBusException.__init__(self, msg)
> self._dbus_error_name = _PRESENCE_INTERFACE + '.NotFound'
>
> - -class DBusGObjectMetaclass(dbus.service.InterfaceType, gobject.GObjectMeta): pass
> - -class DBusGObject(dbus.service.Object, gobject.GObject): __metaclass__ = DBusGObjectMetaclass
> - -
> - -class PresenceService(DBusGObject):
> +class PresenceService(ExportedGObject):
> __gtype_name__ = "PresenceService"
>
> __gsignals__ = {
> @@ -59,8 +61,6 @@ class PresenceService(DBusGObject):
> self._handles_buddies = {} # tp client -> (handle -> Buddy)
> self._activities = {} # activity id -> Activity
>
> - - gobject.GObject.__init__(self)
> - -
> bus = dbus.SessionBus()
> self._bus_name = dbus.service.BusName(_PRESENCE_SERVICE, bus=bus)
>
> @@ -94,7 +94,7 @@ class PresenceService(DBusGObject):
> self._ll_plugin = LinkLocalPlugin(self._registry, self._owner)
> self._handles_buddies[self._ll_plugin] = {}
>
> - - dbus.service.Object.__init__(self, self._bus_name, _PRESENCE_PATH)
> + ExportedGObject.__init__(self, self._bus_name, _PRESENCE_PATH)
>
> def _activity_shared_cb(self, tp, activity, success, exc, async_cb, async_err_cb):
> if success:
> - --
> 1.5.1.3
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.6 (GNU/Linux)
> Comment: OpenPGP key: http://www.pseudorandom.co.uk/2003/contact/ or pgp.net
>
> iD8DBQFGRFX9WSc8zVUw7HYRAvf9AJ9stwZtVzgI36oRTqFmzHcFZwX8HgCg5bUW
> GE8bg3yEBzZT2ruSA/btZNg=
> =59rE
> -----END PGP SIGNATURE-----
> _______________________________________________
> Sugar mailing list
> Sugar at laptop.org
> http://mailman.laptop.org/mailman/listinfo/sugar
More information about the Sugar
mailing list