1 files changed, 110 insertions, 14 deletions

diff --git a/src/gns/gnunet-gns-gtk_zone.c b/src/gns/gnunet-gns-gtk_zone.c
index 0d6fff3a..890ef80d 100644
--- a/src/gns/gnunet-gns-gtk_zone.c
+++ b/src/gns/gnunet-gns-gtk_zone.c
@@ -54,9 +54,101 @@ enum LIST_COLUMNS
 
 
 void
-check_record_validity ()
+check_name_validity_and_commit (struct GNUNET_GNS_Context *gns, gchar *path)
 {
-  GNUNET_break (0);
+  GtkTreeIter it;
+  GtkTreeIter parent;
+  int children;
+  int c;
+  int valid = GNUNET_YES;
+
+  char          *n_name;
+  int           n_type;
+  gboolean      n_public;
+  char          *n_exp_color;
+  guint64       n_exp_time;
+  char          *n_exp_str;
+  gboolean      n_is_relative;
+  char          *n_value;
+  char          *n_value_color;
+
+
+  gtk_tree_model_get_iter_from_string(gns->tm, &it, path);
+
+  if (TRUE == gtk_tree_model_iter_parent (gns->tm, &parent, &it))
+  {
+    GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "Has parent\n");
+  }
+  else
+  {
+    GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "No parent\n");
+    parent = it;
+  }
+
+  children = gtk_tree_model_iter_n_children (gns->tm, &parent);
+  if (children < 1)
+  {
+    GNUNET_break (0);
+    return;
+  }
+
+  struct GNUNET_NAMESTORE_RecordData rd[children];
+
+  if (FALSE == gtk_tree_model_iter_children (gns->tm, &it, &parent))
+  {
+    GNUNET_break (0);
+    return;
+  }
+  for (c = 0; c < children; c++)
+  {
+    gtk_tree_model_get(gns->tm, &it,
+                       TREE_COL_NAME, &n_name,
+                       TREE_COL_RECORD_TYPE, &n_type,
+                       TREE_COL_IS_PUBLIC, &n_public,
+                       TREE_COL_EXP_TIME_COLOR, &n_exp_color,
+                       TREE_COL_EXP_TIME, &n_exp_time,
+                       TREE_COL_EXP_TIME_IS_REL, &n_is_relative,
+                       TREE_COL_EXP_TIME_AS_STR, &n_exp_str,
+                       TREE_COL_VAL_AS_STR, &n_value,
+                       TREE_COL_VAL_COLOR, &n_value_color,
+                       -1);
+
+    /* valid name */
+    if (NULL == n_name)
+        valid = GNUNET_NO;
+
+    /* valid record type */
+    if (0 == n_type)
+      valid = GNUNET_NO;
+
+    /* valid expiration */
+    if ((n_exp_color != NULL) || (NULL == n_exp_str) || (0 == n_exp_time))
+        valid = GNUNET_NO;
+
+    /* valid value */
+    if ((n_value_color != NULL) || (NULL == n_value))
+        valid = GNUNET_NO;
+
+    GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "Child [%u] is %s\n", c, (GNUNET_NO == valid) ? "INVALID" : "VALID");
+
+    g_free (n_name);
+    g_free (n_exp_color);
+    g_free (n_exp_str);
+    g_free (n_value);
+    g_free (n_value_color);
+
+    if ((GNUNET_NO == valid) || (FALSE == gtk_tree_model_iter_next (gns->tm, &it)))
+      break;
+  }
+
+  if (GNUNET_NO == valid)
+  {
+    GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "Not valid\n");
+  }
+  else
+  {
+    GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "Valid\n");
+  }
 }
 
 
@@ -147,8 +239,7 @@ GNUNET_GNS_GTK_type_cellrenderercombo_changed_cb (GtkCellRendererCombo *combo,
       gtk_tree_store_set (gns->ts, &child, TREE_COL_VAL_COLOR, "red", -1);
   }
 
-  check_record_validity ();
-
+  check_name_validity_and_commit (gns, path);
 }
 
 
@@ -168,13 +259,13 @@ GNUNET_GNS_GTK_ispublic_cellrenderertoggle_toggled_cb (GtkCellRendererToggle *ce
   struct GNUNET_GNS_Context *gns = user_data;
   GtkTreeIter it;
   GtkTreeModel *tm = GTK_TREE_MODEL(gns->ts);
-  int value;
+  gboolean value;
 
   gtk_tree_model_get_iter_from_string(tm, &it, path);
   gtk_tree_model_get(tm, &it, TREE_COL_IS_PUBLIC, &value, -1);
   gtk_tree_store_set(gns->ts, &it, TREE_COL_IS_PUBLIC, !value, -1);
 
-  GNUNET_break (0); // FIXME, not implemented
+  check_name_validity_and_commit (gns, path);
 }
 
 
@@ -243,8 +334,8 @@ GNUNET_GNS_GTK_expiration_cellrenderertext_edited_cb (GtkCellRendererText *rende
                         TREE_COL_EXP_TIME, abstime.abs_value,
                         TREE_COL_EXP_TIME_COLOR, NULL,
                         -1);
+    check_name_validity_and_commit (gns, path);
   }
-  check_record_validity ();
 }
 
 
@@ -282,14 +373,13 @@ GNUNET_GNS_GTK_value_cellrenderertext_edited_cb (GtkCellRendererText *renderer,
     {
       gtk_tree_store_set (gns->ts, &it, TREE_COL_VAL_COLOR, NULL, -1);
       gtk_tree_store_set (gns->ts, &it, TREE_COL_VAL_AS_STR, new_text, -1);
+      check_name_validity_and_commit (gns, path);
     }
     else
     {
       gtk_tree_store_set (gns->ts, &it, TREE_COL_VAL_COLOR, "red", -1);
       gtk_tree_store_set (gns->ts, &it, TREE_COL_VAL_AS_STR, new_text, -1);
     }
-
-    check_record_validity ();
   }
 }
 
@@ -317,22 +407,25 @@ GNUNET_GNS_GTK_name_cellrenderertext_edited_cb (GtkCellRendererText *renderer,
   GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "New text for `%s' is `%s'\n", path, new_text);
   if ((0 == strcmp (new_text, NEW_NAME_STR)) || (0 == strcmp (new_text, "")))
     return;
+
   gtk_tree_model_get_iter_from_string(tm, &it, path);
   gtk_tree_model_get(tm, &it, TREE_COL_NOT_DUMMY_ROW, &not_dummy, -1);
 
   if (not_dummy == GNUNET_NO)
   {
     /* update name */
-    gtk_tree_store_set (gns->ts, &it, TREE_COL_NAME, new_text, -1);
-    gtk_tree_store_set (gns->ts, &it, TREE_COL_RECORD_TYPE_AS_STR, _(NEW_RECORD_STR), -1);
-    gtk_tree_store_set (gns->ts, &it, TREE_COL_NOT_DUMMY_ROW, GNUNET_YES, -1);
+    gtk_tree_store_set (gns->ts, &it,
+                        TREE_COL_NAME, new_text,
+                        TREE_COL_RECORD_TYPE, 0,
+                        TREE_COL_RECORD_TYPE_AS_STR, _(NEW_RECORD_STR),
+                        TREE_COL_NOT_DUMMY_ROW, GNUNET_YES,
+                        -1);
 
     /* add a new dummy line */
     gtk_tree_store_insert_with_values (gns->ts, &it,NULL, 0,
         TREE_COL_NAME, _(NEW_NAME_STR),
         TREE_COL_NAME_IS_VISIBLE, TRUE,
         TREE_COL_RECORD_TYPE, 1,
-        TREE_COL_NAME_IS_VISIBLE, 1,
         TREE_COL_NOT_DUMMY_ROW, GNUNET_NO,
         TREE_COL_IS_RECORD_ROW, GNUNET_NO,
         -1);
@@ -342,6 +435,7 @@ GNUNET_GNS_GTK_name_cellrenderertext_edited_cb (GtkCellRendererText *renderer,
     /* update name */
     gtk_tree_store_set (gns->ts, &it, TREE_COL_NAME, new_text, -1);
   }
+  check_name_validity_and_commit (gns, path);
 }
 
 /**
@@ -600,9 +694,10 @@ void GNUNET_GNS_GTK_delete_popup_menu_delete_cb (GtkMenuItem *menuitem,
   GtkTreeSelection * ts = gtk_tree_view_get_selection(tv);
 
   gtk_tree_selection_get_selected (ts, &tm, &it);
+
   gtk_tree_model_get(tm, &it, TREE_COL_NOT_DUMMY_ROW, &not_dummy, -1);
   if (GNUNET_NO == not_dummy)
-    return;
+    return; /* do not delete the dummy line */
 
   /* Has parent? */
   has_parent = gtk_tree_model_iter_parent (tm, &parent, &it);
@@ -712,6 +807,7 @@ void zone_iteration_proc (void *cls,
   gtk_tree_store_set(zc_ctx->gns->ts, &iter_name,
                      TREE_COL_NAME, name,
                      TREE_COL_NAME_IS_VISIBLE, TRUE,
+                     TREE_COL_RECORD_TYPE, 0,
                      TREE_COL_RECORD_TYPE_AS_STR, _(NEW_RECORD_STR),
                      TREE_COL_IS_RECORD_ROW, GNUNET_NO,
                      TREE_COL_NOT_DUMMY_ROW, GNUNET_YES,