From 97e5c2e571f3c91ece50cf0aa3fc6e88e8e86921 Mon Sep 17 00:00:00 2001
From: Anna Thomas <annat21@live.co.uk>
Date: Fri, 3 Oct 2014 16:12:24 +0100
Subject: [PATCH 1/2] Added multi-level property functionality

---
 nodes/core/logic/15-change.html | 10 +++---
 nodes/core/logic/15-change.js   | 64 +++++++++++++++------------------
 2 files changed, 35 insertions(+), 39 deletions(-)

diff --git a/nodes/core/logic/15-change.html b/nodes/core/logic/15-change.html
index 702dd8072..acdefb33e 100644
--- a/nodes/core/logic/15-change.html
+++ b/nodes/core/logic/15-change.html
@@ -36,7 +36,7 @@
     <div class="form-row" id="node-reg-row">
         <label>&nbsp;</label>
         <input type="checkbox" id="node-input-reg" style="display: inline-block; width: auto; vertical-align: top;">
-        <label for="node-input-reg" style="width: 70%;">Use regular expressions ?</label>
+        <label for="node-input-reg" style="width: 70%;">Use regular expressions</label>
     </div>
     <div class="form-tips" id="node-tip"></div>
     <br/>
@@ -47,10 +47,10 @@
 </script>
 
 <script type="text/x-red" data-help-name="change">
-    <p>A simple function node to change, replace, add or delete properties of a message.</p>
+    <p>A simple function node to set, replace or delete properties of a message.</p>
     <p>When a message arrives, the selected property is modified by the defined rules.
     The message is then sent to the output.</p>
-    <p><b>Note:</b> Replace only operates on <b>strings</b>. Anything else will be passed straight through.</p>
+    <p><b>Note:</b> Set and replace only operate using <b>strings</b>. Anything else will be passed straight through.</p>
 </script>
 
 <script type="text/javascript">
@@ -82,8 +82,10 @@
             if (this.name) {
                 return this.name;
             }
-            if (this.action == "replace") {
+            if (this.action === "replace") {
                 return "set msg."+this.property;
+            } else if (this.action === "change") {
+                return "replace msg."+this.property;
             } else {
                 return this.action+" msg."+this.property
             }
diff --git a/nodes/core/logic/15-change.js b/nodes/core/logic/15-change.js
index b7ef62e15..bcb2aa53a 100644
--- a/nodes/core/logic/15-change.js
+++ b/nodes/core/logic/15-change.js
@@ -20,55 +20,49 @@ module.exports = function(RED) {
         RED.nodes.createNode(this, n);
         this.action = n.action;
         this.property = n.property || "";
-        this.from = n.from || " ";
-        this.to = n.to || " ";
+        this.from = n.from || "";
+        this.to = n.to || "";
         this.reg = (n.reg === null || n.reg);
         var node = this;
         if (node.reg === false) {
             this.from = this.from.replace(/[-[\]{}()*+?.,\\^$|#\s]/g, "\\$&");
         }
-        var makeNew = function( stem, path, value ) {
-            var lastPart = (arguments.length === 3) ? path.pop() : false;
-            for (var i = 0; i < path.length; i++) {
-                stem = stem[path[i]] = stem[path[i]] || {};
-            }
-            if (lastPart) { stem = stem[lastPart] = value; }
-            return stem;
-        };
 
         this.on('input', function (msg) {
-            if (node.action == "change") {
+            var propertyParts;
+            var depth = 0;
+
+            if (node.action === "change") {
                 try {
                     node.re = new RegExp(this.from, "g");
                 } catch (e) {
                     node.error(e.message);
                 }
-                if (typeof msg[node.property] === "string") {
-                    msg[node.property] = (msg[node.property]).replace(node.re, node.to);
-                }
-            }
-            //else if (node.action == "replace") {
-                //if (node.to.indexOf("msg.") == 0) {
-                    //msg[node.property] = eval(node.to);
-                //}
-                //else {
-                    //msg[node.property] = node.to;
-                //}
-            //}
-            else if (node.action == "replace") {
-                if (node.to.indexOf("msg.") === 0) {
-                    makeNew( msg, node.property.split("."), eval(node.to) );
-                }
-                else {
-                    makeNew( msg, node.property.split("."), node.to );
-                }
-                //makeNew( msg, node.property.split("."), node.to );
-            }
-            else if (node.action == "delete") {
-                delete(msg[node.property]);
             }
+
+            propertyParts = node.property.split(".");
+            try {
+                propertyParts.reduce(function (obj, i) {
+                    if (++depth === propertyParts.length) {
+                        if (node.action === "change") {
+                            if (typeof obj[i] === "string") {
+                                obj[i] = obj[i].replace(node.re, node.to);
+                            }
+                        } else if (node.action === "replace") {
+                            obj[i] = node.to;
+                        } else if (node.action === "delete") {
+                            delete(obj[i]);
+                        }
+                    } else {
+                        if (!obj[i]) {
+                            obj[i] = {};
+                        }
+                        return obj[i];
+                    }
+                }, msg);
+            } catch (err) {}
             node.send(msg);
         });
     }
     RED.nodes.registerType("change", ChangeNode);
-}
+};

From 986ce8163f394346ce83b9b818889008e2bd8d51 Mon Sep 17 00:00:00 2001
From: Anna Thomas <annat21@live.co.uk>
Date: Tue, 7 Oct 2014 14:46:43 +0100
Subject: [PATCH 2/2] Added tests to check functionality when using multi-level
 properties

---
 test/nodes/core/logic/15-change_spec.js | 207 ++++++++++++++++++++----
 1 file changed, 175 insertions(+), 32 deletions(-)

diff --git a/test/nodes/core/logic/15-change_spec.js b/test/nodes/core/logic/15-change_spec.js
index 5072dfcee..88472a330 100644
--- a/test/nodes/core/logic/15-change_spec.js
+++ b/test/nodes/core/logic/15-change_spec.js
@@ -20,11 +20,11 @@ var changeNode = require("../../../../nodes/core/logic/15-change.js");
 var helper = require("../../helper.js");
 
 describe('ChangeNode', function() {
-    
+
     beforeEach(function(done) {
         helper.startServer(done);
     });
-    
+
     afterEach(function(done) {
         helper.unload();
         helper.stopServer(done);
@@ -38,7 +38,7 @@ describe('ChangeNode', function() {
             done();
         });
     });
-    
+
     it('sets the value of the message property', function(done) {
         var flow = [{"id":"changeNode1","type":"change","action":"replace","property":"payload","from":"","to":"changed","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
                     {id:"helperNode1", type:"helper", wires:[]}];
@@ -56,7 +56,43 @@ describe('ChangeNode', function() {
             changeNode1.receive({payload:"changeMe"});
         });
     });
-    
+
+    it('sets the value of an already set multi-level message property', function(done) {
+        var flow = [{"id":"changeNode1","type":"change","action":"replace","property":"foo.bar","from":"","to":"bar","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
+                    {id:"helperNode1", type:"helper", wires:[]}];
+        helper.load(changeNode, flow, function() {
+            var changeNode1 = helper.getNode("changeNode1");
+            var helperNode1 = helper.getNode("helperNode1");
+            helperNode1.on("input", function(msg) {
+                try {
+                    msg.foo.bar.should.equal("bar");
+                    done();
+                } catch(err) {
+                    done(err);
+                }
+            });
+            changeNode1.receive({foo:{bar:"foo"}});
+        });
+    });
+
+    it('sets the value of an empty multi-level message property', function(done) {
+        var flow = [{"id":"changeNode1","type":"change","action":"replace","property":"foo.bar","from":"","to":"bar","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
+                    {id:"helperNode1", type:"helper", wires:[]}];
+        helper.load(changeNode, flow, function() {
+            var changeNode1 = helper.getNode("changeNode1");
+            var helperNode1 = helper.getNode("helperNode1");
+            helperNode1.on("input", function(msg) {
+                try {
+                    msg.foo.bar.should.equal("bar");
+                    done();
+                } catch(err) {
+                    done(err);
+                }
+            });
+            changeNode1.receive({});
+        });
+    });
+
     it('changes the value of the message property', function(done) {
         var flow = [{"id":"changeNode1","type":"change","action":"change","property":"payload","from":"Hello","to":"Goodbye","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
                     {id:"helperNode1", type:"helper", wires:[]}];
@@ -74,7 +110,61 @@ describe('ChangeNode', function() {
             changeNode1.receive({payload:"Hello World!"});
         });
     });
-    
+
+    it('changes the value of a multi-level message property', function(done) {
+        var flow = [{"id":"changeNode1","type":"change","action":"change","property":"foo.bar","from":"Hello","to":"Goodbye","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
+                    {id:"helperNode1", type:"helper", wires:[]}];
+        helper.load(changeNode, flow, function() {
+            var changeNode1 = helper.getNode("changeNode1");
+            var helperNode1 = helper.getNode("helperNode1");
+            helperNode1.on("input", function(msg) {
+                try {
+                    msg.foo.bar.should.equal("Goodbye World!");
+                    done();
+                } catch(err) {
+                    done(err);
+                }
+            });
+            changeNode1.receive({foo:{bar:"Hello World!"}});
+        });
+    });
+
+    it('sends unaltered message if the changed message property does not exist', function(done) {
+        var flow = [{"id":"changeNode1","type":"change","action":"change","property":"foo","from":"Hello","to":"Goodbye","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
+                    {id:"helperNode1", type:"helper", wires:[]}];
+        helper.load(changeNode, flow, function() {
+            var changeNode1 = helper.getNode("changeNode1");
+            var helperNode1 = helper.getNode("helperNode1");
+            helperNode1.on("input", function(msg) {
+                try {
+                    msg.payload.should.equal("Hello World!");
+                    done();
+                } catch(err) {
+                    done(err);
+                }
+            });
+            changeNode1.receive({payload:"Hello World!"});
+        });
+    });
+
+    it('sends unaltered message if a changed multi-level message property does not exist', function(done) {
+        var flow = [{"id":"changeNode1","type":"change","action":"change","property":"foo.bar","from":"Hello","to":"Goodbye","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
+                    {id:"helperNode1", type:"helper", wires:[]}];
+        helper.load(changeNode, flow, function() {
+            var changeNode1 = helper.getNode("changeNode1");
+            var helperNode1 = helper.getNode("helperNode1");
+            helperNode1.on("input", function(msg) {
+                try {
+                    msg.payload.should.equal("Hello World!");
+                    done();
+                } catch(err) {
+                    done(err);
+                }
+            });
+            changeNode1.receive({payload:"Hello World!"});
+        });
+    });
+
     it('changes the value of the message property based on a regex', function(done) {
         var flow = [{"id":"changeNode1","type":"change","action":"change","property":"payload","from":"\\d+","to":"NUMBER","reg":true,"name":"changeNode","wires":[["helperNode1"]]},
                     {id:"helperNode1", type:"helper", wires:[]}];
@@ -92,7 +182,7 @@ describe('ChangeNode', function() {
             changeNode1.receive({payload:"Replace all numbers 12 and 14"});
         });
     });
-    
+
     it('supports regex groups', function(done) {
         var flow = [{"id":"changeNode1","type":"change","action":"change","property":"payload","from":"(Hello)","to":"$1-$1-$1","reg":true,"name":"changeNode","wires":[["helperNode1"]]},
                     {id:"helperNode1", type:"helper", wires:[]}];
@@ -110,17 +200,15 @@ describe('ChangeNode', function() {
             changeNode1.receive({payload:"Hello World"});
         });
     });
-    
+
     it('Reports invalid regex', function(done) {
-        
         var sinon = require('sinon');
-        
         var flow = [{"id":"changeNode1","type":"change","action":"change","property":"payload","from":"\\+**+","to":"NUMBER","reg":true,"name":"changeNode","wires":[["helperNode1"]]},
                     {id:"helperNode1", type:"helper", wires:[]}];
         helper.load(changeNode, flow, function() {
             var changeNode1 = helper.getNode("changeNode1");
             var helperNode1 = helper.getNode("helperNode1");
-            
+
             sinon.stub(changeNode1, 'error', function(error) {
                 if(error.indexOf("regular expression" > -1)) {
                     done();
@@ -135,7 +223,7 @@ describe('ChangeNode', function() {
             changeNode1.receive({payload:"This is irrelevant"});
         });
     });
-    
+
     it('deletes the value of the message property', function(done) {
         var flow = [{"id":"changeNode1","type":"change","action":"delete","property":"payload","from":"","to":"","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
                     {id:"helperNode1", type:"helper", wires:[]}];
@@ -153,26 +241,81 @@ describe('ChangeNode', function() {
             changeNode1.receive({payload:"This won't get through!"});
         });
     });
-    
-// TODO confirm the behaviour of the change node later,apparently calling eval such that  makeNew( msg, node.property.split("."), eval(node.to) ); is incorrect
-//    it('changes the property name of the message object', function(done) {
-//        var flow = [{"id":"changeNode1","type":"change","action":"replace","property":"payload","from":"","to":"msg.otherProp=10","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
-//                    {id:"helperNode1", type:"helper", wires:[]}];
-//        helper.load(changeNode, flow, function() {
-//            var changeNode1 = helper.getNode("changeNode1");
-//            var helperNode1 = helper.getNode("helperNode1");
-//            helperNode1.on("input", function(msg) {
-//                try {
-//                    msg.otherProp.should.equal(10);
-//                    done();
-//                } catch(err) {
-//                    done(err);
-//                }
-//            });
-//            changeNode1.receive({payload:"changeMe"});
-//        });
-//    });
-    
+
+    it('deletes the value of a multi-level message property', function(done) {
+        var flow = [{"id":"changeNode1","type":"change","action":"delete","property":"foo.bar","from":"","to":"","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
+                    {id:"helperNode1", type:"helper", wires:[]}];
+        helper.load(changeNode, flow, function() {
+            var changeNode1 = helper.getNode("changeNode1");
+            var helperNode1 = helper.getNode("helperNode1");
+            helperNode1.on("input", function(msg) {
+                try {
+                    msg.should.not.have.property('foo.bar');
+                    done();
+                } catch(err) {
+                    done(err);
+                }
+            });
+            changeNode1.receive({payload:"This won't get through!", foo:{bar:"This will be deleted!"}});
+        });
+    });
+
+    it('sends unaltered message if the deleted message property does not exist', function(done) {
+        var flow = [{"id":"changeNode1","type":"change","action":"delete","property":"foo","from":"","to":"","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
+                    {id:"helperNode1", type:"helper", wires:[]}];
+        helper.load(changeNode, flow, function() {
+            var changeNode1 = helper.getNode("changeNode1");
+            var helperNode1 = helper.getNode("helperNode1");
+            helperNode1.on("input", function(msg) {
+                try {
+                    msg.should.not.have.property('foo');
+                    msg.payload.should.equal('payload');
+                    done();
+                } catch(err) {
+                    done(err);
+                }
+            });
+            changeNode1.receive({payload:"payload"});
+        });
+    });
+
+    it('sends unaltered message if a deleted multi-level message property does not exist', function(done) {
+        var flow = [{"id":"changeNode1","type":"change","action":"delete","property":"foo.bar","from":"","to":"","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
+                    {id:"helperNode1", type:"helper", wires:[]}];
+        helper.load(changeNode, flow, function() {
+            var changeNode1 = helper.getNode("changeNode1");
+            var helperNode1 = helper.getNode("helperNode1");
+            helperNode1.on("input", function(msg) {
+                try {
+                    msg.should.not.have.property('foo.bar');
+                    msg.payload.should.equal('payload');
+                    done();
+                } catch(err) {
+                    done(err);
+                }
+            });
+            changeNode1.receive({payload:"payload"});
+        });
+    });
+
+   it('does not change other properties', function(done) {
+       var flow = [{"id":"changeNode1","type":"change","action":"replace","property":"payload","from":"","to":"msg.otherProp=10","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
+                   {id:"helperNode1", type:"helper", wires:[]}];
+       helper.load(changeNode, flow, function() {
+           var changeNode1 = helper.getNode("changeNode1");
+           var helperNode1 = helper.getNode("helperNode1");
+           helperNode1.on("input", function(msg) {
+               try {
+                   msg.payload.should.not.equal(10);
+                   done();
+               } catch(err) {
+                   done(err);
+               }
+           });
+           changeNode1.receive({payload:"changeMe"});
+       });
+   });
+
     it('splits dot delimited properties into objects', function(done) {
         var flow = [{"id":"changeNode1","type":"change","action":"replace","property":"pay.load","from":"","to":"10","reg":false,"name":"changeNode","wires":[["helperNode1"]]},
                     {id:"helperNode1", type:"helper", wires:[]}];
@@ -187,7 +330,7 @@ describe('ChangeNode', function() {
                     done(err);
                 }
             });
-            changeNode1.receive({"pay.load":"changeMe"});
+            changeNode1.receive({pay:{load:"changeMe"}});
         });
     });
 });