[Inject] Use legacy properies by preference

packages/node_modules/@node-red/nodes/core/common/20-inject.html

@@ -168,16 +168,14 @@
         color:"#a6bbcf",
         defaults: {
             name: {value:""},
-            props:{value:[{p:"payload",v:"",vt:"date"},{p:"topic",v:"",vt:"str"}]},
+            props:{value:[{p:"payload"},{p:"topic",vt:"str"}]},
             repeat: {value:"", validate:function(v) { return ((v === "") || (RED.validators.number(v) && (v >= 0) && (v <= 2147483))) }},
             crontab: {value:""},
             once: {value:false},
             onceDelay: {value:0.1},
-            /* Legacy */
             topic: {value:""},
             payload: {value:"", validate: RED.validators.typedInput("payloadType")},
             payloadType: {value:"date"},
-            /* */
         },
         icon: "inject.svg",
         inputs:0,
@@ -206,10 +204,10 @@
                     }
                     lab += props[i].p+": ";
 
-                    var propType = props[i].vt;
+                    var propType = props[i].p === "payload"? this.payloadType : props[i].vt;
                     if (propType === "json") {
                         try {
-                            var parsedProp = JSON.parse(props[i].v);
+                            var parsedProp = JSON.parse(props[i].p === "payload"? this.payload : props[i].v);
                             propType = typeof parsedProp;
                             if (propType === "object" && Array.isArray(parsedProp)) {
                                 propType = "Array";
@@ -237,30 +235,9 @@
                 return this.name+suffix;
             }
 
-            var payloadProperty;
+            var payload = this.payload || "";
-            var topicProperty;
+            var payloadType = this.payloadType || "str";
-            var payload;
+            var topic = this.topic || "";
-            var payloadType;
-            var topic;
-
-            if (Array.isArray(this.props)) {
-                // find the payload & topic if they are set
-                for (var i=0,l=this.props.length; i<l; i++) {
-                    if (this.props[i].p === 'payload') {
-                        payloadProperty = this.props[i];
-                    } else if (this.props[i].p === 'topic' && this.props[i].vt === 'str') {
-                        topicProperty = this.props[i];
-                    }
-                }
-                payload = payloadProperty === undefined ? "" : payloadProperty.v;
-                payloadType = payloadProperty === undefined ? "str" : payloadProperty.vt;
-                topic = topicProperty === undefined ? "" : topicProperty.v;
-            } else {
-                /* Legacy */
-                payload = this.payload || "";
-                payloadType = this.payloadType || "str";
-                topic = this.topic || "";
-            }
 
             if (payloadType === "string" ||
                     payloadType === "str" ||
@@ -490,7 +467,16 @@
 
             for (var i=0; i<this.props.length; i++) {
                 var prop = this.props[i];
-                $("#node-input-property-container").editableList('addItem',prop);
+                var newProp = { p: prop.p, v: prop.v, vt: prop.vt };
+                if (newProp.v === undefined) {
+                    if (prop.p === 'payload') {
+                        newProp.v = this.payload ? this.payload : '';
+                        newProp.vt = this.payloadType ? this.payloadType : 'date';
+                    } else if (prop.p === 'topic' && prop.vt === "str") {
+                        newProp.v =  this.topic ? this.topic : '';
+                    }
+                }
+                $("#node-input-property-container").editableList('addItem',newProp);
             }
 
             $("#inject-time-type-select").trigger("change");
@@ -606,15 +592,14 @@
                     if (p.p === "payload") { // save payload to old "legacy" property
                         node.payloadType = p.vt;
                         node.payload = p.v;
-                    } 
+                        delete p.v;
-                    else if (p.p === "topic" && p.vt === "str") {
+                        delete p.vt;
+                    } else if (p.p === "topic" && p.vt === "str") {
                         node.topic = p.v;
-                        node.props.push(p);
+                        delete p.v;
                     }
-                    else {
                     node.props.push(p);
                 }
-                }
             });
         },
         button: {

packages/node_modules/@node-red/nodes/core/common/20-inject.js

@@ -34,6 +34,15 @@ module.exports = function(RED) {
                 v:n.topic,
                 vt:'str'
             });
+        } else {
+            for (var i=0,l=n.props.length; i<l; i++) {
+                if (n.props[i].p === 'payload' && !n.props[i].hasOwnProperty('v')) {
+                    n.props[i].v = n.payload;
+                    n.props[i].vt = n.payloadType;
+                } else if (n.props[i].p === 'topic' && n.props[i].vt === 'str' && !n.props[i].hasOwnProperty('v')) {
+                    n.props[i].v = n.topic;
+                }
+            }
         }
 
         this.props = n.props;
@@ -43,9 +52,6 @@ module.exports = function(RED) {
         this.onceDelay = (n.onceDelay || 0.1) * 1000;
         this.interval_id = null;
         this.cronjob = null;
-        if (n.payload && n.payloadType) { // load up original payload
-            this.props.unshift({p:"payload",v:n.payload,vt:n.payloadType});
-        }
         var node = this;
 
         node.props.forEach(function (prop) {

test/nodes/core/common/20-inject_spec.js

@@ -510,6 +510,26 @@ describe('inject node', function() {
         });
     });
 
+    it('should inject multiple properties using legacy props if needed', function (done) {
+        var flow = [{id: "n1", type: "inject", payload:"123", payloadType:"num", topic:"foo", props: [{p:"topic", vt:"str"}, {p:"payload"}], wires: [["n2"]], z: "flow"},
+                    {id: "n2", type: "helper"}];
+        helper.load(injectNode, flow, function () {
+            var n1 = helper.getNode("n1");
+            var n2 = helper.getNode("n2");
+            n2.on("input", function (msg) {
+                try {
+                    msg.should.have.property("topic", "foo");
+                    msg.should.have.property("payload", 123);
+                    done();
+                } catch (err) {
+                    done(err);
+                }
+            });
+            n1.receive({});
+        });
+    });
+
+
     it('should report invalid JSONata expression', function (done) {
         var flow = [{id: "n1", type: "inject", props: [{p:"topic", v:"t1", vt:"str"}, {p:"payload", v:"@", vt:"jsonata"}], wires: [["n2"]], z: "flow"},
                     {id: "n2", type: "helper"}];