1 files changed, 52 insertions, 33 deletions

diff --git a/src/gns/gnunet-gns-gtk_zone.c b/src/gns/gnunet-gns-gtk_zone.c
index 10d52dbb..39f6f6ef 100644
--- a/src/gns/gnunet-gns-gtk_zone.c
+++ b/src/gns/gnunet-gns-gtk_zone.c
@@ -26,6 +26,7 @@
 #include "gnunet_gtk.h"
 #include "gnunet-gns-gtk.h"
 
+#define NEW_NAME_STR "<new name>"
 #define NEW_RECORD_STR "<new record>"
 
 enum
@@ -40,7 +41,8 @@ enum
   TREE_COL_VAL_AS_STR,
   TREE_COL_VAL_COLOR,
   TREE_COL_NAME_IS_EDITABLE,
-  TREE_COL_IS_RECORD_ROW = 10
+  TREE_COL_IS_RECORD_ROW,
+  TREE_COL_NOT_DUMMY_ROW
 };
 
 enum
@@ -67,36 +69,48 @@ GNUNET_GNS_GTK_type_cellrenderercombo_changed_cb (GtkCellRendererCombo *combo,
 {
   struct GNUNET_GNS_Context *gns = user_data;
   GtkTreeIter it;
+  GtkTreeIter child;
   GtkTreeModel *tm = GTK_TREE_MODEL(gns->ts);
   guint type;
-//  int is_row;
+  int record_row;
   char *type_str;
   char *value_str;
+  char *name_str;
   void *data;
   size_t data_size;
 
+  gtk_tree_model_get(GTK_TREE_MODEL(gns->ls), new_iter, 0, &type, -1);
+  gtk_tree_model_get(GTK_TREE_MODEL(gns->ls), new_iter, TYPE_LIST_TYPENAME, &type_str, -1);
+
+
   /* check if this is a new record */
-  /*
-  gtk_tree_model_get(tm, &it, TREE_COL_IS_NAME_ROW, &is_row, -1);
+  gtk_tree_model_get_iter_from_string(tm, &it, path);
+  gtk_tree_model_get(tm, &it, TREE_COL_IS_RECORD_ROW, &record_row, -1);
+  gtk_tree_model_get(tm, &it, TREE_COL_NAME, &name_str, -1);
 
-  if (0 == is_row)
+  if (GNUNET_YES == record_row)
   {
-
+    /* Updating an existing record */
+    gtk_tree_store_set(gns->ts, &it,
+                       TREE_COL_RECORD_TYPE, type,
+                       TREE_COL_RECORD_TYPE_AS_STR, type_str,
+                       -1);
   }
-  else
+  else if ((NULL != name_str) && (0 != strcmp (NEW_NAME_STR, name_str)))
   {
+    /* Adding a new record */
+    GNUNET_break (0);
+    gtk_tree_store_insert_with_values(gns->ts, &child , &it, 0,
+        TREE_COL_RECORD_TYPE, type,
+        TREE_COL_RECORD_TYPE_AS_STR, type_str,
+        TREE_COL_IS_RECORD_ROW, GNUNET_YES,
+        TREE_COL_NOT_DUMMY_ROW, GNUNET_YES,
+        -1);
 
   }
-*/
-  gtk_tree_model_get(GTK_TREE_MODEL(gns->ls), new_iter, 0, &type, -1);
-  gtk_tree_model_get(GTK_TREE_MODEL(gns->ls), new_iter, TYPE_LIST_TYPENAME, &type_str, -1);
-
-  gtk_tree_model_get_iter_from_string(tm, &it, path);
-  gtk_tree_store_set(gns->ts, &it,
-                     TREE_COL_RECORD_TYPE, type,
-                     TREE_COL_RECORD_TYPE_AS_STR, type_str,
-                     -1);
   GNUNET_free (type_str);
+  GNUNET_free_non_null (name_str);
+
 
   /* check if value is still valid */
   gtk_tree_model_get(tm, &it, TREE_COL_VAL_AS_STR, &value_str, -1);
@@ -109,8 +123,18 @@ GNUNET_GNS_GTK_type_cellrenderercombo_changed_cb (GtkCellRendererCombo *combo,
       gtk_tree_store_set (gns->ts, &it, TREE_COL_VAL_COLOR, "red", -1);
     else
       gtk_tree_store_set (gns->ts, &it, TREE_COL_VAL_COLOR, NULL, -1);
+    GNUNET_free (value_str);
+  }
+  else if (NULL == value_str)
+  {
+    /* Empty value field */
+    GNUNET_break (0);
+    if (GNUNET_YES == record_row)
+      gtk_tree_store_set (gns->ts, &it, TREE_COL_VAL_COLOR, "red", -1);
+    else
+      gtk_tree_store_set (gns->ts, &child, TREE_COL_VAL_COLOR, "red", -1);
   }
-  GNUNET_free (value_str);
+
 }
 
 
@@ -222,32 +246,23 @@ GNUNET_GNS_GTK_name_cellrenderertext_edited_cb (GtkCellRendererText *renderer,
   GtkTreeModel *tm = GTK_TREE_MODEL(gns->ts);
 
   GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "New text for `%s' is `%s'\n", path, new_text);
-  if ((0 == strcmp (path, "0")) && (0 != strcmp (new_text, NEW_RECORD_STR)) )
+  if ((0 == strcmp (path, "0")) && (0 != strcmp (new_text, NEW_NAME_STR)) )
   {
     GNUNET_break (0);
     /* update name */
     gtk_tree_model_get_iter_from_string(tm, &it, path);
     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);
 
     /* add a new dummy line */
     gtk_tree_store_insert_with_values (gns->ts, &it,NULL, 0,
-        TREE_COL_NAME, _(NEW_RECORD_STR),
+        TREE_COL_NAME, _(NEW_NAME_STR),
         TREE_COL_RECORD_TYPE, 1,
         TREE_COL_NAME_IS_EDITABLE, 1,
-        //TREE_COL_IS_NAME_ROW, 1
+        TREE_COL_NOT_DUMMY_ROW, GNUNET_NO,
+        TREE_COL_IS_RECORD_ROW, GNUNET_NO,
         -1);
-#if 0
-    gtk_tree_store_append(gns->ts, &it, NULL);
-    gtk_tree_store_set(gns->ts, &it,
-                       TREE_COL_NAME, new_text,
-                       TREE_COL_RECORD_TYPE, 1,
-                       -1);
-    /* add a new name */
-    gtk_tree_store_append(gns->ts, &it, NULL);
-    gtk_tree_store_set(gns->ts, &it,
-                       TREE_COL_NAME, new_text,
-                       TREE_COL_RECORD_TYPE, 1, -1);
-#endif
   }
   else
   {
@@ -323,7 +338,9 @@ void zone_iteration_proc (void *cls,
   gtk_tree_store_set(zc_ctx->gns->ts, &iter_name,
                      TREE_COL_NAME, name,
                      TREE_COL_NAME_IS_EDITABLE, 1,
+                     TREE_COL_RECORD_TYPE_AS_STR, _(NEW_RECORD_STR),
                      TREE_COL_IS_RECORD_ROW, GNUNET_NO,
+                     TREE_COL_NOT_DUMMY_ROW, GNUNET_YES,
                      -1);
 
 
@@ -363,6 +380,7 @@ void zone_iteration_proc (void *cls,
                                        TREE_COL_EXP_TIME_IS_REL, time_is_relative,
                                        TREE_COL_VAL_AS_STR, val,
                                        TREE_COL_IS_RECORD_ROW, GNUNET_YES,
+                                       TREE_COL_NOT_DUMMY_ROW, GNUNET_YES,
                                        -1);
 
     GNUNET_free (exp);
@@ -387,10 +405,11 @@ GNUNET_GNS_GTK_main_treeview_realize_cb (GtkWidget *widget,
 
   /* Append a top level row and leave it empty */
   gtk_tree_store_insert_with_values(gns->ts, &toplevel, NULL, 0,
-                                     TREE_COL_NAME, _(NEW_RECORD_STR),
+                                     TREE_COL_NAME, _(NEW_NAME_STR),
                                      TREE_COL_RECORD_TYPE, 1,
                                      TREE_COL_NAME_IS_EDITABLE, 1,
                                      TREE_COL_IS_RECORD_ROW, GNUNET_NO,
+                                     TREE_COL_NOT_DUMMY_ROW, GNUNET_NO,
                                      -1);
 
   zc_ctx = GNUNET_malloc (sizeof (struct ZoneIteration_Context));