Prevent repeated fermenter offline broadcasts

Tue, 12 May 2020 14:24:45 +0200

author
Michiel Broek <mbroek@mbse.eu>
date
Tue, 12 May 2020 14:24:45 +0200
changeset 674
c865e3cdee04
parent 673
9924b1218d39
child 675
a27af02ab16a

Prevent repeated fermenter offline broadcasts

bmsd/fermenters.c file | annotate | diff | comparison | revisions
--- a/bmsd/fermenters.c	Tue May 12 13:16:25 2020 +0200
+++ b/bmsd/fermenters.c	Tue May 12 14:24:45 2020 +0200
@@ -758,34 +758,36 @@
     if (alias) {
     	for (tmpp = fermenters; tmpp; tmpp = tmpp->next) {
 	    if ((strcmp(tmpp->node, edge_node) == 0) && (strcmp(tmpp->alias, alias) == 0)) {
-		if (tmpp->online)
+		if (tmpp->online) {
 		    syslog(LOG_NOTICE, "Offline fermenter %s/%s", tmpp->node, tmpp->alias);
-	    	tmpp->online = false;
-		msg = xstrcpy((char *)"{\"device\":\"fermenter\",\"node\":\"");
-    		msg = xstrcat(msg, edge_node);
-    		msg = xstrcat(msg, (char *)"\",\"unit\":\"");
-    		msg = xstrcat(msg, alias);
-    		msg = xstrcat(msg, (char *)"\",\"online\":0}");
-		ws_broadcast(msg);
-    		free(msg);
-    		msg = NULL;
+	    	    tmpp->online = false;
+		    msg = xstrcpy((char *)"{\"device\":\"fermenter\",\"node\":\"");
+    		    msg = xstrcat(msg, edge_node);
+    		    msg = xstrcat(msg, (char *)"\",\"unit\":\"");
+    		    msg = xstrcat(msg, alias);
+    		    msg = xstrcat(msg, (char *)"\",\"online\":0}");
+		    ws_broadcast(msg);
+    		    free(msg);
+    		    msg = NULL;
+		}
 	    	break;
 	    }
     	}
     } else {
 	for (tmpp = fermenters; tmpp; tmpp = tmpp->next) {
 	    if (strcmp(tmpp->node, edge_node) == 0) {
-		if (tmpp->online)
+		if (tmpp->online) {
 		    syslog(LOG_NOTICE, "Offline fermenter %s/%s", tmpp->node, tmpp->alias);
-		tmpp->online = false;
-		msg = xstrcpy((char *)"{\"device\":\"fermenter\",\"node\":\"");
-                msg = xstrcat(msg, edge_node);
-                msg = xstrcat(msg, (char *)"\",\"unit\":\"");
-                msg = xstrcat(msg, tmpp->alias);
-                msg = xstrcat(msg, (char *)"\",\"online\":0}");
-                ws_broadcast(msg);
-                free(msg);
-                msg = NULL;
+		    tmpp->online = false;
+		    msg = xstrcpy((char *)"{\"device\":\"fermenter\",\"node\":\"");
+                    msg = xstrcat(msg, edge_node);
+                    msg = xstrcat(msg, (char *)"\",\"unit\":\"");
+                    msg = xstrcat(msg, tmpp->alias);
+                    msg = xstrcat(msg, (char *)"\",\"online\":0}");
+                    ws_broadcast(msg);
+                    free(msg);
+                    msg = NULL;
+		}
 	    }
 	}
     }

mercurial