Implemented Confi::path_set_value in db plugin.

diff --git a/plugins/db/plgdb.c b/plugins/db/plgdb.c
index 26725cdaf43fa24b810141a0ad0fd877bfd7d769..57142f1edb5da4c040b28d2a848e5f67e58f0901 100644 (file)
--- a/plugins/db/plgdb.c
+++ b/plugins/db/plgdb.c
@@ -400,6 +400,42 @@ static gchar
        return ret;
 }
 
+gboolean
+confi_db_plugin_path_set_value (ConfiPluggable *pluggable, const gchar *path, const gchar *value)
+{
+       GdaDataModel *dm;
+       gchar *sql;
+       gboolean ret;
+
+       dm = confi_db_plugin_path_get_data_model (pluggable, confi_path_normalize (pluggable, path));
+
+       ConfiDBPluginPrivate *priv = CONFI_DB_PLUGIN_GET_PRIVATE (pluggable);
+
+       ret = FALSE;
+       if (dm != NULL && gda_data_model_get_n_rows (dm) > 0)
+               {
+                       sql = g_strdup_printf ("UPDATE %cvalues%c SET value = '%s' "
+                                                     "WHERE id_configs = %d "
+                                                     "AND id = %d ",
+                                                     priv->chrquot, priv->chrquot,
+                                                     gdaex_strescape (value, NULL),
+                                                     priv->id_config,
+                                                     gdaex_data_model_get_field_value_integer_at (dm, 0, "id"));
+                       ret = (gdaex_execute (priv->gdaex, sql) >= 0);
+                       g_free (sql);
+               }
+       else
+               {
+                       g_warning ("Path %s doesn't exists.", path);
+               }
+       if (dm != NULL)
+               {
+                       g_object_unref (dm);
+               }
+
+       return ret;
+}
+
 static void
 confi_db_plugin_class_init (ConfiDBPluginClass *klass)
 {
@@ -423,6 +459,7 @@ confi_pluggable_iface_init (ConfiPluggableInterface *iface)
        iface->initialize = confi_db_plugin_initialize;
        iface->get_configs_list = confi_db_plugin_get_configs_list;
        iface->path_get_value = confi_db_plugin_path_get_value;
+       iface->path_set_value = confi_db_plugin_path_set_value;
 }
 
 static void

diff --git a/src/confipluggable.c b/src/confipluggable.c
index bf455e93395bd532691899767050896ea6c82b43..e43b06a7f50b5b6d0e3ef2fb776d88c93e0ab696 100644 (file)
--- a/src/confipluggable.c
+++ b/src/confipluggable.c
@@ -152,3 +152,22 @@ gchar
        return iface->path_get_value (pluggable, path);
 }
 
+/**
+ * confi_pluggable_path_set_value:
+ * @pluggable: a #ConfiPluggable object.
+ * @path: the key's path.
+ * @value: the value to set.
+ *
+ */
+gboolean
+confi_pluggable_path_set_value (ConfiPluggable *pluggable, const gchar *path, const gchar *value)
+{
+       ConfiPluggableInterface *iface;
+
+       g_return_val_if_fail (CONFI_IS_PLUGGABLE (pluggable), FALSE);
+
+       iface = CONFI_PLUGGABLE_GET_IFACE (pluggable);
+       g_return_val_if_fail (iface->path_set_value != NULL, FALSE);
+
+       return iface->path_set_value (pluggable, path, value);
+}

diff --git a/src/confipluggable.h b/src/confipluggable.h
index ac52cb0f515ed9f47c623c03a25b19a6ad799acb..71a729136867d97c35964b7d61515d0373ac55d4 100644 (file)
--- a/src/confipluggable.h
+++ b/src/confipluggable.h
@@ -58,6 +58,9 @@ struct _ConfiPluggableInterface {
        GList *(*get_configs_list) (ConfiPluggable *pluggable,
                                    const gchar *filter);
        gchar *(*path_get_value) (ConfiPluggable *pluggable, const gchar *path);
+       gboolean (*path_set_value) (ConfiPluggable *pluggable,
+                                      const gchar *path,
+                                      const gchar *value);
 };
 
 /*
@@ -70,6 +73,9 @@ gboolean confi_pluggable_initialize (ConfiPluggable *pluggable, const gchar *cnc
 GList *confi_pluggable_get_configs_list (ConfiPluggable *pluggable,
                                          const gchar *filter);
 gchar *confi_pluggable_path_get_value (ConfiPluggable *pluggable, const gchar *path);
+gboolean confi_pluggable_path_set_value (ConfiPluggable *pluggable,
+                               const gchar *path,
+                               const gchar *value);
 
 
 G_END_DECLS

diff --git a/src/libconfi.c b/src/libconfi.c
index 1a7edbb221f814eb7a61c2d98d8d3a849222a38e..6eff72947e8ece5c51f36a5736c20a1e8ea254cb 100644 (file)
--- a/src/libconfi.c
+++ b/src/libconfi.c
@@ -615,34 +615,18 @@ gchar
 gboolean
 confi_path_set_value (Confi *confi, const gchar *path, const gchar *value)
 {
-       GdaDataModel *dm;
-       gchar *sql;
        gboolean ret;
 
-       //dm = path_get_data_model (confi, path_normalize (confi, path));
-
        ConfiPrivate *priv = CONFI_GET_PRIVATE (confi);
 
-       ret = FALSE;
-       if (dm != NULL && gda_data_model_get_n_rows (dm) > 0)
+       if (priv->pluggable == NULL)
                {
-                       sql = g_strdup_printf ("UPDATE %cvalues%c SET value = '%s' "
-                                                     "WHERE id_configs = %d "
-                                                     "AND id = %d ",
-                                                     priv->chrquot, priv->chrquot,
-                                                     gdaex_strescape (value, NULL),
-                                                     priv->id_config,
-                                                     gdaex_data_model_get_field_value_integer_at (dm, 0, "id"));
-                       ret = (gdaex_execute (priv->gdaex, sql) >= 0);
-                       g_free (sql);
+                       g_warning ("Not initialized.");
+                       ret = FALSE;
                }
        else
                {
-                       g_warning ("Path %s doesn't exists.", path);
-               }
-       if (dm != NULL)
-               {
-                       g_object_unref (dm);
+                       ret = confi_pluggable_path_set_value (priv->pluggable, path, value);
                }
 
        return ret;

diff --git a/tests/test.c b/tests/test.c
index f7d6fe5971ce9e290cb7c566dc3bcf83caccfac3..59c507b40d5a9a297c8c996d3cc6ad42e70933db 100644 (file)
--- a/tests/test.c
+++ b/tests/test.c
@@ -71,7 +71,11 @@ main (int argc, char **argv)
                        return 0;
                }
 
+       gchar *val = confi_path_get_value (confi, "folder/key1/key1_2");
+       g_printf ("Value from key \"folder/key1/key1_2\"\n%s\n\n", val);
+       confi_path_set_value (confi, "folder/key1/key1_2", "new value programmatically setted");
        g_printf ("Value from key \"folder/key1/key1_2\"\n%s\n\n", confi_path_get_value (confi, "folder/key1/key1_2"));
+       confi_path_set_value (confi, "folder/key1/key1_2", val);
 
        /*g_printf ("Traversing the entire tree\n");
        tree = confi_get_tree (confi);