afm-system-daemon: send signal before the answer
authorJosé Bollo <jose.bollo@iot.bzh>
Fri, 12 Feb 2016 11:56:44 +0000 (12:56 +0100)
committerJosé Bollo <jose.bollo@iot.bzh>
Fri, 12 Feb 2016 11:56:44 +0000 (12:56 +0100)
Change-Id: I96a0886406235a8448e8f07b09a3cdd01aa29021
Signed-off-by: José Bollo <jose.bollo@iot.bzh>
src/afm-system-daemon.c

index 6a242ea..9cb9433 100644 (file)
@@ -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()