2004-07-12 Seth Nickell <seth@gnome.org>

	* python/dbus.py:

	Add message argument to the default object_method_handler
	function.

	* python/dbus_bindings.pyx.in:

	Automatically return NIL when passed an empty list
	(we can't pass back a list since lists are typed
	and we don't have any idea what type the the client
	intended the list to be... :-( )

diff --git a/ChangeLog b/ChangeLog
index 81dac46..93526f5 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2004-07-12  Seth Nickell  <seth@gnome.org>
+
+       * python/dbus.py:
+
+       Add message argument to the default object_method_handler
+       function.
+       
+       * python/dbus_bindings.pyx.in:
+
+       Automatically return NIL when passed an empty list
+       (we can't pass back a list since lists are typed
+       and we don't have any idea what type the the client
+       intended the list to be... :-( )
+       
 2004-07-10  Seth Nickell  <seth@gnome.org>
 
        * python/examples/Makefile.am:

diff --git a/python/dbus.py b/python/dbus.py
index e53bf23..55c6123 100644 (file)
--- a/python/dbus.py
+++ b/python/dbus.py
@@ -337,7 +337,7 @@ class ObjectTree:
         message = dbus_bindings.Signal(object_path, interface, signal_name)
         self._connection.send(message)
         
-    def object_method_called(self, relative_path, method_name, argument_list):
+    def object_method_called(self, message, relative_path, method_name, argument_list):
         """Override this method. Called with, object_path, the relative path of the object
         under the base_path, the name of the method invoked, and a list of arguments
         """

diff --git a/python/dbus_bindings.pyx.in b/python/dbus_bindings.pyx.in
index de90603..6703f7d 100644 (file)
--- a/python/dbus_bindings.pyx.in
+++ b/python/dbus_bindings.pyx.in
@@ -614,19 +614,23 @@ cdef class MessageIter:
         elif value_type == dict:
             retval = self.append_dict(value)
         elif value_type == list:
-            if (len(value) == 0):
-                raise TypeError, "Empty lists are currently not supported, return None instead"
-            list_type = type(value[0])
-            if list_type == str:
-                self.append_string_array(value)
-            elif list_type == int:
-                self.append_int32_array(value)
-            elif list_type == float:
-                self.append_double_array(value)
-            elif isinstance(value[0], ObjectPath):
-                self.append_object_path_array(value)
+            if len(value) == 0:
+                # Empty lists are currently not supported, returning None instead
+                retval = self.append(None)
             else:
-                raise TypeError, "List of unknown type '%s'" % (list_type)
+                list_type = type(value[0])
+                if list_type == str:
+                    self.append_string_array(value)
+                elif list_type == int:
+                    self.append_int32_array(value)
+                elif list_type == float:
+                    self.append_double_array(value)
+                elif isinstance(value[0], ObjectPath):
+                    self.append_object_path_array(value)
+                else:
+                    raise TypeError, "List of unknown type '%s'" % (list_type)
+        elif value_type == None.__class__:
+            retval = self.append_nil()
         elif isinstance(value, ObjectPath):
             retval = self.append_object_path(value)
         else: