[gnome-disk-utility/udisks2-port] Don't show "Configured: No" if we don't know how to configure the device
- From: David Zeuthen <davidz src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-disk-utility/udisks2-port] Don't show "Configured: No" if we don't know how to configure the device
- Date: Thu, 11 Aug 2011 22:19:51 +0000 (UTC)
commit 11e8facace7d0b56d5e6be581feb49bedd1dec9b
Author: David Zeuthen <davidz redhat com>
Date: Thu Aug 11 18:19:21 2011 -0400
Don't show "Configured: No" if we don't know how to configure the device
E.g. LVM2
Signed-off-by: David Zeuthen <davidz redhat com>
src/palimpsest/gduwindow.c | 40 +++++++++++++++++++++++++++++++---------
1 files changed, 31 insertions(+), 9 deletions(-)
---
diff --git a/src/palimpsest/gduwindow.c b/src/palimpsest/gduwindow.c
index 3df0422..9b734ff 100644
--- a/src/palimpsest/gduwindow.c
+++ b/src/palimpsest/gduwindow.c
@@ -1507,6 +1507,9 @@ calculate_configuration_for_display (UDisksBlockDevice *block)
const gchar *config_type;
gboolean mentioned_fstab = FALSE;
gboolean mentioned_crypttab = FALSE;
+ gchar *ret;
+
+ ret = NULL;
/* TODO: could include more details such as whether the
* device is activated at boot time
@@ -1552,11 +1555,27 @@ calculate_configuration_for_display (UDisksBlockDevice *block)
if (str->len == 0)
{
- /* Translators: Shown when the device is not configured */
- g_string_append (str, _("No"));
+ /* No known configuration... show "No" only if we actually
+ * know how to configure the device.
+ */
+ if (g_strcmp0 (udisks_block_device_get_id_usage (block), "filesystem") == 0 ||
+ (g_strcmp0 (udisks_block_device_get_id_usage (block), "other") == 0 &&
+ g_strcmp0 (udisks_block_device_get_id_type (block), "swap") == 0) ||
+ g_strcmp0 (udisks_block_device_get_id_usage (block), "crypto") == 0)
+ {
+ /* Translators: Shown when the device is not configured */
+ g_string_append (str, _("No"));
+ }
+ else
+ {
+ g_string_free (str, TRUE);
+ goto out;
+ }
}
- return g_string_free (str, FALSE);
+ ret = g_string_free (str, FALSE);
+ out:
+ return ret;
}
static gboolean
@@ -1782,12 +1801,15 @@ update_device_page_for_block (GduWindow *window,
}
configuration_for_display = calculate_configuration_for_display (block);
- set_markup (window,
- "devtab-volume-configured-label",
- "devtab-volume-configured-value-label",
- configuration_for_display,
- SET_MARKUP_FLAGS_HYPHEN_IF_EMPTY);
- g_free (configuration_for_display);
+ if (configuration_for_display != NULL)
+ {
+ set_markup (window,
+ "devtab-volume-configured-label",
+ "devtab-volume-configured-value-label",
+ configuration_for_display,
+ SET_MARKUP_FLAGS_HYPHEN_IF_EMPTY);
+ g_free (configuration_for_display);
+ }
}
static void
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]