From e6fd97bc8191c78e9571c9171e31414b059cee51 Mon Sep 17 00:00:00 2001 From: Andrea Zagli Date: Wed, 17 Dec 2014 10:34:26 +0100 Subject: [PATCH] Implemented Confi::path_set_value in db plugin. --- plugins/db/plgdb.c | 37 +++++++++++++++++++++++++++++++++++++ src/confipluggable.c | 19 +++++++++++++++++++ src/confipluggable.h | 6 ++++++ src/libconfi.c | 24 ++++-------------------- tests/test.c | 4 ++++ 5 files changed, 70 insertions(+), 20 deletions(-) diff --git a/plugins/db/plgdb.c b/plugins/db/plgdb.c index 26725cd..57142f1 100644 --- 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 bf455e9..e43b06a 100644 --- 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 ac52cb0..71a7291 100644 --- 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 1a7edbb..6eff729 100644 --- 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 f7d6fe5..59c507b 100644 --- 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); -- 2.49.0