GNOME Bugzilla – Bug 642503
Add DBus methods to nm-applet
Last modified: 2011-03-02 20:26:16 UTC
As discussed on IRC, GNOME Shell in 3.0 will still rely on nm-applet for part of its UI, and needs a method to summon it.
Created attachment 181046 [details] [review] Export DBus methods for wireless dialogs Export two void methods on the session bus, ConnectToHiddenNetwork and CreateWirelessNetwork, that just open the respective wireless dialog. These will be used by GNOME Shell.
Can we get this committed, please ? We are beyond ui freeze, and this blocks the network menu from landing in the shell
Cleaned up and pushed as: afd0ee59a962f6eedc3dc1c41e065f9480b8468e Thanks for the patch! A few notes though: 1) the patch didn't respect the GConf preference for disabling creation of wifi networks which is necessary for managed installs; I've gone ahead and added a check to the D-Bus method that returns an error if this feature has been disabled via GConf or via NM permissions 2) it looks like the original patch would not have actually connected or created networks, as the dialog's "response" signal wasn't handled at all; so I've instead made the dbus method handlers call into the applet-device-wifi.c functions where the response can be handled correctly 3) I changed the bus name, object path, and interface; nm-applet itself, while the default implementation of an applet, isn't a freedesktop project, it's a GNOME project, and I think the bus name should reflect that. Having a bus name of org.freedesktop.NetworkManager.XXX implies the API is somehow standarized or part of the core NM API, and I'd hope this is all temporary until we can get the shell indicator up to full functionality.