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] 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);
-}
+};