From: José Bollo Date: Fri, 12 Feb 2016 11:56:44 +0000 (+0100) Subject: afm-system-daemon: send signal before the answer X-Git-Tag: 2.0.2~68 X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?p=src%2Fapp-framework-main.git;a=commitdiff_plain;h=d7f0ae1fe578e92a1b7c83807683b2b9a9752efe afm-system-daemon: send signal before the answer Change-Id: I96a0886406235a8448e8f07b09a3cdd01aa29021 Signed-off-by: José Bollo --- diff --git a/src/afm-system-daemon.c b/src/afm-system-daemon.c index 6a242ea..9cb9433 100644 --- a/src/afm-system-daemon.c +++ b/src/afm-system-daemon.c @@ -102,16 +102,15 @@ static void on_install(struct jreq *jreq, struct json_object *req) resp = json_object_new_object(); if(!resp || !j_add_string(resp, "added", wgt_info_desc(ifo)->idaver)) jbus_reply_error_s(jreq, "\"out of memory but installed!\""); - else + else { + jbus_send_signal_s(jbus, "changed", "true"); jbus_reply_j(jreq, resp); + } /* clean-up */ wgt_info_unref(ifo); json_object_put(resp); } - - /* still sends the signal */ - jbus_send_signal_s(jbus, "changed", "true"); } static void on_uninstall(struct jreq *jreq, struct json_object *req) @@ -141,11 +140,10 @@ static void on_uninstall(struct jreq *jreq, struct json_object *req) rc = uninstall_widget(idaver, root); if (rc) jbus_reply_error_s(jreq, "\"uninstallation had error\""); - else + else { + jbus_send_signal_s(jbus, "changed", "true"); jbus_reply_s(jreq, "true"); - - /* still sends the signal */ - jbus_send_signal_s(jbus, "changed", "true"); + } } static int daemonize()