Merge in improved wired profile UI

Tweak how wired profile list is built to be simple/more efficent and not cause a dbus error.

wicd/guiutil.py

@@ -46,6 +46,45 @@ def alert(parent, message, block=True):
         dialog.run()
         dialog.destroy()
 
+def string_input(prompt, secondary, textbox_label):
+    # based on a version of a PyGTK text entry from
+    # http://ardoris.wordpress.com/2008/07/05/pygtk-text-entry-dialog/
+
+    def dialog_response(entry, dialog, response):
+        dialog.response(response)
+
+    dialog = gtk.MessageDialog(
+        None,
+        gtk.DIALOG_MODAL,
+        gtk.MESSAGE_QUESTION,
+        gtk.BUTTONS_OK_CANCEL,
+        None)
+
+    # set the text
+    dialog.set_markup("<span size='larger'><b>" + prompt + "</b></span>")
+    # add the secondary text
+    dialog.format_secondary_markup(secondary)
+
+    entry = gtk.Entry()
+    # allow the user to press enter instead of clicking OK
+    entry.connect("activate", dialog_response, dialog, gtk.RESPONSE_OK)
+
+    # create an hbox and pack the label and entry in
+    hbox = gtk.HBox()
+    hbox.pack_start(gtk.Label(textbox_label), False, 4, 4)
+    hbox.pack_start(entry)
+
+    # pack the boxes and show the dialog
+    dialog.vbox.pack_end(hbox, True, True, 0)
+    dialog.show_all()
+
+    if dialog.run() == gtk.RESPONSE_OK:
+        text = entry.get_text()
+        dialog.destroy()
+        return text
+    else:
+        dialog.destroy()
+        return None
 
 class SmallLabel(gtk.Label):
     def __init__(self, text=''):

wicd/netentry.py

@@ -21,7 +21,7 @@ import os
 import misc
 import wpath
 from misc import noneToString, stringToNone, noneToBlankString, to_bool
-from guiutil import error, SmallLabel, LabelEntry, GreyLabel, LeftAlignedLabel
+from guiutil import error, SmallLabel, LabelEntry, GreyLabel, LeftAlignedLabel, string_input
 
 language = misc.get_language_list_gui()
 
@@ -553,11 +553,19 @@ class WiredNetworkEntry(NetworkEntry):
         self.chkbox_default_profile = gtk.CheckButton(language['default_wired'])
                 
         # Build the profile list.
-        self.combo_profile_names = gtk.combo_box_entry_new_text()
+        self.combo_profile_names = gtk.combo_box_new_text() 
         self.profile_list = wired.GetWiredProfileList()
+        default_prof = wired.GetDefaultWiredNetwork()
         if self.profile_list:
-            for x in self.profile_list:
+            starting_index = 0
-                self.combo_profile_names.append_text(x)
+            for x, prof in enumerate(self.profile_list):
+                self.combo_profile_names.append_text(prof)
+                if default_prof == prof:
+                    starting_index = x
+            self.combo_profile_names.set_active(starting_index)
+        else:
+            print "no wired profiles found"
+            self.profile_help.show()
         
         # Format the profile help label.
         self.profile_help.set_justify(gtk.JUSTIFY_LEFT)
@@ -581,31 +589,17 @@ class WiredNetworkEntry(NetworkEntry):
                                             self.toggle_default_profile)
         self.combo_profile_names.connect("changed", self.change_profile)
 
+        # Show everything, but hide the profile help label.
+        self.show_all()
+        self.profile_help.hide()
+        self.advanced_dialog = WiredSettingsDialog(self.combo_profile_names.get_active_text())
+        
         # Toggle the default profile checkbox to the correct state.
         if to_bool(wired.GetWiredProperty("default")):
             self.chkbox_default_profile.set_active(True)
         else:
             self.chkbox_default_profile.set_active(False)
 
-        # Show everything, but hide the profile help label.
-        self.show_all()
-        self.profile_help.hide()
-        self.advanced_dialog = WiredSettingsDialog(self.combo_profile_names.get_active_text())
-        
-        # Display the default profile if it exists.
-        if self.profile_list is not None:
-            prof = wired.GetDefaultWiredNetwork()
-            if prof != None:  # Make sure the default profile gets displayed.
-                i = 0
-                while self.combo_profile_names.get_active_text() != prof:
-                    self.combo_profile_names.set_active(i)
-                    i += 1
-            else:
-                self.combo_profile_names.set_active(0)
-            print "wired profiles found"
-        else:
-            print "no wired profiles found"
-            self.profile_help.show()        
         self.check_enable()
         self.wireddis = self.connect("destroy", self.destroy_called)
         
@@ -642,21 +636,32 @@ class WiredNetworkEntry(NetworkEntry):
     def add_profile(self, widget):
         """ Add a profile to the profile list. """
         print "adding profile"
-        profile_name = self.combo_profile_names.get_active_text()
+
+        response = string_input("Enter a profile name", "The profile name " +
+                                  "will not be used by the computer. It " +
+                                  "allows you to " + 
+                                  "easily distinguish between different network " +
+                                  "profiles.", "Profile name:")
+
+        # if response is "" or None
+        if not response:
+            return
+
+        profile_name = response
         profile_list = wired.GetWiredProfileList()
         if profile_list:
             if profile_name in profile_list:
                 return False
-        if profile_name != "":
+
-            self.profile_help.hide()
+        self.profile_help.hide()
-            wired.CreateWiredNetworkProfile(profile_name, False)
+        wired.CreateWiredNetworkProfile(profile_name, False)
-            self.combo_profile_names.prepend_text(profile_name)
+        self.combo_profile_names.prepend_text(profile_name)
-            self.combo_profile_names.set_active(0)
+        self.combo_profile_names.set_active(0)
-            self.advanced_dialog.prof_name = profile_name
+        self.advanced_dialog.prof_name = profile_name
-            if self.is_full_gui:
+        if self.is_full_gui:
-                self.button_delete.set_sensitive(True)
+            self.button_delete.set_sensitive(True)
-                self.connect_button.set_sensitive(True)
+            self.connect_button.set_sensitive(True)
-                self.advanced_button.set_sensitive(True)
+            self.advanced_button.set_sensitive(True)
 
     def remove_profile(self, widget):
         """ Remove a profile from the profile list. """