Fix return of falsey topic for several widgets

to close #784

Author: Dave Conway-Jones

CHANGELOG.md

@@ -1,6 +1,7 @@
 
 ### 3.2.1: Maintenance Release
 
+ - Fix topic return when topic is falsey. Issue #784
  - Fix notification toast class. Issue #776
  - Fix z-index of dropdown items. Issue #775
 

dist/dashboard.appcache

@@ -1,5 +1,5 @@
 CACHE MANIFEST
-# Time: Fri Oct 28 2022 16:27:22 GMT+0100 (British Summer Time)
+# Time: Sun Nov 20 2022 10:57:19 GMT+0000 (Greenwich Mean Time)
 
 CACHE:
 i18n.js

nodes/ui_colour_picker.js

@@ -49,7 +49,7 @@ module.exports = function(RED) {
                     if (node.format === 'hsv') { msg.payload = pay.toHsv(); }
                 }
                 var t = RED.util.evaluateNodeProperty(config.topic,config.topicType || "str",node,msg) || node.topi;
-                if (t) { msg.topic = t; }
+                if (t !== undefined) { msg.topic = t; }
             },
             convert: function(p,o,m) {
                 if (m.payload === undefined || m.payload === null) { return; }

nodes/ui_date_picker.js

@@ -45,7 +45,7 @@ module.exports = function(RED) {
             },
             beforeSend: function (msg) {
                 var t = RED.util.evaluateNodeProperty(config.topic,config.topicType || "str",node,msg) || node.topi;
-                if (t) { msg.topic = t; }
+                if (t !== undefined) { msg.topic = t; }
             }
         });
         node.on("close", done);

nodes/ui_dropdown.js

@@ -181,7 +181,7 @@ module.exports = function(RED) {
                     msg.payload = emitOptions.value;
                 }
                 var t = RED.util.evaluateNodeProperty(config.topic,config.topicType || "str",node,msg) || node.topi;
-                if (t) { msg.topic = t; }
+                if (t !== undefined) { msg.topic = t; }
                 if (msg.payload === null || msg._dontSend) { node.status({}); }
                 else {
                     var stat = "";

nodes/ui_form.js

@@ -37,7 +37,7 @@ module.exports = function(RED) {
             },
             beforeSend: function (msg) {
                 var t = RED.util.evaluateNodeProperty(config.topic,config.topicType || "str",node,msg) || node.topi;
-                if (t) { msg.topic = t; }
+                if (t !== undefined) { msg.topic = t; }
             }
         });
         node.on("close", done);

nodes/ui_numeric.js

@@ -43,7 +43,7 @@ module.exports = function(RED) {
             beforeSend: function (msg) {
                 msg.payload = parseFloat(msg.payload);
                 var t = RED.util.evaluateNodeProperty(config.topic,config.topicType || "str",node,msg) || node.topi;
-                if (t) { msg.topic = t; }
+                if (t !== undefined) { msg.topic = t; }
                 if (node.pt) {
                     node.status({shape:"dot",fill:"grey",text:msg.payload});
                 }

nodes/ui_slider.js

@@ -39,7 +39,7 @@ module.exports = function(RED) {
             },
             beforeSend: function (msg) {
                 var t = RED.util.evaluateNodeProperty(config.topic,config.topicType || "str",node,msg) || node.topi;
-                if (t) { msg.topic = t; }
+                if (t !== undefined) { msg.topic = t; }
                 if (node.pt) {
                     node.status({shape:"dot",fill:"grey",text:msg.payload});
                 }

nodes/ui_switch.js

@@ -135,7 +135,7 @@ module.exports = function(RED) {
             },
             beforeSend: function (msg) {
                 var t = RED.util.evaluateNodeProperty(config.topic,config.topicType || "str",node,msg) || node.topi;
-                if (t) { msg.topic = t; }
+                if (t !== undefined) { msg.topic = t; }
             }
         });
 

nodes/ui_text_input.js

@@ -41,7 +41,7 @@ module.exports = function(RED) {
                 // if (config.mode === "week") { msg.payload = Date.parse(msg.payload); }
                 // if (config.mode === "month") { msg.payload = Date.parse(msg.payload); }
                 var t = RED.util.evaluateNodeProperty(config.topic,config.topicType || "str",node,msg) || node.topi;
-                if (t) { msg.topic = t; }
+                if (t !== undefined) { msg.topic = t; }
             }
         });
         node.on("close", done);