From 47b4ebc92f6a24b7cfc299b75aa5c1af2c83d457 Mon Sep 17 00:00:00 2001
From: Nick O'Leary <nick.oleary@gmail.com>
Date: Wed, 20 Aug 2014 17:05:01 +0100
Subject: [PATCH] Check for null when undoing history actions

---
 public/red/history.js | 46 ++++++++++++++++++++++++++++---------------
 1 file changed, 30 insertions(+), 16 deletions(-)

diff --git a/public/red/history.js b/public/red/history.js
index b5939f701..3a6d55133 100644
--- a/public/red/history.js
+++ b/public/red/history.js
@@ -34,26 +34,38 @@ RED.history = (function() {
             var i;
             if (ev) {
                 if (ev.t == 'add') {
-                    for (i=0;i<ev.nodes.length;i++) {
-                        RED.nodes.remove(ev.nodes[i]);
+                    if (ev.nodes) {
+                        for (i=0;i<ev.nodes.length;i++) {
+                            RED.nodes.remove(ev.nodes[i]);
+                        }
                     }
-                    for (i=0;i<ev.links.length;i++) {
-                        RED.nodes.removeLink(ev.links[i]);
+                    if (ev.links) {
+                        for (i=0;i<ev.links.length;i++) {
+                            RED.nodes.removeLink(ev.links[i]);
+                        }
                     }
-                    for (i=0;i<ev.workspaces.length;i++) {
-                        RED.nodes.removeWorkspace(ev.workspaces[i].id);
-                        RED.view.removeWorkspace(ev.workspaces[i]);
+                    if (ev.workspaces) {
+                        for (i=0;i<ev.workspaces.length;i++) {
+                            RED.nodes.removeWorkspace(ev.workspaces[i].id);
+                            RED.view.removeWorkspace(ev.workspaces[i]);
+                        }
                     }
                 } else if (ev.t == "delete") {
-                    for (i=0;i<ev.workspaces.length;i++) {
-                        RED.nodes.addWorkspace(ev.workspaces[i]);
-                        RED.view.addWorkspace(ev.workspaces[i]);
+                    if (ev.workspaces) {
+                        for (i=0;i<ev.workspaces.length;i++) {
+                            RED.nodes.addWorkspace(ev.workspaces[i]);
+                            RED.view.addWorkspace(ev.workspaces[i]);
+                        }
                     }
-                    for (i=0;i<ev.nodes.length;i++) {
-                        RED.nodes.add(ev.nodes[i]);
+                    if (ev.nodes) {
+                        for (i=0;i<ev.nodes.length;i++) {
+                            RED.nodes.add(ev.nodes[i]);
+                        }
                     }
-                    for (i=0;i<ev.links.length;i++) {
-                        RED.nodes.addLink(ev.links[i]);
+                    if (ev.links) {
+                        for (i=0;i<ev.links.length;i++) {
+                            RED.nodes.addLink(ev.links[i]);
+                        }
                     }
                 } else if (ev.t == "move") {
                     for (i=0;i<ev.nodes.length;i++) {
@@ -69,8 +81,10 @@ RED.history = (function() {
                         }
                     }
                     RED.editor.updateNodeProperties(ev.node);
-                    for (i=0;i<ev.links.length;i++) {
-                        RED.nodes.addLink(ev.links[i]);
+                    if (ev.links) {
+                        for (i=0;i<ev.links.length;i++) {
+                            RED.nodes.addLink(ev.links[i]);
+                        }
                     }
                     RED.editor.validateNode(ev.node);
                     ev.node.dirty = true;