GNOME Bugzilla – Bug 653207
shellDBus: Add ListExtensions() and GetExtensionInfo()
Last modified: 2011-08-17 13:36:12 UTC
These methods are for SweetTooth: the HTTP proxy talks to these methods to get information about extensions state so it can figure out the right button to display as well as the correct handler for it. In the future, this will be used for extension updating.
Created attachment 190489 [details] [review] shellDBus: Add ListExtensions() and GetExtensionInfo() GetExtensionInfo() takes a UUID and returns a JSON object with information about that extension including their metadata, path and current state. ListExtensions() takes no arguments and returns a JSON object mapping UUIDs to the same information objects described above.
Created attachment 190787 [details] [review] shellDBus: Add ListExtensions() and GetExtensionInfo() GetExtensionInfo() takes a UUID and returns a JSON object with information about that extension including their metadata, path and current state. ListExtensions() takes no arguments and returns a JSON object mapping UUIDs to the same information objects described above. Fix error with GetExtensionInfo that can happen when the extension is uninstalled
Review of attachment 190787 [details] [review]: I must admit that trasmitting the whole contents of all metadata.json over the wire everytime ListExtensions is called looks too much, but it is very clean and extensible, so I'd say go for it!
(In reply to comment #3) > Review of attachment 190787 [details] [review]: > > I must admit that trasmitting the whole contents of all metadata.json over the > wire everytime ListExtensions is called looks too much, but it is very clean > and extensible, so I'd say go for it! Should I convert these to use {sv} and {s{sv}} instead?
Sorry for creating a tangled bug mess. I'll have a more linear patch set for all the DBus hacks I need soon.