From 1d10eba0cce2c6351b082b06b315cfc9452d572a Mon Sep 17 00:00:00 2001
From: Nick O'Leary <nick.oleary@gmail.com>
Date: Fri, 20 Mar 2015 21:20:04 +0000
Subject: [PATCH] Propagate changed flag to parent subflow

---
 public/red/ui/editor.js | 11 ++++++++++-
 public/red/ui/view.js   |  2 +-
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/public/red/ui/editor.js b/public/red/ui/editor.js
index 11381f01b..378a46893 100644
--- a/public/red/ui/editor.js
+++ b/public/red/ui/editor.js
@@ -28,17 +28,22 @@ RED.editor = (function() {
      */
     function validateNode(node) {
         var oldValue = node.valid;
+        var oldChanged = node.changed;
         node.valid = true;
         var subflow;
         var isValid;
+        var hasChanged;
         
         if (node.type.indexOf("subflow:")===0) {
             subflow = RED.nodes.subflow(node.type.substring(8));
             isValid = subflow.valid;
+            hasChanged = subflow.changed;
             if (isValid === undefined) {
                 isValid = validateNode(subflow);
+                hasChanged = subflow.changed;
             }
             node.valid = isValid;
+            node.changed = hasChanged;
         } else if (node._def) {
             node.valid = validateNodeProperties(node, node._def.defaults, node);
             if (node._def._creds) {
@@ -48,15 +53,19 @@ RED.editor = (function() {
             var subflowNodes = RED.nodes.filterNodes({z:node.id});
             for (var i=0;i<subflowNodes.length;i++) {
                 isValid = subflowNodes[i].valid;
+                hasChanged = subflowNodes[i].changed;
                 if (isValid === undefined) {
                     isValid = validateNode(subflowNodes[i]);
+                    hasChanged = subflowNodes[i].changed;
                 }
                 node.valid = node.valid && isValid;
+                node.changed = node.changed || hasChanged;
             }
             var subflowInstances = RED.nodes.filterNodes({type:"subflow:"+node.id});
             var modifiedTabs = {};
             for (i=0;i<subflowInstances.length;i++) {
                 subflowInstances[i].valid = node.valid;
+                subflowInstances[i].changed = node.changed;
                 subflowInstances[i].dirty = true;
                 modifiedTabs[subflowInstances[i].z] = true;
             }
@@ -67,7 +76,7 @@ RED.editor = (function() {
                 }
             });
         }
-        if (oldValue !== node.valid) {
+        if (oldValue !== node.valid || oldChanged !== node.changed) {
             node.dirty = true;
             subflow = RED.nodes.subflow(node.z);
             if (subflow) {
diff --git a/public/red/ui/view.js b/public/red/ui/view.js
index 0a9d70a90..32cb16df3 100644
--- a/public/red/ui/view.js
+++ b/public/red/ui/view.js
@@ -333,7 +333,6 @@ RED.view = (function() {
                 if (!m) {
                     nn.inputs = nn._def.inputs || 0;
                     nn.outputs = nn._def.outputs;
-                    nn.changed = true;
                     
                     for (var d in nn._def.defaults) {
                         if (nn._def.defaults.hasOwnProperty(d)) {
@@ -350,6 +349,7 @@ RED.view = (function() {
                     nn.outputs = subflow.out.length;
                 }
                 
+                nn.changed = true;
                 nn.h = Math.max(node_height,(nn.outputs||0) * 15);
                 RED.history.push({t:'add',nodes:[nn.id],dirty:RED.nodes.dirty()});
                 RED.nodes.add(nn);