compiz - Jan 2007 - DBus setting options broken

Using latest git if you try to set an option to the value it already
has you get an error like 'dbus_bindings.DBusException: Method
"set"
with signature "s" on interface "org.freedesktop.compiz"
doesn't
exist' and setting it to a new value actually changes the setting but
dbus times out waiting for a reply saying so. Setting a list gives the
method doesn't exist error no matter what you do to it.

I think the list one has something to do with dbusGetOptionValue being
called with option->type instead of option->value.list.type but that
change alone doesn't seem to fix it. I've spent about an hour trying
to track this one down and am completely lost, hopefully someone else
can figure it out.

Here is the code I'm using to set a list option:
import dbus
bus = dbus.SessionBus()
obj = bus.get_object('org.freedesktop.compiz',
'/org/freedesktop/compiz/core/allscreens/active_plugins')
iface = dbus.Interface(obj, 'org.freedesktop.compiz')
value = iface.get()
value.remove('place')
iface.set(value)

-- 
Travis Watkins
http://www.realistanew.com

On Mon, 2007-01-01 at 19:35 -0600, Travis Watkins wrote:
> Using latest git if you try to set an option to the value it already
> has you get an error like 'dbus_bindings.DBusException: Method
"set"
> with signature "s" on interface
"org.freedesktop.compiz" doesn't
> exist' and setting it to a new value actually changes the setting but
> dbus times out waiting for a reply saying so. Setting a list gives the
> method doesn't exist error no matter what you do to it.
I've fixed the problem with an error being generated when setting an
option to the existing value. Dbus plugin is not sending any reply
messages so it's supposed to time out unless a error is generated.
> 
> I think the list one has something to do with dbusGetOptionValue being
> called with option->type instead of option->value.list.type but that
> change alone doesn't seem to fix it. I've spent about an hour
trying
> to track this one down and am completely lost, hopefully someone else
> can figure it out.
I'm hoping that this is fixed now, not sure though. Let me know how it
works.

-David