From 7c82710dbf4604c31a4f95417be0e485cabc40e7 Mon Sep 17 00:00:00 2001 From: Tom MacWright Date: Fri, 7 Dec 2012 10:11:42 -0500 Subject: [PATCH] Fix deleting areas --- js/id/actions/delete_way.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/js/id/actions/delete_way.js b/js/id/actions/delete_way.js index 86fe3a3bc..a16309e56 100644 --- a/js/id/actions/delete_way.js +++ b/js/id/actions/delete_way.js @@ -11,6 +11,10 @@ iD.actions.DeleteWay = function(wayId) { way.nodes.forEach(function (nodeId) { var node = graph.entity(nodeId); + // Circular ways include nodes more than once, so they + // can be deleted on earlier iterations of this loop. + if (!node) return; + graph = iD.actions.RemoveWayNode(wayId, nodeId)(graph); if (!graph.parentWays(nodeId).length && !graph.parentRelations(nodeId).length) {