Merge lp:~chipaca/ubuntuone-client/nautilus-ppage into lp:ubuntuone-client

[John Lenton]

these are not the property pages you are looking for.

[Rodrigo Moya (rodrigo-moya)]

yay! But a few things:

ubuntuone-nautilus.c is too crowded, so could you move the property page creation/code to its own file, like location-widget.c does?

If the data is not editable, why not just use a label? See the 'Audio' property page for MP3's, it has sections (in bold) and then the different data in labels, with nice italic strings for the "field" names. Something like that would look much better.

Also, the 'Ubuntu One' page shows for every folder/file, whether it's a UDF/inside a UDF or not.

review: Needs Fixing

Unmerged revisions

627. By John Lenton on 2010-08-11

second half of the ppage work

626. By John Lenton on 2010-08-11

whee, a page provider

=== modified file 'nautilus/ubuntuone-nautilus.c'
--- nautilus/ubuntuone-nautilus.c	2010-08-06 23:31:50 +0000
+++ nautilus/ubuntuone-nautilus.c	2010-08-11 07:07:44 +0000
@@ -31,6 +31,7 @@
 #include <libnautilus-extension/nautilus-info-provider.h>
 #include <libnautilus-extension/nautilus-menu-provider.h>
 #include <libnautilus-extension/nautilus-location-widget-provider.h>
+#include <libnautilus-extension/nautilus-property-page-provider.h>
 #include <libsyncdaemon/libsyncdaemon.h>
 #include "ubuntuone-nautilus.h"
 #include "location-widget.h"
@@ -585,6 +586,132 @@
   iface->get_background_items = ubuntuone_nautilus_get_bg_menu_items;
 }
 
+static void ubuntuone_nautilus_put_label_and_value (GtkWidget *box, const char *label, const char *value) {
+  GtkWidget *widget;
+
+  widget = gtk_label_new (g_markup_printf_escaped ("<b>%s:</b>", label));
+  gtk_label_set_use_markup (GTK_LABEL (widget), TRUE);
+  gtk_misc_set_alignment (GTK_MISC (widget), 0., 0.5);
+  gtk_box_pack_start (GTK_BOX (box), widget, FALSE, FALSE, 2);
+
+  widget = gtk_entry_new ();
+  gtk_entry_set_text (GTK_ENTRY (widget), value);
+  gtk_entry_set_editable (GTK_ENTRY (widget), FALSE);
+  gtk_misc_set_alignment (GTK_MISC (widget), 0.5, 0.5);
+  gtk_box_pack_start (GTK_BOX (box), widget, FALSE, FALSE, 0);
+}
+
+typedef struct _PPdata {
+  UbuntuOneNautilus *uon;
+  GtkWidget *props;
+  GtkWidget *spinner;
+  gchar *path;
+} _PPdata;
+
+static void ubuntuone_nautilus_got_metainfo_for_ppage (SyncdaemonFilesystemInterface *interface,
+                                                       gboolean success,
+                                                       SyncdaemonMetadata *metadata,
+                                                       gpointer user_data) {
+  _PPdata *data = (_PPdata *) user_data;
+  gchar *public_url;
+
+  /* kill the spinner */
+  gtk_spinner_stop (GTK_SPINNER (data->spinner));
+  gtk_widget_destroy (data->spinner);
+
+  if (success) {
+    if (data->uon && data->uon->public) {
+      public_url = g_hash_table_lookup (data->uon->public, data->path);
+      if (!public_url)
+        public_url = _("(not published)");
+    } else {
+      public_url = _("(information not yet available)");
+    }
+
+    gtk_box_pack_start (GTK_BOX (data->props), gtk_label_new (_("Ubuntu One data->props")), FALSE, FALSE, 4);
+    gtk_box_pack_start (GTK_BOX (data->props), gtk_hseparator_new (), FALSE, FALSE, 0);
+
+    ubuntuone_nautilus_put_label_and_value (data->props, _("Public URL"), public_url);
+    ubuntuone_nautilus_put_label_and_value (data->props, _("Currently synchronized"), syncdaemon_metadata_get_is_synced (metadata) ? _("Yes") : _("No"));
+
+    gtk_box_pack_start (GTK_BOX (data->props), gtk_hseparator_new (), FALSE, FALSE, 0);
+
+    ubuntuone_nautilus_put_label_and_value (data->props, _("Local hash"), syncdaemon_metadata_get_local_hash (metadata));
+    ubuntuone_nautilus_put_label_and_value (data->props, _("Server hash"), syncdaemon_metadata_get_server_hash (metadata));
+    ubuntuone_nautilus_put_label_and_value (data->props, _("Share ID"), syncdaemon_metadata_get_share_id (metadata));
+    ubuntuone_nautilus_put_label_and_value (data->props, _("Node ID"), syncdaemon_metadata_get_node_id (metadata));
+  } else {
+    gtk_box_pack_start (GTK_BOX (data->props), gtk_label_new (_("No information available")), TRUE, FALSE, 0);
+  }
+
+  g_free (data->path);
+  g_free (data);
+  gtk_widget_show_all (data->props);
+}
+
+static GList * ubuntuone_nautilus_get_ppages (NautilusPropertyPageProvider *provider, GList *files) {
+  GList *pages;
+  NautilusPropertyPage *page;
+  NautilusFileInfo *file;
+  SyncdaemonFilesystemInterface *interface;
+  UbuntuOneNautilus * uon;
+  gchar *uri, *filename;
+  _PPdata *data;
+
+  if (!files)
+    return NULL;
+
+  /* for now, only support single files */
+  if (files->next)
+    return NULL;
+
+  file = NAUTILUS_FILE_INFO (files->data);
+
+  uon = UBUNTUONE_NAUTILUS(provider);
+  interface = (SyncdaemonFilesystemInterface *) syncdaemon_daemon_get_filesystem_interface (uon->syncdaemon);
+  if (!interface)
+    return NULL;
+
+  uri = nautilus_file_info_get_uri (file);
+  if (!uri) {
+    g_warning ("no URI in file?");
+    return NULL;
+  }
+  filename = g_filename_from_uri (uri, NULL, NULL);
+  if (!filename) {
+    g_warning ("no filename from URI?");
+    return NULL;
+  }
+  g_free (uri);
+
+  /* OK, it looks like we're doing this thing */
+  data = g_new0 (struct _PPdata, 1);
+  data->path = filename;
+  data->props = gtk_vbox_new (FALSE, 8);
+  data->spinner = gtk_spinner_new ();
+  data->uon = uon;
+  gtk_spinner_start (GTK_SPINNER (data->spinner));
+  gtk_box_pack_start (GTK_BOX (data->props), data->spinner, TRUE, FALSE, 0);
+  gtk_widget_show_all (data->props);
+
+  page = nautilus_property_page_new ("ubuntuone-ppage",
+                                     gtk_label_new("Ubuntu One"),
+                                     data->props);
+
+  syncdaemon_filesystem_interface_get_metadata_async (interface,
+                                                      filename,
+                                                      nautilus_file_info_is_directory (file),
+                                                      (SyncdaemonGotMetadataFunc) ubuntuone_nautilus_got_metainfo_for_ppage,
+                                                      data);
+
+  return g_list_append (NULL, page);
+}
+
+static void
+ubuntuone_nautilus_ppage_provider_iface_init (NautilusPropertyPageProviderIface *iface) {
+  iface->get_pages = ubuntuone_nautilus_get_ppages;
+}
+
 /* GType and nautilus module stuff */
 static GType un_type = 0;
 
@@ -722,6 +849,12 @@
     NULL
   };
 
+  static const GInterfaceInfo ppage_provider_iface_info = {
+    (GInterfaceInitFunc) ubuntuone_nautilus_ppage_provider_iface_init,
+    NULL,
+    NULL
+  };
+
   un_type = g_type_module_register_type (module, 
 					 G_TYPE_OBJECT,
 					 "UbuntuOneNautilus",
@@ -742,6 +875,11 @@
 			       un_type,
 			       NAUTILUS_TYPE_MENU_PROVIDER,
 			       &menu_provider_iface_info);
+
+  g_type_module_add_interface (module,
+                               un_type,
+                               NAUTILUS_TYPE_PROPERTY_PAGE_PROVIDER,
+                               &ppage_provider_iface_info);
 }