-another step towards custom edit dialogs for records

1 files changed, 125 insertions, 21 deletions

diff --git a/src/setup/gnunet-setup-gns.c b/src/setup/gnunet-setup-gns.c
index d359796b..ebd87389 100644
--- a/src/setup/gnunet-setup-gns.c
+++ b/src/setup/gnunet-setup-gns.c
@@ -25,6 +25,7 @@
  */
 #include "gnunet_gtk.h"
 #include "gnunet-setup-gns.h"
+#include "gnunet-setup-gns-edit.h"
 #include <gnunet/gnunet_namestore_service.h>
 #include <gnunet/gnunet_dnsparser_lib.h>
 
@@ -142,12 +143,16 @@ enum GNSTreestoreColumn
    */
   GNS_TREESTORE_COL_NAME_COLOR,
 
-
   /**
    * A gboolean; TRUE if the 'type' column can still be changed;
    * FALSE once we have edited the value.
    */
-  GNS_TREESTORE_COL_TYPE_IS_EDITABLE
+  GNS_TREESTORE_COL_TYPE_IS_EDITABLE,
+
+  /**
+   * A gboolean; TRUE if the value is a shadow record.
+   */
+  GNS_TREESTORE_COL_IS_SHADOW
 };
 
 
@@ -1021,7 +1026,7 @@ update_treemodel_after_remove (void *cls,
  * the namestore).   If the given path identifies an entire 'name',
  * remove all records under that name.
  * 
- * @param path identifying record(s) to remove
+ * @param path identifying record(s) to remove; FIXME: consider doing this with an 'iter' instead
  */
 static void
 remove_records_by_path (const gchar *path)
@@ -1089,9 +1094,89 @@ remove_records_by_path (const gchar *path)
 
 
 /**
+ * The edit dialog completed; update the namestore and the 
+ * view based on the new values in 'edc'.
+ *
+ * @param edc editing context information
+ * @param ret return code of the dialog
+ */
+static void
+edit_dialog_continuation (struct EditDialogContext *edc,
+                          GtkResponseType ret)
+{
+  char *path;
+
+  /* FIXME: logic to move records between zones is missing here!
+     (#2473) */
+  switch (ret)
+  {
+  case GTK_RESPONSE_REJECT: /* code for 'delete' */
+    if (GNUNET_YES == edc->old_record_in_namestore)
+    {
+      /* remove item from tree view and namestore */
+      path = gtk_tree_model_get_string_from_iter (tm, &edc->it);
+      remove_records_by_path (path);
+      g_free (path);  
+    }
+    else
+    {
+      /* invalid choice, 'delete' should have been hidden... */
+      GNUNET_break (0);
+      gtk_tree_store_remove (ts, &edc->it);
+    }
+    break;
+  case GTK_RESPONSE_CANCEL:
+  case GTK_RESPONSE_DELETE_EVENT: /* window deletion counts as 'cancel' */
+    if (GNUNET_NO == edc->old_record_in_namestore)
+    {
+      /* re-enable type selection */
+      gtk_tree_store_set (ts, &edc->it,
+                          GNS_TREESTORE_COL_TYPE_IS_EDITABLE, TRUE,
+                          GNS_TREESTORE_COL_VAL_COLOR, "red",
+                          -1);
+    }
+    else
+    {
+      /* do nothing */    
+    }
+    break;
+  case GTK_RESPONSE_OK:
+    /* update model */
+    gtk_tree_store_set (ts, &edc->it,
+                        GNS_TREESTORE_COL_NAME, edc->n_name,
+                        GNS_TREESTORE_COL_IS_PUBLIC, edc->n_public,
+                        GNS_TREESTORE_COL_EXP_TIME, edc->n_exp_time,
+                        GNS_TREESTORE_COL_EXP_TIME_IS_REL, edc->n_is_relative,
+                        GNS_TREESTORE_COL_IS_SHADOW, edc->n_is_shadow,
+                        GNS_TREESTORE_COL_VAL_AS_STR, edc->n_value,
+                        GNS_TREESTORE_COL_VAL_COLOR, NULL,
+                        -1);    
+    if (GNUNET_YES == edc->old_record_in_namestore)
+    {
+      /* replace record in database with that from model */
+      check_name_validity_and_commit (&edc->it, edc->n_name);
+    }
+    else
+    {
+      /* add record in database based on model */
+      check_name_validity_and_commit (&edc->it, NULL);
+    }
+    break;
+  default:
+    GNUNET_break (0);
+    break;
+  }
+  g_free (edc->n_name);
+  g_free (edc->n_new_name);
+  g_free (edc->n_value);
+  GNUNET_free (edc);
+}
+
+
+/**
  * Edit the record at the currently selected row.  If the old record
  * exists, allow the user to modify or delete it; if it does not
- * exist, remove it _only_ from the model (by offering the user the
+ * exist, make the record type editable again (by offering the user the
  * 'cancel' option; hide 'delete' in this case).
  *
  * @param old_record_in_namestore GNUNET_YES if the old record exists in the namestore,
@@ -1101,33 +1186,52 @@ static void
 edit_selected_row (int old_record_in_namestore)
 {
   GtkTreeSelection *sel;
-  GtkTreeIter it;
-  gchar *n_name;
   gint n_type;
-  gboolean n_public;
-  guint64 n_exp_time;
-  gboolean n_is_relative;
-  gchar *n_value;
+  struct EditDialogContext *edc;
 
   sel = gtk_tree_view_get_selection (tv);
-  if (! gtk_tree_selection_get_selected (sel, NULL, &it))
+  edc = GNUNET_malloc (sizeof (struct EditDialogContext));
+  if (! gtk_tree_selection_get_selected (sel, NULL, &edc->it))
   {
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
                 "No row selected\n");
+    GNUNET_free (edc);
     return;
   }
-  gtk_tree_model_get (tm, &it,
-                      GNS_TREESTORE_COL_NAME, &n_name,
+  gtk_tree_model_get (tm, &edc->it,
+                      GNS_TREESTORE_COL_NAME, &edc->n_name,
                       GNS_TREESTORE_COL_RECORD_TYPE, &n_type,
-                      GNS_TREESTORE_COL_IS_PUBLIC, &n_public,
-                      GNS_TREESTORE_COL_EXP_TIME, &n_exp_time,
-                      GNS_TREESTORE_COL_EXP_TIME_IS_REL, &n_is_relative,
-                      GNS_TREESTORE_COL_VAL_AS_STR, &n_value,
+                      GNS_TREESTORE_COL_IS_PUBLIC, &edc->n_public,
+                      GNS_TREESTORE_COL_EXP_TIME, &edc->n_exp_time,
+                      GNS_TREESTORE_COL_EXP_TIME_IS_REL, &edc->n_is_relative,
+                      GNS_TREESTORE_COL_IS_SHADOW, &edc->n_is_shadow,
+                      GNS_TREESTORE_COL_VAL_AS_STR, &edc->n_value,
                       -1);
-  // FIXME - implement...
-  GNUNET_break (0);
-  g_free (n_name);
-  g_free (n_value);
+  edc->old_record_in_namestore = old_record_in_namestore;
+  edc->new_zone_option = current_zone_option;
+  edc->n_new_name = g_strdup (edc->n_name);
+  edc->cont = &edit_dialog_continuation;
+  switch (n_type)
+  {
+  case GNUNET_DNSPARSER_TYPE_A:
+    GNS_edit_dialog_a (edc);
+    break;
+  case GNUNET_DNSPARSER_TYPE_NS:
+  case GNUNET_DNSPARSER_TYPE_CNAME:
+  case GNUNET_DNSPARSER_TYPE_SOA:
+  case GNUNET_DNSPARSER_TYPE_PTR:
+  case GNUNET_DNSPARSER_TYPE_MX:
+  case GNUNET_DNSPARSER_TYPE_TXT:
+  case GNUNET_DNSPARSER_TYPE_AAAA:
+  case GNUNET_DNSPARSER_TYPE_SRV:
+  case GNUNET_DNSPARSER_TYPE_TLSA:
+  case GNUNET_NAMESTORE_TYPE_PKEY:
+  case GNUNET_NAMESTORE_TYPE_LEHO:
+  case GNUNET_NAMESTORE_TYPE_VPN:
+    GNUNET_break (0);   /* FIXME - implement (#2465) */
+    edc->cont (edc, GTK_RESPONSE_CANCEL);  /* treat as 'cancel' */
+    break;
+  }
 }