[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [PATCH] Add DCB option to text mode FCoE setup (#513011)



On Thu, Nov 26, 2009 at 6:20 AM, Hans de Goede <hdegoede redhat com> wrote:
> @@ -232,7 +233,15 @@ class PartitionTypeWindow:
> -        self.anaconda.id.storage.fcoe.addSan(nic)
> +        dcb = entries[1].strip().lower()
> +        if dcb != _("yes").lower() and dcb != _("no").lower():

Expecting lower to work like this breaks for some languages (eg,
Icelandic).  If this is an entry which should be either yes or no, we
really should have it be as a radio button/checkbox rather than an
entry

- Jeremy


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]